cleanup old tools for sanities sake
authorEileen McNaughton <eileen@fuzion.co.nz>
Tue, 13 Jan 2015 00:18:35 +0000 (13:18 +1300)
committerEileen McNaughton <eileen@fuzion.co.nz>
Tue, 13 Jan 2015 00:18:35 +0000 (13:18 +1300)
18 files changed:
tools/CRM/Auction/BAO/Auction.php
tools/CRM/Auction/BAO/Item.php
tools/CRM/Auction/Form/Auction.php
tools/CRM/Auction/Form/Item.php
tools/CRM/Auction/Form/ItemAccount.php
tools/CRM/Auction/Form/SearchAuction.php
tools/CRM/Auction/Form/SearchItem.php
tools/CRM/Auction/Info.php
tools/CRM/Auction/Page/AddItem.php
tools/CRM/Auction/Page/DashBoard.php
tools/CRM/Auction/Page/Item.php
tools/CRM/Auction/Page/Manage.php
tools/CRM/Auction/Page/ManageItem.php
tools/CRM/Touchstone/Info.php
tools/CRM/Touchstone/PseudoConstant.php
tools/CRM/Touchstone/Task.php
tools/extensions/org.civicrm.payment.googlecheckout/GoogleCheckout.php
tools/extensions/org.civicrm.payment.googlecheckout/GoogleIPN.php

