comment fixes
authoreileenmcnaugton <eileen@fuzion.co.nz>
Fri, 23 Oct 2015 01:23:01 +0000 (14:23 +1300)
committereileenmcnaugton <eileen@fuzion.co.nz>
Fri, 23 Oct 2015 02:08:06 +0000 (15:08 +1300)
59 files changed:
CRM/Campaign/Selector/Search.php
CRM/Case/XMLRepository.php
CRM/Contact/BAO/SavedSearch.php
CRM/Contact/Form/Search/Custom/ActivitySearch.php
CRM/Contact/Form/Search/Custom/TagContributions.php
CRM/Contact/Import/Parser/Contact.php
CRM/Contact/Page/DedupeMerge.php
CRM/Contact/Selector.php
CRM/Core/BAO/ActionSchedule.php
CRM/Core/BAO/EntityTag.php
CRM/Core/BAO/UFGroup.php
CRM/Core/Config.php
CRM/Core/Form.php
CRM/Core/Form/RecurringEntity.php
CRM/Core/Menu.php
CRM/Core/Payment/AuthorizeNet.php
CRM/Core/Payment/FirstData.php
CRM/Core/Payment/Form.php
CRM/Core/Payment/PayJunction.php
CRM/Core/Payment/PaymentExpressIPN.php
CRM/Core/Permission/Base.php
CRM/Core/State.php
CRM/Dedupe/Merger.php
CRM/Event/BAO/Event.php
CRM/Event/BAO/Participant.php
CRM/Event/Badge.php
CRM/Event/Form/Registration/Register.php
CRM/Event/Import/Parser/Participant.php
CRM/Export/BAO/Export.php
CRM/Financial/BAO/FinancialItem.php
CRM/Grant/Selector/Search.php
CRM/Import/DataSource.php
CRM/Import/DataSource/CSV.php
CRM/Import/DataSource/SQL.php
CRM/Mailing/BAO/Mailing.php
CRM/Mailing/Event/BAO/Queue.php
CRM/PCP/BAO/PCP.php
CRM/Profile/Selector/Listings.php
CRM/Report/Interface.php
CRM/SMS/Provider.php
CRM/Upgrade/Incremental/Base.php
CRM/Upgrade/Incremental/php/FourThree.php
CRM/Upgrade/Snapshot/V4p2/Price/DAO/Field.php
CRM/Utils/Array.php
CRM/Utils/Hook.php
CRM/Utils/Migrate/ExportJSON.php
CRM/Utils/Number.php
CRM/Utils/Pager.php
CRM/Utils/ReCAPTCHA.php
CRM/Utils/Token.php
Civi/ActionSchedule/RecipientBuilder.php
Civi/Core/Resolver.php
Civi/Core/SettingsManager.php
Civi/Core/SettingsMetadata.php
Civi/Token/AbstractTokenSubscriber.php
Civi/Token/TokenRow.php
sql/GenerateData.php
tests/phpunit/CiviTest/CiviUnitTestCase.php
tests/phpunit/api/v3/JobProcessMailingTest.php

