some comment block fixes
authorEileen McNaughton <eileen@fuzion.co.nz>
Mon, 19 May 2014 02:56:50 +0000 (14:56 +1200)
committerEileen McNaughton <eileen@fuzion.co.nz>
Mon, 19 May 2014 02:56:50 +0000 (14:56 +1200)
20 files changed:
CRM/Core/Form.php
api/Exception.php
api/Wrapper.php
api/api.php
api/class.api.php
api/v3/Constant.php
api/v3/Contact.php
api/v3/Contribution.php
api/v3/DashboardContact.php
api/v3/Event.php
api/v3/Generic/Getactions.php
api/v3/Location.php
api/v3/MailingContact.php
api/v3/MailingGroup.php
api/v3/MessageTemplate.php
api/v3/OptionGroup.php
api/v3/Pledge.php
api/v3/PledgePayment.php
api/v3/Profile.php
api/v3/ReportTemplate.php

index ae09c486bd64850e004db49673199f4529dd4b0a..0cbcedd9905c6333d15597afc460220b9bdb2b18 100644 (file)
@@ -1398,17 +1398,17 @@ class CRM_Core_Form extends HTML_QuickForm_Page {
     }
   }
 
-/**
- * Get contact if for a form object. Prioritise
- *  - cid in URL if 0 (on behalf on someoneelse)
- *      (@todo consider setting a variable if onbehalf for clarity of downstream 'if's
- *  - logged in user id if it matches the one in the cid in the URL
- *  - contact id validated from a checksum from a checksum
- *  - cid from the url if the caller has ACL permission to view
- *  - fallback is logged in user (or ? NULL if no logged in user) (@todo wouldn't 0 be more intuitive?)
- *
- * @return Ambigous <mixed, NULL, value, unknown, array, number>|unknown
- */
+  /**
  * Get contact if for a form object. Prioritise
  *  - cid in URL if 0 (on behalf on someoneelse)
  *      (@todo consider setting a variable if onbehalf for clarity of downstream 'if's
  *  - logged in user id if it matches the one in the cid in the URL
  *  - contact id validated from a checksum from a checksum
  *  - cid from the url if the caller has ACL permission to view
  *  - fallback is logged in user (or ? NULL if no logged in user) (@todo wouldn't 0 be more intuitive?)
  *
+   * @return mixed NULL|integer
  */
   function getContactID() {
     $tempID = CRM_Utils_Request::retrieve('cid', 'Positive', $this);
     if(isset($this->_params) && isset($this->_params['select_contact_id'])) {
index 48d89e7572e38097ac56c65b6ac64a6ffee3ddbc..953fafdbe99e97481b0ca72d98914c9ddcb276a6 100644 (file)
@@ -25,6 +25,13 @@ class API_Exception extends Exception
   const NOT_IMPLEMENTED = 'not-found';
 
   private $extraParams = array();
+
+  /**
+   * @param string $message
+   * @param int $error_code
+   * @param array $extraParams
+   * @param Exception $previous
+   */
   public function __construct($message, $error_code = 0, $extraParams = array(),Exception $previous = null) {
     if (is_numeric ($error_code)) // using int for error code "old way")
       $code = $error_code;
@@ -35,14 +42,23 @@ class API_Exception extends Exception
   }
 
   // custom string representation of object
+  /**
+   * @return string
+   */
   public function __toString() {
     return __CLASS__ . ": [{$this->code}]: {$this->message}\n";
   }
 
+  /**
+   * @return array
+   */
   public function getExtraParams() {
     return $this->extraParams;
   }
 
+  /**
+   * @return array
+   */
   public function getErrorCodes(){
     return array(
         2000 => '$params was not an array',
@@ -64,12 +80,22 @@ class API_Exception extends Exception
 class CiviCRM_API3_Exception extends Exception
 {
   private $extraParams = array();
+
+  /**
+   * @param string $message
+   * @param int $error_code
+   * @param array $extraParams
+   * @param Exception $previous
+   */
   public function __construct($message, $error_code, $extraParams = array(),Exception $previous = null) {
     parent::__construct(ts($message));
     $this->extraParams = $extraParams + array('error_code' => $error_code);
   }
 
   // custom string representation of object
+  /**
+   * @return string
+   */
   public function __toString() {
     return __CLASS__ . ": [{$this->extraParams['error_code']}: {$this->message}\n";
   }
@@ -77,6 +103,10 @@ class CiviCRM_API3_Exception extends Exception
   public function getErrorCode() {
     return $this->extraParams['error_code'];
   }
+
+  /**
+   * @return array
+   */
   public function getExtraParams() {
     return $this->extraParams;
   }
index 73bd8043ecc8370830cb564cb378640c5de0b743..5693e7153ca26fb1ff2018ddd9ac1083f3a660f0 100644 (file)
@@ -1,5 +1,8 @@
 <?php
 
+/**
+ * Interface API_Wrapper
+ */
 interface API_Wrapper {
 
   /**
index 893805895f76a165e60c38612a07500c3179c2ae..c09d220249775dfe0cc439890f1aa65e28676fb1 100644 (file)
@@ -98,6 +98,12 @@ function civicrm_error($result) {
   return FALSE;
 }
 
+/**
+ * @param $entity
+ * @param null $version
+ *
+ * @return string
+ */
 function _civicrm_api_get_camel_name($entity, $version = NULL) {
   $fragments = explode('_', $entity);
   foreach ($fragments as & $fragment) {
index 04032c876961affcc1babd7799c62ff94551bb0a..93dda31ec1b2c266dfa9dd3a47616d6652d09de1 100644 (file)
@@ -198,6 +198,13 @@ class civicrm_api3 {
     return $res;
   }
 
+  /**
+   * @param $entity
+   * @param string $action
+   * @param array $params
+   *
+   * @return bool
+   */
   function call($entity, $action = 'Get', $params = array()) {
     if (is_int($params)) {
       $params = array('id' => $params);
index 27552311b5d4e0139d9c6bd36fe4786710ca2803..f7dcd5e276a161643fe352ac412a6144ea9448a5 100644 (file)
@@ -128,6 +128,9 @@ function civicrm_api3_constant_get($params) {
   return civicrm_api3_create_error('Unknown civicrm constant or method not callable');
 }
 
+/**
+ * @param $params
+ */
 function _civicrm_api3_constant_get_spec(&$params) {
 
   $params = (array
index 438544a5aeb9fd7999ec8de0c38ab234040d8e2f..90729ff875a3d30c7337c04490a0a3ecc4654fe7 100644 (file)
@@ -154,6 +154,11 @@ function civicrm_api3_contact_get($params) {
   return civicrm_api3_create_success($contacts, $params, 'contact');
 }
 
+/**
+ * @param $params
+ *
+ * @return int
+ */
 function civicrm_api3_contact_getcount($params) {
   $options = array();
   _civicrm_api3_contact_get_supportanomalies($params, $options);
@@ -286,6 +291,17 @@ function civicrm_api3_contact_delete($params) {
 }
 
 
+/**
+ * @param $params
+ * @param bool $dupeCheck
+ * @param bool $dupeErrorArray
+ * @param bool $obsoletevalue
+ * @param null $dedupeRuleGroupID
+ *
+ * @return null
+ * @throws API_Exception
+ * @throws CiviCRM_API3_Exception
+ */
 function _civicrm_api3_contact_check_params( &$params, $dupeCheck = true, $dupeErrorArray = false, $obsoletevalue = true, $dedupeRuleGroupID = null )
 {
 
@@ -857,12 +873,21 @@ function civicrm_api3_contact_merge($params) {
   }
 }
 
+/**
+ * @param $params
+ */
 function _civicrm_api3_contact_proximity_spec(&$params) {
   $params['latitude']['api.required'] = 1;
   $params['longitude']['api.required'] = 1;
   $params['unit']['api.default'] = 'meter';
 }
 
+/**
+ * @param $params
+ *
+ * @return array
+ * @throws Exception
+ */
 function civicrm_api3_contact_proximity($params) {
   $latitude  = CRM_Utils_Array::value('latitude', $params);
   $longitude = CRM_Utils_Array::value('longitude', $params);
index c1b967838094cf467096623478db35058cf8eddd..5ef18c90869468810db3b2aeec937bf6253901e4 100644 (file)
@@ -403,6 +403,9 @@ function civicrm_api3_contribution_completetransaction(&$params) {
   }
 }
 
+/**
+ * @param $params
+ */
 function _civicrm_api3_contribution_completetransaction(&$params) {
   $params['id'] = array(
     'title' => 'Contribution ID',
index 0bb8ee893e6e2154aca773863c3fd5cf8372b37b..4df7815f4b84ede66aaedd72925016569729d45f 100644 (file)
@@ -83,6 +83,11 @@ function _civicrm_api3_dashboard_contact_create_spec(&$params) {
   unset($params['version']);
 }
 
+/**
+ * @param $params
+ *
+ * @return array|null
+ */
 function _civicrm_api3_dashboard_contact_check_params(&$params) {
   $dashboard_id = CRM_Utils_Array::value('dashboard_id', $params);
   if ($dashboard_id) {
index 478d28460ed320780a6849698c1fda05a651e579..93eb30c93e6f6bce74697ae57f73aab6eaf916c9 100644 (file)
@@ -220,6 +220,10 @@ function civicrm_api3_event_delete($params) {
  * @param int $event_id Id of the event to be updated
  *
  */
+/**
+ * @param $event
+ * @param $event_id
+ */
 function _civicrm_api3_event_getisfull(&$event, $event_id) {
   $eventFullResult = CRM_Event_BAO_Participant::eventFull($event_id, 1);
   if (!empty($eventFullResult) && is_int($eventFullResult)) {
index 9cfbbd67a95d7235558ccb9459f49d4e3ca550dd..e3520c13a273ae4fa9fe44437f50c8ce044f87e6 100644 (file)
 +--------------------------------------------------------------------+
 */
 
+/**
+ * @param $apiRequest
+ *
+ * @return array
+ * @throws API_Exception
+ */
 function civicrm_api3_generic_getActions($apiRequest) {
   civicrm_api3_verify_mandatory($apiRequest, NULL, array('entity'));
   $mfp = \Civi\Core\Container::singleton()->get('magic_function_provider');
index d9a22ad8ba12f04ae9eff1819cc6d21fc8c008d4..45e81ccadfa4a4b84dccb56f448431554dcea902 100644 (file)
@@ -9,10 +9,20 @@ function civicrm_api3_location_create($params) {
   return civicrm_api3_create_error("API (Location, Create) does not exist, use the Address/Phone/Email/Website API instead", array('obsoleted' => TRUE));
 }
 
+/**
+ * @param $params
+ *
+ * @return array
+ */
 function civicrm_api3_location_get($params) {
   return civicrm_api3_create_error("API (Location, Get) does not exist, use the Address/Phone/Email/Website API instead", array('obsoleted' => TRUE));
 }
 
+/**
+ * @param $params
+ *
+ * @return array
+ */
 function civicrm_api3_location_delete($params) {
   return civicrm_api3_create_error("API (Location, Delete) does not exist, use the Address/Phone/Email/Website API instead", array('obsoleted' => TRUE));
 }
index 6b5765c7a4e0623e1098deb28371baabbfd6cc08..8e342e0c82a852293d4d257ceba3f163dca5f1f6 100644 (file)
@@ -92,6 +92,19 @@ function _civicrm_api3_mailing_contact_get_spec(&$params) {
   );
 }
 
+/**
+ * @param $type
+ * @param $contactID
+ * @param $offset
+ * @param $limit
+ * @param $selectFields
+ * @param $fromClause
+ * @param $whereClause
+ * @param $sort
+ * @param $getCount
+ *
+ * @return array
+ */
 function _civicrm_api3_mailing_contact_query(
   $type,
   $contactID,
@@ -194,6 +207,15 @@ LIMIT %2, %3
   return $results;
 }
 
+/**
+ * @param $contactID
+ * @param $offset
+ * @param $limit
+ * @param $sort
+ * @param $getCount
+ *
+ * @return array
+ */
 function _civicrm_api3_mailing_contact_get_delivered(
   $contactID,
   $offset,
@@ -225,6 +247,15 @@ AND        meb.id IS NULL
   );
 }
 
+/**
+ * @param $contactID
+ * @param $offset
+ * @param $limit
+ * @param $sort
+ * @param $getCount
+ *
+ * @return array
+ */
 function _civicrm_api3_mailing_contact_get_bounced(
   $contactID,
   $offset,
index 5fe96dd2f2ef7b2a0c9a24cb0208c9b417f7491f..c042a393cf9b19100f8966c34b7b1c1583974998 100644 (file)
@@ -86,6 +86,11 @@ function civicrm_api3_mailing_group_event_subscribe($params) {
   return civicrm_api('mailing_event_subscribe', 'create', $params);
 }
 
+/**
+ * @param $params
+ *
+ * @return array
+ */
 function civicrm_api3_mailing_group_getfields($params) {
   $dao = _civicrm_api3_get_DAO('Subscribe');
   $d = new $dao();
index fcfd05fdcb5a3591776b6f222b0d6de572942c40..809db8fcbe79e444557ce3131e83f4b1159ff367 100644 (file)
@@ -65,6 +65,9 @@ function civicrm_api3_message_template_delete($params) {
   return _civicrm_api3_basic_delete(_civicrm_api3_get_BAO(__FUNCTION__), $params);
 }
 
+/**
+ * @param $params
+ */
 function _civicrm_api3_message_template_get_spec(&$params) {
 }
 
index bfe9590ee9b25be6ba4d4b9f29be3b05961ea152..27d0cb570840e50e0f7c35206482c08976d4d04c 100644 (file)
@@ -1,5 +1,10 @@
 <?php
 
+/**
+ * @param $params
+ *
+ * @return array
+ */
 function civicrm_api3_option_group_get($params) {
   return _civicrm_api3_basic_get(_civicrm_api3_get_BAO(__FUNCTION__), $params);
 }
index 9955b91299a74072a9d0cf1a74afbf6d0a72ae74..88b55be7d0225b4a2dcae3bb562fd44961eb0c4b 100644 (file)
@@ -83,6 +83,9 @@ function civicrm_api3_pledge_delete($params) {
   }
 }
 
+/**
+ * @param $params
+ */
 function _civicrm_api3_pledge_delete_spec(&$params) {
   // set as not required as pledge_id also acceptable & no either/or std yet
   $params['id']['api.aliases'] = array('pledge_id');
index 81b6fe59bba4108175b4fbd3fe883774f0940552..722dc400c1a33945f6b44bb002cf4960cb6471b0 100644 (file)
@@ -128,6 +128,13 @@ function civicrm_api3_pledge_payment_get($params) {
   return _civicrm_api3_basic_get(_civicrm_api3_get_BAO(__FUNCTION__), $params);
 }
 
+/**
+ * @param $pledgeId
+ * @param $paymentStatusId
+ * @param $paymentIds
+ *
+ * @return mixed
+ */
 function updatePledgePayments($pledgeId, $paymentStatusId, $paymentIds) {
 
   $result = updatePledgePayments($pledgeId, $paymentStatusId, $paymentIds = NULL);
index 9f5849e303980cadc7d1686834b8c9177d039a90..4f337da33c0f1596ec8d4f15b096205cd6c8a644 100644 (file)
@@ -138,6 +138,9 @@ function civicrm_api3_profile_get($params) {
   }
 }
 
+/**
+ * @param $params
+ */
 function _civicrm_api3_profile_get_spec(&$params) {
   $params['profile_id']['api.required'] = TRUE;
   $params['contact_id']['description'] = 'If no contact is specified an array of defaults will be returned';
@@ -599,6 +602,12 @@ function _civicrm_api3_buildprofile_submitfields($profileID, $optionsBehaviour =
   return $profileFields[$profileID];
 }
 
+/**
+ * @param $a
+ * @param $b
+ *
+ * @return bool
+ */
 function _civicrm_api3_order_by_weight($a, $b) {
   return CRM_Utils_Array::value('weight', $b) < CRM_Utils_Array::value('weight', $a) ? TRUE : FALSE;
 }
index 2976162973770cadaabf9c7943b22f647e2b23a9..ad618ac654d699c7b80b58a5d3d9ab6ef35acf11 100644 (file)
@@ -96,6 +96,13 @@ function civicrm_api3_report_template_getrows($params) {
   return civicrm_api3_create_success($rows, $params, 'report_template', 'getrows', CRM_Core_DAO::$_nullObject, $metadata);
 }
 
+/**
+ * @param $params
+ *
+ * @return array
+ * @throws API_Exception
+ * @throws CiviCRM_API3_Exception
+ */
 function _civicrm_api3_report_template_getrows($params) {
   if(empty($params['report_id'])) {
     $params['report_id'] = civicrm_api3('report_instance', 'getvalue', array('id' => $params['instance_id'], 'return' => 'report_id'));
@@ -141,6 +148,11 @@ function _civicrm_api3_report_template_getrows($params) {
   return array($rows, $reportInstance, $metadata);
 }
 
+/**
+ * @param $params
+ *
+ * @return array
+ */
 function civicrm_api3_report_template_getstatistics($params) {
   list($rows, $reportInstance, $metadata) = _civicrm_api3_report_template_getrows($params);
   $stats = $reportInstance->statistics($rows);