a few comment fixes
authorEileen McNaughton <eileen@fuzion.co.nz>
Mon, 2 Feb 2015 02:15:48 +0000 (15:15 +1300)
committerEileen McNaughton <eileen@fuzion.co.nz>
Mon, 2 Feb 2015 02:45:31 +0000 (15:45 +1300)
25 files changed:
CRM/Contact/Form/Search/Interface.php
CRM/Contribute/BAO/Contribution.php
CRM/Contribute/Form/Contribution.php
CRM/Contribute/StateMachine/Search.php
CRM/Core/BAO/FinancialTrxn.php
CRM/Core/BAO/PrevNextCache.php
CRM/Core/ClassLoader.php
CRM/Core/CodeGen/Util/Smarty.php
CRM/Core/Component.php
CRM/Core/DAO.php
CRM/Core/Form.php
CRM/Core/Form/RecurringEntity.php
CRM/Core/I18n/NativeGettext.php
CRM/Core/InnoDBIndexer.php
CRM/Core/Menu.php
CRM/Core/Payment/AuthorizeNetIPN.php
CRM/Core/Payment/PayPalIPN.php
CRM/Core/Payment/PayPalProIPN.php
CRM/Core/Payment/PaymentExpress.php
CRM/Core/QuickForm/GroupMultiSelect.php
CRM/Custom/Import/Controller.php
CRM/Custom/Import/Parser/Api.php
CRM/Event/Cart/BAO/EventInCart.php
CRM/Event/Controller/Registration.php
CRM/Event/Form/ManageEvent/Repeat.php