index bc9dc5e176f38ad0674fe3fa702a23dd1717c1fe..888ebd96e86fcfb28e305cadd7205c08b2aa6cba 100755 (executable)
@@ -199,7 +199,7 @@ class CRM_Campaign_Selector_Search extends CRM_Core_Selector_Base implements CRM
   /**
    * Returns total number of rows for the query.
    *
-   * @param
+   * @param string $action
    *
    * @return int
    *   Total number of rows
index d8169b7359bb58925161c8043e98d3287dba2b4d..7e3af8c479e2b32f371d2e4c96aac74ef864bd19 100644 (file)
@@ -64,7 +64,10 @@ class CRM_Case_XMLRepository {
   }
 
   /**
-   * @param array <String,SimpleXMLElement> $xml
+   * Class constructor.
+   *
+   * @param array $allCaseTypes
+   * @param array $xml
    */
   public function __construct($allCaseTypes = NULL, $xml = array()) {
     $this->allCaseTypes = $allCaseTypes;
@@ -72,8 +75,12 @@ class CRM_Case_XMLRepository {
   }
 
   /**
+   * Retrieve case.
+   *
    * @param string $caseType
-   * @return SimpleXMLElement|FALSE
+   *
+   * @return FALSE|\SimpleXMLElement
+   * @throws \CRM_Core_Exception
    */
   public function retrieve($caseType) {
     // check if xml definition is defined in db
@@ -106,6 +113,8 @@ class CRM_Case_XMLRepository {
   }
 
   /**
+   * Retrieve file.
+   *
    * @param string $caseType
    * @return SimpleXMLElement|FALSE
    */
@@ -136,6 +145,8 @@ class CRM_Case_XMLRepository {
   }
 
   /**
+   * Find xml file.
+   *
    * @param string $caseType
    * @return null|string
    *   file path
index 2b910db03bcd426a48f1d7fdbc3380c107f87e46..e01b41c7342b468bf331b826c14e9c4b3582bc78 100644 (file)
@@ -83,7 +83,7 @@ class CRM_Contact_BAO_SavedSearch extends CRM_Contact_DAO_SavedSearch {
   }
 
   /**
-   * Given an id, extract the formValues of the saved search
+   * Given an id, extract the formValues of the saved search.
    *
    * @param int $id
    *   The id of the saved search.
@@ -165,6 +165,8 @@ class CRM_Contact_BAO_SavedSearch extends CRM_Contact_DAO_SavedSearch {
   }
 
   /**
+   * Get search parameters.
+   *
    * @param int $id
    *
    * @return array
@@ -210,6 +212,8 @@ class CRM_Contact_BAO_SavedSearch extends CRM_Contact_DAO_SavedSearch {
   }
 
   /**
+   * Contact IDS Sql (whatever that means!).
+   *
    * @param int $id
    *
    * @return string
@@ -234,6 +238,8 @@ WHERE  $where";
   }
 
   /**
+   * Get from where email (whatever that means!).
+   *
    * @param int $id
    *
    * @return array
@@ -266,8 +272,7 @@ LEFT JOIN civicrm_email ON (contact_a.id = civicrm_email.contact_id AND civicrm_
   }
 
   /**
-   * Given a saved search compute the clause and the tables
-   * and store it for future use
+   * Given a saved search compute the clause and the tables and store it for future use.
    */
   public function buildClause() {
     $fv = unserialize($this->form_values);
@@ -291,6 +296,11 @@ LEFT JOIN civicrm_email ON (contact_a.id = civicrm_email.contact_id AND civicrm_
     }
   }
 
+  /**
+   * Save the search.
+   *
+   * @param bool $hook
+   */
   public function save($hook = TRUE) {
     // first build the computed fields
     $this->buildClause();
@@ -321,6 +331,10 @@ LEFT JOIN civicrm_email ON (contact_a.id = civicrm_email.contact_id AND civicrm_
   /**
    * Given a label and a set of normalized POST
    * formValues, create a smart group with that
+   *
+   * @param array $params
+   *
+   * @return \CRM_Contact_DAO_SavedSearch
    */
   public static function create(&$params) {
     $savedSearch = new CRM_Contact_DAO_SavedSearch();
@@ -343,6 +357,13 @@ LEFT JOIN civicrm_email ON (contact_a.id = civicrm_email.contact_id AND civicrm_
     return $savedSearch;
   }
 
+  /**
+   * Assign test value.
+   *
+   * @param string $fieldName
+   * @param array $fieldDef
+   * @param int $counter
+   */
   protected function assignTestValue($fieldName, &$fieldDef, $counter) {
     if ($fieldName == 'form_values') {
       // A dummy value for form_values.
index 8bb4e8106244da63f86693ad437dba543c2d83a0..340c76bb460ae726ba95884b26aa2ec498e07253 100644 (file)
@@ -156,6 +156,14 @@ class CRM_Contact_Form_Search_Custom_ActivitySearch extends CRM_Contact_Form_Sea
 
   /**
    * Construct the search query.
+   *
+   * @param int $offset
+   * @param int $rowcount
+   * @param null $sort
+   * @param bool $includeContactIDs
+   * @param bool $justIDs
+   *
+   * @return string
    */
   public function all(
     $offset = 0, $rowcount = 0, $sort = NULL,
index 193774f065c9dc936e3dcae52a3141ad694d51d8..8263773cdf4cfe040b74416216e79ff21c583bd0 100644 (file)
@@ -94,6 +94,14 @@ class CRM_Contact_Form_Search_Custom_TagContributions extends CRM_Contact_Form_S
 
   /**
    * Construct the search query.
+   *
+   * @param int $offset
+   * @param int $rowcount
+   * @param string $sort
+   * @param bool $includeContactIDs
+   * @param bool $onlyIDs
+   *
+   * @return string
    */
   public function all(
     $offset = 0, $rowcount = 0, $sort = NULL,
index b3f3b8ef59c3769596e89b93839d9076ec5c6dca..37fe638b3911181891f40c928c71965e1a57ee93 100644 (file)
@@ -99,6 +99,19 @@ class CRM_Contact_Import_Parser_Contact extends CRM_Contact_Import_Parser {
 
   /**
    * Class constructor.
+   *
+   * @param array $mapperKeys
+   * @param int $mapperLocType
+   * @param int $mapperPhoneType
+   * @param int $mapperImProvider
+   * @param int $mapperRelated
+   * @param int $mapperRelatedContactType
+   * @param array $mapperRelatedContactDetails
+   * @param int $mapperRelatedContactLocType
+   * @param int $mapperRelatedContactPhoneType
+   * @param int $mapperRelatedContactImProvider
+   * @param int $mapperWebsiteType
+   * @param int $mapperRelatedContactWebsiteType
    */
   public function __construct(
     &$mapperKeys, $mapperLocType = NULL, $mapperPhoneType = NULL, $mapperImProvider = NULL, $mapperRelated = NULL, $mapperRelatedContactType = NULL, $mapperRelatedContactDetails = NULL, $mapperRelatedContactLocType = NULL, $mapperRelatedContactPhoneType = NULL, $mapperRelatedContactImProvider = NULL,
@@ -122,7 +135,7 @@ class CRM_Contact_Import_Parser_Contact extends CRM_Contact_Import_Parser {
   }
 
   /**
-   * The initializer code, called before the processing
+   * The initializer code, called before the processing.
    */
   public function init() {
     $contactFields = CRM_Contact_BAO_Contact::importableFields($this->_contactType);
@@ -1640,6 +1653,15 @@ class CRM_Contact_Import_Parser_Contact extends CRM_Contact_Import_Parser {
 
   /**
    * Method for creating contact.
+   *
+   * @param array $formatted
+   * @param array $contactFields
+   * @param int $onDuplicate
+   * @param int $contactId
+   * @param bool $requiredCheck
+   * @param int $dedupeRuleGroupID
+   *
+   * @return array|bool|\CRM_Contact_BAO_Contact|\CRM_Core_Error|null
    */
   public function createContact(&$formatted, &$contactFields, $onDuplicate, $contactId = NULL, $requiredCheck = TRUE, $dedupeRuleGroupID = NULL) {
     $dupeCheck = FALSE;
index 27c3f5c0d1bee760bc03f09e6e466e6b4546bf5e..eb0213eb127055b534ae0ed63258550678517b35 100644 (file)
@@ -123,6 +123,16 @@ class CRM_Contact_Page_DedupeMerge extends CRM_Core_Page {
 
   /**
    * Carry out batch merges.
+   *
+   * @param \CRM_Queue_TaskContext $ctx
+   * @param int $rgid
+   * @param int $gid
+   * @param string $mode
+   * @param bool $autoFlip
+   * @param int $batchLimit
+   * @param int $isSelected
+   *
+   * @return int
    */
   public static function callBatchMerge(CRM_Queue_TaskContext $ctx, $rgid, $gid = NULL, $mode = 'safe', $autoFlip = TRUE, $batchLimit = 1, $isSelected = 2) {
     $result = CRM_Dedupe_Merger::batchMerge($rgid, $gid, $mode, $autoFlip, $batchLimit, $isSelected);
index 629de9456000cff224d697ceabb162dc47ce3f3a..36d4a94f15309b55826091328d06e17d82d7e797 100644 (file)
@@ -491,7 +491,7 @@ class CRM_Contact_Selector extends CRM_Core_Selector_Base implements CRM_Core_Se
   /**
    * Returns total number of rows for the query.
    *
-   * @param
+   * @param int $action
    *
    * @return int
    *   Total number of rows
index 6d7c3c880dde02794150e52591889f35e02a9d15..88cf6f61a99db6a529e01ac84265b2f247c93abb 100755 (executable)
@@ -662,8 +662,11 @@ FROM civicrm_action_schedule cas
   }
 
   /**
-   * @param $dao
-   * @return string|NULL
+   * Pick SMS phone number.
+   *
+   * @param int $smsToContactId
+   *
+   * @return NULL|string
    */
   protected static function pickSmsPhoneNumber($smsToContactId) {
     $toPhoneNumbers = CRM_Core_BAO_Phone::allPhones($smsToContactId, FALSE, 'Mobile', array(
index 9745cb3778179578f53a1f1d190c0b4a99f3277d..f66afd0fdf0670fa086ad2b44d942971d041a292 100644 (file)
@@ -274,6 +274,11 @@ class CRM_Core_BAO_EntityTag extends CRM_Core_DAO_EntityTag {
 
   /**
    * Get contact tags.
+   *
+   * @param int $contactID
+   * @param bool $count
+   *
+   * @return array
    */
   public static function getContactTags($contactID, $count = FALSE) {
     $contactTags = array();
index 97951229a7065c525094e389e270a5ff3a5df142..016f192bd118567a8944b0866d4e5d61782c3873 100644 (file)
@@ -273,8 +273,11 @@ class CRM_Core_BAO_UFGroup extends CRM_Core_DAO_UFGroup {
    * @param string $orderBy
    * @param null $orderProfiles
    *
+   * @param bool $eventProfile
+   *
    * @return array
-   *   the fields that belong to this ufgroup(s)
+   *   The fields that belong to this ufgroup(s)
+   * @throws \Exception
    */
   public static function getFields(
     $id,
index e4313de4b745d2013e1bb0b7957b442d7cc23e3d..f6aaf031d13e7445ae6fc08e4651f7cc8fe29b57 100644 (file)
@@ -358,6 +358,10 @@ class CRM_Core_Config extends CRM_Core_Config_MagicMerge {
 
   /**
    * Check if running in upgrade mode.
+   *
+   * @param string $path
+   *
+   * @return bool
    */
   public static function isUpgradeMode($path = NULL) {
     if (defined('CIVICRM_UPGRADE_ACTIVE')) {
index 62daf32187e5c0e4aab8f0143a323efa9fb6df0d..d36c6f1ca3a4657d636d499202d586a8f21cd684 100644 (file)
@@ -1550,6 +1550,7 @@ class CRM_Core_Form extends HTML_QuickForm_Page {
    * @param array $entities
    * @param bool $default
    *   //CRM-15427.
+   * @param string $usedFor
    */
   public function addProfileSelector($name, $label, $allowCoreTypes, $allowSubTypes, $entities, $default = FALSE, $usedFor = NULL) {
     // Output widget
@@ -1777,6 +1778,11 @@ class CRM_Core_Form extends HTML_QuickForm_Page {
 
   /**
    *  Function that will add date and time.
+   *
+   * @param string $name
+   * @param string $label
+   * @param bool $required
+   * @param null $attributes
    */
   public function addDateTime($name, $label, $required = FALSE, $attributes = NULL) {
     $addTime = array('addTime' => TRUE);
@@ -1826,6 +1832,12 @@ class CRM_Core_Form extends HTML_QuickForm_Page {
 
   /**
    * Add currency element to the form.
+   *
+   * @param string $name
+   * @param null $label
+   * @param bool $required
+   * @param string $defaultCurrency
+   * @param bool $freezeCurrency
    */
   public function addCurrency(
     $name = 'currency',
index 45a35bb0be3bdc54cb3076c5e41de77fa73bd7af..7ef0ea19a7afa790b0d0f504f128a49963c54a9d 100644 (file)
@@ -326,6 +326,12 @@ class CRM_Core_Form_RecurringEntity {
 
   /**
    * Process the form submission.
+   *
+   * @param array $params
+   * @param string $type
+   * @param array $linkedEntities
+   *
+   * @throws \CiviCRM_API3_Exception
    */
   public static function postProcess($params = array(), $type, $linkedEntities = array()) {
     //Check entity_id not present in params take it from class variable
index 36641585b1c6dd6afd1cd9c063ba4cd7e135e33e..14f61852803316ff7c82c76b4a1acbae80627024 100644 (file)
@@ -69,7 +69,7 @@ class CRM_Core_Menu {
   /**
    * This function fetches the menu items from xml and xmlMenu hooks.
    *
-   * @param boolen $fetchFromXML
+   * @param boolean $fetchFromXML
    *   Fetch the menu items from xml and not from cache.
    *
    * @return array
index 8336d954b6ac7fb71da49b2462cd7ef70febaf60..a112cd5066b9a1dd6a9dd31067f8255b0cc35cd9 100644 (file)
@@ -538,6 +538,12 @@ class CRM_Core_Payment_AuthorizeNet extends CRM_Core_Payment {
    *
    * Function is from Authorize.Net sample code, and used to avoid using
    * PHP5 XML functions
+   *
+   * @param string $haystack
+   * @param string $start
+   * @param string $end
+   *
+   * @return bool|string
    */
   public function _substring_between(&$haystack, $start, $end) {
     if (strpos($haystack, $start) === FALSE || strpos($haystack, $end) === FALSE) {
index f59889a6da45b513347412386b78f8c0437bf95e..b4fea7aa5374e3a89946407615c244cd0da64a27 100644 (file)
@@ -80,10 +80,16 @@ class CRM_Core_Payment_FirstData extends CRM_Core_Payment {
   }
 
   /**
+   * Map fields from params array.
+   *
    * This function is set up and put here to make the mapping of fields
-   * from the params object  as visually clear as possible for easy editing
+   * as visually clear as possible for easy editing
    *
    *  Comment out irrelevant fields
+   *
+   * @param array $params
+   *
+   * @return array
    */
   public function mapProcessorFieldstoParams($params) {
     /*concatenate full customer name first  - code from EWAY gateway
@@ -142,10 +148,15 @@ class CRM_Core_Payment_FirstData extends CRM_Core_Payment {
   /**
    * This function sends request and receives response from
    * the processor
+   *
+   * @param array $params
+   *
+   * @return array|object
+   * @throws \Exception
    */
   public function doDirectPayment(&$params) {
     if ($params['is_recur'] == TRUE) {
-      CRM_Core_Error::fatal(ts('%1 - recurring payments not implemented', array(1 => $paymentProcessor)));
+      CRM_Core_Error::fatal(ts('First Data - recurring payments not implemented'));
     }
 
     if (!defined('CURLOPT_SSLCERT')) {
index 07a1e446571cf1cd2d92357646971bd9b7babd48..410e820b6fd7e26456ed1e58295f73034dc7c0cd 100644 (file)
@@ -261,9 +261,15 @@ class CRM_Core_Payment_Form {
   }
 
   /**
-   * Validate the payment instrument values before passing it to the payment processor
-   * We want this to be overrideable by the payment processor, and default to using
+   * Validate the payment instrument values before passing it to the payment processor.
+   *
+   * We want this to be able to be overridden by the payment processor, and default to using
    * this object's validCreditCard for credit cards (implemented as the default in the Payment class).
+   *
+   * @param int $payment_processor_id
+   * @param array $values
+   * @param array $errors
+   * @param int $billing_profile_id
    */
   public static function validatePaymentInstrument($payment_processor_id, $values, &$errors, $billing_profile_id) {
     $payment = Civi\Payment\System::singleton()->getById($payment_processor_id);
index 5901fb600194dfceee871b699a06685ce62707b8..2b14306c6dcc52203b2db4d429cb7eef773a6b6d 100644 (file)
@@ -172,10 +172,11 @@ class CRM_Core_Payment_PayJunction extends CRM_Core_Payment {
   }
   // end function doDirectPayment
 
-
   /**
    * This function checks the PayJunction response code.
    *
+   * @param array $response
+   *
    * @return bool
    */
   public function isError(&$response) {
index dc6c681d0521064f618cbad546e37adfe511b79b..fcac1ca7d5270e5e596b7e1c9b0e01d9923daeb4 100644 (file)
@@ -259,11 +259,20 @@ class CRM_Core_Payment_PaymentExpressIPN extends CRM_Core_Payment_BaseIPN {
   }
 
   /**
-   * This method is handles the response that will be invoked by the
-   * notification or request sent by the payment processor.
+   * Main notification processing method.
+   *
    * hex string from paymentexpress is passed to this function as hex string. Code based on googleIPN
    * mac_key is only passed if the processor is pxaccess as it is used for decryption
    * $dps_method is either pxaccess or pxpay
+   *
+   * @param string $dps_method
+   * @param array $rawPostData
+   * @param string $dps_url
+   * @param string $dps_user
+   * @param string $dps_key
+   * @param string $mac_key
+   *
+   * @throws \Exception
    */
   public static function main($dps_method, $rawPostData, $dps_url, $dps_user, $dps_key, $mac_key) {
 
index 2eb2ee258d8997873b5695f552e97940dc82644c..c687962f2e297d8b6d7f1f6a7f60d1d81c8dce60 100644 (file)
@@ -253,6 +253,8 @@ class CRM_Core_Permission_Base {
    * Get the permissions defined in the hook_civicrm_permission implementation
    * in all enabled CiviCRM module extensions.
    *
+   * @param bool $descriptions
+   *
    * @return array
    *   Array of permissions, in the same format as CRM_Core_Permission::getCorePermissions().
    */
index 0b385e3ffb2edc17cb57a71dbb5ca97d2fa2a236..6b014902812df42a08101cc97fc165b1c9e77ee1 100644 (file)
@@ -176,6 +176,8 @@ class CRM_Core_State {
 
   /**
    * Setter for name.
+   *
+   * @param string $name
    */
   public function setName($name) {
     $this->_name = $name;
index a138102bd387ca29ed0b4603e7b9e9bf48ed952d..3b127bf8c63ba6acd8b0f3dc4115c21cb0820cb6 100644 (file)
@@ -319,6 +319,12 @@ WHERE
 
   /**
    * Return payment update Query.
+   *
+   * @param string $tableName
+   * @param int $mainContactId
+   * @param int $otherContactId
+   *
+   * @return array
    */
   public static function paymentSql($tableName, $mainContactId, $otherContactId) {
     $sqls = array();
@@ -415,6 +421,10 @@ INNER JOIN  civicrm_membership membership2 ON membership1.membership_type_id = m
    * Based on the provided two contact_ids and a set of tables, move the
    * belongings of the other contact to the main one.
    *
+   * @param int $mainId
+   * @param int $otherId
+   * @param bool $tables
+   * @param array $tableOperations
    */
   public static function moveContactBelongings($mainId, $otherId, $tables = FALSE, $tableOperations = array()) {
     $cidRefs = self::cidRefs();
index 0e1feb7ff27423a38cb1bd37f78cc9ed3d1e4959..15a2869043818d6e164d2cd9117dd9b5f1c75489 100644 (file)
@@ -1048,6 +1048,10 @@ WHERE civicrm_event.is_active = 1
    * This is sometimes called in a loop (during event search).
    *
    * We cache the values to prevent repeated calls to the db.
+   *
+   * @param int $id
+   *
+   * @return bool
    */
   public static function usesPriceSet($id) {
     static $usesPriceSet = array();
index ebc1ff95a82392b874a00d161a73301694aa389e..0ed1ba4f9096272bfb3f477e51b7caa69d2f336c 100644 (file)
@@ -2065,9 +2065,16 @@ WHERE (entity_table = 'civicrm_participant' AND entity_id = {$participantId} AND
   }
 
   /**
-   * @param $updatedAmount
-   * @param $paidAmount
+   * Record adjusted amount.
+   *
+   * @param int $updatedAmount
+   * @param int $paidAmount
    * @param int $contributionId
+   *
+   * @param int $taxAmount
+   * @param bool $updateAmountLevel
+   *
+   * @return bool|\CRM_Core_BAO_FinancialTrxn
    */
   public static function recordAdjustedAmt($updatedAmount, $paidAmount, $contributionId, $taxAmount = NULL, $updateAmountLevel = NULL) {
     $pendingAmount = CRM_Core_BAO_FinancialTrxn::getBalanceTrxnAmt($contributionId);
index 8f5dd088ef882e0606407a4743f1a818af761958..f9aa53014865f5a6cd7804cf7c0aca8f291a10b9 100644 (file)
@@ -75,12 +75,11 @@ class CRM_Event_Badge {
   }
 
   /**
-   * Create the labels (pdf)
+   * Create the labels (pdf).
+   *
    * It assumes the participants are from the same event
    *
    * @param array $participants
-   *
-   * @return;
    */
   public function run(&$participants) {
     // fetch the 1st participant, and take her event to retrieve its attributes
index 988190f01c4f13d194ba616008cf42fa1c5921bb..7dc007f0b90d175f4e6e87e23d2bce918d4277bd 100644 (file)
@@ -1358,7 +1358,7 @@ class CRM_Event_Form_Registration_Register extends CRM_Event_Form_Registration {
    * @param bool $useDedupeRules
    *   Force usage of dedupe rules.
    *
-   * @return void
+   * @return int
    */
   public static function checkRegistration($fields, &$self, $isAdditional = FALSE, $returnContactId = FALSE, $useDedupeRules = FALSE) {
     // CRM-3907, skip check for preview registrations
index 1fb233a4a622315655481af08fe4541f2733fcf4..6ce58cadb3e003e5c8c3d3295921870cd2c383c2 100644 (file)
@@ -59,6 +59,10 @@ class CRM_Event_Import_Parser_Participant extends CRM_Event_Import_Parser {
 
   /**
    * Class constructor.
+   *
+   * @param array $mapperKeys
+   * @param null $mapperLocType
+   * @param null $mapperPhoneType
    */
   public function __construct(&$mapperKeys, $mapperLocType = NULL, $mapperPhoneType = NULL) {
     parent::__construct();
@@ -66,9 +70,7 @@ class CRM_Event_Import_Parser_Participant extends CRM_Event_Import_Parser {
   }
 
   /**
-   * The initializer code, called before the processing
-   *
-   * @return void
+   * The initializer code, called before the processing.
    */
   public function init() {
     $fields = CRM_Event_BAO_Participant::importableFields($this->_contactType, FALSE);
index adc47cadee6e892e4173551f793d9fc60e5e8ee8..44e150f1722bc6fa05312e2a9778c90da4638dc1 100644 (file)
@@ -1718,8 +1718,16 @@ WHERE  id IN ( $deleteIDString )
 
   /**
    * The function unsets static part of the string, if token is the dynamic part.
+   *
    * Example: 'Hello {contact.first_name}' => converted to => '{contact.first_name}'
    * i.e 'Hello Alan' => converted to => 'Alan'
+   *
+   * @param string $parsedString
+   * @param string $defaultGreeting
+   * @param bool $addressMergeGreetings
+   * @param string $greetingType
+   *
+   * @return mixed
    */
   public static function _trimNonTokens(
     &$parsedString, $defaultGreeting,
index daed65932948086ef8622015d4625f35b62054f8..cf14e860d972851c7cbf9e17b6457cc8356f7a2c 100644 (file)
@@ -68,7 +68,9 @@ class CRM_Financial_BAO_FinancialItem extends CRM_Financial_DAO_FinancialItem {
    *   Contribution object.
    * @param bool $taxTrxnID
    *
-   * @return void
+   * @param int $trxnId
+   *
+   * @return CRM_Financial_DAO_FinancialItem
    */
   public static function add($lineItem, $contribution, $taxTrxnID = FALSE, $trxnId = NULL) {
     $contributionStatuses = CRM_Contribute_PseudoConstant::contributionStatus(NULL, 'name');
@@ -128,7 +130,7 @@ class CRM_Financial_BAO_FinancialItem extends CRM_Financial_DAO_FinancialItem {
   }
 
   /**
-   * Create the financial Items and financial enity trxn.
+   * Create the financial Items and financial entity trxn.
    *
    * @param array $params
    *   Associated array to create financial items.
@@ -137,7 +139,7 @@ class CRM_Financial_BAO_FinancialItem extends CRM_Financial_DAO_FinancialItem {
    * @param array $trxnIds
    *   Financial item ids.
    *
-   * @return object
+   * @return CRM_Financial_DAO_FinancialItem
    */
   public static function create(&$params, $ids = NULL, $trxnIds = NULL) {
     $financialItem = new CRM_Financial_DAO_FinancialItem();
index 3546de9f0685235eff2af455065b1ce00c162728..c973e061d2dd25bb3f7e841cf8c8141e605483ab 100644 (file)
@@ -239,7 +239,7 @@ class CRM_Grant_Selector_Search extends CRM_Core_Selector_Base implements CRM_Co
   /**
    * Returns total number of rows for the query.
    *
-   * @param
+   * @param int $action
    *
    * @return int
    *   Total number of rows
index 3e8258b91c7546b634ad8b243e92215b82f4e311..4fc206855fc8cab5ca6f33f167d633abc5ba8bad 100644 (file)
@@ -49,23 +49,26 @@ abstract class CRM_Import_DataSource {
 
   /**
    * Set variables up before form is built.
+   *
+   * @param CRM_Core_Form $form
    */
   abstract public function preProcess(&$form);
 
   /**
-   * This is function is called by the form object to get the DataSource's
-   * form snippet. It should add all fields necesarry to get the data
-   * uploaded to the temporary table in the DB.
+   * This is function is called by the form object to get the DataSource's form snippet.
    *
-   * @param CRM_Core_Form $form
+   * It should add all fields necessary to get the data uploaded to the temporary table in the DB.
    *
-   * @return void
-   *   (operates directly on form argument)
+   * @param CRM_Core_Form $form
    */
   abstract public function buildQuickForm(&$form);
 
   /**
    * Process the form submission.
+   *
+   * @param array $params
+   * @param string $db
+   * @param CRM_Core_Form $form
    */
   abstract public function postProcess(&$params, &$db, &$form);
 
index 3960d1fc6343e6b5af6c908cfac7cf2a2b1ecc95..3bb3deacb9dc0ebd3b80e23e392833126302b68f 100644 (file)
@@ -82,6 +82,10 @@ class CRM_Import_DataSource_CSV extends CRM_Import_DataSource {
 
   /**
    * Process the form submission.
+   *
+   * @param array $params
+   * @param string $db
+   * @param \CRM_Core_Form $form
    */
   public function postProcess(&$params, &$db, &$form) {
     $file = $params['uploadFile']['name'];
index e6b8b5588046fd6aab148ec269b4dd0cb85fa9c4..9b91bac7ff239fed923c88f6f34e63eaa4a12486 100644 (file)
@@ -91,6 +91,10 @@ class CRM_Import_DataSource_SQL extends CRM_Import_DataSource {
 
   /**
    * Process the form submission.
+   *
+   * @param array $params
+   * @param string $db
+   * @param \CRM_Core_Form $form
    */
   public function postProcess(&$params, &$db, &$form) {
     $importJob = new CRM_Contact_Import_ImportJob(
index 4c15b921307c13c564181a34af479a71c5a9f231..aeff5dd80f2de022496bcd23a4503d64e9a1343d 100644 (file)
@@ -1393,9 +1393,11 @@ ORDER BY   civicrm_email.is_bulkmail DESC
   }
 
   /**
+   * Replace tokens.
    *
-   * get mailing object and replaces subscribeInvite,
-   * domain and mailing tokens
+   * Get mailing object and replaces subscribeInvite, domain and mailing tokens.
+   *
+   * @param array $mailing
    */
   public static function tokenReplace(&$mailing) {
     $domain = CRM_Core_BAO_Domain::getDomain();
@@ -1416,9 +1418,16 @@ ORDER BY   civicrm_email.is_bulkmail DESC
   }
 
   /**
+   * Get data to resolve tokens.
+   *
+   * @param array $token_a
+   * @param bool $html
+   * @param array $contact
+   * @param string $verp
+   * @param array $urls
+   * @param int $event_queue_id
    *
-   *  getTokenData receives a token from an email
-   *  and returns the appropriate data for the token
+   * @return bool|mixed|null|string
    */
   private function getTokenData(&$token_a, $html = FALSE, &$contact, &$verp, &$urls, $event_queue_id) {
     $type = $token_a['type'];
index 0d356a2c71e35a30ec26c573e0625844c4e8f652..7e0a6db2c097174d2a848b1410660e14e3ea2b9c 100644 (file)
@@ -59,7 +59,9 @@ class CRM_Mailing_Event_BAO_Queue extends CRM_Mailing_Event_DAO_Queue {
   }
 
   /**
-   * Create a security hash from the job, email and contact ids
+   * Create a security hash from the job, email and contact ids.
+   *
+   * @param array $params
    *
    * @return int
    *   The hash
@@ -75,7 +77,7 @@ class CRM_Mailing_Event_BAO_Queue extends CRM_Mailing_Event_DAO_Queue {
   }
 
   /**
-   * Verify that a queue event exists with the specified id/job id/hash
+   * Verify that a queue event exists with the specified id/job id/hash.
    *
    * @param int $job_id
    *   The job ID of the event to find.
index 638994babebbfcec6a36ab0a3c81b348e03151fc..4c387888fc4ce6675035efc33389c9fd9f10097a 100644 (file)
@@ -348,10 +348,8 @@ WHERE pcp.id = %1 AND cc.contribution_status_id =1 AND cc.is_test = 0";
   /**
    * Delete the campaign page.
    *
-   * @param int $id
+   * @param int
    *   Campaign page id.
-   *
-   * @return;
    */
   public static function deleteById($id) {
     CRM_Utils_Hook::pre('delete', 'Campaign', $id, CRM_Core_DAO::$_nullArray);
@@ -580,14 +578,12 @@ WHERE pcp.id = %1 AND cc.contribution_status_id =1 AND cc.is_test = 0";
   }
 
   /**
-   * Approve / Reject the campaign page
+   * Approve / Reject the campaign page.
    *
    * @param int $id
    *   Campaign page id.
    *
    * @param $is_active
-   *
-   * @return;
    */
   public static function setIsActive($id, $is_active) {
     switch ($is_active) {
index 4927be34ac6d1eeea99e63c1ce39c844af48ae4e..6d222c3a2ca31511d368b912594b5347d72c2b14 100644 (file)
@@ -384,7 +384,7 @@ class CRM_Profile_Selector_Listings extends CRM_Core_Selector_Base implements CR
   /**
    * Returns total number of rows for the query.
    *
-   * @param
+   * @param int $action
    *
    * @return int
    *   Total number of rows
index b1dc8523a9d9b3afb1eea07f52c26eb439ab9b76..eb3532de10d4fdda34d7a184b393da53fe2387a1 100644 (file)
@@ -67,15 +67,29 @@ interface CRM_Report_Interface {
   public function summary();
 
   /**
+   * Get contact IDs.
+   *
    * 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 string $sort
+   *
+   * @return
    */
   public function contactIDs($offset = 0, $rowcount = 0, $sort = NULL);
 
   /**
-   * Retrieve all the values that match the current input parameters
-   * Used by the selector
+   * Retrieve all the values that match the current input parameters used by the selector.
+   *
+   * @param int $offset
+   * @param int $rowcount
+   * @param string $sort
+   * @param bool $includeContactIDs
+   *
+   * @return
    */
   public function all(
     $offset = 0, $rowcount = 0, $sort = NULL,
@@ -98,6 +112,8 @@ interface CRM_Report_Interface {
 
   /**
    * The where clause for the query.
+   *
+   * @param bool $includeContactIDs
    */
   public function where($includeContactIDs = FALSE);
 
index ce9d2788fc41132e845aff308bcc9a3bda2e039b..4476aac705705acff5ea0b000ef0ce6f97e741ad 100644 (file)
@@ -88,6 +88,11 @@ abstract class CRM_SMS_Provider {
 
   /**
    * Send an SMS Message via the API Server.
+   *
+   * @param array $recipients
+   * @param string $header
+   * @param string $message
+   * @param int $dncID
    */
   abstract public function send($recipients, $header, $message, $dncID = NULL);
 
index 5b19b3b20c6c4368dceb93897c82e3f73714daba..858e1a5adcc61162843c954ac58933c4b135b2d5 100644 (file)
@@ -67,9 +67,13 @@ class CRM_Upgrade_Incremental_Base {
   public function setPostUpgradeMessage(&$postUpgradeMessage, $rev) {
   }
 
-
   /**
    * (Queue Task Callback)
+   *
+   * @param \CRM_Queue_TaskContext $ctx
+   * @param string $rev
+   *
+   * @return bool
    */
   public static function runSql(CRM_Queue_TaskContext $ctx, $rev) {
     $upgrade = new CRM_Upgrade_Form();
@@ -79,11 +83,15 @@ class CRM_Upgrade_Incremental_Base {
   }
 
   /**
-   * Syntactic sugar for adding a task which (a) is in this class and (b) has
-   * a high priority.
+   * Syntactic sugar for adding a task.
+   *
+   * Task is (a) in this class and (b) has a high priority.
    *
    * After passing the $funcName, you can also pass parameters that will go to
    * the function. Note that all params must be serializable.
+   *
+   * @param string $title
+   * @param string $funcName
    */
   protected function addTask($title, $funcName) {
     $queue = CRM_Queue_Service::singleton()->load(array(
index 30e5379f49179c2b5705c62a06593359cc2d88f9..f506978967b8b290e5c392379988503269c5b852 100644 (file)
@@ -971,6 +971,10 @@ ALTER TABLE civicrm_financial_account
 
   /**
    * Change index and add missing constraints for civicrm_contribution_recur.
+   *
+   * @param \CRM_Queue_TaskContext $ctx
+   *
+   * @return bool
    */
   public function addMissingConstraints(CRM_Queue_TaskContext $ctx) {
     $query = "SHOW KEYS FROM `civicrm_contribution_recur` WHERE key_name = 'UI_contrib_payment_instrument_id'";
@@ -1002,6 +1006,10 @@ ALTER TABLE civicrm_financial_account
   /**
    * Update financial_account_id for bad data in financial_trxn table.
    * CRM-12844
+   *
+   * @param \CRM_Queue_TaskContext $ctx
+   *
+   * @return bool
    */
   public function updateFinancialTrxnData(CRM_Queue_TaskContext $ctx) {
     $upgrade = new CRM_Upgrade_Form();
@@ -1127,7 +1135,14 @@ AND cli.entity_table = 'civicrm_contribution' AND cli.id IN (" . implode(',', $v
 
   /**
    * Replace contribution_type to financial_type in table.
-   * civicrm_saved_search and Structure civicrm_report_instance
+   *
+   * Civicrm_saved_search and Structure civicrm_report_instance
+   *
+   * @param \CRM_Queue_TaskContext $ctx
+   * @param string $query
+   * @param string $table
+   *
+   * @return bool
    */
   public function replaceContributionTypeId(CRM_Queue_TaskContext $ctx, $query, $table) {
     $dao = CRM_Core_DAO::executeQuery($query);
index ffcd7cdcfe6b291e7cbe7dc344d76f0433b26309..cc8167006a2a714aae33b10472869b5161ec7d63 100644 (file)
@@ -361,7 +361,9 @@ class CRM_Upgrade_Snapshot_V4p2_Price_DAO_Field extends CRM_Core_DAO {
   }
 
   /**
-   * returns the list of fields that can be exported.
+   * Returns the list of fields that can be exported.
+   *
+   * @param bool $prefix
    *
    * @return array
    */
index 57c03e139101c7eab2726a0024b73c45fddf1e80..6ea8b5780b5ac66c36969aff726b56a19c0281e1 100644 (file)
@@ -132,7 +132,6 @@ class CRM_Utils_Array {
    * @param string $seperator
    *   (optional) String to be appended after open/close tags.
    *
-   *
    * @return string
    *   XML fragment representing $list.
    */
@@ -345,8 +344,11 @@ class CRM_Utils_Array {
   }
 
   /**
-   * @param $subset
-   * @param $superset
+   * Is array A a subset of array B.
+   *
+   * @param array $subset
+   * @param array $superset
+   *
    * @return bool
    *   TRUE if $subset is a subset of $superset
    */
@@ -388,12 +390,18 @@ class CRM_Utils_Array {
   }
 
   /**
-   * convert associative array names to values.
-   * and vice-versa.
+   * Convert associative array names to values and vice-versa.
    *
    * This function is used by both the web form layer and the api. Note that
    * the api needs the name => value conversion, also the view layer typically
    * requires value => name conversion
+   *
+   * @param array $defaults
+   * @param string $property
+   * @param $lookup
+   * @param $reverse
+   *
+   * @return bool
    */
   public static function lookupValue(&$defaults, $property, $lookup, $reverse) {
     $id = $property . '_id';
index 950ec296e99f2476fc48eba4bd20a0554b2bf958..15bc0bf93bfebc43c65374e2cb7ec803ddbd14f9 100644 (file)
@@ -1966,6 +1966,10 @@ abstract class CRM_Utils_Hook {
 
   /**
    * This hook is called when a query string of the CSV Batch export is generated.
+   *
+   * @param string $query
+   *
+   * @return mixed
    */
   public static function batchQuery(&$query) {
     return self::singleton()->invoke(1, $query, self::$_nullObject,
@@ -1976,6 +1980,11 @@ abstract class CRM_Utils_Hook {
 
   /**
    * This hook is called when the entries of the CSV Batch export are mapped.
+   *
+   * @param array $results
+   * @param array $items
+   *
+   * @return mixed
    */
   public static function batchItems(&$results, &$items) {
     return self::singleton()->invoke(2, $results, $items,
index e205cd1a5728819fa056923ea2df50597abc3292..9f07954514c3df6bcda2462a6617966b2ead529c 100644 (file)
@@ -59,6 +59,10 @@ class CRM_Utils_Migrate_ExportJSON {
 
   /**
    * Split a large array of contactIDs into more manageable smaller chunks.
+   *
+   * @param array $contactIDs
+   *
+   * @return array
    */
   public function &splitContactIDs(&$contactIDs) {
     // contactIDs could be a real large array, so we split it up into
@@ -88,6 +92,9 @@ class CRM_Utils_Migrate_ExportJSON {
 
   /**
    * Given a set of contact IDs get the values.
+   *
+   * @param array $contactIDs
+   * @param array $additionalContactIDs
    */
   public function getValues(&$contactIDs, &$additionalContactIDs) {
 
index d1858b3857eea3af7ff67e6157375356b46906a2..7eecb8cd01dfc08713befacd1a547080266550b2 100644 (file)
@@ -79,6 +79,11 @@ class CRM_Utils_Number {
 
   /**
    * Some kind of numbery-looky-printy thing.
+   *
+   * @param string $size
+   * @param bool $checkForPostMax
+   *
+   * @return int
    */
   public static function formatUnitSize($size, $checkForPostMax = FALSE) {
     if ($size) {
index 222222f6758b47fc9e392cd09c1afe99682a22e5..7b6d3c71338bd1422a83e6b3d54e71ab75378438 100644 (file)
@@ -309,6 +309,11 @@ class CRM_Utils_Pager extends Pager_Sliding {
 
   /**
    * Build a url for pager links.
+   *
+   * @param string $key
+   * @param string $value
+   *
+   * @return string
    */
   public function makeURL($key, $value) {
     $href = CRM_Utils_System::makeURL($key, TRUE);
index 8d53f596ad51e885cceeb495bc7a8df6f47b755b..8b211d0252f2e84ba4bd22f8232a252f7dbc1f8e 100644 (file)
@@ -67,6 +67,8 @@ class CRM_Utils_ReCAPTCHA {
 
   /**
    * Add element to form.
+   *
+   * @param CRM_Core_Form $form
    */
   public static function add(&$form) {
     $error = NULL;
index e4eeb41fa122af3ba00787dd3aa5a53347083b73..2b4269e205d1febcd525c9f626a7e15dbee9f2c8 100644 (file)
@@ -1412,6 +1412,12 @@ class CRM_Utils_Token {
 
   /**
    * Replace existing greeting tokens in message/subject.
+   *
+   * @param string $tokenString
+   * @param array $contactDetails
+   * @param int $contactId
+   * @param string $className
+   * @param bool $escapeSmarty
    */
   public static function replaceGreetingTokens(&$tokenString, $contactDetails = NULL, $contactId = NULL, $className = NULL, $escapeSmarty = FALSE) {
 
index d20dee60d15d0d28174eead5e3f7fbb421d94d46..fb0ade6ace31a1af6c18b82a32c926e950bdbea6 100644 (file)
@@ -356,7 +356,8 @@ class RecipientBuilder {
   }
 
   /**
-   * @param $actionSchedule
+   * Parse repetition interval.
+   *
    * @return int|string
    */
   protected function parseRepetitionInterval() {
@@ -411,8 +412,9 @@ class RecipientBuilder {
   }
 
   /**
-   * @param $actionSchedule
-   * @param $contactTableAlias
+   * Prepare language filter.
+   *
+   * @param string $contactTableAlias
    * @return string
    */
   protected function prepareLanguageFilter($contactTableAlias) {
index 6bb47fab6143d3e47538913f6c92b9f45d32ffe7..70654bac29a43cbf64d6a114cdadf4150a074eb5 100644 (file)
@@ -267,6 +267,8 @@ class ResolverGlobalCallback {
   /**
    * Invoke function.
    *
+   * @param mixed $arg1
+   *
    * @return mixed
    */
   public function __invoke($arg1 = NULL) {
index fe47d5e41ddfb40fe53ffafb9e89d9a0ef6c1157..84bb6c632da45f5486494e6257a213f059fed10e 100644 (file)
@@ -325,6 +325,8 @@ class SettingsManager {
    * during bootstrap -- in particular, defaults cannot be loaded. For a very small number of settings,
    * we must define defaults before the system bootstraps.
    *
+   * @param string $entity
+   *
    * @return array
    */
   private static function getSystemDefaults($entity) {
index 1fe89496b3e7b016966a8a6ac5186f61b8447df4..c0b3806705c84c03decd6c5d00e9afa9c21b3e07 100644 (file)
@@ -120,6 +120,10 @@ class SettingsMetadata {
 
   /**
    * Load up settings metadata from files.
+   *
+   * @param array $metaDataFolder
+   *
+   * @return array
    */
   protected static function loadSettingsMetadata($metaDataFolder) {
     $settingMetaData = array();
index 5e5472ba654d7287bcc00dda6af295813d3f7baf..3566fdf453bbab455a05c8960097544ad5026bea 100644 (file)
@@ -154,6 +154,8 @@ abstract class AbstractTokenSubscriber implements EventSubscriberInterface {
    * To perform a bulk lookup before rendering tokens, override this
    * function and return the prefetched data.
    *
+   * @param \Civi\Token\Event\TokenValueEvent $e
+   *
    * @return mixed
    */
   public function prefetch(TokenValueEvent $e) {
index 4409d68a6a9cd287d45bca9fd8af74ec34125d06..d9347d0f6d3dd50f58dee2035ec4d926b53c8377 100644 (file)
@@ -169,6 +169,10 @@ class TokenRow {
 
   /**
    * Auto-convert between different formats
+   *
+   * @param string $format
+   *
+   * @return $this
    */
   public function fill($format = NULL) {
     if ($format === NULL) {
index a11faae369c7c9c8036887cec81ef531bacd058f..f58da8f5ce47afc8cad2c5367813ae8558e85b57 100644 (file)
@@ -267,7 +267,11 @@ class CRM_GCD {
    */
 
   /**
-   * get a randomly generated string
+   * Get a randomly generated string.
+   *
+   * @param int $size
+   *
+   * @return string
    */
   private function randomString($size = 32) {
     $string = "";
index 352d6d68fe6fa269cb4d33ceeeff304af0d8e17e..dd95ae263e4c54d9e16b75efc794f07439377514 100755 (executable)
@@ -3316,7 +3316,9 @@ AND    ( TABLE_NAME LIKE 'civicrm_value_%' )
   }
 
   /**
-   * @param $exists
+   * Assert the attachment exists.
+   *
+   * @param bool $exists
    * @param array $apiResult
    */
   protected function assertAttachmentExistence($exists, $apiResult) {
index 6a914d08cd53dc9f60fa0146f9a086bcfe84bbba..de91bcb1ebf0e56e305282f452459f0f7d361a36 100644 (file)
@@ -273,8 +273,11 @@ class api_v3_JobProcessMailingTest extends CiviUnitTestCase {
   }
 
   /**
+   * Create contacts in group.
+   *
    * @param int $count
    * @param int $groupID
+   * @param string $domain
    */
   public function createContactsInGroup($count, $groupID, $domain = 'nul.example.com') {
     for ($i = 1; $i <= $count; $i++) {