a few random comment fixes
authorEileen McNaughton <eileen@fuzion.co.nz>
Mon, 12 Jan 2015 22:49:22 +0000 (11:49 +1300)
committerEileen McNaughton <eileen@fuzion.co.nz>
Mon, 12 Jan 2015 22:49:22 +0000 (11:49 +1300)
36 files changed:
CRM/Activity/Controller/Search.php
CRM/Activity/Form/Activity.php
CRM/Activity/Import/Controller.php
CRM/Activity/Import/Field.php
CRM/Activity/Import/Parser/Activity.php
CRM/Activity/StateMachine/Search.php
CRM/Campaign/Form/Petition.php
CRM/Report/Form/Contribute/History.php
api/class.api.php
api/v3/ActionSchedule.php
api/v3/CaseType.php
api/v3/Contact.php
api/v3/Contribution.php
api/v3/CustomField.php
api/v3/CustomGroup.php
api/v3/DashboardContact.php
api/v3/Domain.php
api/v3/Entity.php
api/v3/EntityTag.php
api/v3/Event.php
api/v3/File.php
api/v3/GroupContact.php
api/v3/Mailing.php
api/v3/MailingAB.php
api/v3/MailingJob.php
api/v3/MessageTemplate.php
api/v3/Participant.php
api/v3/PaymentProcessorType.php
api/v3/Pledge.php
api/v3/Premium.php
api/v3/RelationshipType.php
api/v3/Setting.php
api/v3/SurveyRespondant.php
api/v3/UFField.php
api/v3/WordReplacement.php
api/v3/utils.php

