Clean up code for batch 19
authorpratikshad <pratiksha.dubey@webaccess.co.in>
Fri, 16 Jan 2015 05:36:39 +0000 (11:06 +0530)
committerpratikshad <pratiksha.dubey@webaccess.co.in>
Fri, 16 Jan 2015 05:36:39 +0000 (11:06 +0530)
64 files changed:
CRM/ACL/BAO/ACL.php
CRM/ACL/Form/ACLBasic.php
CRM/Activity/BAO/Query.php
CRM/Activity/Form/Task/RemoveFromTag.php
CRM/Admin/Form/PaymentProcessor.php
CRM/Admin/Form/Setting/Miscellaneous.php
CRM/Admin/Form/Setting/Search.php
CRM/Campaign/Form/Search/Campaign.php
CRM/Campaign/Form/Survey.php
CRM/Campaign/Form/SurveyType.php
CRM/Campaign/Form/Task/Release.php
CRM/Campaign/Page/Vote.php
CRM/Campaign/Selector/Search.php
CRM/Case/Form/Case.php
CRM/Case/Page/Tab.php
CRM/Contact/BAO/Contact/Location.php
CRM/Contact/BAO/RelationshipType.php
CRM/Contact/BAO/SearchCustom.php
CRM/Contact/Page/DedupeFind.php
CRM/Contribute/Page/DashBoard.php
CRM/Core/Lock.php
CRM/Event/Cart/Form/Checkout/Payment.php
CRM/Event/Form/ManageEvent/Conference.php
CRM/Event/Form/Task/Batch.php
CRM/Friend/BAO/Friend.php
CRM/Profile/Form/Search.php
CRM/Queue/ErrorPolicy.php
CRM/Report/Form/Contribute/SoftCredit.php
CRM/Upgrade/Snapshot/V4p2/Price/BAO/FieldValue.php
CRM/Utils/Pager.php
CRM/Utils/System/Drupal8.php
CRM/Utils/VersionCheck.php
api/v3/Activity.php
api/v3/CustomSearch.php
api/v3/Domain.php
api/v3/Grant.php
api/v3/LocBlock.php
api/v3/MailSettings.php
api/v3/MailingContact.php
api/v3/MailingEventSubscribe.php
api/v3/Phone.php
api/v3/PriceFieldValue.php
api/v3/PriceSet.php
api/v3/Survey.php
api/v3/Tag.php
api/v3/UFField.php
bin/deprecated/RespondentProcessor.php
tests/extensions/test.extension.manager.reporttest/main.php
tests/phpunit/CRM/Contact/BAO/GroupContactCacheTest.php
tests/phpunit/CRM/Core/MenuTest.php
tests/phpunit/CRM/Core/Payment/AuthorizeNetIPNTest.php
tests/phpunit/CRM/Core/PseudoConstantTest.php
tests/phpunit/CRM/Queue/QueueTest.php
tests/phpunit/CRM/Utils/API/MatchOptionTest.php
tests/phpunit/CRM/Utils/Cache/AllTests.php
tests/phpunit/CiviTest/Custom.php
tests/phpunit/CiviTest/Event.php
tests/phpunit/CiviTest/PaypalPro.php
tests/phpunit/WebTest/Event/AddEventTest.php
tests/phpunit/WebTest/Import/ContactSubtypeTest.php
tests/phpunit/WebTest/Import/ContactTest.php
tests/phpunit/WebTest/Import/DuplicateMatchingTest.php
tests/phpunit/api/v3/EntityTagTest.php
tests/phpunit/api/v3/MembershipTest.php

index 18b60cdd85ae106a4cf084269b959e2e8a1571fc..b8cef2a1b36bd28266c8a880c047581b4eea1c11 100644 (file)
@@ -112,7 +112,7 @@ class CRM_ACL_BAO_ACL extends CRM_ACL_DAO_ACL {
     $object_table = NULL, $object_id = NULL,
     $acl_id = NULL, $acl_role = FALSE
   ) {
-    $dao = new CRM_ACL_DAO_ACL;
+    $dao = new CRM_ACL_DAO_ACL();
 
     $t = array(
       'ACL' => self::getTableName(),
@@ -821,11 +821,11 @@ SELECT g.*
         }
 
         if ($staticGroupIDs) {
-          $clauses[] = '( `civicrm_group_contact-ACL`.group_id IN (' . join(', ', $staticGroupIDs) . ') AND `civicrm_group_contact-ACL`.status IN ("Added") )';
+          $clauses[] = '( `civicrm_group_contact-ACL`.group_id IN (' . implode(', ', $staticGroupIDs) . ') AND `civicrm_group_contact-ACL`.status IN ("Added") )';
         }
 
         if ($cachedGroupIDs) {
-          $clauses[] = '`civicrm_group_contact_cache-ACL`.group_id IN (' . join(', ', $cachedGroupIDs) . ')';
+          $clauses[] = '`civicrm_group_contact_cache-ACL`.group_id IN (' . implode(', ', $cachedGroupIDs) . ')';
         }
       }
     }
