tidy up some unused params

diff --git a/api/v3/Address.php b/api/v3/Address.php
index f1f9185682ce01ed66fcc02bb2c6d78e12b08e4a..ac3bc0a3df387f9d19459d4c3768c3f3db1045d0 100644 (file)
--- a/api/v3/Address.php
+++ b/api/v3/Address.php
@@ -85,7 +85,6 @@ function civicrm_api3_address_create(&$params) {
     return civicrm_api3_create_error("Address is not created or updated ");
   }
   else {
-    $values = array();
     $values = _civicrm_api3_dao_to_array($addressBAO, $params);
     return civicrm_api3_create_success($values, $params, 'address', $addressBAO);
   }

diff --git a/api/v3/Contact.php b/api/v3/Contact.php
index d4179d0f81ba547478dfbd4965a24c162d20be52..b06e8d9e65ecc722cdf0d9a2a09f3e7839012015 100644 (file)
--- a/api/v3/Contact.php
+++ b/api/v3/Contact.php
@@ -401,8 +401,7 @@ function _civicrm_api3_contact_check_params(&$params, $dupeCheck = TRUE, $dupeEr
  * @param int $contactID
  *   If present the contact with that ID is updated.
  *
- * @return CRM_Contact_BAO_Contact
- * @access public
+ * @return CRM_Contact_BAO_Contact|CRM_Core_Error
  */
 function _civicrm_api3_contact_update($params, $contactID = NULL) {
   //@todo - doesn't contact create support 'id' which is already set- check & remove

diff --git a/api/v3/EntityTag.php b/api/v3/EntityTag.php
index cf265e226b3522cc02ab1fc780b4f7f6cf9d3b0c..249515640d38a8a1d3696d66dd2bb0c3a2a014e5 100644 (file)
--- a/api/v3/EntityTag.php
+++ b/api/v3/EntityTag.php
@@ -120,7 +120,6 @@ function _civicrm_api3_entity_tag_delete_spec(&$params) {
 function _civicrm_api3_entity_tag_common($params, $op = 'add') {
 
   $entityIDs   = array();
-  $tagsIDs     = array();
   $entityTable = 'civicrm_contact';
   if (is_array($params)) {
     foreach ($params as $n => $v) {

diff --git a/api/v3/MailingJob.php b/api/v3/MailingJob.php
index 05c027bef1e8b0dea1bccfa1b89b3d40690d3a1e..9d1a70270669bb022c44e01e1f5fe0579c7d3d89 100644 (file)
--- a/api/v3/MailingJob.php
+++ b/api/v3/MailingJob.php
@@ -64,13 +64,12 @@ function _civicrm_api3_mailing_job_create_spec(&$params) {
  *
  * @param array $params
  *   Array of one or more valid.
- * @param array $ids
  *
  * @return array
  *   API return Array of matching mailing jobs
  * {@getfields mailing_job_get}
  */
-function civicrm_api3_mailing_job_get($params, $ids = array()) {
+function civicrm_api3_mailing_job_get($params) {
   return _civicrm_api3_basic_get(_civicrm_api3_get_BAO(__FUNCTION__), $params);
 }
 

diff --git a/api/v3/MailingRecipients.php b/api/v3/MailingRecipients.php
index b2ea3a4fee5b865b985e26f1980a0b5f5cc7b87a..99042f32d77696fd533a61518b678f2bbea5f1ce 100644 (file)
--- a/api/v3/MailingRecipients.php
+++ b/api/v3/MailingRecipients.php
@@ -42,12 +42,10 @@
  *
  * @param array $params
  *   Array of one or more valid.
- * @param array $ids
  *
  * @return array
  *   API return Array of matching mailing jobs
- * {@getfields mailing_recipient_get}
  */
-function civicrm_api3_mailing_recipients_get($params, $ids = array()) {
+function civicrm_api3_mailing_recipients_get($params)) {
   return _civicrm_api3_basic_get(_civicrm_api3_get_BAO(__FUNCTION__), $params);
 }

diff --git a/api/v3/PaymentProcessorType.php b/api/v3/PaymentProcessorType.php
index aa2a9e5ac9100220bf459c08b8730443ad74dfc3..370361eb3e3a1335df2b02c4850aa4a7f8605b5b 100644 (file)
--- a/api/v3/PaymentProcessorType.php
+++ b/api/v3/PaymentProcessorType.php
@@ -49,19 +49,17 @@
  * @return array
  */
 function civicrm_api3_payment_processor_type_create($params) {
-  $ids = array();
   if (isset($params['id']) && !CRM_Utils_Rule::integer($params['id'])) {
     return civicrm_api3_create_error('Invalid value for payment_processor type ID');
   }
 
-  $payProcType = new CRM_Financial_BAO_PaymentProcessorType();
-  $payProcType = CRM_Financial_BAO_PaymentProcessorType::create($params);
+  $paymentProcessorType = CRM_Financial_BAO_PaymentProcessorType::create($params);
 
   $relType = array();
 
-  _civicrm_api3_object_to_array($payProcType, $relType[$payProcType->id]);
+  _civicrm_api3_object_to_array($paymentProcessorType, $relType[$paymentProcessorType->id]);
 
-  return civicrm_api3_create_success($relType, $params, 'payment_processor_type', 'create', $payProcType);
+  return civicrm_api3_create_success($relType, $params, 'payment_processor_type', 'create', $paymentProcessorType);
 }
 
 /**

diff --git a/api/v3/Pledge.php b/api/v3/Pledge.php
index 5767a97744fe3b365276ccd1f62b38a598d8607f..c5b0e357549321d9380b25fa52f3caaca479b25a 100644 (file)
--- a/api/v3/Pledge.php
+++ b/api/v3/Pledge.php
@@ -164,11 +164,10 @@ function _civicrm_api3_pledge_get_defaults() {
  *
  * @param array $values
  *   The reformatted properties that we can use internally.
- * @param bool $create
  *
- * @return array|CRM_Error
+ * @return array|CRM_Core_Error
  */
-function _civicrm_api3_pledge_format_params(&$values, $create = FALSE) {
+function _civicrm_api3_pledge_format_params(&$values) {
 
   // probably most of the below can be removed.... just needs a little more review
   if (array_key_exists('original_installment_amount', $values)) {