index 43f515369d893d4bb75948a08f49ef75c655f11b..ca5f8cb3fcb0c6568590fae46eb1b3c8bed0aad1 100644 (file)
@@ -47,6 +47,9 @@ class CRM_Activity_Controller_Search extends CRM_Core_Controller {
 
   /**
    * Class constructor
+   * @param null $title
+   * @param bool $modal
+   * @param int|mixed|null $action
    */
   public function __construct($title = NULL, $modal = TRUE, $action = CRM_Core_Action::NONE) {
 
index b10b90cb23fd463f194e86e7c3f6b22bd73d29fc..1a9c9ce3458cd27a7b412449bd0166dd1548a698 100644 (file)
@@ -1148,6 +1148,8 @@ class CRM_Activity_Form_Activity extends CRM_Contact_Form_Task {
 
   /**
    * Shorthand for getting id by display name (makes code more readable)
+   * @param $displayName
+   * @return null|string
    */
   protected function _getIdByDisplayName($displayName) {
     return CRM_Core_DAO::getFieldValue('CRM_Contact_DAO_Contact',
@@ -1159,6 +1161,8 @@ class CRM_Activity_Form_Activity extends CRM_Contact_Form_Task {
 
   /**
    * Shorthand for getting display name by id (makes code more readable)
+   * @param $id
+   * @return null|string
    */
   protected function _getDisplayNameById($id) {
     return CRM_Core_DAO::getFieldValue('CRM_Contact_DAO_Contact',
index e838df67b5b73d0f2a150d25991aed40709dedb2..dbec3aa18d2c726022d469a857ce73e27220c4b9 100644 (file)
@@ -36,6 +36,9 @@ class CRM_Activity_Import_Controller extends CRM_Core_Controller {
 
   /**
    * 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 b865151ab4254b5d4d8532510b7296df56a78427..532b667f772f56c52c98c4576c1389b251b306d4 100644 (file)
@@ -108,6 +108,7 @@ class CRM_Activity_Import_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 $value
    */
   public function setValue($value) {
     $this->_value = $value;
index da723f0ec1262beb5e7d93677be456bc52537cba..bf077530dceb78199c970fc0d93915e842702d5c 100644 (file)
@@ -56,6 +56,9 @@ class CRM_Activity_Import_Parser_Activity extends CRM_Activity_Import_Parser {
 
   /**
    * Class constructor
+   * @param $mapperKeys
+   * @param null $mapperLocType
+   * @param null $mapperPhoneType
    */
   public function __construct(&$mapperKeys, $mapperLocType = NULL, $mapperPhoneType = NULL) {
     parent::__construct();
index d9713b8c29ca411544365258522d2c4073c11c3b..3525c173569a1ad8bcc55bbbaa8b28c5cb940108 100644 (file)
@@ -42,6 +42,8 @@ class CRM_Activity_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 86ee1ccda2797998fc5c5724a622523850397a91..973b6e29b9ae5694fdabd0f3c20c7caf3ba3a3b9 100644 (file)
@@ -253,6 +253,10 @@ class CRM_Campaign_Form_Petition extends CRM_Core_Form {
 
   /**
    * Global validation rules for the form
+   * @param $fields
+   * @param $files
+   * @param $form
+   * @return array|bool
    */
   public static function formRule($fields, $files, $form) {
     $errors = array();
index fe8ee105af890a3780e9a1d4f68b2d2df0224335..9ae238833fc25203ab98dde10ed3886ded362c23 100644 (file)
@@ -519,6 +519,7 @@ class CRM_Report_Form_Contribute_History extends CRM_Report_Form {
   }
 
   /**
+   * @param $sql
    * @param $rows
    */
   public function buildRows($sql, &$rows) {
index 7112996141c197594ec90edccb20bd48b5b620cc..8b2c2145ee00b7b2183e25320c4d8cf6aa1a7618 100644 (file)
@@ -134,6 +134,9 @@ class civicrm_api3 {
 
   /**
    * Perform action
+   * @param $action
+   * @param $params
+   * @return bool
    */
   public function __call($action, $params) {
     // @TODO Check if it's a valid action.
@@ -147,6 +150,8 @@ class civicrm_api3 {
 
   /**
    *  As of PHP 5.3.0
+   * @param $name
+   * @param $arguments
    */
   public static function __callStatic($name, $arguments) {
     // Should we implement it ?
@@ -155,7 +160,10 @@ class civicrm_api3 {
 
   /**
    * Call via rest
-   * @return stdClass
+   * @param $entity
+   * @param $action
+   * @param array $params
+   * @return \stdClass
    */
   function remoteCall($entity, $action, $params = array()) {
     $fields = "key={$this->key}&api_key={$this->api_key}";
@@ -268,6 +276,8 @@ class civicrm_api3 {
   }
 
   /**
+   * @param $name
+   * @param null $value
    * @return $this
    */
   public function attr($name, $value = NULL) {
@@ -298,6 +308,7 @@ class civicrm_api3 {
   }
 
   /**
+   * @param $name
    * @return $this
    */
   public function __get($name) {
index 2f5c4c929bae8fa23a84ba2e3ad02bfc391a23e3..f9e6dc80abddc63a0161e7784d8250d41a4d240b 100644 (file)
@@ -38,7 +38,8 @@
 /**
  * Get CiviCRM Action Schedule details
  * {@getfields action_schedule_create}
- *
+ * @param $params
+ * @return array
  */
 function civicrm_api3_action_schedule_get($params) {
   return _civicrm_api3_basic_get(_civicrm_api3_get_BAO(__FUNCTION__), $params, 'action_schedule');
index b24b7be45b63e86368a02d97cea11c26b4f82dcc..3d6508c7d958f1e9bef248c517e186f88c028273 100644 (file)
@@ -90,8 +90,9 @@ function civicrm_api3_case_type_get($params) {
 /**
  * format definition
  *
- * @param $caseTypes
+ * @param array $result
  * @return mixed
+ * @throws \CRM_Core_Exception
  */
 function _civicrm_api3_case_type_get_formatResult(&$result) {
   foreach ($result['values'] as $key => $caseType) {
index 0bdcffdeaf117fe1593b24ed5b635bd566bfb74a..7029bc5f716452f1bd80f9b85b4a99609f903bb7 100644 (file)
@@ -560,7 +560,9 @@ function _civicrm_api3_greeting_format_params($params) {
  * @deprecated
  *
  * {@example ContactGetquick.php 0}
- *
+ * @param $params
+ * @return array
+ * @throws \API_Exception
  */
 function civicrm_api3_contact_getquick($params) {
   civicrm_api3_verify_mandatory($params, NULL, array('name'));
index cdaa8f056ae043cc5e33118ba05c7b2f704bae32..fc73fe82d476ca7a911682dc226e6f90097937bc 100644 (file)
@@ -151,6 +151,7 @@ function _civicrm_api3_contribution_create_spec(&$params) {
  * The main purpose of the API is to provide integrators a level of stability not provided by
  * the core code or schema - this means we have to provide support for api calls (where possible)
  * across schema changes.
+ * @param $params
  */
 function _civicrm_api3_contribution_create_legacy_support_45(&$params) {
   //legacy soft credit handling - recommended approach is chaining
@@ -196,6 +197,7 @@ function civicrm_api3_contribution_delete($params) {
 
 /**
  * modify metadata. Legacy support for contribution_id
+ * @param $params
  */
 function _civicrm_api3_contribution_delete_spec(&$params) {
   $params['id']['api.aliases'] = array('contribution_id');
@@ -239,6 +241,7 @@ function civicrm_api3_contribution_get($params) {
  * This function is used to format the soft credit for backward compatibility
  * as of v4.4 we support multiple soft credit, so now contribution returns array with 'soft_credit' as key
  * but we still return first soft credit as a part of contribution array
+ * @param $contribution
  */
 function _civicrm_api3_format_soft_credit(&$contribution) {
   if (!empty($contribution['soft_credit'])) {
index 86e507998a7bce01581823c5527e66424abe147e..13bd9d4dc70825084a4a51341cb2dd2981463b7e 100644 (file)
@@ -262,6 +262,8 @@ SELECT count(*)
 
 /**
  * CRM-15191 - Hack to ensure the cache gets cleared after updating a custom field
+ * @param $params
+ * @return array
  */
 function civicrm_api3_custom_field_setvalue($params) {
   require_once 'api/v3/Generic/Setvalue.php';
index 011d821c514408687afeef25a731de90021e28f7..e089ed30a1fee4e08b6a39e5bea330927e43d069 100644 (file)
@@ -131,6 +131,8 @@ function civicrm_api3_custom_group_get($params) {
 
 /**
  * CRM-15191 - Hack to ensure the cache gets cleared after updating a custom group
+ * @param $params
+ * @return array
  */
 function civicrm_api3_custom_group_setvalue($params) {
   require_once 'api/v3/Generic/Setvalue.php';
index da1b0fbe5cdb8a5593956535c20ffb58b0c90a65..bb3b1f5b390211474164203653eceabce0492405 100644 (file)
@@ -108,6 +108,9 @@ function _civicrm_api3_dashboard_contact_check_params(&$params) {
  *
  * {@getfields dashboard_contact_delete}
  * @access public
+ * @param $params
+ * @return array
+ * @throws \API_Exception
  */
 function civicrm_api3_dashboard_contact_delete($params) {
   return _civicrm_api3_basic_delete(_civicrm_api3_get_BAO(__FUNCTION__), $params);
index 6265eace6cb43e6a6a30b4b60f0f411a407f2b9a..5c74b5c929edc447ada1904056d1aab05509a561 100644 (file)
@@ -40,6 +40,9 @@
  * Get CiviCRM domain details
  * {@getfields domain_create}
  * @example DomainGet.php
+ * @param $params
+ * @return array
+ * @throws \API_Exception
  */
 function civicrm_api3_domain_get($params) {
 
index 1346b66aa09c058ed7c1dfa12ac0781da5f00bd4..5b70b6e5ee2e26d7a6d5ea41621ed8c1c5547993 100644 (file)
@@ -20,6 +20,8 @@ function _civicrm_api3_entity_deprecation($entities) {
 
 /**
  *  Placeholder function. This should never be called, as it doesn't have any meaning
+ * @param $params
+ * @return array
  */
 function civicrm_api3_entity_create($params) {
   return civicrm_api3_create_error("API (Entity, Create) does not exist Creating a new entity means modifying the source code of civiCRM.");
@@ -27,6 +29,8 @@ function civicrm_api3_entity_create($params) {
 
 /**
  *  Placeholder function. This should never be called, as it doesn't have any meaning
+ * @param $params
+ * @return array
  */
 function civicrm_api3_entity_delete($params) {
   return civicrm_api3_create_error("API (Entity, Delete) does not exist Deleting an entity means modifying the source code of civiCRM.");
@@ -34,6 +38,8 @@ function civicrm_api3_entity_delete($params) {
 
 /**
  *  Placeholder function. This should never be called, as it doesn't have any meaning
+ * @param $params
+ * @return array
  */
 function civicrm_api3_entity_getfields($params) {
   // we return an empty array so it makes it easier to write generic getdefaults / required tests
index ec5794da16070bf292c4b7e2a2db8c9f2d28223a..8aab9012569602f360d8b90eabea1ee77dca703c 100644 (file)
@@ -102,6 +102,7 @@ function civicrm_api3_entity_tag_delete($params) {
 
 /**
  * modify metadata
+ * @param $params
  */
 function _civicrm_api3_entity_tag_delete_spec(&$params) {
   // set as not required as tag_id also acceptable & no either/or std yet
index 410e37068f46c513897a34be9296c3bf520c15dc..3a3251bc32d40c8d48dc4451faedf1afe9793d26 100644 (file)
@@ -85,6 +85,7 @@ function _civicrm_api3_event_create_spec(&$params) {
  * The main purpose of the API is to provide integrators a level of stability not provided by
  * the core code or schema - this means we have to provide support for api calls (where possible)
  * across schema changes.
+ * @param $params
  */
 function _civicrm_api3_event_create_legacy_support_42(&$params) {
   if (!empty($params['payment_processor_id'])) {
@@ -171,6 +172,8 @@ function _civicrm_api3_event_get_spec(&$params) {
  * The main purpose of the API is to provide integrators a level of stability not provided by
  * the core code or schema - this means we have to provide support for api calls (where possible)
  * across schema changes.
+ * @param $event
+ * @param $event_id
  */
 function _civicrm_api3_event_get_legacy_support_42(&$event, $event_id) {
   if (!empty($event[$event_id]['payment_processor'])) {
@@ -186,17 +189,14 @@ function _civicrm_api3_event_get_legacy_support_42(&$event, $event_id) {
  *
  * This API is used for deleting a event
  *
- * @param array $paramsArray containing event_id to be deleted.
- *   Array containing event_id to be deleted.
+ * @param $params
+ * @return array
  *
- * @return boolean
- *   true if success, error otherwise
  * @access public
  *   note API has legacy support for 'event_id'
  *  {@getfields event_delete}
  */
 function civicrm_api3_event_delete($params) {
-
   return CRM_Event_BAO_Event::del($params['id']) ? civicrm_api3_create_success() : civicrm_api3_create_error(ts('Error while deleting event'));
 }
 /*
index cacdb7a46fb9b595e233889c39fa557337252b7a..b0867261e88486b405cd99edd67b3b0b04d36b67 100644 (file)
@@ -100,12 +100,8 @@ function civicrm_api3_file_get($params) {
  * This api is used for updating an existing file.
  * Required parameters : id of a file
  *
- * @param array $paramsAn array of name/value property values of civicrm_file.
- *   An array of name/value property values of civicrm_file.
- *
+ * @param array $params
  * @return array
- *   Array of updated file object property values
- * @access public
  */
 function civicrm_api3_file_update($params) {
 
index fc661239084e226f302b6c20442a5e6c8dab516b..1c01e9d6f32644f91b29f15b89d7a07f62815414 100644 (file)
@@ -139,6 +139,7 @@ function civicrm_api3_group_contact_delete($params) {
 
 /**
  * modify metadata
+ * @param $params
  */
 function _civicrm_api3_group_contact_delete_spec(&$params) {
   // set as not required no either/or std yet
@@ -228,6 +229,9 @@ function _civicrm_api3_group_contact_common($params, $op = 'Added') {
 
 /**
  * @deprecated - this should be part of create but need to know we aren't missing something
+ * @param $params
+ * @return bool
+ * @throws \API_Exception
  */
 function civicrm_api3_group_contact_update_status($params) {
 
index 45102e56958885957a29095836134cea62dbbbff..2a1119c2199c85136f1feb5c5e52932effccd60a 100755 (executable)
@@ -583,6 +583,8 @@ function civicrm_api3_mailing_stats($params) {
  * Fix the reset dates on the email record based on when a mail was last delivered
  * We only consider mailings that were completed and finished in the last 3 to 7 days
  * Both the min and max days can be set via the params
+ * @param $params
+ * @return array
  */
 function civicrm_api3_mailing_update_email_resetdate($params) {
   CRM_Mailing_Event_BAO_Delivered::updateEmailResetDate(
index 9615f081f12e70fbc52fca43e37c6377f8f7b475..bd825f52899280df5867a1424e86fec6a7423c76 100755 (executable)
@@ -40,7 +40,6 @@
  * Handle a create mailing ab testing
  *
  * @param array $params
- * @param array $ids
  *
  * @return array
  *   API Success Array
@@ -53,7 +52,6 @@ function civicrm_api3_mailing_a_b_create($params) {
  * Handle a delete event.
  *
  * @param array $params
- * @param array $ids
  *
  * @return array
  *   API Success Array
index 11f10779ab98ca3a76f7d9b96ea28d84ba47f67b..eeaee7681e74e3bfe8961d21581f757ac85b02ed 100644 (file)
@@ -38,6 +38,9 @@
 
 /**
  * Handle creation of a Mailing Job for a Mailing.
+ * @param $params
+ * @return array
+ * @throws \API_Exception
  */
 function civicrm_api3_mailing_job_create($params) {
   return _civicrm_api3_basic_create(_civicrm_api3_get_BAO(__FUNCTION__), $params);
@@ -74,6 +77,9 @@ function civicrm_api3_mailing_job_get($params, $ids = array()) {
 
 /**
  * Handle deletion of a Mailing Job for a Mailing.
+ * @param $params
+ * @return array
+ * @throws \API_Exception
  */
 function civicrm_api3_mailing_job_delete($params) {
   return _civicrm_api3_basic_delete(_civicrm_api3_get_BAO(__FUNCTION__), $params);
index 6bd144b6e9b08407be91daa5f1dbf5081ecd5faa..d72c1ef8dfbc81d156e7efc32dad964215e549b2 100644 (file)
@@ -33,6 +33,9 @@
 
 /**
  * @access public
+ * @param $params
+ * @return array
+ * @throws \API_Exception
  */
 function civicrm_api3_message_template_create($params) {
   return _civicrm_api3_basic_create(_civicrm_api3_get_BAO(__FUNCTION__), $params);
@@ -94,6 +97,7 @@ function civicrm_api3_message_template_get($params) {
 
 /**
  * Sends a template.
+ * @param $params
  */
 function civicrm_api3_message_template_send($params) {
   CRM_Core_BAO_MessageTemplates::sendTemplate($params);
index e19d028916219f85b6542c0fd8290fb2fae369c0..a1cb21aca2c504e7b1a9e7d2523055bab948a918 100644 (file)
@@ -77,6 +77,9 @@ function civicrm_api3_participant_create($params) {
 /**
  * @todo this should be done in the BAO not the api
  * Create a default participant line item
+ * @param $params
+ * @param $participant
+ * @throws \CiviCRM_API3_Exception
  */
 function _civicrm_api3_participant_createlineitem(&$params, $participant) {
   // it is possible that a fee level contains information about multiple
index 6012a6a07c86fa37f849b9af553fbf68828f8cdb..1cdac8dd31d15dfdee93ca7d5ec5d514fab1f091 100644 (file)
@@ -90,6 +90,8 @@ function _civicrm_api3_payment_processor_type_create_spec(&$params) {
  * @access  public
  * {@getfields PaymentProcessorType_get}
  * @example PaymentProcessorTypeGet.php
+ * @param $params
+ * @return array
  */
 function civicrm_api3_payment_processor_type_get($params) {
   return _civicrm_api3_basic_get(_civicrm_api3_get_BAO(__FUNCTION__), $params);
index cdc79d7cbc8c387bcc045630cec9a58f27e53b15..958f38d30513834ade23606ae83d29ab31658104 100644 (file)
@@ -96,6 +96,7 @@ function _civicrm_api3_pledge_delete_spec(&$params) {
 
 /**
  * return field specification specific to get requests
+ * @param $params
  */
 function _civicrm_api3_pledge_get_spec(&$params) {
   $params['next_pay_date'] = array(
@@ -112,6 +113,7 @@ function _civicrm_api3_pledge_get_spec(&$params) {
 
 /**
  * return field specification specific to get requests
+ * @param $params
  */
 function _civicrm_api3_pledge_create_spec(&$params) {
 
index 46f791561686bb2371f303070cd0f910d09bc5ba..348bab99cd9ff1681eb46559253e30e874940aaf 100644 (file)
@@ -88,6 +88,7 @@ function civicrm_api3_premium_delete($params) {
 
 /**
  * return field specification specific to get requests
+ * @param $params
  */
 function _civicrm_api3_premium_get_spec(&$params) {
   $params['premiums_active']['api.aliases'] = array('is_active');
@@ -95,6 +96,7 @@ function _civicrm_api3_premium_get_spec(&$params) {
 
 /**
  * return field specification specific to create requests
+ * @param $params
  */
 function _civicrm_api3_premium_create_spec(&$params) {
   $params['premiums_active']['api.aliases'] = array('is_active');
index 6d13a07b3d08b087f07d9650bad4d1c5169a702b..aa72f1366598057873fbbafa5742d7eebc572c7f 100644 (file)
@@ -98,6 +98,8 @@ function _civicrm_api3_relationship_type_create_spec(&$params) {
  * @access  public
  * {@getfields RelationshipType_get}
  * @example RelationshipTypeGet.php
+ * @param $params
+ * @return array
  */
 function civicrm_api3_relationship_type_get($params) {
   return _civicrm_api3_basic_get(_civicrm_api3_get_BAO(__FUNCTION__), $params);
index ba35cdd86605b5eb64b8b4a2222ae73a581770d8..0d0d3cdeb3ff8b1b20aadcd45725d89cfa1c1066 100644 (file)
@@ -32,7 +32,8 @@
  * @subpackage API_Settings
  * @copyright CiviCRM LLC (c) 2004-2014
  * @version $Id: Settings.php
- *
+ * @param $params
+ * @return array
  */
 
 function civicrm_api3_setting_getfields($params) {
@@ -73,6 +74,7 @@ function civicrm_api3_setting_getfields($params) {
 
 /**
  * Alter metadata for getfields functions
+ * @param $params
  */
 function _civicrm_api3_setting_getfields_spec(&$params) {
   $params['filters'] = array('title' => 'Fields you wish to filter by e.g. array("group_name" => "CiviCRM Preferences")');
@@ -84,6 +86,10 @@ function _civicrm_api3_setting_getfields_spec(&$params) {
  * Return default values for settings. We will domain key this as it could vary by domain (ie. urls)
  * as we will be creating the option for a function rather than an value to be in the defaults
  * Note that is not in place as yet
+ * @param $params
+ * @return array
+ * @throws \CiviCRM_API3_Exception
+ * @throws \Exception
  */
 function civicrm_api3_setting_getdefaults(&$params) {
   $settings = civicrm_api3('setting', 'getfields', $params);
@@ -124,6 +130,9 @@ function _civicrm_api3_setting_getdefaults_spec(&$params) {
 
 /**
  * Revert settings to defaults
+ * @param $params
+ * @return array
+ * @throws \Exception
  */
 function civicrm_api3_setting_revert(&$params) {
   $defaults = civicrm_api('setting', 'getdefaults', $params);
@@ -146,6 +155,7 @@ function civicrm_api3_setting_revert(&$params) {
 
 /**
  * Alter metadata for getfields functions
+ * @param $params
  */
 function _civicrm_api3_setting_revert_spec(&$params) {
   $params['name'] = array('title' => 'Setting Name belongs to');
@@ -160,6 +170,10 @@ function _civicrm_api3_setting_revert_spec(&$params) {
 
 /**
  * Revert settings to defaults
+ * @param $params
+ * @return array
+ * @throws \CiviCRM_API3_Exception
+ * @throws \Exception
  */
 function civicrm_api3_setting_fill(&$params) {
   $defaults = civicrm_api3('setting', 'getdefaults', $params);
@@ -181,6 +195,7 @@ function civicrm_api3_setting_fill(&$params) {
 
 /**
  * Alter metadata for getfields functions
+ * @param $params
  */
 function _civicrm_api3_setting_fill_spec(&$params) {
   $params['name'] = array('title' => 'Setting Name belongs to');
@@ -331,6 +346,9 @@ function _civicrm_api3_setting_getvalue_spec(&$params) {
  * Really domain_id should always be set but doing an empty check because at the moment
  * using crm-editable will pass an id & default won't be applied
  * we did talk about id being a pseudonym for domain_id in this api so applying it here
+ * @param $params
+ * @return array
+ * @throws \Exception
  */
 function _civicrm_api3_setting_getDomainArray(&$params) {
   if (empty($params['domain_id']) && isset($params['id'])) {
index 23288b93ff54743b2c70a8dbb8da00e14fc374c9..31b900ef3a49ea3093032b628f34123c8d914aff 100644 (file)
@@ -81,6 +81,8 @@ function civicrm_api3_survey_respondant_get(&$params) {
 
 /**
  * @deprecated - api currently not supported
+ * @param $params
+ * @return array
  */
 function &civicrm_api3_survey_respondant_count($params) {
 
index c9bd9027cf1b33cb19808be65be5faf67a51e246..122179e9b2aa9af7a4a33fdf95264dd0f108c4cc 100644 (file)
@@ -174,8 +174,10 @@ function civicrm_api3_uf_field_delete($params) {
 
   return civicrm_api3_create_success($result, $params);
 }
+
 /**
  * field id accepted for backward compatibility - unset required on id
+ * @param $params
  */
 function _civicrm_api3_uf_field_delete_spec(&$params) {
   // legacy support for field_id
index 72e6f82feaac03313a277a88c7dadb2a9377cbb7..d019bd4cb92e14c516db75f2739c5b546d8ff633 100644 (file)
@@ -39,7 +39,9 @@
 /**
  * Get CiviCRM Word Replacement details
  * {@getfields word_replacement_create}
- *
+ * @param $params
+ * @return array
+ * @throws \API_Exception
  */
 function civicrm_api3_word_replacement_get($params) {
   $bao = new CRM_Core_BAO_WordReplacement();
index 087aa74c874d05ff75bac4badbe41d37ae0e510e..53076f73e64bf57f3b0d8f791486b05ed2894105 100644 (file)
@@ -135,11 +135,9 @@ function civicrm_api3_verify_mandatory($params, $daoName = NULL, $keys = array()
 
 /**
  *
+ * @param $msg
  * @param array $data
- *
- * @throws API_Exception
  * @return array
- *   <type>
  */
 function civicrm_api3_create_error($msg, $data = array()) {
   $data['is_error'] = 1;
@@ -272,6 +270,8 @@ function civicrm_api3_create_success($values = 1, $params = array(), $entity = N
 
 /**
  * Load the DAO of the entity
+ * @param $entity
+ * @return bool
  */
 function _civicrm_api3_load_DAO($entity) {
   $dao = _civicrm_api3_get_DAO($entity);
@@ -375,7 +375,7 @@ function _civicrm_api3_get_BAO($name) {
 
 /**
  *  Recursive function to explode value-separated strings into arrays
- *
+ * @param $values
  */
 function _civicrm_api3_separate_values(&$values) {
   $sp = CRM_Core_DAO::VALUE_SEPARATOR;
@@ -825,6 +825,9 @@ function _civicrm_api3_apply_options_to_dao(&$params, &$dao, $entity) {
 /**
  * build fields array. This is the array of fields as it relates to the given DAO
  * returns unique fields as keys by default but if set but can return by DB fields
+ * @param $bao
+ * @param bool $unique
+ * @return
  */
 function _civicrm_api3_build_fields_array(&$bao, $unique = TRUE) {
   $fields = $bao->fields();
@@ -958,6 +961,9 @@ function _civicrm_api3_object_to_array(&$dao, &$values, $uniqueFields = FALSE) {
 
 /**
  * Wrapper for _civicrm_object_to_array when api supports unique fields
+ * @param $dao
+ * @param $values
+ * @return array
  */
 function _civicrm_api3_object_to_array_unique_fields(&$dao, &$values) {
   return _civicrm_api3_object_to_array($dao, $values, TRUE);
@@ -1519,13 +1525,12 @@ function _civicrm_api3_getValidDate($dateValue, $fieldName, $fieldType) {
 /**
  * Validate foreign constraint fields being passed into API.
  *
- * @param array $params
- *   Params from civicrm_api.
+ * @param mixed $fieldValue
  * @param string $fieldName
  *   Uniquename of field being checked.
  * @param array $fieldInfo
  *   Array of fields from getfields function.
- * @throws Exception
+ * @throws \API_Exception
  */
 function _civicrm_api3_validate_constraint(&$fieldValue, &$fieldName, &$fieldInfo) {
   $dao = new $fieldInfo['FKClassName'];
@@ -1533,7 +1538,7 @@ function _civicrm_api3_validate_constraint(&$fieldValue, &$fieldName, &$fieldInf
   $dao->selectAdd();
   $dao->selectAdd('id');
   if (!$dao->find()) {
-    throw new Exception("$fieldName is not valid : " . $fieldValue);
+    throw new API_Exception("$fieldName is not valid : " . $fieldValue);
   }
 }
 
@@ -1713,6 +1718,9 @@ function _civicrm_api_get_fields($entity, $unique = FALSE, &$params = array()) {
 /**
  * Return an array of fields for a given entity - this is the same as the BAO function but
  * fields are prefixed with 'custom_' to represent api params
+ * @param $entity
+ * @param $params
+ * @return array
  */
 function _civicrm_api_get_custom_fields($entity, &$params) {
   $entity = _civicrm_api_get_camel_name($entity);
@@ -1742,8 +1750,11 @@ function _civicrm_api_get_custom_fields($entity, &$params) {
   }
   return $ret;
 }
+
 /**
  * Translate the custom field data_type attribute into a std 'type'
+ * @param $dataType
+ * @return
  */
 function _getStandardTypeFromCustomDataType($dataType) {
   $mapping = array(
@@ -1769,6 +1780,8 @@ function _getStandardTypeFromCustomDataType($dataType) {
  * If multiple aliases the last takes precedence
  *
  * Function also swaps unique fields for non-unique fields & vice versa.
+ * @param $apiRequest
+ * @param $fields
  */
 function _civicrm_api3_swap_out_aliases(&$apiRequest, $fields) {
   foreach ($fields as $field => $values) {
@@ -1972,10 +1985,11 @@ function _civicrm_api3_validate_string(&$params, &$fieldName, &$fieldInfo, $enti
 /**
  * Validate & swap out any pseudoconstants / options
  *
- * @param array $params: api parameters
- * @param string $entity: api entity name
- * @param string $fieldName: field name used in api call (not necessarily the canonical name)
- * @param array $fieldInfo: getfields meta-data
+ * @param mixed $fieldValue
+ * @param string $entity : api entity name
+ * @param string $fieldName : field name used in api call (not necessarily the canonical name)
+ * @param array $fieldInfo : getfields meta-data
+ * @throws \API_Exception
  */
 function _civicrm_api3_api_match_pseudoconstant(&$fieldValue, $entity, $fieldName, $fieldInfo) {
   $options = CRM_Utils_Array::value('options', $fieldInfo);