comment fixes
authoreileenmcnaugton <eileen@fuzion.co.nz>
Fri, 23 Oct 2015 03:39:15 +0000 (16:39 +1300)
committereileenmcnaugton <eileen@fuzion.co.nz>
Fri, 23 Oct 2015 03:53:41 +0000 (16:53 +1300)
60 files changed:
CRM/Admin/Form/ScheduleReminders.php
CRM/Campaign/BAO/Survey.php
CRM/Contact/BAO/GroupContact.php
CRM/Contact/Form/Contact.php
CRM/Contact/Form/Search/Basic.php
CRM/Contact/Form/Search/Builder.php
CRM/Contact/Import/Field.php
CRM/Contact/Page/AJAX.php
CRM/Contribute/BAO/ContributionRecur.php
CRM/Core/BAO/EntityTag.php
CRM/Core/BAO/Navigation.php
CRM/Core/BAO/Phone.php
CRM/Core/BAO/Setting.php
CRM/Core/BAO/UFField.php
CRM/Core/CodeGen/Specification.php
CRM/Core/Component.php
CRM/Core/Component/Info.php
CRM/Core/Config.php
CRM/Core/DAO.php
CRM/Core/Error.php
CRM/Core/IDS.php
CRM/Core/Menu.php
CRM/Core/Payment/FirstData.php
CRM/Core/Payment/GoogleIPN.php
CRM/Core/Payment/PaymentExpressIPN.php
CRM/Core/Payment/Realex.php
CRM/Core/Payment/eWAY.php
CRM/Custom/Form/ChangeFieldType.php
CRM/Dedupe/BAO/RuleGroup.php
CRM/Event/BAO/Event.php
CRM/Event/Badge.php
CRM/Grant/Form/Task.php
CRM/Logging/Schema.php
CRM/Member/Selector/Search.php
CRM/Pledge/BAO/Query.php
CRM/Price/BAO/LineItem.php
CRM/Profile/Selector/Listings.php
CRM/Report/Utils/Report.php
CRM/Upgrade/Incremental/php/FourTwo.php
CRM/Upgrade/Snapshot/V4p2/Price/DAO/Field.php
CRM/Upgrade/Snapshot/V4p2/Price/DAO/FieldValue.php
CRM/Upgrade/Snapshot/V4p2/Price/DAO/LineItem.php
CRM/Upgrade/Snapshot/V4p2/Price/DAO/Set.php
CRM/Upgrade/Snapshot/V4p2/Price/DAO/SetEntity.php
CRM/Utils/Check.php
CRM/Utils/DeprecatedUtils.php
CRM/Utils/File.php
CRM/Utils/Mail/Incoming.php
CRM/Utils/Rule.php
CRM/Utils/System/Drupal.php
CRM/Utils/System/Drupal6.php
CRM/Utils/System/DrupalBase.php
CRM/Utils/System/Joomla.php
CRM/Utils/System/WordPress.php
api/v3/UFJoin.php
tests/phpunit/Civi/ActionSchedule/AbstractMappingTest.php
tests/phpunit/CiviTest/CiviSeleniumTestCase.php
tests/phpunit/CiviTest/CiviUnitTestCase.php
tests/phpunit/WebTest/Event/AddEventTest.php
tests/phpunit/WebTest/Member/OnlineMembershipCreateTest.php