index d38cc2735c2bd1e8d9433d263648c4d20932dd5c..2391478d6822d1ba977aaee99dc0fb541074819a 100644 (file)
@@ -95,9 +95,9 @@ SELECT object_table
 
     $label = ts('Role');
     $role = array(
-        '-1' => ts('- select role -'),
-        '0' => ts('Everyone'),
-      ) + CRM_Core_OptionGroup::values('acl_role');
+      '-1' => ts('- select role -'),
+      '0' => ts('Everyone'),
+    ) + CRM_Core_OptionGroup::values('acl_role');
     $entityID = &$this->add('select', 'entity_id', $label, $role, TRUE);
 
     if ($this->_id) {
index cdc2ff6bf882f06c4030dad05690ec541ae7cb1f..2e9213e5f66762f91c26c34adcac72bc182b5b77 100644 (file)
@@ -460,11 +460,12 @@ class CRM_Activity_BAO_Query {
     CRM_Core_Form_Tag::buildQuickForm($form, $parentNames, 'civicrm_activity', NULL, TRUE, TRUE);
 
     $surveys = CRM_Campaign_BAO_Survey::getSurveys(TRUE, FALSE, FALSE, TRUE);
-    if ($surveys) $form->add('select', 'activity_survey_id', ts('Survey / Petition'),
-      array('' => ts('- none -')) + $surveys, FALSE,
-      array('class' => 'crm-select2')
-    );
-
+    if ($surveys) {
+      $form->add('select', 'activity_survey_id', ts('Survey / Petition'),
+                 array('' => ts('- none -')) + $surveys, FALSE,
+                 array('class' => 'crm-select2')
+                 );
+    }
     $extends = array('Activity');
     $groupDetails = CRM_Core_BAO_CustomGroup::getGroupDetail(NULL, TRUE, $extends);
     if ($groupDetails) {
index c97fb65285881532dc98416ca91f63a5e67a9c1f..1e67b893f2689b8aa945b5253e4590b605e31d16 100644 (file)
@@ -137,13 +137,13 @@ class CRM_Activity_Form_Task_RemoveFromTag extends CRM_Activity_Form_Task {
       $status = array(
         ts('%count activities un-tagged', array(
             'count' => $removed,
-            'plural' => '%count activities un-tagged'
-          ))
+            'plural' => '%count activities un-tagged',
+          )),
       );
       if ($notRemoved) {
         $status[] = ts('1 activity already did not have this tag', array(
             'count' => $notRemoved,
-            'plural' => '%count activities already did not have this tag'
+            'plural' => '%count activities already did not have this tag',
           ));
       }
       $status = '<ul><li>' . implode('</li><li>', $status) . '</li></ul>';
index 1afa4544c8d046fc3796df495e5d674cdd2085a8..247f7880ba18484b275842cb85ed9eb9b9f72d5e 100644 (file)
@@ -183,7 +183,7 @@ class CRM_Admin_Form_PaymentProcessor extends CRM_Admin_Form {
 
     $this->addRule('name', ts('Name already exists in Database.'), 'objectExists', array(
         'CRM_Financial_DAO_PaymentProcessor',
-        $this->_id
+        $this->_id,
       ));
 
     $this->add('text', 'description', ts('Description'),
@@ -348,7 +348,7 @@ class CRM_Admin_Form_PaymentProcessor extends CRM_Admin_Form {
     if ($this->_action & CRM_Core_Action::DELETE) {
       CRM_Financial_BAO_PaymentProcessor::del($this->_id);
       CRM_Core_Session::setStatus("", ts('Payment Processor Deleted.'), "success");
-      return;
+      return NULL;
     }
 
     $values = $this->controller->exportValues($this->_name);
@@ -371,7 +371,7 @@ class CRM_Admin_Form_PaymentProcessor extends CRM_Admin_Form {
    * @param int $domainID
    * @param $test
    *
-   * @return Void
+   * @return void
    */
   public function updatePaymentProcessor(&$values, $domainID, $test) {
     $dao = new CRM_Financial_DAO_PaymentProcessor();
index fc964512d7cc82e945c1ac8bd7aa4689d03262a2..77008585f0f03fa3b9a78826436fa582f6938ddc 100644 (file)
@@ -58,7 +58,6 @@ class CRM_Admin_Form_Setting_Miscellaneous extends CRM_Admin_Form_Setting {
   /**
    * Basic setup
    */
-
   public function preProcess() {
     $config = CRM_Core_Config::singleton();
     $this->_uploadMaxSize = (int) ini_get('upload_max_filesize');
@@ -78,7 +77,7 @@ class CRM_Admin_Form_Setting_Miscellaneous extends CRM_Admin_Form_Setting {
     $validTriggerPermission = CRM_Core_DAO::checkTriggerViewPermission(FALSE);
 
     // FIXME: for now, disable logging for multilingual sites OR if triggers are not permittted
-    $domain = new CRM_Core_DAO_Domain;
+    $domain = new CRM_Core_DAO_Domain();
     $domain->find(TRUE);
     $attribs = $domain->locales || !$validTriggerPermission ? array('disabled' => 'disabled') : array();
 
@@ -166,7 +165,7 @@ class CRM_Admin_Form_Setting_Miscellaneous extends CRM_Admin_Form_Setting {
     parent::postProcess();
 
     if ($config->logging != $values['logging']) {
-      $logging = new CRM_Logging_Schema;
+      $logging = new CRM_Logging_Schema();
       if ($values['logging']) {
         $logging->enableLogging();
       }
index 8e7868031565cbb128d2f4575c5a9f1f2f3179b6..aaa858c5580fe8faa7b5660f915022e1bacd4788 100644 (file)
@@ -72,8 +72,8 @@ class CRM_Admin_Form_Setting_Search extends CRM_Admin_Form_Setting {
 
     // Autocomplete for Contact Search (quick search etc.)
     $options = array(
-        ts('Contact Name') => 1
-      ) + array_flip(CRM_Core_OptionGroup::values('contact_autocomplete_options',
+      ts('Contact Name') => 1,
+    ) + array_flip(CRM_Core_OptionGroup::values('contact_autocomplete_options',
         FALSE, FALSE, TRUE
       ));
     $this->addCheckBox('autocompleteContactSearch', ts('Autocomplete Contact Search'), $options,
@@ -84,8 +84,8 @@ class CRM_Admin_Form_Setting_Search extends CRM_Admin_Form_Setting {
 
     // Autocomplete for Contact Reference (custom fields)
     $optionsCR = array(
-        ts('Contact Name') => 1
-      ) + array_flip(CRM_Core_OptionGroup::values('contact_reference_options',
+      ts('Contact Name') => 1,
+    ) + array_flip(CRM_Core_OptionGroup::values('contact_reference_options',
         FALSE, FALSE, TRUE
       ));
     $this->addCheckBox('autocompleteContactReference', ts('Contact Reference Options'), $optionsCR,
index c210b70c95792aed240b7c248f8e6bb0c318fb19..f909e34bd6da9979b4c8d71ff5c01731a4d25cea 100755 (executable)
@@ -50,7 +50,7 @@ class CRM_Campaign_Form_Search_Campaign extends CRM_Core_Form {
    *
    * @return void
    */
-  function preProcess() {
+  public function preProcess() {
     $this->_search = CRM_Utils_Array::value('search', $_GET);
     $this->_force = CRM_Utils_Request::retrieve('force', 'Boolean', $this, FALSE, FALSE);
     $this->_searchTab = CRM_Utils_Request::retrieve('type', 'String', $this, FALSE, 'campaign');
@@ -95,7 +95,7 @@ class CRM_Campaign_Form_Search_Campaign extends CRM_Core_Form {
     $campaignTypes = CRM_Campaign_PseudoConstant::campaignType();
     $this->add('select', 'campaign_type_id', ts('Campaign Type'),
       array(
-        '' => ts('- select -')
+        '' => ts('- select -'),
       ) + $campaignTypes
     );
 
@@ -106,7 +106,7 @@ class CRM_Campaign_Form_Search_Campaign extends CRM_Core_Form {
     $campaignStatus = CRM_Campaign_PseudoConstant::campaignStatus();
     $this->addElement('select', 'status_id', ts('Campaign Status'),
       array(
-        '' => ts('- select -')
+        '' => ts('- select -'),
       ) + $campaignStatus
     );
     $this->set('campaignStatus', $campaignStatus);
index 4d6e5075f41df2aa0a85c92924ca79015ea999fb..bf554f725bd84d03145a80c488f0794269816085 100644 (file)
@@ -92,8 +92,6 @@ class CRM_Campaign_Form_Survey extends CRM_Core_Form {
   /**
    * Build the form object
    *
-   * @param null
-   *
    * @return void
    */
   public function buildQuickForm() {
@@ -128,11 +126,10 @@ class CRM_Campaign_Form_Survey extends CRM_Core_Form {
         ),
       );
     }
-    $buttons[] =
-      array(
-        'type' => 'cancel',
-        'name' => ts('Cancel'),
-      );
+    $buttons[] = array(
+      'type' => 'cancel',
+      'name' => ts('Cancel'),
+    );
     $this->addButtons($buttons);
 
     $url = CRM_Utils_System::url('civicrm/campaign', 'reset=1&subPage=survey');
@@ -151,10 +148,10 @@ class CRM_Campaign_Form_Survey extends CRM_Core_Form {
 
       $this->postProcessHook();
 
-      if ($this->_action & CRM_Core_Action::ADD)
+      if ($this->_action & CRM_Core_Action::ADD) {
         CRM_Utils_System::redirect(CRM_Utils_System::url("civicrm/survey/configure/questions",
           "action=update&reset=1&id={$this->_surveyId}"));
-
+      }
       if ($this->controller->getButtonName('submit') == "_qf_{$className}_upload_done") {
         CRM_Utils_System::redirect(CRM_Utils_System::url('civicrm/campaign', 'reset=1&subPage=survey'));
       }
index 2d57b2298eb7d326fe5ee7b08efb843a3ddbd6e3..a1e41ccd716c8c2e0e1f05dfd0a1a65af6b394ed 100644 (file)
@@ -64,11 +64,9 @@ class CRM_Campaign_Form_SurveyType extends CRM_Admin_Form {
   /**
    * Set variables up before form is built
    *
-   * @param null
-   *
    * @return void
    */
-  function preProcess() {
+  public function preProcess() {
     $this->_action = CRM_Utils_Request::retrieve('action', 'String', $this);
 
     if ($this->_action & (CRM_Core_Action::UPDATE | CRM_Core_Action::DELETE)) {
@@ -98,8 +96,6 @@ class CRM_Campaign_Form_SurveyType extends CRM_Admin_Form {
    * Set default values for the form.
    * the default values are retrieved from the database.
    *
-   * @param null
-   *
    * @return array
    *   array of default values
    */
index bbb84df5e2f6493f9069479c185a7f722f99f8a2..cb408656a6b807793893f3b7df3d8a7b16c494c5 100644 (file)
@@ -67,14 +67,14 @@ class CRM_Campaign_Form_Task_Release extends CRM_Campaign_Form_Task {
    *
    * @return void
    */
-  function preProcess() {
+  public function preProcess() {
     $this->_interviewToRelease = $this->get('interviewToRelease');
     if ($this->_interviewToRelease) {
       //user came from interview form.
       foreach (array(
                  'surveyId',
                  'contactIds',
-                 'interviewerId'
+                 'interviewerId',
                ) as $fld) {
         $this->{"_$fld"} = $this->get($fld);
       }
@@ -107,7 +107,7 @@ class CRM_Campaign_Form_Task_Release extends CRM_Campaign_Form_Task {
     $activityStatus = CRM_Core_PseudoConstant::activityStatus('name');
     $statusIds = array();
     foreach (array(
-               'Scheduled'
+               'Scheduled',
              ) as $name) {
       if ($statusId = array_search($name, $activityStatus)) {
         $statusIds[] = $statusId;
index c322e9cd3b26c9ad2136699e0ada1dbec9bded70..2ff97eed26a44fcf2f8aaeb2eceee4db71870221 100644 (file)
@@ -98,7 +98,7 @@ class CRM_Campaign_Page_Vote extends CRM_Core_Page {
       ->addSetting(array(
         'tabSettings' => array(
           'active' => strtolower(CRM_Utils_Array::value('subPage', $_GET, 'reserve')),
-        )
+        ),
       ));
   }
 
@@ -119,8 +119,8 @@ class CRM_Campaign_Page_Vote extends CRM_Core_Page {
           array(
             'manage campaign',
             'administer CiviCampaign',
-            "{$name} campaign contacts"
-          )
+            "{$name} campaign contacts",
+          ),
         ))
       ) {
         continue;
index 6928b2c23bd0ec0fc9b20546e2e18a33341565a6..71dac101b44c688fa313b19b17fbf4b2a22edbaa 100755 (executable)
@@ -145,7 +145,7 @@ class CRM_Campaign_Selector_Search extends CRM_Core_Selector_Base implements CRM
    *
    * @return \CRM_Campaign_Selector_Search
    */
-  function __construct(
+  public function __construct(
     &$queryParams,
     $action = CRM_Core_Action::NONE,
     $surveyClause = NULL,
@@ -183,8 +183,7 @@ class CRM_Campaign_Selector_Search extends CRM_Core_Selector_Base implements CRM
    *
    * @return array
    */
-  static
-  public function &links() {
+  static public function &links() {
     return self::$_links = array();
   }
 
@@ -307,7 +306,7 @@ FROM {$from}
   /**
    * @return array
    *   which contains an array of strings
-   **/
+   */
   public function getQILL() {
     return $this->_query->qill();
   }
index c065454fee2a31566fc3c19b6e93a3fbf1ca62ff..a324b78ff0e0f9ed66f0bdb11782431d1e29cbaa 100644 (file)
@@ -159,8 +159,7 @@ class CRM_Case_Form_Case extends CRM_Core_Form {
       CRM_Core_Error::fatal(ts('You are not authorized to access this page.'));
     }
 
-    if ($this->_activityTypeFile =
-      CRM_Activity_BAO_Activity::getFileForActivityTypeId($this->_activityTypeId,
+    if ($this->_activityTypeFile = CRM_Activity_BAO_Activity::getFileForActivityTypeId($this->_activityTypeId,
         'Case'
       )
     ) {
@@ -261,7 +260,7 @@ class CRM_Case_Form_Case extends CRM_Core_Form {
     }
     $this->add('text', 'activity_subject', ts('Subject'),
       array_merge($s, array(
-        'maxlength' => '128'
+        'maxlength' => '128',
       )), TRUE
     );
 
index f02272dc480655e4df56c01df9ef799322d80b83..6eeecf3b84dbbc06f0a7ee160a700a045ee107a1 100644 (file)
@@ -117,8 +117,7 @@ class CRM_Case_Page_Tab extends CRM_Core_Page {
     $this->assign('caseId', $this->_id);
     $output = CRM_Core_Selector_Controller::SESSION;
     $selector = new CRM_Activity_Selector_Activity($this->_contactId, $this->_permission, FALSE, 'case');
-    $controller =
-      new CRM_Core_Selector_Controller(
+    $controller = new CRM_Core_Selector_Controller(
         $selector,
         $this->get(CRM_Utils_Pager::PAGE_ID),
         NULL,
@@ -140,7 +139,7 @@ class CRM_Case_Page_Tab extends CRM_Core_Page {
   /**
    * called when action is browse
    *
-   * @return null
+   * @return void
    */
   public function browse() {
 
@@ -225,8 +224,7 @@ class CRM_Case_Page_Tab extends CRM_Core_Page {
    * @return array
    *   (reference) of action links
    */
-  static
-  public function &links() {
+  static public function &links() {
     $config = CRM_Core_Config::singleton();
 
     if (!(self::$_links)) {
index 665dcd9969c23a7eaacbfee9b0d6520332f3d452..63d6ee122a4d4675d8dac36b155e82808307fc21 100644 (file)
@@ -52,7 +52,6 @@ class CRM_Contact_BAO_Contact_Location {
       $primaryClause = " AND civicrm_email.is_primary = 1";
     }
 
-
     $locationClause = NULL;
     if ($locationTypeID) {
       $locationClause = " AND civicrm_email.location_type_id = $locationTypeID";
@@ -96,7 +95,6 @@ WHERE     civicrm_contact.id = %1";
       $cond = " AND civicrm_phone.phone_type_id = '$type'";
     }
 
-
     $sql = "
    SELECT civicrm_contact.display_name, civicrm_phone.phone, civicrm_contact.do_not_sms
      FROM civicrm_contact
@@ -197,8 +195,7 @@ AND civicrm_contact.id IN $idString ";
       $location['displayAddress'] = str_replace('<br />', ', ', addslashes($address));
       $location['url'] = CRM_Utils_System::url('civicrm/contact/view', 'reset=1&cid=' . $dao->contact_id);
       $location['location_type'] = $dao->location_type;
-      $location['image'] = CRM_Contact_BAO_Contact_Utils::getImage(isset($dao->contact_sub_type) ?
-          $dao->contact_sub_type : $dao->contact_type, $imageUrlOnly, $dao->contact_id
+      $location['image'] = CRM_Contact_BAO_Contact_Utils::getImage(isset($dao->contact_sub_type) ? $dao->contact_sub_type : $dao->contact_type, $imageUrlOnly, $dao->contact_id
       );
       $locations[] = $location;
     }
index 39b0e965121bbefb5806b0fb4611852354d77c36..d3327a7685930f144b784cabb3a700eb7c11a363 100644 (file)
@@ -147,7 +147,6 @@ class CRM_Contact_BAO_RelationshipType extends CRM_Contact_DAO_RelationshipType
       throw new CRM_Core_Exception(ts('Invalid relationship type'));
     }
 
-
     //check dependencies
 
     // delete all relationships
@@ -164,8 +163,8 @@ UPDATE civicrm_membership_type
     $params = array(
       1 => array(
         CRM_Core_DAO::VALUE_SEPARATOR . $relationshipTypeId . CRM_Core_DAO::VALUE_SEPARATOR,
-        'String'
-      )
+        'String',
+      ),
     );
     CRM_Core_DAO::executeQuery($query, $params);
 
index 1ea22a7d2d5a26c7f96e79196c2a4031cb9aa19c..849277c34c6d18b1a7f70093463f9b7094334ff7 100644 (file)
@@ -161,7 +161,6 @@ class CRM_Contact_BAO_SearchCustom {
     $from = $customClass->from();
     $where = $customClass->where();
 
-
     return array($from, $where);
   }
 }
index 36c93485075bb9cd7985071b55b3ed0baf320c2a..a1a965f6665dfe135df196d6c55ff8af83d20373 100644 (file)
@@ -51,7 +51,7 @@ class CRM_Contact_Page_DedupeFind extends CRM_Core_Page_Basic {
   /**
    * Get action Links
    *
-   * @return array
+   * @return void
    *   (reference) of action links
    */
   public function &links() {
index 518f9e909c08e6de3c4e59d930ff62116d96d35a..559e5275e71839b37c99d2a3908ff549120990c7 100644 (file)
@@ -57,7 +57,7 @@ class CRM_Contribute_Page_DashBoard extends CRM_Core_Page {
     foreach (array(
                'now',
                'yearDate',
-               'monthDate'
+               'monthDate',
              ) as $date) {
       $$date = $dates[$date];
     }
index eb42b3c20715dd1a8f8c3bd3a16ff05d7155ed4a..2bde33175231a8e8f16fa3f056b27b454f0219a3 100644 (file)
@@ -148,7 +148,7 @@ class CRM_Core_Lock {
    *
    * @param string $jobLog
    * @throws CRM_Core_Exception
-   * @return boolean
+   * @return bool
    */
   public function hackyHandleBrokenCode($jobLog) {
     if (stristr($this->_name, 'job')) {
index 9933061ad8133279c3c923d2e262e213724f2d83..d787a3a45867a5dffc0e63fae7ca94e52be5ed70 100644 (file)
@@ -585,7 +585,7 @@ class CRM_Event_Cart_Form_Checkout_Payment extends CRM_Event_Cart_Form_Cart {
     if (is_a($result, 'CRM_Core_Error')) {
       CRM_Core_Error::displaySessionError($result);
       CRM_Utils_System::redirect(CRM_Utils_System::url('civicrm/event/cart_checkout', "_qf_Payment_display=1&qfKey={$this->controller->_key}", TRUE, NULL, FALSE));
-      return;
+      return NULL;
     }
     elseif (!$result['trxn_id']) {
       CRM_Core_Error::fatal(ts("Financial institution didn't return a transaction id."));
index a1ba39b20e25264f4985751c7a0dc8854938c613..6dbe42fb18cd524e948b8a1217064d11c85dcf13 100644 (file)
@@ -81,7 +81,7 @@ class CRM_Event_Form_ManageEvent_Conference extends CRM_Event_Form_ManageEvent {
       'slot_label_id',
       ts('Conference Slot'),
       array(
-        '' => ts('- select -')
+        '' => ts('- select -'),
       ) + $slots,
       FALSE
     );
index f5d9d01710a9c83929fc27e01a39032c4987ded5..60555e282877cccaa4cf9e17e85e6b65c47e14d9 100644 (file)
@@ -139,7 +139,6 @@ class CRM_Event_Form_Task_Batch extends CRM_Event_Form_Task {
       )
     );
 
-
     $this->assign('profileTitle', $this->_title);
     $this->assign('componentIds', $this->_participantIds);
     $fileFieldExists = FALSE;
@@ -350,7 +349,7 @@ class CRM_Event_Form_Task_Batch extends CRM_Event_Form_Task {
    */
   public static function updatePendingOnlineContribution($participantId, $statusId) {
     if (!$participantId || !$statusId) {
-      return;
+      return NULL;
     }
 
     $contributionId = CRM_Contribute_BAO_Contribution::checkOnlinePendingContribution($participantId,
index f9062a3cbc59830c0884b958b42152719133f511..604132359f1b386c0a057aef82632263a38c0398 100644 (file)
@@ -91,7 +91,7 @@ class CRM_Friend_BAO_Friend extends CRM_Friend_DAO_Friend {
    * @param array $params
    *   (reference ) an assoc array of name/value pairs.
    *
-   * @return CRM_Contact_BAO_Contact
+   * @return void
    */
   public static function create(&$params) {
     $transaction = new CRM_Core_Transaction();
index 20e36938a394421fc3dc11ffefd5368405df3bd4..f93cdcc0545d9708485edb1adcee117e998ec07c 100644 (file)
@@ -89,7 +89,7 @@ class CRM_Profile_Form_Search extends CRM_Profile_Form {
       }
       elseif (in_array($key, array(
         'birth_date',
-        'deceased_date'
+        'deceased_date',
       ))) {
         list($value) = CRM_Utils_Date::setDateDefaults($value);
       }
index d5fb08d6ea34daf331a93a900043ade914d3e3e3..f642ea8e66ec85f67c70af09c13f1feb70129bd3 100644 (file)
@@ -65,7 +65,7 @@ class CRM_Queue_ErrorPolicy {
     foreach (array(
                'display_errors',
                'html_errors',
-               'xmlrpc_errors'
+               'xmlrpc_errors',
              ) as $key) {
       $this->backup[$key] = ini_get($key);
       ini_set($key, 0);
@@ -84,7 +84,7 @@ class CRM_Queue_ErrorPolicy {
     foreach (array(
                'display_errors',
                'html_errors',
-               'xmlrpc_errors'
+               'xmlrpc_errors',
              ) as $key) {
       ini_set($key, $this->backup[$key]);
     }
index e68006ace1e03decc14139f1ec6c2c9a258e841e..88c0859d42a092585fd16dc65bbc581fcab920b2 100644 (file)
@@ -419,8 +419,7 @@ GROUP BY   {$this->_aliases['civicrm_contribution']}.currency
     $count = 0;
     $totalAmount = $average = array();
     while ($dao->fetch()) {
-      $totalAmount[] =
-        CRM_Utils_Money::format($dao->amount, $dao->currency) . '(' .
+      $totalAmount[] = CRM_Utils_Money::format($dao->amount, $dao->currency) . '(' .
         $dao->count . ')';
       $average[] = CRM_Utils_Money::format($dao->avg, $dao->currency);
       $count += $dao->count;
index 35b597ad901999b52a4aa0783fe232324d7cff88..6d81879fce2a536c6fec247b571e88748961fff1 100644 (file)
@@ -80,7 +80,7 @@ class CRM_Upgrade_Snapshot_V4p2_Price_BAO_FieldValue extends CRM_Upgrade_Snapsho
   public static function create(&$params, $ids) {
 
     if (!is_array($params) || empty($params)) {
-      return;
+      return NULL;
     }
 
     if ($id = CRM_Utils_Array::value('id', $ids)) {
@@ -190,7 +190,7 @@ class CRM_Upgrade_Snapshot_V4p2_Price_BAO_FieldValue extends CRM_Upgrade_Snapsho
    * @param int $fieldId
    *   Price field id.
    *
-   * @return boolean
+   * @return bool
    *
    */
   public static function deleteValues($fieldId) {
@@ -209,7 +209,7 @@ class CRM_Upgrade_Snapshot_V4p2_Price_BAO_FieldValue extends CRM_Upgrade_Snapsho
    * @param int $id
    *   Id.
    *
-   * @return boolean
+   * @return bool
    *
    */
   public static function del($id) {
index 3c9960de440f7352b915560b14883ded9c53f5b3..c9f40a79aa4a054b208b2a80a70502c8dfc753af 100644 (file)
@@ -126,11 +126,11 @@ class CRM_Utils_Pager extends Pager_Sliding {
      */
     $this->_response['titleTop'] = ts('Page %1 of %2', array(
         1 => '<input size="2" maxlength="3" name="' . self::PAGE_ID . '" type="text" value="' . $this->_response['currentPage'] . '" />',
-        2 => $this->_response['numPages']
+        2 => $this->_response['numPages'],
       ));
     $this->_response['titleBottom'] = ts('Page %1 of %2', array(
         1 => '<input size="2" maxlength="3" name="' . self::PAGE_ID_BOTTOM . '" type="text" value="' . $this->_response['currentPage'] . '" />',
-        2 => $this->_response['numPages']
+        2 => $this->_response['numPages'],
       ));
   }
 
@@ -252,8 +252,6 @@ class CRM_Utils_Pager extends Pager_Sliding {
   /**
    * Use the pager class to get the pageId and Offset
    *
-   * @param void
-   *
    * @return array
    *   an array of the pageID and offset
    */
index d233ca0cd38395e71288571813e1dc89f666138f..2f957e4f9b77777d065c31de5546d41f739a7503 100644 (file)
@@ -422,8 +422,6 @@ class CRM_Utils_System_Drupal8 extends CRM_Utils_System_DrupalBase {
    * This function does nothing in Drupal 8. Changes to the base_url should be made
    * in settings.php directly.
    *
-   * @param null
-   *
    * @return void
    */
   public function mapConfigToSSL() {
@@ -482,7 +480,7 @@ class CRM_Utils_System_Drupal8 extends CRM_Utils_System_DrupalBase {
    * @return array|bool
    *   [contactID, ufID, uniqueString] if success else false if no auth
    *
-   * This always bootstraps Drupal
+   *   This always bootstraps Drupal
    */
   public function authenticate($name, $password, $loadCMSBootstrap = FALSE, $realPath = NULL) {
     (new CRM_Utils_System_Drupal8())->loadBootStrap(array(), FALSE);
index f70da825a40888fcdf954d0da53b33e20ae86969..874d8290951039b3faf125e40504580faab09068 100644 (file)
@@ -196,7 +196,7 @@ class CRM_Utils_VersionCheck {
    * Get the latest version number if it's newer than the local one
    *
    * @return string|null
-   * Returns version number of the latest release if it is greater than the local version
+   *   Returns version number of the latest release if it is greater than the local version
    */
   public function isNewerVersionAvailable() {
     $newerVersion = NULL;
@@ -309,7 +309,7 @@ class CRM_Utils_VersionCheck {
    * Get active payment processor types
    */
   private function getPayProcStats() {
-    $dao = new CRM_Financial_DAO_PaymentProcessor;
+    $dao = new CRM_Financial_DAO_PaymentProcessor();
     $dao->is_active = 1;
     $dao->find();
     $ppTypes = array();
@@ -353,7 +353,7 @@ class CRM_Utils_VersionCheck {
       'CRM_Pledge_DAO_PledgeBlock' => NULL,
     );
     foreach ($tables as $daoName => $where) {
-      $dao = new $daoName;
+      $dao = new $daoName();
       if ($where) {
         $dao->whereAdd($where);
       }
index 0709d6ea013625f25e32719ce5150bf80636716c..c4a95986e8679bac5145e4775eddc5089258b0e2 100644 (file)
@@ -222,7 +222,7 @@ function _civicrm_api3_activity_create_spec(&$params) {
  *
  * @return array
  *
- * {@getfields activity_get}
+ *   {@getfields activity_get}
  * @example ActivityGet.php Basic example
  * @example Activity/DateTimeHigh.php Example get with date filtering
  * {@example ActivityGet.php 0}
index 2b37825c6fe44b2d79f3351ba205fcf3ef613afa..2469de097f438c1bf2d68e2fa44735a41d808f87 100644 (file)
@@ -12,7 +12,7 @@
  *
  * @return array
  *   details of found Option Values
- * {@getfields OptionValue_get}
+ *   {@getfields OptionValue_get}
  */
 function civicrm_api3_custom_search_get($params) {
   require_once 'api/v3/OptionValue.php';
@@ -33,7 +33,7 @@ function civicrm_api3_custom_search_get($params) {
  *
  * @return array
  *   Array of newly created option_value property values.
- * {@getfields OptionValue_create}
+ *   {@getfields OptionValue_create}
  */
 function civicrm_api3_custom_search_create($params) {
   require_once 'api/v3/OptionValue.php';
@@ -72,7 +72,7 @@ function _civicrm_api3_custom_search_create_spec(&$params) {
  *
  * @return array
  *   Api result
- * {@getfields ReportTemplate_create}
+ *   {@getfields ReportTemplate_create}
  */
 function civicrm_api3_custom_search_delete($params) {
   require_once 'api/v3/OptionValue.php';
index da334b8f28527f27501206a7a331b3073039c423..e06da4281125f6164e5d0d9cf367c7915af5e38c 100644 (file)
@@ -87,11 +87,11 @@ function civicrm_api3_domain_get($params) {
             'phone_type_id',
           $values['location']['phone'][1]
         )
-        ),
-        'phone' => CRM_Utils_Array::value(
+          ),
+          'phone' => CRM_Utils_Array::value(
           'phone',
         $values['location']['phone'][1]
-        ),
+          ),
         );
       }
 
index 8a21f412effc425587c5392bd10b8dfba797c00d..684940f69f4621edff803e575a4adcce92b50ce1 100644 (file)
@@ -46,7 +46,7 @@
  *
  * @return array
  *   grant array
- * {@getfields grant_create}
+ *   {@getfields grant_create}
  */
 function civicrm_api3_grant_create($params) {
   return _civicrm_api3_basic_create(_civicrm_api3_get_BAO(__FUNCTION__), $params, 'Grant');
@@ -74,7 +74,7 @@ function _civicrm_api3_grant_create_spec(&$params) {
  *
  * @return array
  *   Array of matching grants
- * {@getfields grant_get}
+ *   {@getfields grant_get}
  */
 function civicrm_api3_grant_get($params) {
   return _civicrm_api3_basic_get(_civicrm_api3_get_BAO(__FUNCTION__), $params, TRUE, 'Grant');
@@ -92,7 +92,7 @@ function civicrm_api3_grant_get($params) {
  *
  * @return array
  *   API Result Array
- * {@getfields grant_delete}
+ *   {@getfields grant_delete}
  */
 function civicrm_api3_grant_delete($params) {
   return _civicrm_api3_basic_delete(_civicrm_api3_get_BAO(__FUNCTION__), $params);
index a9af5405f29a54a2cb02e1ca070be7de00cd7c18..464ad93518707e8283948b2189c359ae231335b9 100644 (file)
@@ -44,7 +44,7 @@
  *
  * @return array
  *   api result array
- * {@getfields loc_block_create}
+ *   {@getfields loc_block_create}
  */
 function civicrm_api3_loc_block_create($params) {
   $entities = array();
@@ -95,7 +95,7 @@ function civicrm_api3_loc_block_create($params) {
  *
  * @return array
  *   Array of matching loc_blocks
- * {@getfields loc_block_get}
+ *   {@getfields loc_block_get}
  */
 function civicrm_api3_loc_block_get($params) {
   $options = _civicrm_api3_get_options_from_params($params);
@@ -133,7 +133,7 @@ function civicrm_api3_loc_block_get($params) {
  *
  * @return array
  *   API result array
- * {@getfields loc_block_delete}
+ *   {@getfields loc_block_delete}
  */
 function civicrm_api3_loc_block_delete($params) {
   return _civicrm_api3_basic_delete('CRM_Core_DAO_LocBlock', $params);
index d136b8a66891d0e601d48ddf2c71b9f30ff2fa3f..235d8709cc730a28b0954fce065dba33f10766cf 100644 (file)
@@ -43,7 +43,7 @@
  *
  * @return array
  *   api result array
- * {@getfields mail_settings_create}
+ *   {@getfields mail_settings_create}
  */
 function civicrm_api3_mail_settings_create($params) {
   return _civicrm_api3_basic_create(_civicrm_api3_get_BAO(__FUNCTION__), $params);
@@ -70,7 +70,7 @@ function _civicrm_api3_mail_settings_create_spec(&$params) {
  *
  * @return array
  *   Array of matching mail_settings
- * {@getfields mail_settings_get}
+ *   {@getfields mail_settings_get}
  */
 function civicrm_api3_mail_settings_get($params) {
   return _civicrm_api3_basic_get(_civicrm_api3_get_BAO(__FUNCTION__), $params);
@@ -88,7 +88,7 @@ function civicrm_api3_mail_settings_get($params) {
  *
  * @return array
  *   api result array
- * {@getfields mail_settings_delete}
+ *   {@getfields mail_settings_delete}
  */
 function civicrm_api3_mail_settings_delete($params) {
   return _civicrm_api3_basic_delete(_civicrm_api3_get_BAO(__FUNCTION__), $params);
index 7bf53d370a870fe51bc33e0c6807899cd4d7638b..f58575e605918af19f388aa0144300f62c7d2ea6 100644 (file)
@@ -60,7 +60,7 @@ function _civicrm_api3_mailing_contact_getresults($params, $count) {
     //ie. because the api is an anomaly & passing in id is not valid
     throw new Exception('This api call does not accept api as a parameter');
   }
-  $options  = _civicrm_api3_get_options_from_params($params, TRUE,'contribution','get');
+  $options  = _civicrm_api3_get_options_from_params($params, TRUE, 'contribution', 'get');
   $fnName = '_civicrm_api3_mailing_contact_get_' . strtolower($params['type']);
   return $fnName(
       $params['contact_id'],
@@ -81,7 +81,7 @@ function _civicrm_api3_mailing_contact_get_spec(&$params) {
   $params['contact_id']['title'] = 'Contact ID';
   $params['type'] = array(
     'api.default' => 'Delivered',
-    'title' => 'Type',// doesn't really explain the field - but not sure I understand it to explain it better
+    'title' => 'Type', // doesn't really explain the field - but not sure I understand it to explain it better
     'type' => CRM_Utils_Type::T_STRING,
     'options' => array(
       'Delivered' => 'Delivered',
index 1ff9f8e14d87f048274918c3050b88fff5b0a1bf..c90f514d756cf1ffaebfeccefb3068aed5589e74 100644 (file)
@@ -46,7 +46,7 @@
  * @throws API_Exception
  * @return array
  *   api result array
- * {@getfields mailing_event_subscribe_create}
+ *   {@getfields mailing_event_subscribe_create}
  */
 function civicrm_api3_mailing_event_subscribe_create($params) {
   $email      = $params['email'];
@@ -55,7 +55,7 @@ function civicrm_api3_mailing_event_subscribe_create($params) {
 
   $group            = new CRM_Contact_DAO_Group();
   $group->is_active = 1;
-  $group->id        = (int)$group_id;
+  $group->id        = (int) $group_id;
   if (!$group->find(TRUE)) {
     throw new API_Exception('Invalid Group id');
   }
@@ -65,7 +65,6 @@ function civicrm_api3_mailing_event_subscribe_create($params) {
   if ($subscribe !== NULL) {
     /* Ask the contact for confirmation */
 
-
     $subscribe->send_confirm_request($email);
 
     $values = array();
index 982521889da2e9141ecf327edca31ed819e8e2f6..17581d1c734f41e8b09b05b2b2f1977e39a73069 100644 (file)
@@ -72,7 +72,7 @@ function _civicrm_api3_phone_create_spec(&$params) {
  *
  * @return array
  *   Api Result
- * {@getfields phone_delete}
+ *   {@getfields phone_delete}
  * @example PhoneDelete.php
  */
 function civicrm_api3_phone_delete($params) {
index 5aee7f913823f3f3ff5b6a0c063c2a347fb3b8a9..ae023d38b5eb8bd57869c3274bac666008016278 100644 (file)
@@ -44,7 +44,7 @@
  *
  * @return array
  *   api result array
- * {@getfields price_field_value_create}
+ *   {@getfields price_field_value_create}
  */
 function civicrm_api3_price_field_value_create($params) {
   $ids = array();
@@ -83,7 +83,7 @@ function _civicrm_api3_price_field_value_create_spec(&$params) {
  *
  * @return array
  *   Array of matching price_field_values
- * {@getfields price_field_value_get}
+ *   {@getfields price_field_value_get}
  */
 function civicrm_api3_price_field_value_get($params) {
   return _civicrm_api3_basic_get(_civicrm_api3_get_BAO(__FUNCTION__), $params);
@@ -101,7 +101,7 @@ function civicrm_api3_price_field_value_get($params) {
  *
  * @return array
  *   API result array
- * {@getfields price_field_value_delete}
+ *   {@getfields price_field_value_delete}
  */
 function civicrm_api3_price_field_value_delete($params) {
   return _civicrm_api3_basic_delete(_civicrm_api3_get_BAO(__FUNCTION__), $params);
index 082b22807544fd4871a5440c571451a266140c3d..78e457d40bc7a5e271d95d61e71eba7c82601491 100644 (file)
@@ -44,7 +44,7 @@
  *
  * @return array
  *   api result array
- * {@getfields price_set_create}
+ *   {@getfields price_set_create}
  */
 function civicrm_api3_price_set_create($params) {
   $result = _civicrm_api3_basic_create(_civicrm_api3_get_BAO(__FUNCTION__), $params);
@@ -84,7 +84,7 @@ function _civicrm_api3_price_set_create_spec(&$params) {
  *
  * @return array
  *   Array of matching price_sets
- * {@getfields price_set_get}
+ *   {@getfields price_set_get}
  */
 function civicrm_api3_price_set_get($params) {
   // hack to make getcount work. - not sure the best approach here
@@ -112,7 +112,7 @@ function civicrm_api3_price_set_get($params) {
  *
  * @return array
  *   API result array
- * {@getfields price_set_delete}
+ *   {@getfields price_set_delete}
  */
 function civicrm_api3_price_set_delete($params) {
   return _civicrm_api3_basic_delete(_civicrm_api3_get_BAO(__FUNCTION__), $params);
index 1012f1a3b0a87b1c047cc8c54cae6309719f56d1..722476d676589a5096d0ab7254db978448c69cdf 100644 (file)
@@ -44,7 +44,7 @@
  *
  * @return array
  *   api result array
- * {@getfields survey_create}
+ *   {@getfields survey_create}
  */
 function civicrm_api3_survey_create($params) {
   return _civicrm_api3_basic_create(_civicrm_api3_get_BAO(__FUNCTION__), $params, 'Survey');
@@ -71,7 +71,7 @@ function _civicrm_api3_survey_create_spec(&$params) {
  *
  * @return array
  *   API result Array of matching surveys
- * {@getfields survey_get}
+ *   {@getfields survey_get}
  */
 function civicrm_api3_survey_get($params) {
   return _civicrm_api3_basic_get(_civicrm_api3_get_BAO(__FUNCTION__), $params, TRUE, 'Survey');
@@ -89,7 +89,7 @@ function civicrm_api3_survey_get($params) {
  *
  * @return array
  *   api result array
- * {@getfields survey_delete}
+ *   {@getfields survey_delete}
  */
 function civicrm_api3_survey_delete($params) {
   return _civicrm_api3_basic_delete(_civicrm_api3_get_BAO(__FUNCTION__), $params);
index 01f6efed539739ad531019ba430d056c015af413..34472fcc96611a771f69c557ce209583a5405446 100644 (file)
@@ -46,7 +46,7 @@
  *
  * @return array
  *   Array of newly created tag property values.
- * {@getfields tag_create}
+ *   {@getfields tag_create}
  */
 function civicrm_api3_tag_create($params) {
   return _civicrm_api3_basic_create(_civicrm_api3_get_BAO(__FUNCTION__), $params);
@@ -73,7 +73,7 @@ function _civicrm_api3_tag_create_spec(&$params) {
  *
  * @return array
  *   API result array
- * {@getfields tag_delete}
+ *   {@getfields tag_delete}
  */
 function civicrm_api3_tag_delete($params) {
   return _civicrm_api3_basic_delete(_civicrm_api3_get_BAO(__FUNCTION__), $params);
@@ -92,7 +92,7 @@ function civicrm_api3_tag_delete($params) {
  *
  * @return array
  *   details of found tags else error
- * {@getfields tag_get}
+ *   {@getfields tag_get}
  */
 function civicrm_api3_tag_get($params) {
 
index 7027417cdf0a5f4e3cc75d4cc64a2dd02275a773..75daf73c28bb0b03748eb937b9e1f9a77214b044 100644 (file)
@@ -59,7 +59,7 @@ function civicrm_api3_uf_field_create($params) {
   $location_type_id = CRM_Utils_Array::value('location_type_id', $params);
   $phone_type       = CRM_Utils_Array::value('phone_type_id', $params, CRM_Utils_Array::value('phone_type', $params));
 
-  if (! CRM_Core_BAO_UFField::isValidFieldName($field_name)) {
+  if (!CRM_Core_BAO_UFField::isValidFieldName($field_name)) {
     throw new API_Exception('The field_name is not valid');
   }
   $params['field_name'] = array($field_type, $field_name, $location_type_id, $phone_type);
@@ -110,7 +110,7 @@ function civicrm_api3_uf_field_create($params) {
  *
  * @param array $params
  *
- * @return array
+ * @return void
  *   fields valid for other functions
  */
 function _civicrm_api3_uf_field_create_spec(&$params) {
@@ -134,7 +134,7 @@ function _civicrm_api3_uf_field_create_spec(&$params) {
  *
  * @return array
  *   (reference) Array
- * {@getfields UFField_get
+ *   {@getfields UFField_get
  * @example UFFieldGet.php
  */
 function civicrm_api3_uf_field_get($params) {
index 7f73f669a6a701246f5062e5c92aac2ddffe9135..608e1e95ba0641f4ff46e66b8ea318ab4cd542d7 100644 (file)
@@ -40,7 +40,7 @@ require_once 'CRM/Core/Config.php';
 class CRM_RespondentProcessor {
   /**
    */
-  function __construct() {
+  public function __construct() {
     $config = CRM_Core_Config::singleton();
 
     //this does not return on failure
index 4edf0d4acf903f6357d09b9f855fcf769c3d246f..0c9e14731ec8e92e6e6f3026bb501488965c30a9 100644 (file)
@@ -6,8 +6,8 @@
 class test_extension_manager_reporttest extends CRM_Core_Report {
   /**
    */
-  function __construct() {
-    $logging        = new CRM_Logging_Schema;
+  public function __construct() {
+    $logging        = new CRM_Logging_Schema();
     $this->tables[] = 'civicrm_contact';
     $this->tables   = array_merge($this->tables, array_keys($logging->customDataLogTables()));
     $this->tables[] = 'civicrm_email';
@@ -25,7 +25,7 @@ class test_extension_manager_reporttest extends CRM_Core_Report {
     parent::__construct();
   }
 
-  function buildQuickForm() {
+  public function buildQuickForm() {
     $layout = CRM_Utils_Request::retrieve('layout', 'String', $this);
     $this->assign('layout', $layout);
 
index a9374611acb41cd2a1fb20adfdc355a9ab29185a..8de1b603dab5cc03902e3ceef6f7188d60a2f48d 100644 (file)
@@ -89,7 +89,7 @@ class CRM_Contact_BAO_GroupContactCacheTest extends CiviUnitTestCase {
       array(/* deceased[0], */
         $deceased[1]->id,
         $deceased[2]->id,
-        $living[0]->id
+        $living[0]->id,
       ),
       $group->id
     );
@@ -152,7 +152,7 @@ class CRM_Contact_BAO_GroupContactCacheTest extends CiviUnitTestCase {
     $this->assertCacheMatches(
       array(/* deceased[0], */
         $deceased[1]->id,
-        $deceased[2]->id
+        $deceased[2]->id,
       ),
       $parent->id
     );
@@ -224,7 +224,7 @@ class CRM_Contact_BAO_GroupContactCacheTest extends CiviUnitTestCase {
    * @param int $numObjects
    * @param bool $createOnly
    */
-  function createTestObject($daoName, $params = array(), $numObjects = 1, $createOnly = FALSE) {
+  public function createTestObject($daoName, $params = array(), $numObjects = 1, $createOnly = FALSE) {
     $objects = CRM_Core_DAO::createTestObject($daoName, $params, $numObjects, $createOnly);
     if (is_array($objects)) {
       $this->registerTestObjects($objects);
index 7ce665f52acd8c9b28acbc135edc3237baeb7c6a..a9e487cc1668c481611c20eff772f4b95411e74d 100644 (file)
@@ -17,7 +17,7 @@ class CRM_Core_MenuTest extends CiviUnitTestCase {
     $cases[] = array('mode=256', array('mode' => '256'));
     $cases[] = array(
       'mode=256,addSequence=1,attachUpload=1',
-      array('mode' => '256', 'addSequence' => '1', 'attachUpload' => 1)
+      array('mode' => '256', 'addSequence' => '1', 'attachUpload' => 1),
     );
     $cases[] = array(
       'mode=256,urlToSession=a:b:c:d',
@@ -26,7 +26,7 @@ class CRM_Core_MenuTest extends CiviUnitTestCase {
         'urlToSession' => array(
           array('urlVar' => 'a', 'sessionVar' => 'b', 'type' => 'c', 'default' => 'd'),
         ),
-      )
+      ),
     );
     $cases[] = array(
       'mode=256,urlToSession=a:b:c:d;z:y:x:w',
@@ -36,7 +36,7 @@ class CRM_Core_MenuTest extends CiviUnitTestCase {
           array('urlVar' => 'a', 'sessionVar' => 'b', 'type' => 'c', 'default' => 'd'),
           array('urlVar' => 'z', 'sessionVar' => 'y', 'type' => 'x', 'default' => 'w'),
         ),
-      )
+      ),
     );
     $cases[] = array('url=whiz!;.:#=%/|+bang?', array('url' => 'whiz!;.:#=%/|+bang?'));
     return $cases;
index e894ad66a0dcbb7b96b7e0e663d9119b15b3a254..52e5be603a8e777e4370e14f471dcfe83083116b 100644 (file)
@@ -51,7 +51,7 @@ class CRM_Core_Payment_AuthorizeNetIPNTest extends CiviUnitTestCase {
     $paypalIPN->main();
     $contribution = $this->callAPISuccess('contribution', 'get', array(
         'contribution_recur_id' => $this->_contributionRecurID,
-        'sequential' => 1
+        'sequential' => 1,
       ));
     $this->assertEquals(2, $contribution['count']);
     $this->assertEquals('second_one', $contribution['values'][1]['trxn_id']);
@@ -75,14 +75,14 @@ class CRM_Core_Payment_AuthorizeNetIPNTest extends CiviUnitTestCase {
     $paypalIPN->main();
     $contribution = $this->callAPISuccess('contribution', 'get', array(
         'contribution_recur_id' => $this->_contributionRecurID,
-        'sequential' => 1
+        'sequential' => 1,
       ));
     $this->assertEquals(2, $contribution['count']);
     $this->assertEquals('second_one', $contribution['values'][1]['trxn_id']);
     $this->callAPISuccessGetSingle('membership_payment', array('contribution_id' => $contribution['values'][1]['id']));
     $this->callAPISuccessGetSingle('line_item', array(
         'contribution_id' => $contribution['values'][1]['id'],
-        'entity_table' => 'civicrm_membership'
+        'entity_table' => 'civicrm_membership',
       ));
   }
 
index e0327a41fdb935aaa986ed3178e04bd8e92e6a47..eee1c4306ae000d7cdc1427fc892a5b8a833786d 100644 (file)
@@ -70,7 +70,7 @@ class CRM_Core_PseudoConstantTest extends CiviUnitTestCase {
           'value' => 'foo',
           'is_active' => 1,
           'weight' => 0,
-        )
+        ),
       ),
     );
     $result = civicrm_api3('custom_field', 'create', $api_params);
@@ -1083,14 +1083,14 @@ class CRM_Core_PseudoConstantTest extends CiviUnitTestCase {
     // But we can also fetch by ID
     $result = CRM_Core_PseudoConstant::get('CRM_Contact_DAO_Contact', 'contact_type', array(
         'keyColumn' => 'id',
-        'labelColumn' => 'name'
+        'labelColumn' => 'name',
       ));
     $this->assertEquals($byId, $result);
     // Make sure flip param works
     $result = CRM_Core_PseudoConstant::get('CRM_Contact_DAO_Contact', 'contact_type', array(
         'keyColumn' => 'id',
         'labelColumn' => 'name',
-        'flip' => TRUE
+        'flip' => TRUE,
       ));
     $this->assertEquals(array_flip($byId), $result);
   }
index 24c49021315223275a9d698102f16141f4c9a511..50d8d4b2a5dd4828490e4d1cea428c90f5b2215b 100644 (file)
@@ -46,18 +46,20 @@ class CRM_Queue_QueueTest extends CiviUnitTestCase {
       array(
         'type' => 'Sql',
         'name' => 'test-queue',
-      )
+      ),
     );
     $queueSpecs[] = array(
       array(
         'type' => 'Memory',
         'name' => 'test-queue',
-      )
+      ),
     );
     return $queueSpecs;
   }
 
-  /* ----------------------- Per-provider tests ----------------------- */
+  /**
+   * Per-provider tests
+   */
   public function setUp() {
     parent::setUp();
     $this->queueService = CRM_Queue_Service::singleton(TRUE);
index a6c2e9a802f0518ff8ae3d502c498edda23b60fd..ea9e52578f5ed6c3f56b6e3b1d071ceffbfbdcad 100644 (file)
@@ -189,7 +189,7 @@ class CRM_Utils_API_MatchOptionTest extends CiviUnitTestCase {
           array(
             'location_type_id' => 2,
             'email' => 'j2@example.com',
-            'signature_text' => 'You know, a lotta ins, a lotta outs, a lotta what-have-yous.'
+            'signature_text' => 'You know, a lotta ins, a lotta outs, a lotta what-have-yous.',
           ),
         ),
       ),
@@ -269,12 +269,12 @@ class CRM_Utils_API_MatchOptionTest extends CiviUnitTestCase {
           array(
             'location_type_id' => 1,
             'street_address' => 'j1-a Example Ave',
-            'supplemental_address_1' => 'The Dude abides.'
+            'supplemental_address_1' => 'The Dude abides.',
           ),
           array(
             'location_type_id' => 2,
             'street_address' => 'j2 Example Ave',
-            'supplemental_address_1' => 'You know, a lotta ins, a lotta outs, a lotta what-have-yous.'
+            'supplemental_address_1' => 'You know, a lotta ins, a lotta outs, a lotta what-have-yous.',
           ),
         ),
       ),
index 03108aed0eb3bbde825eaa91afa3934819729f6a..2fba80953a98b9edc6fd477dae76db778bbee598 100644 (file)
@@ -47,7 +47,7 @@ class CRM_Utils_Cache_AllTests extends CiviTestSuite {
    */
   private static function getInstance() {
     if (is_null(self::$instance)) {
-      self::$instance = new self;
+      self::$instance = new self();
     }
     return self::$instance;
   }
index 699efaa87623db4ccf1ab53ddadaeb1bf7b6eb8a..8e85ce566049738ff0d81abebe9d600e54882913 100644 (file)
@@ -101,7 +101,6 @@ class Custom extends CiviUnitTestCase {
   /**
    * Helper function to delete custom field
    * @deprecated use function on parent class
-   * @param object of Custom Field to delete
    */
   public static function deleteField($params) {
     require_once 'CRM/Core/BAO/CustomField.php';
@@ -111,8 +110,7 @@ class Custom extends CiviUnitTestCase {
   /**
    * Helper function to delete custom group
    * @deprecated use function on parent class
-   * @param object Custom Group to delete
-   * @return boolean
+   * @return bool
    *   true if Group deleted, false otherwise
    */
   public static function deleteGroup($params) {
index bd393f36c5d0602aba432e4e8e42d38673aaca74..b243622c320f7ab000cd3b1c440f85f151391f7b 100644 (file)
@@ -37,7 +37,7 @@ class Event extends PHPUnit_Framework_Testcase {
    * Helper function to delete an Event
    *
    * @param int $eventId
-   * @return boolean
+   * @return bool
    *   true if event deleted, false otherwise
    */
   public static function delete($eventId) {
index 1a9ce0a411299c0fd136bc620dbfb16b4c23f84d..c01369f90e7cebb8b0090c2bf94a3fc4858cfff8 100644 (file)
@@ -11,8 +11,8 @@ class PaypalPro extends PHPUnit_Framework_Testcase {
    *
    * @return int
    *   $paymentProcessor id of created payment processor@todo this appears not to be working but because it doesn't extend the test class
-   * callAPISuccess won't work
-   * I have duplicated this on the main test class as a work around
+   *   callAPISuccess won't work
+   *   I have duplicated this on the main test class as a work around
    */
   public static function create() {
 
@@ -45,7 +45,7 @@ class PaypalPro extends PHPUnit_Framework_Testcase {
    * @param int $id
    *   Id of the PayPal Pro payment processor.
    *   to be deleted
-   * @return boolean
+   * @return bool
    *   true if payment processor deleted, false otherwise
    */
   public static function delete($id) {
index b592be535d5fb92f12e3498e94550da4b41d20c9..543d11418a35bb95369665137c7a8f4bf9d7cc64 100644 (file)
@@ -763,8 +763,7 @@ class WebTest_Event_AddEventTest extends CiviSeleniumTestCase {
 
     $participantEmails = array($primaryParticipant, $secParticipant, $thirdParticipant);
     $addtlPart = array($secParticipant, $thirdParticipant);
-    $primaryParticipantInfo =
-      $this->_testOnlineRegistration($registerUrl, 2, $anonymous, FALSE, $participantEmails, "Test Processor");
+    $primaryParticipantInfo = $this->_testOnlineRegistration($registerUrl, 2, $anonymous, FALSE, $participantEmails, "Test Processor");
     $primaryDisplayName = "{$primaryParticipantInfo['first_name']} {$primaryParticipantInfo['last_name']}";
     $this->webtestLogin();
     $this->openCiviPage("event/search", "reset=1");
index 18ed1c3b4baa208b9acc0dfd170f2f75bd435faf..3c1d05edee95d2755cdf8f2125ea17540e6f7289 100644 (file)
@@ -35,7 +35,7 @@ class WebTest_Import_ContactSubtypeTest extends ImportCiviSeleniumTestCase {
     parent::setUp();
   }
 
-  /*
+  /**
    *  Test contact import for Individuals Subtype.
    */
   public function testIndividualSubtypeImport() {
@@ -117,7 +117,7 @@ class WebTest_Import_ContactSubtypeTest extends ImportCiviSeleniumTestCase {
     $this->importContacts($headers, $rows, 'Individual', 'No Duplicate Checking', array(), $other);
   }
 
-  /*
+  /**
    *  Test contact import for Organization Subtype.
    */
   public function testOrganizationSubtypeImport() {
@@ -193,7 +193,7 @@ class WebTest_Import_ContactSubtypeTest extends ImportCiviSeleniumTestCase {
     $this->importContacts($headers, $rows, 'Organization', 'No Duplicate Checking', array(), $other);
   }
 
-  /*
+  /**
    *  Test contact import for Household Subtype.
    */
   public function testHouseholdSubtypeImport() {
index d105e069710d2113783fff392d91110f9ab3ce5f..e012887a55b6f666c418509e56a42af2b4168ce5 100644 (file)
@@ -35,7 +35,7 @@ class WebTest_Import_ContactTest extends ImportCiviSeleniumTestCase {
     parent::setUp();
   }
 
-  /*
+  /**
    *  Test contact import for Individuals.
    */
   public function testIndividualImport() {
@@ -122,7 +122,7 @@ class WebTest_Import_ContactTest extends ImportCiviSeleniumTestCase {
     $this->importContacts($headers, $rows, 'Individual', 'No Duplicate Checking');
   }
 
-  /*
+  /**
    *  Test contact import for Organization.
    */
   public function testOrganizationImport() {
@@ -203,7 +203,7 @@ class WebTest_Import_ContactTest extends ImportCiviSeleniumTestCase {
     $this->importContacts($headers, $rows, 'Organization', 'No Duplicate Checking');
   }
 
-  /*
+  /**
    *  Test contact import for Household.
    */
   public function testHouseholdImport() {
index 5a5e213bdcf1b01db3bacdcb295a5d6dc9fe7b66..6e3923fb09172ba6c7950847f75b91d843a401fd 100644 (file)
@@ -35,7 +35,7 @@ class WebTest_Import_DuplicateMatchingTest extends ImportCiviSeleniumTestCase {
     parent::setUp();
   }
 
-  /*
+  /**
    *  Test contact import for Individuals Duplicate Matching.
    */
   public function testIndividualDuplicateMatchingImport() {
@@ -136,7 +136,7 @@ class WebTest_Import_DuplicateMatchingTest extends ImportCiviSeleniumTestCase {
     $this->importContacts($headers, $rows, 'Individual', 'No Duplicate Checking');
   }
 
-  /*
+  /**
    *  Test contact import for Organization Duplicate Matching.
    */
   public function testOrganizationDuplicateMatchingImport() {
@@ -231,7 +231,7 @@ class WebTest_Import_DuplicateMatchingTest extends ImportCiviSeleniumTestCase {
     $this->importContacts($headers, $rows, 'Organization', 'No Duplicate Checking');
   }
 
-  /*
+  /**
    *  Test contact import for Household Duplicate Matching.
    */
   public function testHouseholdDuplicateMatchingImport() {
index a6938f343253211c03c329538971e00f07f2a0e0..f2f99ce4c13d4351506b68997f64b2d64bd1468a 100644 (file)
@@ -119,7 +119,9 @@ class api_v3_EntityTagTest extends CiviUnitTestCase {
     $this->assertEquals($result['not_added'], 1);
   }
 
-  ///////////////// civicrm_entity_tag_get methods
+  /**
+   * civicrm_entity_tag_get methods
+   */
   public function testGetNoEntityID() {
     $ContactId = $this->_individualID;
     $tagID = $this->_tagID;
@@ -178,7 +180,9 @@ class api_v3_EntityTagTest extends CiviUnitTestCase {
     $entity = $this->callAPISuccess('entity_tag', 'get', $paramsEntity);
   }
 
-  ///////////////// civicrm_entity_tag_Delete methods
+  /**
+   * civicrm_entity_tag_Delete methods
+   */
   public function testEntityTagDeleteNoTagId() {
     $entityTagParams = array(
       'contact_id_i' => $this->_individualID,
@@ -252,8 +256,9 @@ class api_v3_EntityTagTest extends CiviUnitTestCase {
     $this->assertEquals($result['not_removed'], 1);
   }
 
-  ///////////////// civicrm_tag_entities_get methods
-
+  /**
+   * civicrm_tag_entities_get methods
+   */
   public function testCommonContactEntityTagAdd() {
     $params = array(
       'contact_id' => $this->_individualID,
index 0a1089266f959fc2ee2976a2dfa239f32165cac8..595d0769abed0a000d46375e65d08f42c4639ad9 100644 (file)
@@ -626,9 +626,9 @@ class api_v3_MembershipTest extends CiviUnitTestCase {
     $this->assertEquals($result['id'], $result['values'][$result['id']]['id'], " in line " . __LINE__);
   }
 
-  /*
-      * Check for useful message if contact doesn't exist
-      */
+  /**
+   * Check for useful message if contact doesn't exist
+   */
   public function testMembershipCreateWithInvalidContact() {
     $params = array(
       'contact_id' => 999,
@@ -833,7 +833,7 @@ class api_v3_MembershipTest extends CiviUnitTestCase {
   /**
    * per CRM-15746 check that the id can be altered in an update hook
    */
-  function testMembershipUpdateCreateHookCRM15746() {
+  public function testMembershipUpdateCreateHookCRM15746() {
     $this->hookClass->setHook('civicrm_pre', array($this, 'hook_civicrm_pre_update_create_membership'));
     $result = $this->callAPISuccess('membership', 'create', $this->_params);
     $this->callAPISuccess('membership', 'create', array('id' => $result['id'], 'end_date' => '1 year ago'));
@@ -843,7 +843,7 @@ class api_v3_MembershipTest extends CiviUnitTestCase {
     $this->callAPISuccessGetCount('membership', array(), 2);
   }
 
-  function hook_civicrm_pre_update_create_membership($op, $objectName, $id, &$params) {
+  public function hook_civicrm_pre_update_create_membership($op, $objectName, $id, &$params) {
     if ($objectName == 'Membership' && $op == 'edit') {
       $existingMembership = $this->callAPISuccessGetSingle('membership', array('id' => $params['id']));
       unset($params['id'], $params['membership_id']);