index f99e40190b1f1d46ad23140adcc6426db9d6cf9b..08f25e5f2d7770cbc54d941f4765bad5459591f1 100644 (file)
 interface CRM_Contact_Form_Search_Interface {
 
   /**
-   * The constructor gets the submitted form values
+   * The constructor gets the submitted form values.
    *
-   * @param $formValues
+   * @param array $formValues
    */
   public function __construct(&$formValues);
 
   /**
-   * Builds the quickform for this search
+   * Builds the quickform for this search.
+   *
+   * @param CRM_Core_Form $form
+   * @return
    */
   public function buildForm(&$form);
 
@@ -54,27 +57,45 @@ interface CRM_Contact_Form_Search_Interface {
    */
 
   /**
-   * Count of records that match the current input parameters
-   * Used by pager
+   * Count of records that match the current input parameters.
+   *
+   * Used by pager.
    */
   public function count();
 
   /**
-   * Summary information for the query that can be displayed in the template
+   * Summary information for the query that can be displayed in the template.
+   *
    * This is useful to pass total / sub total information if needed
    */
   public function summary();
 
   /**
-   * List of contact ids that match the current input parameters
+   * List of contact ids that match the current input parameters.
+   *
    * Used by different tasks. Will be also used to optimize the
    * 'all' query below to avoid excessive LEFT JOIN blowup
+   *
+   * @param int $offset
+   * @param int $rowcount
+   * @param null $sort
+   *
+   * @return
    */
   public function contactIDs($offset = 0, $rowcount = 0, $sort = NULL);
 
   /**
-   * Retrieve all the values that match the current input parameters
+   * Retrieve all the values that match the current input parameters.
+   *
    * Used by the selector
+   *
+   * @param int $offset
+   * @param int $rowcount
+   * @param null $sort
+   * @param bool $includeContactIDs
+   * @param bool $justIDs
+   *
+   * @return
    */
   public function all($offset = 0, $rowcount = 0, $sort = NULL, $includeContactIDs = FALSE, $justIDs = FALSE);
 
@@ -88,22 +109,26 @@ interface CRM_Contact_Form_Search_Interface {
    */
 
   /**
-   * The from clause for the query
+   * The from clause for the query.
    */
   public function from();
 
   /**
-   * The where clause for the query
+   * The where clause for the query.
+   *
+   * @param bool $includeContactIDs
+   *
+   * @return
    */
   public function where($includeContactIDs = FALSE);
 
   /**
-   * The template FileName to use to display the results
+   * The template FileName to use to display the results.
    */
   public function templateFile();
 
   /**
-   * Returns an array of column headers and field names and sort options
+   * Returns an array of column headers and field names and sort options.
    */
   public function &columns();
 
index a936639b5e8048fe660e9b4f63056bd2bc20f9de..97d53a1dbeba1181620fa8210178e569fae5757f 100644 (file)
@@ -1739,6 +1739,10 @@ LEFT JOIN  civicrm_contribution contribution ON ( componentPayment.contribution_
 
   /**
    * Returns all contribution related object ids.
+   *
+   * @param $contributionId
+   *
+   * @return array
    */
   public static function getComponentDetails($contributionId) {
     $componentDetails = $pledgePayment = array();
@@ -3561,10 +3565,18 @@ WHERE con.id = {$contributionId}
     return CRM_Utils_Array::value('financial_account_id', $result);
   }
 
+  /**
+   * Check tax amount.
+   *
+   * @param array $params
+   * @param bool $isLineItem
+   *
+   * @return mixed
+   */
   public static function checkTaxAmount($params, $isLineItem = FALSE) {
     $taxRates = CRM_Core_PseudoConstant::getTaxRates();
 
-    // Update contribution
+    // Update contribution.
     if (!empty($params['id'])) {
       $id = $params['id'];
       $values = $ids = array();
index 67300ced8c4dffdcb7a5c1259fd6ade01eb50331..e19f9068b0d1a3c3be2df68c38918476d5a91dd1 100644 (file)
@@ -1812,7 +1812,7 @@ class CRM_Contribute_Form_Contribution extends CRM_Contribute_Form_AbstractEditP
 
   /**
    * @param array $paymentParams
-   *
+   * @param $message
    */
   public function cleanupDBAfterPaymentFailure($paymentParams, $message) {
     //make sure to cleanup db for recurring case.
index da6a677ec9cbf6be0cbf7632d41d7844b9585554..1c0dc22631664aacee32ba61b0cff0cc1f2fd98d 100644 (file)
@@ -43,6 +43,9 @@ class CRM_Contribute_StateMachine_Search extends CRM_Core_StateMachine {
 
   /**
    * Class constructor
+   *
+   * @param object $controller
+   * @param \const|int $action
    */
   public function __construct($controller, $action = CRM_Core_Action::NONE) {
     parent::__construct($controller, $action);
index 869a633c5d1abef50233a969be34853660ac4990..0d98f34580dab28d4fcc7f5f000485bd3f37dd7e 100644 (file)
@@ -360,6 +360,7 @@ WHERE ceft.entity_id = %1";
    * @param array $params
    *   To create trxn entries.
    *
+   * @return bool
    */
   public static function recordFees($params) {
     $expenseTypeId = key(CRM_Core_PseudoConstant::accountOptionValues('account_relationship', NULL, " AND v.name LIKE 'Expense Account is' "));
index cb5bcef2e9a94abdf50ad34052e4e24beab22b02..9a2b23122a6e31972911df82113623b3c7db0ea0 100644 (file)
@@ -249,6 +249,8 @@ WHERE cacheKey $op %1
    * @param int $rgid
    * @param int $gid
    * @param NULL $cacheKeyString
+   *
+   * @return bool
    */
   public static function refillCache($rgid = NULL, $gid = NULL, $cacheKeyString = NULL) {
     if (!$cacheKeyString && $rgid) {
index e92bf888dc90ea5ee9261c69a4834c6ea865bb32..b68cc3d6b911cb3932c64623ecaec690fe05b8f0 100644 (file)
@@ -101,6 +101,11 @@ class CRM_Core_ClassLoader {
     require_once "$civicrm_base_path/vendor/autoload.php";
   }
 
+  /**
+   * Initialize HTML purifier class.
+   *
+   * @param string $prepend
+   */
   public function initHtmlPurifier($prepend) {
     if (class_exists('HTMLPurifier_Bootstrap')) {
       // HTMLPurifier is already initialized, e.g. by the Drupal module.
index eff08a87824888101a76fced261799a79ca3a37f..8ab0b2adcb0c2388a8b370a38477164103c368c2 100644 (file)
@@ -44,6 +44,11 @@ class CRM_Core_CodeGen_Util_Smarty {
     $this->smarty = NULL;
   }
 
+  /**
+   * Get templates_c directory.
+   *
+   * @return string
+   */
   public function getCompileDir() {
     if ($this->compileDir === NULL) {
       $this->compileDir = CRM_Core_CodeGen_Util_File::createTempDir('templates_c_');
@@ -51,6 +56,11 @@ class CRM_Core_CodeGen_Util_Smarty {
     return $this->compileDir;
   }
 
+  /**
+   * Get smarty instance.
+   *
+   * @return \Smarty
+   */
   public function getSmarty() {
     if ($this->smarty === NULL) {
       require_once 'Smarty/Smarty.class.php';
index 85c69308675466cdbdfeec609a8ae5ea497de122..0cc72ce62ccda3394ae3f8a727f6ef6affa39244 100644 (file)
@@ -222,6 +222,8 @@ class CRM_Core_Component {
   /**
    * @param $config
    * @param bool $oldMode
+   *
+   * @return null
    */
   public static function addConfig(&$config, $oldMode = FALSE) {
     $info = self::_info();
index 32fb178dd98f1ccfbd981c2c60a6986f8d808045..5a96a775d1443f6eb8128d9936d3f0f909ad7605 100644 (file)
@@ -1408,6 +1408,16 @@ FROM   civicrm_domain
     return $newObject;
   }
 
+  /**
+   * Cascade update through related entities.
+   *
+   * @param string $daoName
+   * @param $fromId
+   * @param $toId
+   * @param array $newData
+   *
+   * @return null
+   */
   public static function cascadeUpdate($daoName, $fromId, $toId, $newData = array()) {
     $object = new $daoName();
     $object->id = $fromId;
index cbe49b178cdf0f13c7edcd6812cdf3be4742daea..1ecbbede88734d9c028a39123c08580e2f606b53 100644 (file)
@@ -1696,6 +1696,8 @@ class CRM_Core_Form extends HTML_QuickForm_Page {
    * that small pieces of duplication are not being refactored into separate functions because their only shared parent
    * is this form. Inserting a class FrontEndForm.php between the contribution & event & this class would allow functions like this
    * and a dozen other small ones to be refactored into a shared parent with the reduction of much code duplication
+   *
+   * @param $onlinePaymentProcessorEnabled
    */
   public function addCIDZeroOptions($onlinePaymentProcessorEnabled) {
     $this->assign('nocid', TRUE);
index 40f96a5b46babb65792d42ffc9205f4d9ce3f7e3..8dbed40139449f3e3b493784280bd22cea1df304 100644 (file)
@@ -74,6 +74,9 @@ class CRM_Core_Form_RecurringEntity {
    */
   public static $_hasParent = FALSE;
 
+  /**
+   * @param $entityTable
+   */
   public static function preProcess($entityTable) {
     self::$_entityId = (int) CRM_Utils_Request::retrieve('id', 'Positive');
     self::$_entityTable = $entityTable;
index 0f721512ae0f3e5dd69f879aa14169ef40139977..996d1192b6f380e3e9848fb5420e2ab2b36e919b 100644 (file)
@@ -45,6 +45,11 @@ class CRM_Core_I18n_NativeGettext {
 
   /**
    * Based on php-gettext, since native gettext does not support this as is.
+   *
+   * @param $context
+   * @param $text
+   *
+   * @return string
    */
   public function pgettext($context, $text) {
     $key = $context . chr(4) . $text;
index 5b0dafa38178a47eccfcd540a65395c7de988914..7751aa5632b14ee641896c66bf604ca04d60527c 100644 (file)
@@ -108,15 +108,26 @@ class CRM_Core_InnoDBIndexer {
    */
   protected $isActive;
 
+  /**
+   * Class constructor.
+   *
+   * @param $isActive
+   * @param $indices
+   */
   public function __construct($isActive, $indices) {
     $this->isActive = $isActive;
     $this->indices = $this->normalizeIndices($indices);
   }
 
+  /**
+   * Fix schema differences.
+   *
+   * Limitation: This won't pick up stale indices on tables which are not
+   * declared in $this->indices. That's not much of an issue for now b/c
+   * we have a static list of tables.
+   */
   public function fixSchemaDifferences() {
-    // Limitation: This won't pick up stale indices on tables which are not
-    // declared in $this->indices. That's not much of an issue for now b/c
-    // we have a static list of tables.
+    //
     foreach ($this->indices as $tableName => $ign) {
       $todoSqls = $this->reconcileIndexSqls($tableName);
       foreach ($todoSqls as $todoSql) {
index 59359e353fbe18e2edffe8ac8044f68bf6acd3f6..8338850db70ab1e638f1399d4542423c0bb7ad73 100644 (file)
@@ -164,6 +164,7 @@ class CRM_Core_Menu {
    * @param boolen $fetchFromXML
    *   Fetch the menu items from xml and not from cache.
    *
+   * @return array
    */
   public static function &items($fetchFromXML = FALSE) {
     return self::xmlItems($fetchFromXML);
index b794d2f66ed2c4b0114498fcd734369f4b413ce6..011fb7f408cffa921404b89a9726214b4b532675 100644 (file)
@@ -206,6 +206,8 @@ class CRM_Core_Payment_AuthorizeNetIPN extends CRM_Core_Payment_BaseIPN {
   /**
    * @param $input
    * @param $ids
+   *
+   * @return bool
    */
   public function getInput(&$input, &$ids) {
     $input['amount'] = $this->retrieve('x_amount', 'String');
index d8c26ec843cca460ac5be599d976234ee933edb0..0d2996af739697e6180bbd82b761089038026e07 100644 (file)
@@ -69,6 +69,8 @@ class CRM_Core_Payment_PayPalIPN extends CRM_Core_Payment_BaseIPN {
    * @param $ids
    * @param $objects
    * @param $first
+   *
+   * @return bool
    */
   public function recur(&$input, &$ids, &$objects, $first) {
     if (!isset($input['txnType'])) {
@@ -225,6 +227,8 @@ class CRM_Core_Payment_PayPalIPN extends CRM_Core_Payment_BaseIPN {
    * @param $objects
    * @param bool $recur
    * @param bool $first
+   *
+   * @return bool
    */
   public function single(
     &$input, &$ids, &$objects,
@@ -342,6 +346,8 @@ class CRM_Core_Payment_PayPalIPN extends CRM_Core_Payment_BaseIPN {
   /**
    * @param $input
    * @param $ids
+   *
+   * @return bool
    */
   public function getInput(&$input, &$ids) {
     if (!$this->getBillingID($ids)) {
index 2e433b95f0c37afb2391a881d9d08441b5eaaf6d..e4f05d177376a2481840ed5aa9611c3485e1c3c0 100644 (file)
@@ -327,6 +327,8 @@ class CRM_Core_Payment_PayPalProIPN extends CRM_Core_Payment_BaseIPN {
    * @param $objects
    * @param bool $recur
    * @param bool $first
+   *
+   * @return bool
    */
   public function single(&$input, &$ids, &$objects, $recur = FALSE, $first = FALSE) {
     $contribution = &$objects['contribution'];
index 0aba51b29e4777db47bb73386d5aea5a4ec622b0..ad7244aab7aa72437bfbccc136a46999b4c2dd40 100644 (file)
@@ -128,6 +128,8 @@ class CRM_Core_Payment_PaymentExpress extends CRM_Core_Payment {
    *
    * @param array $params
    *   Assoc array of input parameters for this transaction.
+   *
+   * @return array|void
    */
   public function doDirectPayment(&$params) {
     CRM_Core_Error::fatal(ts('This function is not implemented'));
index 2a4c15edf169db76f3af407d0dff6eb9f7c80fa8..8bc800e59db0a3a42205156864df0bd4defd6051 100644 (file)
@@ -35,7 +35,7 @@
  */
 class CRM_Core_QuickForm_GroupMultiSelect extends CRM_Core_QuickForm_NestedAdvMultiSelect {
   /**
-   * Returns the HTML generated for the advanced mutliple select component
+   * Returns the HTML generated for the advanced multiple select component
    *
    * @return string
    * @since      version 0.4.0 (2005-06-25)
index 5db234e1bc5d7adcb68d150946dafae162ceb69b..a5053f34eff86b3472fc62f7ff621ce96bb9c858 100644 (file)
@@ -5,7 +5,11 @@
  */
 class CRM_Custom_Import_Controller extends CRM_Core_Controller {
   /**
-   * Class constructor
+   * Class constructor.
+   *
+   * @param string $title
+   * @param bool|int $action
+   * @param bool $modal
    */
   public function __construct($title = NULL, $action = CRM_Core_Action::NONE, $modal = TRUE) {
     parent::__construct($title, $modal);
index 71f6e225c5f0642ebdfaee9cbc73064237b700dd..aa439a46e15cd76bf661e42144d3c885b1f29fc8 100644 (file)
@@ -18,7 +18,11 @@ class CRM_Custom_Import_Parser_Api extends CRM_Custom_Import_Parser {
   protected $_params = array();
 
   /**
-   * Class constructor
+   * Class constructor.
+   *
+   * @param array $mapperKeys
+   * @param null $mapperLocType
+   * @param null $mapperPhoneType
    */
   public function __construct(&$mapperKeys, $mapperLocType = NULL, $mapperPhoneType = NULL) {
     parent::__construct();
index 25aae6aad8bf450d17a2491af5d919085ffda452..26d5721b745499918cd5c1a2fe045c55309509dc 100644 (file)
@@ -47,6 +47,8 @@ class CRM_Event_Cart_BAO_EventInCart extends CRM_Event_Cart_DAO_EventInCart impl
 
   /**
    * @param bool $useWhere
+   *
+   * @return mixed|void
    */
   public function delete($useWhere = FALSE) {
     $this->load_associations();
index 085196b0a67e082c97092bfda7aefa1763b89324..46248826383b23dcd4ab807a4fb95344172a5f62 100644 (file)
 class CRM_Event_Controller_Registration extends CRM_Core_Controller {
 
   /**
-   * Class constructor
+   * Class constructor.
+   *
+   * @param null $title
+   * @param bool|int $action
+   * @param bool $modal
    */
   public function __construct($title = NULL, $action = CRM_Core_Action::NONE, $modal = TRUE) {
     parent::__construct($title, $modal);
index f57ebb00e684d5db8b55aae7b3498f439e2cc87d..3e1103f37f795c6cb0352e82e1049849acc1ba2c 100644 (file)
@@ -193,7 +193,10 @@ class CRM_Event_Form_ManageEvent_Repeat extends CRM_Event_Form_ManageEvent {
   /**
    * This function checks if there was any registraion for related event ids,
    * and returns array of ids with no regsitrations
+   *
    * @param string or int or object... $eventID
+   *
+   * @return array
    */
   public static function checkRegistrationForEvents($eventID) {
     $eventIdsWithNoRegistration = array();