index 144cbe803ccc0fe09267f50220f01f45c2238f72..61fca0e171b18d80a4e827ac9cceeab4cff4ffc9 100644 (file)
@@ -294,10 +294,11 @@ class CRM_Admin_Form_ScheduleReminders extends CRM_Admin_Form {
    *
    * @param array $fields
    *   The input form values.
+   * @param array $files
    * @param CRM_Admin_Form_ScheduleReminders $self
    *
-   * @return bool|array
-   *   true if no errors, else array of errors
+   * @return array|bool
+   *   True if no errors, else array of errors
    */
   public static function formRule($fields, $files, $self) {
     $errors = array();
index 0ec174d5dfc7ed62e5b87d19658b135e30eed8ae..f68a4d9eb17f0f2d89573f328ef599c01ec44469 100644 (file)
@@ -301,6 +301,9 @@ SELECT  survey.id    as id,
 
   /**
    * Get Survey activity types.
+   *
+   * @param string $returnColumn
+   * @param bool $includePetitionActivityType
    */
   public static function getSurveyActivityType($returnColumn = 'label', $includePetitionActivityType = FALSE) {
     static $activityTypes;
@@ -914,6 +917,10 @@ INNER JOIN  civicrm_contact contact_a ON ( activityTarget.contact_id = contact_a
 
   /**
    * Decides the contact type for given survey.
+   *
+   * @param int $surveyId
+   *
+   * @return null|string
    */
   public static function getSurveyContactType($surveyId) {
     $contactType = NULL;
index 7306a4c0d0b8ca887269c9ce90970f8d949659be..2e52d1316ef3e7103cd5577b33a975909a7c02af 100644 (file)
@@ -320,9 +320,10 @@ class CRM_Contact_BAO_GroupContact extends CRM_Contact_DAO_GroupContact {
    *
    * @param bool $excludeHidden
    *
-   * @return array (reference)|int $values
-   *   the relevant data object values for the contact or
-   *   the total count when $count is TRUE
+   * @param int $groupId
+   *
+   * @return array|int $values
+   *   the relevant data object values for the contact or the total count when $count is TRUE
    */
   public static function &getContactGroup(
     $contactId,
index 31f06684caba7733ade6644ad88bc20a88967dc7..4d30d0c01988c7d3253e772edcfcb339d5c7c5a6 100644 (file)
@@ -464,8 +464,9 @@ class CRM_Contact_Form_Contact extends CRM_Core_Form {
   }
 
   /**
-   * Do the set default related to location type id,
-   * primary location,  default country
+   * Do the set default related to location type id, primary location,  default country.
+   *
+   * @param array $defaults
    */
   public function blockSetDefaults(&$defaults) {
     $locationTypeKeys = array_filter(array_keys(CRM_Core_PseudoConstant::get('CRM_Core_DAO_Address', 'location_type_id')), 'is_int');
index 5dea92c50d94d290e022ac335090aaf335894f1b..d8d27abf834910766012994a5941fdf0bc1a578d 100644 (file)
@@ -194,8 +194,13 @@ class CRM_Contact_Form_Search_Basic extends CRM_Contact_Form_Search {
   }
 
   /**
-   * Add a form rule for this form. If Go is pressed then we must select some checkboxes
-   * and an action
+   * Add a form rule for this form.
+   *
+   * If Go is pressed then we must select some checkboxes and an action.
+   *
+   * @param array $fields
+   *
+   * @return array|bool
    */
   public static function formRule($fields) {
     // check actionName and if next, then do not repeat a search, since we are going to the next page
index b42fad974ba2325c2788a1808cbad207f98c999e..dfb2767d184201e6a00298e81ade5ec14e159464 100644 (file)
@@ -465,10 +465,14 @@ class CRM_Contact_Form_Search_Builder extends CRM_Contact_Form_Search {
   }
 
   /**
-   * CRM-10338
-   * tags and groups use array keys for selection list.
+   * CRM-10338 tags and groups use array keys for selection list.
+   *
    * if using IS NULL/NOT NULL, an array with no array key is created
    * convert that to simple NULL so processing can proceed
+   *
+   * @param string $val
+   *
+   * @return null
    */
   public static function checkArrayKeyEmpty($val) {
     if (is_array($val)) {
index 1edcdb65b9f0897c1b05c3b92333eb6b7671d1d6..fd4f6329fa23097ef9990588f8b207b55b4677f7 100644 (file)
@@ -164,14 +164,20 @@ class CRM_Contact_Import_Field {
   }
 
   /**
-   * The value is in string format. convert the value to the type of this field
+   * The value is in string format.
+   *
+   * Convert the value to the type of this field
    * and set the field value with the appropriate type
+   *
+   * @param mixed $value
    */
   public function setValue($value) {
     $this->_value = $value;
   }
 
   /**
+   * Validate something we didn't document.
+   *
    * @return bool
    */
   public function validate() {
index 9621dbc329fa45cb61e4e620b623f5dd6ad40e16..ddab661e09f79b720d542de0821e5df2284e21b1 100644 (file)
@@ -880,6 +880,8 @@ LIMIT {$offset}, {$rowCount}
 
   /**
    * Swap contacts in a dupe pair i.e main with duplicate contact.
+   *
+   * @param int $prevNextId
    */
   public static function flipDupePairs($prevNextId = NULL) {
     if (!$prevNextId) {
@@ -887,7 +889,7 @@ LIMIT {$offset}, {$rowCount}
     }
     $query = "
       UPDATE civicrm_prevnext_cache cpc
-      INNER JOIN civicrm_prevnext_cache old on cpc.id = old.id 
+      INNER JOIN civicrm_prevnext_cache old on cpc.id = old.id
       SET cpc.entity_id1 = cpc.entity_id2, cpc.entity_id2 = old.entity_id1 ";
     if (is_array($prevNextId) && !CRM_Utils_Array::crmIsEmptyArray($prevNextId)) {
       $prevNextId = implode(', ', $prevNextId);
index d4b938336c05205341ddc5837ff1c3d42bc1e377..ce853c5472e5637fa9e297cfc0dc0309155f6f44 100644 (file)
@@ -457,11 +457,13 @@ INNER JOIN civicrm_contribution       con ON ( con.id = mp.contribution_id )
 
   /**
    * CRM-16285 - Function to handle validation errors on form, for recurring contribution field.
+   *
    * @param array $fields
    *   The input form values.
    * @param array $files
    *   The uploaded files if any.
-   * @param $self
+   * @param CRM_Core_Form $self
+   * @param array $errors
    */
   public static function validateRecurContribution($fields, $files, $self, &$errors) {
     if (!empty($fields['is_recur'])) {
index f66afd0fdf0670fa086ad2b44d942971d041a292..6cebea4c5cd8e4990eeb71fdb5efe077bc0249c4 100644 (file)
@@ -312,6 +312,12 @@ class CRM_Core_BAO_EntityTag extends CRM_Core_DAO_EntityTag {
 
   /**
    * Get child contact tags given parentId.
+   *
+   * @param int $parentId
+   * @param int $entityId
+   * @param string $entityTable
+   *
+   * @return array
    */
   public static function getChildEntityTags($parentId, $entityId, $entityTable = 'civicrm_contact') {
     $entityTags = array();
@@ -334,6 +340,11 @@ class CRM_Core_BAO_EntityTag extends CRM_Core_DAO_EntityTag {
 
   /**
    * Merge two tags: tag B into tag A.
+   *
+   * @param int $tagAId
+   * @param int $tagBId
+   *
+   * @return array
    */
   public function mergeTags($tagAId, $tagBId) {
     $queryParams = array(
index c2047bf107e441522c7d0b3a235c96919111b6fc..ab9ec65a492d9f2a0322c9188e9e757725076aa8 100644 (file)
@@ -486,6 +486,8 @@ ORDER BY parent_id, weight";
    * @param array $nodes
    *   Each key is a numeral; each value is a node in
    *   the menu tree (with keys "child" and "attributes").
+   * @param int $maxNavID
+   * @param int $parentID
    */
   private static function _fixNavigationMenu(&$nodes, &$maxNavID, $parentID) {
     $origKeys = array_keys($nodes);
index 796129085f3dea2d98977972d9f102eef39458b7..00c6cdacaf5e2fc8fdc211fd44d6aa17b53d944c 100644 (file)
@@ -260,6 +260,10 @@ ORDER BY ph.is_primary DESC, phone_id ASC ";
 
   /**
    * Call common delete function.
+   *
+   * @param int $id
+   *
+   * @return bool
    */
   public static function del($id) {
     // Ensure mysql phone function exists
index 9fe452586057e994b1236dc18728757e509b23df..0794209bb8d26c208a7d99c5b7ec8ea614ee82d2 100644 (file)
@@ -305,10 +305,13 @@ class CRM_Core_BAO_Setting extends CRM_Core_DAO_Setting {
   }
 
   /**
-   * Validate & convert settings input - translate True False to 0 or 1
+   * Validate & convert settings input - translate True False to 0 or 1.
    *
-   * @value mixed value of the setting to be set
-   * @fieldSpec array Metadata for given field (drawn from the xml)
+   * @param mixed $value value of the setting to be set
+   * @param array $fieldSpec Metadata for given field (drawn from the xml)
+   *
+   * @return bool
+   * @throws \api_Exception
    */
   public static function validateBoolSetting(&$value, $fieldSpec) {
     if (!CRM_Utils_Rule::boolean($value)) {
index d8190c4d17dc63b87e5baac5ca42c5dd4d91b2ad..c348c37b0357030876015f048ed03664634660c0 100644 (file)
@@ -378,6 +378,10 @@ WHERE cf.id IN (" . $customFieldIds . ") AND is_multiple = 1 LIMIT 0,1";
   /**
    * Find out whether given profile group using Activity
    * Profile fields with contact fields
+   *
+   * @param int $ufGroupId
+   *
+   * @return bool
    */
   public static function checkContactActivityProfileType($ufGroupId) {
     $ufGroup = new CRM_Core_DAO_UFGroup();
index 1d6b98b2e5bfcd18d9ad112415c11b177ad85150..86c211afcf274f3d2384a0f2bb14158c680030b1 100644 (file)
@@ -660,6 +660,10 @@ class CRM_Core_CodeGen_Specification {
 
   /**
    * Sets the size property of a textfield.
+   *
+   * @param string $fieldXML
+   *
+   * @return null|string
    */
   protected function getSize($fieldXML) {
     // Extract from <size> tag if supplied
index b7cdb3fb9488c8d8acfc87e843ec499d9888aae4..5e5ce0c6d9f101f26d128ca7d3affa9d282655d2 100644 (file)
@@ -436,6 +436,10 @@ class CRM_Core_Component {
 
   /**
    * Get components info from info file.
+   *
+   * @param string $crmFolderDir
+   *
+   * @return array
    */
   public static function getComponentsFromFile($crmFolderDir) {
     $components = array();
index f2b600c7204ffe13dd50b04a920aa1ec7d513480..05a0689eb27a136427a2498a9df2adf3caa07514 100644 (file)
@@ -266,6 +266,8 @@ abstract class CRM_Core_Component_Info {
 
   /**
    * Builds advanced search form's component specific pane.
+   *
+   * @param CRM_Core_Form $form
    */
   public function buildAdvancedSearchPaneForm(&$form) {
     $bao = $this->getBAOQueryObject();
index 460e011c08c40f40a682fe91f90ac9fb0e124cdc..ace4979140472f10f3ad37bec142b0d151ce383b 100644 (file)
@@ -239,6 +239,8 @@ class CRM_Core_Config extends CRM_Core_Config_MagicMerge {
   /**
    * Do general cleanup of caches, temp directories and temp tables
    * CRM-8739
+   *
+   * @param bool $sessionReset
    */
   public function cleanupCaches($sessionReset = TRUE) {
     // cleanup templates_c directory
@@ -425,6 +427,10 @@ class CRM_Core_Config extends CRM_Core_Config_MagicMerge {
 
   /**
    * @deprecated
+   *
+   * @param string $defaultCurrency
+   *
+   * @return string
    */
   public function defaultCurrencySymbol($defaultCurrency = NULL) {
     return CRM_Core_BAO_Country::defaultCurrencySymbol($defaultCurrency);
index baa3033c9d862d25ff756b9e1143d61144072f2c..335a3fd50c03e8d7c324987d98f7b35e9a919afb 100644 (file)
@@ -431,6 +431,10 @@ class CRM_Core_DAO extends DB_DataObject {
   }
 
   /**
+   * Save DAO object.
+   *
+   * @param bool $hook
+   *
    * @return $this
    */
   public function save($hook = TRUE) {
index 82b0c8d7fb30a1c8b455010fa47390e615350505..6bf012854ec8125106d3cc6132cd73aedd7035a1 100644 (file)
@@ -291,6 +291,10 @@ class CRM_Core_Error extends PEAR_ErrorStack {
    * Also, if we do not return any value the PEAR_ErrorStack::push() then does the
    * action of PEAR_ERRORSTACK_PUSHANDLOG which displays the errors on the screen,
    * since the logger set for this error stack is 'display' - see CRM_Core_Config::getLog();
+   *
+   * @param mixed $pearError
+   *
+   * @return int
    */
   public static function handlePES($pearError) {
     return PEAR_ERRORSTACK_PUSH;
@@ -563,16 +567,16 @@ class CRM_Core_Error extends PEAR_ErrorStack {
   /**
    * Display the error message on terminal.
    *
-   * @param $message
+   * @param string $message
    * @param bool $out
    *   Should we log or return the output.
    *
    * @param string $comp
    *   Message to be output.
-   * @return string
-   *   format of the backtrace
-   *
+   * @param string $priority
    *
+   * @return string
+   *   Format of the backtrace
    */
   public static function debug_log_message($message, $out = FALSE, $comp = '', $priority = NULL) {
     $config = CRM_Core_Config::singleton();
index 5204255f0de763e64c04bc7c979a02e84b3a9915..85180ef6beaa0c6a43a54a3bdb69877095168057 100644 (file)
@@ -29,8 +29,6 @@
  *
  * @package CRM
  * @copyright CiviCRM LLC (c) 2004-2015
- * $Id$
- *
  */
 class CRM_Core_IDS {
 
@@ -49,6 +47,8 @@ class CRM_Core_IDS {
   private $init = NULL;
 
   /**
+   * Check function.
+   *
    * This function includes the IDS vendor parts and runs the
    * detection routines on the request array.
    *
@@ -64,7 +64,7 @@ class CRM_Core_IDS {
       return NULL;
     }
 
-    #add request url and user agent
+    // Add request url and user agent.
     $_REQUEST['IDS_request_uri'] = $_SERVER['REQUEST_URI'];
     if (isset($_SERVER['HTTP_USER_AGENT'])) {
       $_REQUEST['IDS_user_agent'] = $_SERVER['HTTP_USER_AGENT'];
@@ -171,8 +171,7 @@ class CRM_Core_IDS {
   }
 
   /**
-   * This function rects on the values in
-   * the incoming results array.
+   * This function reacts on the values in the incoming results array.
    *
    * Depending on the impact value certain actions are
    * performed.
@@ -204,8 +203,7 @@ class CRM_Core_IDS {
   }
 
   /**
-   * This function writes an entry about the intrusion
-   * to the intrusion database
+   * This function writes an entry about the intrusion to the database.
    *
    * @param array $result
    * @param int $reaction
@@ -238,14 +236,22 @@ class CRM_Core_IDS {
   }
 
   /**
-   * //todo
+   * Warn about IDS.
+   *
+   * @param array $result
+   *
+   * @return array
    */
   private function warn($result) {
     return $result;
   }
 
   /**
-   *  //todo
+   * Kick (whatever that means!).
+   *
+   * @param array $result
+   *
+   * @throws \Exception
    */
   private function kick($result) {
     $session = CRM_Core_Session::singleton();
index ff45aeebb37b87192385696de755755ce7c4d725..e45e737cfe308c790cef80c7dd7e0c1f59aea2bf 100644 (file)
@@ -248,6 +248,8 @@ class CRM_Core_Menu {
    * 2. Compute local tasks value if any
    * 3. Propagate access argument, access callback, page callback to the menu item
    * 4. Build the global navigation block
+   *
+   * @param array $menu
    */
   public static function build(&$menu) {
     foreach ($menu as $path => $menuItems) {
index b4fea7aa5374e3a89946407615c244cd0da64a27..daaaf465a4904b3dfc9734ed96b81669a708e685 100644 (file)
@@ -313,6 +313,11 @@ class CRM_Core_Payment_FirstData extends CRM_Core_Payment {
 
   /**
    * Produces error message and returns from class.
+   *
+   * @param int $errorCode
+   * @param string $errorMessage
+   *
+   * @return object
    */
   public function &errorExit($errorCode = NULL, $errorMessage = NULL) {
     $e = CRM_Core_Error::singleton();
index ec61179aee51078c39c1a76cb52c1551473a5399..2e107a62dfdf0b60317324375a87b316de8ae71c 100644 (file)
@@ -692,6 +692,10 @@ WHERE  contribution_recur_id = {$ids['contributionRecur']}
   /**
    * Converts the comma separated name-value pairs in <merchant-private-data>
    * to an array of name-value pairs.
+   *
+   * @param string $str
+   *
+   * @return array
    */
   public static function stringToArray($str) {
     $vars = $labels = array();
index fcac1ca7d5270e5e596b7e1c9b0e01d9923daeb4..079f81b1aa5a6295d57656717f4c1ee6feae5c0f 100644 (file)
@@ -432,8 +432,11 @@ class CRM_Core_Payment_PaymentExpressIPN extends CRM_Core_Payment_BaseIPN {
   }
 
   /**
-   * Converts the comma separated name-value pairs in <TxnData2>
-   * to an array of values.
+   * Converts the comma separated name-value pairs in <TxnData2> to an array of values.
+   *
+   * @param string $str
+   *
+   * @return array
    */
   public static function stringToArray($str) {
     $vars = $labels = array();
index 6f148380bb1cdcc966e4d4e598164cdbafe43e12..077db4b72bb1ce662ba1e518c6ca52b97e905a20 100644 (file)
@@ -296,7 +296,13 @@ class CRM_Core_Payment_Realex extends CRM_Core_Payment {
   }
 
   /**
-   *  Format the params from the form ready for sending to Realex.  Also perform some validation
+   * Format the params from the form ready for sending to Realex.
+   *
+   * Also perform some validation
+   *
+   * @param array $params
+   *
+   * @return bool
    */
   public function setRealexFields(&$params) {
     if ((int) $params['amount'] <= 0) {
index 37582a5b5c511697b9d936f0f37d09eba1846376..12bbb7bc0123b31978ae59421aa2b1889aba18cc 100644 (file)
@@ -427,9 +427,12 @@ class CRM_Core_Payment_eWAY extends CRM_Core_Payment {
   }
 
   /**
-   * ************************************************
-   * Produces error message and returns from class
-   * *************************************************
+   * Produces error message and returns from class.
+   *
+   * @param int $errorCode
+   * @param string $errorMessage
+   *
+   * @return object
    */
   public function &errorExit($errorCode = NULL, $errorMessage = NULL) {
     $e = CRM_Core_Error::singleton();
index ea4a8398f2e16c1284cd57f91f8ef45c3d38ff25..92ffa093436b1aa22f10c44ea6ae22d98cf5168f 100644 (file)
@@ -267,6 +267,9 @@ class CRM_Custom_Form_ChangeFieldType extends CRM_Core_Form {
   /**
    * Take a single-value column (eg: a Radio or Select etc ) and convert
    * value to the multi listed value (eg:"^Foo^")
+   *
+   * @param string $table
+   * @param string $column
    */
   public function firstValueToFlatten($table, $column) {
     $selectSql = "SELECT id, $column FROM $table WHERE $column IS NOT NULL";
@@ -288,6 +291,9 @@ class CRM_Custom_Form_ChangeFieldType extends CRM_Core_Form {
   /**
    * Take a multi-value column (e.g. a Multi-Select or CheckBox column), and convert
    * all values (of the form "^^" or "^Foo^" or "^Foo^Bar^") to the first listed value ("Foo")
+   *
+   * @param string $table
+   * @param string $column
    */
   public function flattenToFirstValue($table, $column) {
     $selectSql = "SELECT id, $column FROM $table WHERE $column IS NOT NULL";
index ec82cfb7671e2aedbe92d0a2a16f537836c733cd..1616392f6d04371c248867deb714c4586d5834a0 100644 (file)
@@ -344,6 +344,10 @@ class CRM_Dedupe_BAO_RuleGroup extends CRM_Dedupe_DAO_RuleGroup {
    * default is to always check permissioning but public pages for example might not want
    * permission to be checked for anonymous users. Refer CRM-6211. We might be beaking
    * Multi-Site dedupe for public pages.
+   *
+   * @param bool $checkPermission
+   *
+   * @return string
    */
   public function thresholdQuery($checkPermission = TRUE) {
     $this->_aclFrom = '';
index 15a2869043818d6e164d2cd9117dd9b5f1c75489..cd8a27bb0b1458a1268b64df929ee0a6a82ee724 100644 (file)
@@ -1032,6 +1032,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
    */
   public static function isMonetary($id) {
     static $isMonetary = array();
index b5bb98351e36d7c65cf136261b7c12e31577d170..e0ab8d5f1487e94c0fc9544ad56090962c7f52a2 100644 (file)
@@ -170,7 +170,9 @@ class CRM_Event_Badge {
   }
 
   /**
-   * This is supposed to be overrided.
+   * This is supposed to be overridden.
+   *
+   * @param array $participant
    */
   public function generateLabel($participant) {
     $txt = "{$this->event['title']}
index cc77b99cdee7fff1169ceded1dd92b8f72767717..6b805ab25db15d82c21754f2cadc3a59f3ca1146 100644 (file)
@@ -153,7 +153,7 @@ class CRM_Grant_Form_Task extends CRM_Core_Form {
    * @param string $nextType
    * @param string $backType
    *
-   * @return void
+   * @param bool $submitOnce
    */
   public function addDefaultButtons($title, $nextType = 'next', $backType = 'back', $submitOnce = FALSE) {
     $this->addButtons(array(
index b36635eef5c20cc87db0ca3b8240a7dd043971f2..0725ae270b14efc2adb662a76ffade4321a0a336 100644 (file)
@@ -159,6 +159,10 @@ AND    TABLE_NAME LIKE 'log_civicrm_%'
 
   /**
    * Return custom data tables for specified entity / extends.
+   *
+   * @param string $extends
+   *
+   * @return array
    */
   public function entityCustomDataLogTables($extends) {
     $customGroupTables = array();
@@ -409,6 +413,11 @@ AND    TABLE_NAME LIKE 'log_civicrm_%'
 
   /**
    * Get an array of column names of the given table.
+   *
+   * @param string$table
+   * @param bool $force
+   *
+   * @return array
    */
   private function columnsOf($table, $force = FALSE) {
     static $columnsOf = array();
@@ -432,6 +441,10 @@ AND    TABLE_NAME LIKE 'log_civicrm_%'
 
   /**
    * Get an array of columns and their details like DATA_TYPE, IS_NULLABLE, COLUMN_DEFAULT for the given table.
+   *
+   * @param string $table
+   *
+   * @return array
    */
   private function columnSpecsOf($table) {
     static $columnSpecs = array(), $civiDB = NULL;
@@ -534,6 +547,8 @@ WHERE  table_schema IN ('{$this->db}', '{$civiDB}')";
 
   /**
    * Create a log table with schema mirroring the given table’s structure and seeding it with the given table’s contents.
+   *
+   * @param string $table
    */
   private function createLogTableFor($table) {
     $dao = CRM_Core_DAO::executeQuery("SHOW CREATE TABLE $table", CRM_Core_DAO::$_nullArray, TRUE, NULL, FALSE, FALSE);
index 18713e1528c74fe682497ebf0e26b0d508b71468..9bb9f496bbc11a1941a2eb2af969f156ad8a5596 100644 (file)
@@ -290,7 +290,7 @@ class CRM_Member_Selector_Search extends CRM_Core_Selector_Base implements CRM_C
   /**
    * Returns total number of rows for the query.
    *
-   * @param
+   * @param int $action.
    *
    * @return int
    *   Total number of rows
index d9af00c1a424b03a9f79f8164271b3aa34d9af26..5d96048138640094411a7b0aca3c452fce2f098d 100644 (file)
@@ -458,6 +458,10 @@ class CRM_Pledge_BAO_Query {
 
   /**
    * This includes any extra fields that might need for export etc.
+   *
+   * @param string $mode
+   *
+   * @return array|null
    */
   public static function extraReturnProperties($mode) {
     $properties = NULL;
index 98fea33b52d2d9cb52b02f69f01ed67bfbbb9660..0e8dccce32d53fcd6a03799e90a927433d196deb 100644 (file)
@@ -475,6 +475,7 @@ AND li.entity_id = {$entityId}
 
   /**
    * Build line items array.
+   *
    * @param array $params
    *   Form values.
    *
@@ -484,7 +485,7 @@ AND li.entity_id = {$entityId}
    * @param string $entityTable
    *   Entity Table.
    *
-   * @return void
+   * @param bool $isRelatedID
    */
   public static function getLineItemArray(&$params, $entityId = NULL, $entityTable = 'contribution', $isRelatedID = FALSE) {
 
index 6d222c3a2ca31511d368b912594b5347d72c2b14..565ee4b9583d3d114f3ee49ead484d9d498d1dc8 100644 (file)
@@ -438,6 +438,8 @@ class CRM_Profile_Selector_Listings extends CRM_Core_Selector_Base implements CR
    * @param string $output
    *   What should the result set include (web/email/csv).
    *
+   * @param string $extraWhereClause
+   *
    * @return int
    *   the total number of rows for this action
    */
@@ -743,8 +745,11 @@ class CRM_Profile_Selector_Listings extends CRM_Core_Selector_Base implements CR
   }
 
   /**
-   *  set the _multiRecordTableName to display the result set.
-   *  according to multi record custom field values
+   *  Set the _multiRecordTableName to display the result set.
+   *
+   *  (according to multi record custom field values).
+   *
+   * @param array $fields
    */
   public function setMultiRecordTableName($fields) {
     $customGroupId = $multiRecordTableName = NULL;
@@ -772,7 +777,7 @@ class CRM_Profile_Selector_Listings extends CRM_Core_Selector_Base implements CR
             $this->_multiRecordTableName
               = $multiRecordTableName = CRM_Core_DAO::getFieldValue('CRM_Core_DAO_CustomGroup', $customGroupId, 'table_name');
             if ($multiRecordTableName) {
-              return NULL;
+              return;
             }
           }
 
@@ -784,7 +789,7 @@ class CRM_Profile_Selector_Listings extends CRM_Core_Selector_Base implements CR
     }
 
     if (!isset($customGroupId) || !$customGroupId) {
-      return NULL;
+      return;
     }
 
     //if the field is in selector and not a searchable field
index 4999acc9114c299e5960c48d4eacecbbb27056be..2337edb9599ede3cf14db39b21c1963ca3fb1c42 100644 (file)
@@ -235,6 +235,11 @@ WHERE  inst.report_id = %1";
   /**
    * Utility function for export2csv and CRM_Report_Form::endPostProcess
    * - make CSV file content and return as string.
+   *
+   * @param CRM_Core_Form $form
+   * @param array $rows
+   *
+   * @return string
    */
   public static function makeCsv(&$form, &$rows) {
     $config = CRM_Core_Config::singleton();
index 53769370e5c01b99fae6a5e7bd25a4ea7f36e58d..8811da71c01fc5e09f89d8d13f06a28a4548982d 100644 (file)
@@ -377,6 +377,11 @@ HAVING COUNT(cpse.price_set_id) > 1 AND MIN(cpse1.id) <> cpse.id ";
    * (Queue Task Callback)
    *
    * Upgrade code to create priceset for contribution pages and events
+   *
+   * @param \CRM_Queue_TaskContext $ctx
+   * @param string $rev
+   *
+   * @return bool
    */
   public static function task_4_2_alpha1_createPriceSets(CRM_Queue_TaskContext $ctx, $rev) {
     $upgrade = new CRM_Upgrade_Form();
@@ -428,8 +433,11 @@ WHERE     cpse.price_set_id IS NULL";
   }
 
   /**
+   * Create price sets.
    *
-   * create price sets
+   * @param string $daoName
+   * @param string $addTo
+   * @param array $options
    */
   public static function createPriceSet($daoName, $addTo, $options = array()) {
     $query = "SELECT title FROM {$addTo[0]} where id =%1";
@@ -781,6 +789,10 @@ AND       cli.entity_id IS NULL AND cp.fee_amount IS NOT NULL";
    * (Queue Task Callback)
    *
    * Create an event registration profile with a single email field CRM-9587
+   *
+   * @param \CRM_Queue_TaskContext $ctx
+   *
+   * @return bool
    */
   public static function task_4_2_alpha1_eventProfile(CRM_Queue_TaskContext $ctx) {
     $upgrade = new CRM_Upgrade_Form();
index cc8167006a2a714aae33b10472869b5161ec7d63..51c1ae45f952bbd2dc57c23a0204e4652dfb643d 100644 (file)
@@ -340,6 +340,8 @@ class CRM_Upgrade_Snapshot_V4p2_Price_DAO_Field extends CRM_Core_DAO {
   /**
    * returns the list of fields that can be imported.
    *
+   * @param bool $prefix
+   *
    * @return array
    */
   static function &import($prefix = FALSE) {
index df113703401f59b4d2151ab1a02e2c613f60fde1..90807e0a97e99d90aa2363cd1a2dfbf0383b197c 100644 (file)
@@ -306,7 +306,9 @@ class CRM_Upgrade_Snapshot_V4p2_Price_DAO_FieldValue 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 221e1fda28b314452c8a71b2f9764c8532277fe8..a6767500da486146106377053872c7e4c2894157 100644 (file)
@@ -259,6 +259,8 @@ class CRM_Upgrade_Snapshot_V4p2_Price_DAO_LineItem extends CRM_Core_DAO {
   /**
    * returns the list of fields that can be imported.
    *
+   * @param bool $prefix
+   *
    * @return array
    */
   static function &import($prefix = FALSE) {
@@ -280,7 +282,9 @@ class CRM_Upgrade_Snapshot_V4p2_Price_DAO_LineItem 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 3a169ccae522e2b5f6fb5f10022abce45b761aa6..786feb93196898c380a9007d0293df2304b65bd4 100644 (file)
@@ -281,7 +281,9 @@ class CRM_Upgrade_Snapshot_V4p2_Price_DAO_Set extends CRM_Core_DAO {
   }
 
   /**
-   * returns the list of fields that can be imported.
+   * Returns the list of fields that can be imported.
+   *
+   * @param bool $prefix
    *
    * @return array
    */
index e4b6eaee98b58f96087fec96977038b68826e8e5..7cf6624dcda7cdc48986baef9ea5d1ddcd801fa2 100644 (file)
@@ -182,7 +182,9 @@ class CRM_Upgrade_Snapshot_V4p2_Price_DAO_SetEntity extends CRM_Core_DAO {
   }
 
   /**
-   * returns the list of fields that can be imported.
+   * Returns the list of fields that can be imported.
+   *
+   * @param bool $prefix
    *
    * @return array
    */
index b72f71ca65e13e9e16f1a8852a531c0a83a02fd9..f1b5cd281be5830097074595dc7b9ceadb0306f6 100644 (file)
@@ -232,8 +232,11 @@ class CRM_Utils_Check {
   /**
    * Evaluate if a system check should be hushed/snoozed.
    *
+   * @param string $message
+   *
    * @return bool
    *   TRUE means hush/snooze, FALSE means display.
+   * @throws \CiviCRM_API3_Exception
    */
   public static function checkHushSnooze($message) {
     $statusPreferenceParams = array(
index c1da326b6084a675b7257fad0c6117b85ed4ecbc..2f01ac8e7ef22f68fcb1f8e8a3ca5a25dad2c6d8 100644 (file)
@@ -605,7 +605,11 @@ function _civicrm_api3_deprecated_formatted_param($params, &$values, $create = F
 }
 
 /**
- * check duplicate contacts based on de-deupe parameters
+ * Check duplicate contacts based on de-dupe parameters.
+ *
+ * @param array $params
+ *
+ * @return array
  */
 function _civicrm_api3_deprecated_check_contact_dedupe($params) {
   static $cIndieFields = NULL;
index 5404a8f6adf171cd1046ebc74fe93ae0c6009e8d..f869d7604ef9f592331d2a0c10844ff7f0875989 100644 (file)
@@ -271,7 +271,7 @@ class CRM_Utils_File {
    */
   public static function addTrailingSlash($path, $slash = NULL) {
     if (!$slash) {
-      // FIXME: Defaulting to backslash on windows systems can produce 
+      // FIXME: Defaulting to backslash on windows systems can produce
       // unexpected results, esp for URL strings which should always use forward-slashes.
       // I think this fn should default to forward-slash instead.
       $slash = DIRECTORY_SEPARATOR;
@@ -381,8 +381,11 @@ class CRM_Utils_File {
   }
 
   /**
-   * Remove the 32 bit md5 we add to the fileName
-   * also remove the unknown tag if we added it
+   * Remove the 32 bit md5 we add to the fileName also remove the unknown tag if we added it.
+   *
+   * @param $name
+   *
+   * @return mixed
    */
   public static function cleanFileName($name) {
     // replace the last 33 character before the '.' with null
@@ -520,6 +523,8 @@ HTACCESS;
   /**
    * Determine if a path is absolute.
    *
+   * @param string $path
+   *
    * @return bool
    *   TRUE if absolute. FALSE if relative.
    */
index 5b3ddb821949fe6e53a20fb75b587e899e4b67f9..5550706803f93daea5e166203df4eddc0aedcb8f 100644 (file)
@@ -407,7 +407,15 @@ class CRM_Utils_Mail_Incoming {
 
   /**
    * Retrieve a contact ID and if not present.
-   * create one with this email
+   *
+   * Create one with this email
+   *
+   * @param string $email
+   * @param string $name
+   * @param bool $create
+   * @param string $mail
+   *
+   * @return int|null
    */
   public static function getContactID($email, $name = NULL, $create = TRUE, &$mail) {
     $dao = CRM_Contact_BAO_Contact::matchContactOnEmail($email, 'Individual');
index 12bcb6280fc3ab633250ee37bca6225a8f2f4f24..ff361257a1619df6ac08ab4f07852353c2efdc9f 100644 (file)
@@ -569,8 +569,10 @@ class CRM_Utils_Rule {
    * See how file rules are written in HTML/QuickForm/file.php
    * Checks to make sure the uploaded file is ascii
    *
+   * @param string $elementValue
+   *
    * @return bool
-   *   true if file has been uploaded, false otherwise
+   *   True if file has been uploaded, false otherwise
    */
   public static function asciiFile($elementValue) {
     if ((isset($elementValue['error']) && $elementValue['error'] == 0) ||
@@ -584,8 +586,10 @@ class CRM_Utils_Rule {
   /**
    * Checks to make sure the uploaded file is in UTF-8, recodes if it's not
    *
+   * @param array $elementValue
+   *
    * @return bool
-   *   whether file has been uploaded properly and is now in UTF-8
+   *   Whether file has been uploaded properly and is now in UTF-8.
    */
   public static function utf8File($elementValue) {
     $success = FALSE;
@@ -612,8 +616,10 @@ class CRM_Utils_Rule {
    * See how file rules are written in HTML/QuickForm/file.php
    * Checks to make sure the uploaded file is html
    *
+   * @param array $elementValue
+   *
    * @return bool
-   *   true if file has been uploaded, false otherwise
+   *   True if file has been uploaded, false otherwise
    */
   public static function htmlFile($elementValue) {
     if ((isset($elementValue['error']) && $elementValue['error'] == 0) ||
index f98b0a2ebe48c9d362310cf18d085e8dddd52b49..497a625219da2c633735c6da8389e7ef0a2e292c 100644 (file)
@@ -559,6 +559,11 @@ AND    u.status = 1
   }
 
   /**
+   * Get CMS root path.
+   *
+   * @param string $scriptFilename
+   *
+   * @return null|string
    */
   public function cmsRootPath($scriptFilename = NULL) {
     $cmsRoot = $valid = NULL;
index 60da7ae2c4e4c12d8bd37ec9776b38619fc381a8..c11b66b2b490eaa2baf95b12db05fae0a12d9a33 100644 (file)
@@ -509,6 +509,11 @@ class CRM_Utils_System_Drupal6 extends CRM_Utils_System_DrupalBase {
   }
 
   /**
+   * Get CMS root path.
+   *
+   * @param string $scriptFilename
+   *
+   * @return null|string
    */
   public function cmsRootPath($scriptFilename = NULL) {
     $cmsRoot = $valid = NULL;
index ed9ea9099a892fff1c5dd3de3810651efb4161f2..fce73f6113382e6d484e38d6c4a2a971a9c39a58 100644 (file)
@@ -278,6 +278,8 @@ abstract class CRM_Utils_System_DrupalBase extends CRM_Utils_System_Base {
 
   /**
    * Append Drupal js to coreResourcesList.
+   *
+   * @param array $list
    */
   public function appendCoreResources(&$list) {
     $list[] = 'js/crm.drupal.js';
index 8e6917e23b4e4d784d462aef92c098b0f1b22d4e..46b4b54bf10fdc63841c26bc8891bea11b6874b2 100644 (file)
@@ -424,12 +424,18 @@ class CRM_Utils_System_Joomla extends CRM_Utils_System_Base {
 
   /**
    * FIXME: Do something
+   *
+   * @param string $message
    */
   public function setMessage($message) {
   }
 
   /**
    * FIXME: Do something
+   *
+   * @param \obj $user
+   *
+   * @return bool
    */
   public function loadUser($user) {
     return TRUE;
index 3ca0033d12f34493465371f9a8a6d0fa35f6d8fe..dea048da3bd14799fdbb00835d587dfd111e0997 100644 (file)
@@ -301,12 +301,18 @@ class CRM_Utils_System_WordPress extends CRM_Utils_System_Base {
 
   /**
    * FIXME: Do something
+   *
+   * @param string $message
    */
   public function setMessage($message) {
   }
 
   /**
    * FIXME: Do something
+   *
+   * @param \obj $user
+   *
+   * @return bool
    */
   public function loadUser($user) {
     return TRUE;
index db5b3d10f980d3ee34ba26f801aab7f487c17872..4f5922531f509dbac7386ddbcf1f6e9d6704842a 100644 (file)
@@ -75,10 +75,13 @@ function civicrm_api3_uf_join_get($params) {
 }
 
 /**
- * Delete a CiviCRM UF_Join
+ * Delete a CiviCRM UF_Join.
  *
  * @param array $params
  *   Array of name/value pairs.
+ *
+ * @return array
+ *   API result array.
  */
 function civicrm_api3_uf_join_delete($params) {
   return _civicrm_api3_basic_delete('CRM_Core_BAO_UFJoin', $params);
index 2f7774cba999badf8d8a6bc57facc5f3d582275b..5835e95758b237820a2ca84f8e1cc7cb4bb2cc39 100644 (file)
@@ -188,6 +188,12 @@ abstract class AbstractMappingTest extends \CiviUnitTestCase {
    * Execute the default schedule, without any special recipient selections.
    *
    * @dataProvider createTestCases
+   *
+   * @param string $targetDate
+   * @param string $setupFuncs
+   * @param array $expectMessages
+   *
+   * @throws \Exception
    */
   public function testDefault($targetDate, $setupFuncs, $expectMessages) {
     $this->targetDate = $targetDate;
index eb27b5909a75615cb6538cec042a7384a2a6ff8a..33b2bfa2a70ea63394033fb8d6ae5abc06663be9 100644 (file)
@@ -2290,8 +2290,9 @@ class CiviSeleniumTestCase extends PHPUnit_Extensions_SeleniumTestCase {
 
   /**
    * Type and select first occurrence of autocomplete.
+   *
    * @param $fieldName
-   * @param $label
+   * @param string $labels
    * @param bool $multiple
    * @param bool $xpath
    */
index dd95ae263e4c54d9e16b75efc794f07439377514..3fde130f28d622607276dc3b0ce8faab23970544 100755 (executable)
@@ -3028,6 +3028,8 @@ AND    ( TABLE_NAME LIKE 'civicrm_value_%' )
    *  $this->createLoggedInUser();
    *   $this->_permissionedDisabledGroup = $this->groupCreate(array('title' => 'pick-me-disabled', 'is_active' => 0, 'name' => 'pick-me-disabled'));
    *   $this->_permissionedGroup = $this->groupCreate(array('title' => 'pick-me-active', 'is_active' => 1, 'name' => 'pick-me-active'));
+   *
+   * @param bool $isProfile
    */
   public function setupACL($isProfile = FALSE) {
     global $_REQUEST;
index bf330f09d793f5a4dbf7b86a1745b15ce8b30141..be9845a633799315be120ebd0ceb8ba985c9c084 100644 (file)
@@ -965,7 +965,9 @@ WHERE ceft.entity_id = %1 AND ceft.entity_table = 'civicrm_contribution'";
   }
 
   /**
-   * @param $status
+   * Test enabling participant statuses.
+   *
+   * @param int $statusId
    */
   public function _testEnableParticipantStatuses($statusId) {
     // enable participant status
index d715a75b8100093afb0597ee41e40edcc0e929bb..8d57aa977527a81d4c7da080c18df64974dbb3af 100644 (file)
@@ -200,13 +200,17 @@ class WebTest_Member_OnlineMembershipCreateTest extends CiviSeleniumTestCase {
   }
 
   /**
+   * Test online membership signup.
+   *
    * @param int $pageId
    * @param int $memTypeId
    * @param string $firstName
    * @param string $lastName
-   * @param $payLater
-   * @param $hash
+   * @param bool $payLater
+   * @param string $hash
    * @param bool $otherAmount
+   * @param bool $amountSection
+   * @param bool $freeMembership
    */
   public function _testOnlineMembershipSignup($pageId, $memTypeId, $firstName, $lastName, $payLater, $hash, $otherAmount = FALSE, $amountSection = TRUE, $freeMembership = FALSE) {
     //Open Live Contribution Page