index c4e69c1a8a14783c2dfa292920a9cffe3fa26d06..d0a0dcf4a7812f9195003090cda3d5bba5da18f0 100644 (file)
@@ -55,11 +55,8 @@ class CRM_Auction_BAO_Auction extends CRM_Auction_DAO_Auction {
    * @param array $defaults (reference ) an assoc array to hold the flattened values
    *
    * @return CRM_Auction_BAO_Auction object
-   * @access public
-   * @static
    */
-  static
-  function retrieve(&$params, &$defaults) {
+  static function retrieve(&$params, &$defaults) {
     $auction = new CRM_Auction_DAO_Auction();
     $auction->copyValues($params);
     if ($auction->find(TRUE)) {
@@ -76,10 +73,8 @@ class CRM_Auction_BAO_Auction extends CRM_Auction_DAO_Auction {
    * @param boolean  $is_active value we want to set the is_active field
    *
    * @return Object             DAO object on sucess, null otherwise
-   * @static
    */
-  static
-  function setIsActive($id, $is_active) {
+  static function setIsActive($id, $is_active) {
     return CRM_Core_DAO::setFieldValue('CRM_Auction_DAO_Auction', $id, 'is_active', $is_active);
   }
 
@@ -87,16 +82,9 @@ class CRM_Auction_BAO_Auction extends CRM_Auction_DAO_Auction {
    * add the auction
    *
    * @param array $params reference array contains the values submitted by the form
-   *
-   * @access public
-   * @static
-   *
    * @return object
    */
-  static
-  function add(&$params) {
-    require_once 'CRM/Utils/Hook.php';
-
+  static function add(&$params) {
     if (!empty($params['id'])) {
       CRM_Utils_Hook::pre('edit', 'Auction', $params['id'], $params);
     }
@@ -125,12 +113,8 @@ class CRM_Auction_BAO_Auction extends CRM_Auction_DAO_Auction {
    * @param array $params reference array contains the values submitted by the form
    *
    * @return object
-   * @access public
-   * @static
-   *
    */
   public static function create(&$params) {
-    require_once 'CRM/Core/Transaction.php';
     $transaction = new CRM_Core_Transaction();
 
     $auction = self::add($params);
@@ -150,13 +134,8 @@ class CRM_Auction_BAO_Auction extends CRM_Auction_DAO_Auction {
    *
    * @param int $id  auction id
    *
-   * @access public
-   * @static
-   *
    */
-  static
-  function del($id) {
-    require_once 'CRM/Auction/DAO/Auction.php';
+  static function del($id) {
     $auction     = new CRM_Auction_DAO_Auction();
     $auction->id = $id;
     $result      = $auction->delete();
@@ -171,10 +150,8 @@ class CRM_Auction_BAO_Auction extends CRM_Auction_DAO_Auction {
    * @param bool $id
    *
    * @return array
-   * @static
    */
-  static
-  function getAuctions($all = FALSE, $id = FALSE) {
+  static function getAuctions($all = FALSE, $id = FALSE) {
     $query = "SELECT `id`, `title`, `start_date` FROM `civicrm_auction`";
 
     if (!$all) {
index 319a2df64dd3b0cbbcfd298a2b6d3b5200190be7..26fa5de1d23d6ebfacaddfe49dd20cfca17109d7 100644 (file)
@@ -55,11 +55,8 @@ class CRM_Auction_BAO_Item extends CRM_Auction_DAO_Auction {
    * @param array $defaults (reference ) an assoc array to hold the flattened values
    *
    * @return CRM_Auction_BAO_Item object
-   * @access public
-   * @static
    */
-  static
-  function retrieve(&$params, &$defaults) {
+  static function retrieve(&$params, &$defaults) {
     $auction = new CRM_Auction_DAO_Item();
     $auction->copyValues($params);
     if ($auction->find(TRUE)) {
@@ -76,10 +73,8 @@ class CRM_Auction_BAO_Item extends CRM_Auction_DAO_Auction {
    * @param boolean  $is_active value we want to set the is_active field
    *
    * @return Object             DAO object on sucess, null otherwise
-   * @static
    */
-  static
-  function setIsActive($id, $is_active) {
+  static function setIsActive($id, $is_active) {
     return CRM_Core_DAO::setFieldValue('CRM_Auction_DAO_Item', $id, 'is_active', $is_active);
   }
 
@@ -87,16 +82,9 @@ class CRM_Auction_BAO_Item extends CRM_Auction_DAO_Auction {
    * add the auction
    *
    * @param array $params reference array contains the values submitted by the form
-   *
-   * @access public
-   * @static
-   *
    * @return object
    */
-  static
-  function add(&$params) {
-    require_once 'CRM/Utils/Hook.php';
-
+  static function add(&$params) {
     if (!empty($params['id'])) {
       CRM_Utils_Hook::pre('edit', 'Auction_Item', $params['id'], $params);
     }
@@ -137,12 +125,8 @@ class CRM_Auction_BAO_Item extends CRM_Auction_DAO_Auction {
    * @param array $params reference array contains the values submitted by the form
    *
    * @return object
-   * @access public
-   * @static
-   *
    */
   public static function create(&$params) {
-    require_once 'CRM/Core/Transaction.php';
     $transaction = new CRM_Core_Transaction();
 
     $auction = self::add($params);
@@ -161,14 +145,8 @@ class CRM_Auction_BAO_Item extends CRM_Auction_DAO_Auction {
    * delete the auction
    *
    * @param int $id  auction id
-   *
-   * @access public
-   * @static
-   *
    */
-  static
-  function del($id) {
-    require_once 'CRM/Auction/DAO/Item.php';
+  static function del($id) {
     $auction     = new CRM_Auction_DAO_Item();
     $auction->id = $id;
     $result      = $auction->delete();
@@ -183,11 +161,8 @@ class CRM_Auction_BAO_Item extends CRM_Auction_DAO_Auction {
    * @internal param int $id profile id
    *
    * @return boolean
-   * @access public
-   * @static
    */
-  static
-  function isEmailInProfile($profileId) {
+  static function isEmailInProfile($profileId) {
     $query = "
 SELECT field_name
 FROM civicrm_uf_field
index 6aae416921c3ee08cfa58d9d2b4230373a2a6932..061bb88e292c8d0deb1b99525aa86150d690cbc1 100644 (file)
@@ -34,7 +34,6 @@
  *
  */
 
-require_once 'CRM/Core/Form.php';
 
 /**
  * This class manages the auction form
@@ -43,7 +42,7 @@ require_once 'CRM/Core/Form.php';
 class CRM_Auction_Form_Auction extends CRM_Core_Form {
 
   /**
-   * the id of the auction we are proceessing
+   * the id of the auction we are processsing
    *
    * @var int
    */
@@ -59,7 +58,6 @@ class CRM_Auction_Form_Auction extends CRM_Core_Form {
    * set variables up before form is built
    *
    * @return void
-   * @access public
    */
   function preProcess() {
     $this->_action = CRM_Utils_Request::retrieve('action', 'String', $this, FALSE);
@@ -83,13 +81,9 @@ class CRM_Auction_Form_Auction extends CRM_Core_Form {
   /**
    * Set default values for the form.
    * the default values are retrieved from the database
-   *
-   * @access public
-   *
-   * @return None
+   * @return array
    */
   function setDefaultValues() {
-    require_once 'CRM/Auction/BAO/Auction.php';
 
     $defaults = array();
 
@@ -107,8 +101,7 @@ class CRM_Auction_Form_Auction extends CRM_Core_Form {
   /**
    * Build the form object
    *
-   * @return None
-   * @access public
+   * @return array
    */
   public function buildQuickForm() {
     $this->applyFilter('__ALL__', 'trim');
@@ -211,14 +204,9 @@ class CRM_Auction_Form_Auction extends CRM_Core_Form {
    * @param array $fields the input form values
    * @param array $files the uploaded files if any
    * @param $self
-   *
-   *
    * @return true if no errors, else array of errors
-   * @access public
-   * @static
    */
-  static
-  function formRule($fields, $files, $self) {
+  static function formRule($fields, $files, $self) {
     $errors = array();
 
     // add rules to validate dates and overlap
@@ -227,10 +215,6 @@ class CRM_Auction_Form_Auction extends CRM_Core_Form {
 
   /**
    * Process the form submission
-   *
-   * @access public
-   *
-   * @return None
    */
   public function postProcess() {
     if ($this->_action & CRM_Core_Action::VIEW) {
index 202eeaaf402a91b1f12f1ac53e1402a0f6f613c6..50fc0abf4d8c34f14c56c57361ba3ef8af852b12 100644 (file)
@@ -67,7 +67,6 @@ class CRM_Auction_Form_Item extends CRM_Core_Form {
    * set variables up before form is built
    *
    * @return void
-   * @access public
    */
   function preProcess() {
     $this->_action = CRM_Utils_Request::retrieve('action', 'String', $this, FALSE, 'add');
@@ -111,14 +110,9 @@ class CRM_Auction_Form_Item extends CRM_Core_Form {
   /**
    * Set default values for the form.
    * the default values are retrieved from the database
-   *
-   * @access public
-   *
-   * @return None
+   * @return array
    */
   function setDefaultValues() {
-    require_once 'CRM/Auction/BAO/Item.php';
-
     $defaults = array();
 
     if (isset($this->_id)) {
@@ -135,9 +129,6 @@ class CRM_Auction_Form_Item extends CRM_Core_Form {
 
   /**
    * Build the form object
-   *
-   * @return None
-   * @access public
    */
   public function buildQuickForm() {
     $this->applyFilter('__ALL__', 'trim');
@@ -264,14 +255,9 @@ class CRM_Auction_Form_Item extends CRM_Core_Form {
    * @param array $fields the input form values
    * @param array $files the uploaded files if any
    * @param $self
-   *
-   *
    * @return true if no errors, else array of errors
-   * @access public
-   * @static
    */
-  static
-  function formRule($fields, $files, $self) {
+  static function formRule($fields, $files, $self) {
     $errors = array();
 
     if (isset($files['attachFile_1'])) {
@@ -286,10 +272,6 @@ class CRM_Auction_Form_Item extends CRM_Core_Form {
 
   /**
    * Process the form submission
-   *
-   * @access public
-   *
-   * @return None
    */
   public function postProcess() {
     if ($this->_action & CRM_Core_Action::VIEW) {
index 55635a5e5ca985b3e449e9891119cae1a2712c8c..c7d4462622e57d850bee7d23bc27bf891100730c 100644 (file)
@@ -75,11 +75,7 @@ class CRM_Auction_Form_ItemAccount extends CRM_Core_Form {
   /**
    * This virtual function is used to set the default values of
    * various form elements
-   *
-   * access        public
-   *
    * @return array reference to the array of default values
-   *
    */
   function setDefaultValues() {
     if (!$this->_donorID) {
@@ -110,9 +106,6 @@ class CRM_Auction_Form_ItemAccount extends CRM_Core_Form {
 
   /**
    * Build the form object
-   *
-   * @return None
-   * @access public
    */
   public function buildQuickForm() {
     $profileID = CRM_Core_DAO::getFieldValue('CRM_Auction_DAO_Auction',
@@ -183,15 +176,10 @@ class CRM_Auction_Form_ItemAccount extends CRM_Core_Form {
    * @param array $files the uploaded files if any
    * @param $self
    *
-   *
-   * @return true if no errors, else array of errors
-   * @access public
-   * @static
+   * @return bool|array
    */
-  static
-  function formRule($fields, $files, $self) {
+  static function formRule($fields, $files, $self) {
     $errors = array();
-    require_once "CRM/Utils/Rule.php";
     foreach ($fields as $key => $value) {
       if (strpos($key, 'email-') !== FALSE) {
         $ufContactId = CRM_Core_DAO::getFieldValue('CRM_Core_DAO_UFMatch', $value, 'contact_id', 'uf_name');
@@ -205,10 +193,6 @@ class CRM_Auction_Form_ItemAccount extends CRM_Core_Form {
 
   /**
    * Process the form submission
-   *
-   * @access public
-   *
-   * @return None
    */
   public function postProcess() {
     $params = $this->controller->exportValues($this->getName());
index b0f4021db525f475b93190ff46dd604f3594bd1e..70eb0c502efe286371b820e9cc350e06aaa98034 100644 (file)
@@ -65,9 +65,6 @@ class CRM_Auction_Form_SearchAuction extends CRM_Core_Form {
 
   /**
    * Build the form object
-   *
-   * @access public
-   *
    * @return void
    */
   public function buildQuickForm() {
index eba5409c0337fda6135b41195d9931bd12c94c01..4800e4d086cce3eb3dbb753b2248bfba0301949e 100644 (file)
@@ -44,10 +44,7 @@ class CRM_Auction_Form_SearchItem extends CRM_Core_Form {
    * This virtual function is used to set the default values of
    * various form elements
    *
-   * access        public
-   *
    * @return array reference to the array of default values
-   *
    */
   function setDefaultValues() {
     $defaults = array();
@@ -58,9 +55,6 @@ class CRM_Auction_Form_SearchItem extends CRM_Core_Form {
 
   /**
    * Build the form object
-   *
-   * @access public
-   *
    * @return void
    */
   public function buildQuickForm() {
@@ -82,6 +76,9 @@ class CRM_Auction_Form_SearchItem extends CRM_Core_Form {
       ));
   }
 
+  /**
+   *
+   */
   function postProcess() {
     $params = $this->controller->exportValues($this->_name);
     $parent = $this->controller->getParent();
index ed317f0f7955ac2eeb7ebc3cf52cd4014edbc8a8..4a5612e0e604574fa13fa3143737a255de5adc14 100644 (file)
@@ -40,18 +40,14 @@ require_once 'CRM/Core/Component/Info.php';
  */
 class CRM_Auction_Info extends CRM_Core_Component_Info {
 
-  // docs inherited from interface
   protected $keyword = 'auction';
 
-  // docs inherited from interface
   /**
    * Provides base information about the component.
    * Needs to be implemented in component's information
    * class.
    *
    * @return array collection of required component settings
-   * @access public
-   *
    */
   public function getInfo() {
     return array('name' => 'CiviAuction',
@@ -76,15 +72,12 @@ class CRM_Auction_Info extends CRM_Core_Component_Info {
     );
   }
 
-  // docs inherited from interface
   /**
    * Provides information about user dashboard element
    * offered by this component.
    *
-   * @return array|null collection of required dashboard settings,
+   * @return array collection of required dashboard settings,
    *                    null if no element offered
-   * @access public
-   *
    */
   public function getUserDashboardElement() {
     return array('name' => ts('Auctions'),
@@ -94,15 +87,12 @@ class CRM_Auction_Info extends CRM_Core_Component_Info {
     );
   }
 
-  // docs inherited from interface
   /**
    * Provides information about user dashboard element
    * offered by this component.
    *
    * @return array|null collection of required dashboard settings,
    *                    null if no element offered
-   * @access public
-   *
    */
   public function registerTab() {
     return array('title' => ts('Auctions'),
@@ -112,15 +102,12 @@ class CRM_Auction_Info extends CRM_Core_Component_Info {
     );
   }
 
-  // docs inherited from interface
   /**
    * Provides information about advanced search pane
    * offered by this component.
    *
-   * @return array|null collection of required pane settings,
+   * @return array collection of required pane settings,
    *                    null if no element offered
-   * @access public
-   *
    */
   public function registerAdvancedSearchPane() {
     return array('title' => ts('Auctions'),
@@ -128,7 +115,6 @@ class CRM_Auction_Info extends CRM_Core_Component_Info {
     );
   }
 
-  // docs inherited from interface
   /**
    * Provides potential activity types that this
    * component might want to register in activity history.
@@ -136,15 +122,12 @@ class CRM_Auction_Info extends CRM_Core_Component_Info {
    * class.
    *
    * @return array|null collection of activity types
-   * @access public
-   *
    */
   public function getActivityTypes() {
     $types = array();
     return $types;
   }
 
-  // add shortcut to Create New
   /**
    * @param $shortCuts
    */
index 2c19ace71d3c276dad85e248f5aeb40fa9683ccd..51ea348f1adbfd18b04e25b7590ca23113e0570f 100644 (file)
@@ -33,8 +33,6 @@
  *
  */
 
-require_once 'CRM/Core/Page.php';
-
 /**
  * Page to decide the flow of adding an item.
  */
@@ -48,8 +46,6 @@ class CRM_Auction_Page_AddItem extends CRM_Core_Page {
    * Finally it calls the parent's run method.
    *
    * @return void
-   * @access public
-   *
    */
   function run() {
     // get the requested action
@@ -91,7 +87,6 @@ class CRM_Auction_Page_AddItem extends CRM_Core_Page {
       }
     }
 
-    // parent run
     parent::run();
   }
 }
index 7a320f8d5f9f6623dc8fd675e57e94e9915af620..7fa1da78a8000f7b5d9e502c78645d5736876247 100644 (file)
@@ -45,7 +45,6 @@ class CRM_Auction_Page_DashBoard extends CRM_Core_Page {
    * it decides the which action has to be taken for the page.
    *
    * return null
-   * @access public
    */
   function run() {
     parent::run();
index eda19ca9f33ab246dbfd380a622f98b17e58d116..d37932376f0550ce46b722495727ca6d8afa4c9d 100644 (file)
@@ -59,8 +59,6 @@ class CRM_Auction_Page_Item extends CRM_Core_Page {
    * Finally it calls the parent's run method.
    *
    * @return void
-   * @access public
-   *
    */
   function run() {
     // get the requested action
@@ -118,10 +116,7 @@ class CRM_Auction_Page_Item extends CRM_Core_Page {
   /**
    * Browse all auctions
    *
-   *
    * @return void
-   * @access public
-   * @static
    */
   function browse() {
     if ($this->_aid) {
index d5a89c4fdaf5f21982a8034c9e21d82a885bd826..cadf0cfce0ed7badf5fbdcf04c4508f99139393e 100644 (file)
@@ -33,8 +33,6 @@
  *
  */
 
-require_once 'CRM/Core/Page.php';
-
 /**
  * Page for displaying list of auctions
  */
@@ -44,7 +42,6 @@ class CRM_Auction_Page_Manage extends CRM_Core_Page {
    * The action links that we need to display for the browse screen
    *
    * @var array
-   * @static
    */
   static $_actionLinks = NULL;
 
@@ -119,8 +116,6 @@ class CRM_Auction_Page_Manage extends CRM_Core_Page {
    * Finally it calls the parent's run method.
    *
    * @return void
-   * @access public
-   *
    */
   function run() {
     // get the requested action
@@ -179,10 +174,7 @@ class CRM_Auction_Page_Manage extends CRM_Core_Page {
   /**
    * Browse all auctions
    *
-   *
    * @return void
-   * @access public
-   * @static
    */
   function browse() {
 
@@ -251,6 +243,9 @@ ORDER BY start_date desc
     $this->assign('rows', $auctions);
   }
 
+  /**
+   *
+   */
   function search() {
     $form = new CRM_Core_Controller_Simple('CRM_Auction_Form_SearchAuction',
       ts('Search Items'), CRM_Core_Action::ADD
@@ -353,4 +348,3 @@ SELECT count(id)
     $this->assign('aToZ', $aToZBar);
   }
 }
-
index 6da9723ce954952bed15bb86bd9dd9e4c2280e9e..b326292a170cd099e28192349ee9d61708536e2a 100644 (file)
  * @package CRM
  * @copyright CiviCRM LLC (c) 2004-2014
  * $Id$
- *
  */
 
-require_once 'CRM/Core/Page.php';
 
 /**
  * Page for displaying list of auctions
@@ -51,7 +49,6 @@ class CRM_Auction_Page_ManageItem extends CRM_Core_Page {
    * The action links that we need to display for the browse screen
    *
    * @var array
-   * @static
    */
   static $_actionLinks = NULL;
 
@@ -113,8 +110,6 @@ class CRM_Auction_Page_ManageItem extends CRM_Core_Page {
    * Finally it calls the parent's run method.
    *
    * @return void
-   * @access public
-   *
    */
   function run() {
     // get the requested action
@@ -175,10 +170,7 @@ class CRM_Auction_Page_ManageItem extends CRM_Core_Page {
   /**
    * Browse all auctions
    *
-   *
    * @return void
-   * @access public
-   * @static
    */
   function browse() {
     $this->assign('newItemURL', CRM_Utils_System::url('civicrm/auction/item/add',
@@ -274,7 +266,6 @@ class CRM_Auction_Page_ManageItem extends CRM_Core_Page {
    * all the fields in the event wizard
    *
    * @return void
-   * @access public
    */
   function copy() {
     $id = CRM_Utils_Request::retrieve('id', 'Positive', $this, TRUE, 0, 'GET');
index cabab603e3cad6205d6ad6fd6b6e677b738af05b..188d446793d58f0fa6145197d0befc99c21a44e4 100644 (file)
@@ -37,22 +37,17 @@ require_once 'CRM/Core/Component/Info.php';
  * @package CRM
  * @copyright CiviCRM LLC (c) 2004-2014
  * $Id$
- *
  */
 class CRM_Touchstone_Info extends CRM_Core_Component_Info {
 
-  // docs inherited from interface
   protected $keyword = 'touchstone';
 
-  // docs inherited from interface
   /**
    * Provides base information about the component.
    * Needs to be implemented in component's information
    * class.
    *
    * @return array collection of required component settings
-   * @access public
-   *
    */
   public function getInfo() {
     return array('name' => 'CiviTouchstone',
@@ -62,7 +57,6 @@ class CRM_Touchstone_Info extends CRM_Core_Component_Info {
     );
   }
 
-  // docs inherited from interface
   /**
    * @return array
    */
@@ -70,15 +64,12 @@ class CRM_Touchstone_Info extends CRM_Core_Component_Info {
     return array('access CiviTouchstone');
   }
 
-  // docs inherited from interface
   /**
    * Provides information about user dashboard element
    * offered by this component.
    *
    * @return array|null collection of required dashboard settings,
    *                    null if no element offered
-   * @access public
-   *
    */
   public function getUserDashboardElement() {
     return array('name' => ts('Touchstone'),
@@ -88,15 +79,12 @@ class CRM_Touchstone_Info extends CRM_Core_Component_Info {
     );
   }
 
-  // docs inherited from interface
   /**
    * Provides information about user dashboard element
    * offered by this component.
    *
    * @return array|null collection of required dashboard settings,
    *                    null if no element offered
-   * @access public
-   *
    */
   public function registerTab() {
     return array('title' => ts('Touchstone'),
@@ -105,15 +93,12 @@ class CRM_Touchstone_Info extends CRM_Core_Component_Info {
     );
   }
 
-  // docs inherited from interface
   /**
    * Provides information about advanced search pane
    * offered by this component.
    *
    * @return array|null collection of required pane settings,
    *                    null if no element offered
-   * @access public
-   *
    */
   public function registerAdvancedSearchPane() {
     return array('title' => ts('Touchstone'),
@@ -121,7 +106,6 @@ class CRM_Touchstone_Info extends CRM_Core_Component_Info {
     );
   }
 
-  // docs inherited from interface
   /**
    * Provides potential activity types that this
    * component might want to register in activity history.
@@ -129,14 +113,11 @@ class CRM_Touchstone_Info extends CRM_Core_Component_Info {
    * class.
    *
    * @return array|null collection of activity types
-   * @access public
-   *
    */
   public function getActivityTypes() {
     return NULL;
   }
 
-  // add shortcut to Create New
   /**
    * @param $shortCuts
    */
index 02c590a5626ef6f1d1a92bf9654deeeac8819166..c2286313c3991f462214f3be74f0f5fbf3d6970d 100644 (file)
@@ -41,15 +41,12 @@ class CRM_Touchstone_PseudoConstant extends CRM_Core_PseudoConstant {
   /**
    * pseudoconstant definition
    * @var array
-   * @static
    */
   private static $pseudoConst;
 
   /**
    * Get all values for $pseudoConst pseudoconstant
    *
-   * @access public
-   *
    * @param null $id
    *
    * @return array - array reference of all $pseudoConst values
index 1e63e1ca9e84524c0c1582c1d888f9000baa71e8..bc6917109240844e2bc9635ca703257d72215fff 100644 (file)
@@ -45,7 +45,6 @@ class CRM_Touchstone_Task {
    * the task array
    *
    * @var array
-   * @static
    */
   static $_tasks = NULL;
 
@@ -53,7 +52,6 @@ class CRM_Touchstone_Task {
    * the optional task array
    *
    * @var array
-   * @static
    */
   static $_optionalTasks = NULL;
 
@@ -62,11 +60,8 @@ class CRM_Touchstone_Task {
    * on a contact / group of contacts
    *
    * @return array the set of tasks for a group of contacts
-   * @static
-   * @access public
    */
-  static
-  function &tasks() {
+  static function &tasks() {
     if (!(self::$_tasks)) {
       self::$_tasks = array(
         1 => ts('Perform First Task'),
index fc9b70332b211039002b03cae7d740edf2193cf1..a274bad6d1615456e1024cd9c0d392274c4ed625 100644 (file)
@@ -47,7 +47,6 @@ class org_civicrm_payment_googlecheckout extends CRM_Core_Payment {
    * pattern and cache the instance in this variable
    *
    * @var object
-   * @static
    */
   static private $_singleton = NULL;
 
@@ -55,7 +54,6 @@ class org_civicrm_payment_googlecheckout extends CRM_Core_Payment {
    * mode of operation: live or test
    *
    * @var object
-   * @static
    */
   static protected $_mode = NULL;
 
@@ -81,7 +79,6 @@ class org_civicrm_payment_googlecheckout extends CRM_Core_Payment {
    *
    * @param object $paymentProcessor
    * @return object
-   * @static
    */
   static
   function &singleton($mode, &$paymentProcessor) {
@@ -139,7 +136,6 @@ class org_civicrm_payment_googlecheckout extends CRM_Core_Payment {
    * @param $component
    * @throws Exception
    * @return void
-   * @access public
    */
   function doTransferCheckout(&$params, $component) {
     $component = strtolower($component);
index a2f1ccc80bb92ffccd7d0f8e3ead3b296c6d2369..42b7ed5719fc4e690816225b58940e30b7776028 100644 (file)
@@ -48,7 +48,6 @@ class org_civicrm_payment_googlecheckout_GoogleIPN extends CRM_Core_Payment_Base
    * pattern and cache the instance in this variable
    *
    * @var object
-   * @static
    */
   static private $_singleton = NULL;
 
@@ -56,7 +55,6 @@ class org_civicrm_payment_googlecheckout_GoogleIPN extends CRM_Core_Payment_Base
    * mode of operation: live or test
    *
    * @var object
-   * @static
    */
   static protected $_mode = NULL;
 
@@ -275,10 +273,8 @@ class org_civicrm_payment_googlecheckout_GoogleIPN extends CRM_Core_Payment_Base
    * @param $component
    * @param $paymentProcessor
    * @return object
-   * @static
    */
-  static
-  function &singleton($mode, $component, &$paymentProcessor) {
+  static function &singleton($mode, $component, &$paymentProcessor) {
     if (self::$_singleton === NULL) {
       self::$_singleton = new org_civicrm_payment_googlecheckout_GoogleIPN($mode, $paymentProcessor);
     }
@@ -291,7 +287,6 @@ class org_civicrm_payment_googlecheckout_GoogleIPN extends CRM_Core_Payment_Base
    * @param int $orderNo <order-total> send by google
    *
    * @return amount
-   * @access public
    */
   function getAmount($orderNo) {
     require_once 'CRM/Contribute/DAO/Contribution.php';
@@ -314,12 +309,8 @@ class org_civicrm_payment_googlecheckout_GoogleIPN extends CRM_Core_Payment_Base
    * @param string  $root           root of xml-response
    *
    * @return array context of this call (test, module, payment processor id)
-   * @static
    */
-  static
-  function getContext($xml_response, $privateData, $orderNo, $root) {
-    require_once 'CRM/Contribute/DAO/Contribution.php';
-
+  static function getContext($xml_response, $privateData, $orderNo, $root) {
     $isTest = NULL;
     $module = NULL;
     if ($root == 'new-order-notification') {