Fixed code after cleanup test
authorpratikshad <pratiksha.dubey@webaccess.co.in>
Thu, 15 Jan 2015 07:43:11 +0000 (13:13 +0530)
committerpratikshad <pratiksha.dubey@webaccess.co.in>
Thu, 15 Jan 2015 07:43:11 +0000 (13:13 +0530)
42 files changed:
CRM/Campaign/Form/Survey/Results.php
CRM/Case/StateMachine/Search.php
CRM/Contact/Form/Edit/Email.php
CRM/Contact/Form/Search/Custom/ActivitySearch.php
CRM/Contact/Form/Task/AddToHousehold.php
CRM/Contact/Form/Task/Print.php
CRM/Contact/Form/Task/SaveSearch.php
CRM/Contribute/Page/ContributionRecur.php
CRM/Core/Action.php
CRM/Core/InnoDBIndexer.php
CRM/Core/Page/Angular.php
CRM/Core/Page/Basic.php
CRM/Core/Payment/Elavon.php
CRM/Core/Payment/FirstData.php
CRM/Core/Payment/PayPalProIPN.php
CRM/Event/BAO/ParticipantStatusType.php
CRM/Event/Badge.php
CRM/Event/Form/ManageEvent/Registration.php
CRM/Event/Import/Form/Summary.php
CRM/Mailing/BAO/Recipients.php
CRM/Mailing/Event/BAO/Delivered.php
CRM/Member/Import/Parser/Membership.php
CRM/PCP/BAO/PCP.php
CRM/PCP/Form/Event.php
CRM/Price/BAO/PriceField.php
CRM/Profile/Form.php
CRM/Report/Form/Contribute/LoggingDetail.php
CRM/Report/Form/Extended.php
CRM/Utils/Array.php
CRM/Utils/Date.php
CRM/Utils/Migrate/Export.php
CRM/Utils/REST.php
CRM/Utils/System/Base.php
CRM/Utils/System/Soap.php
Civi/Core/Transaction/Manager.php
api/v3/Contact.php
api/v3/UFMatch.php
tests/phpunit/AllTests.php
tests/phpunit/CRM/Bridge/OG/DrupalTest.php
tests/phpunit/WebTest/Member/OfflineMembershipAddPricesetTest.php
tests/phpunit/WebTest/Member/StandaloneAddTest.php
tests/phpunit/api/v3/UFJoinTest.php

index 70e49c6571781bfeeecdc804251359d0bc9821da..039ef71573b46d1b1d331da567a326fa3d503489 100644 (file)
@@ -77,8 +77,6 @@ class CRM_Campaign_Form_Survey_Results extends CRM_Campaign_Form_Survey {
    * Set default values for the form. Note that in edit/view mode
    * the default values are retrieved from the database
    *
-   * @param null
-   *
    * @return array
    *   array of default values
    */
@@ -100,8 +98,6 @@ class CRM_Campaign_Form_Survey_Results extends CRM_Campaign_Form_Survey {
   /**
    * Build the form object
    *
-   * @param null
-   *
    * @return void
    */
   public function buildQuickForm() {
@@ -344,8 +340,6 @@ class CRM_Campaign_Form_Survey_Results extends CRM_Campaign_Form_Survey {
   /**
    * Process the form
    *
-   * @param null
-   *
    * @return void
    */
   public function postProcess() {
@@ -415,23 +409,21 @@ class CRM_Campaign_Form_Survey_Results extends CRM_Campaign_Form_Survey {
     if (!$this->_reportId && $survey->id && !empty($params['create_report'])) {
       $activityStatus = CRM_Core_PseudoConstant::activityStatus('name');
       $activityStatus = array_flip($activityStatus);
-      $this->_params =
-        array(
-          'name' => "survey_{$survey->id}",
-          'title' => $params['report_title'] ? $params['report_title'] : $this->_values['title'],
-          'status_id_op' => 'eq',
-          'status_id_value' => $activityStatus['Scheduled'], // reserved status
-          'survey_id_value' => array($survey->id),
-          'description' => ts('Detailed report for canvassing, phone-banking, walk lists or other surveys.'),
-        );
+      $this->_params = array(
+        'name' => "survey_{$survey->id}",
+        'title' => $params['report_title'] ? $params['report_title'] : $this->_values['title'],
+        'status_id_op' => 'eq',
+        'status_id_value' => $activityStatus['Scheduled'], // reserved status
+        'survey_id_value' => array($survey->id),
+        'description' => ts('Detailed report for canvassing, phone-banking, walk lists or other surveys.'),
+      );
       //Default value of order by
-      $this->_params['order_bys'] =
-        array(
-          1 => array(
-            'column' => 'sort_name',
-            'order' => 'ASC',
-          ),
-        );
+      $this->_params['order_bys'] = array(
+        1 => array(
+          'column' => 'sort_name',
+          'order' => 'ASC',
+        ),
+      );
       // for WalkList or default
       $displayFields = array(
         'id',
@@ -445,25 +437,24 @@ class CRM_Campaign_Form_Survey_Results extends CRM_Campaign_Form_Survey {
       if (CRM_Core_OptionGroup::getValue('activity_type', 'WalkList') ==
         $this->_values['activity_type_id']
       ) {
-        $this->_params['order_bys'] =
-          array(
-            1 => array(
-              'column' => 'street_name',
-              'order' => 'ASC',
-            ),
-            2 => array(
-              'column' => 'street_number_odd_even',
-              'order' => 'ASC',
-            ),
-            3 => array(
-              'column' => 'street_number',
-              'order' => 'ASC',
-            ),
-            4 => array(
-              'column' => 'sort_name',
-              'order' => 'ASC',
-            ),
-          );
+        $this->_params['order_bys'] = array(
+          1 => array(
+            'column' => 'street_name',
+            'order' => 'ASC',
+          ),
+          2 => array(
+            'column' => 'street_number_odd_even',
+            'order' => 'ASC',
+          ),
+          3 => array(
+            'column' => 'street_number',
+            'order' => 'ASC',
+          ),
+          4 => array(
+            'column' => 'sort_name',
+            'order' => 'ASC',
+          ),
+        );
       }
       elseif (CRM_Core_OptionGroup::getValue('activity_type', 'PhoneBank') ==
         $this->_values['activity_type_id']
index 26bee4b885325e1b8773ca31e06ce23b15e63e31..4e460c7e3669d76e70cc8901e3e1f0c8f421280b 100644 (file)
@@ -44,7 +44,7 @@ class CRM_Case_StateMachine_Search extends CRM_Core_StateMachine {
   /**
    * Class constructor
    */
-  function __construct($controller, $action = CRM_Core_Action::NONE) {
+  public function __construct($controller, $action = CRM_Core_Action::NONE) {
     parent::__construct($controller, $action);
 
     $this->_pages = array();
index 003d1b6e1b8c3f806aa4dfaa4122ccab6548152f..916f22b041da15935dbb055d15a5a4da76a189aa 100644 (file)
@@ -69,7 +69,7 @@ class CRM_Contact_Form_Edit_Email {
       $form->addSelect("email[$blockId][location_type_id]", array(
           'entity' => 'email',
           'class' => 'eight',
-          'placeholder' => NULL
+          'placeholder' => NULL,
         ));
 
       $multipleBulk = CRM_Core_BAO_Email::isMultipleBulkMail();
index 54ed4e595d2224c50bf3ff91eca73d14695ce93f..e21c6479ece0f6b562a7ff8661e3701732976f6f 100644 (file)
@@ -158,7 +158,7 @@ class CRM_Contact_Form_Search_Custom_ActivitySearch implements CRM_Contact_Form_
   /**
    * Construct the search query
    */
-  function all(
+  public function all(
     $offset = 0, $rowcount = 0, $sort = NULL,
     $includeContactIDs = FALSE, $justIDs = FALSE
   ) {
index cad57bfc0b2c39075e822cece526472eeb76e036..f537e23ca0c6d74d65682772f56fdfd69024a1f9 100644 (file)
@@ -62,7 +62,7 @@ class CRM_Contact_Form_Task_AddToHousehold extends CRM_Contact_Form_Task {
 
     $this->add('select', 'relationship_type_id', ts('Relationship Type'),
       array(
-        '' => ts('- select -')
+        '' => ts('- select -'),
       ) +
       CRM_Contact_BAO_Relationship::getRelationType("Household"), TRUE
     );
@@ -164,27 +164,27 @@ class CRM_Contact_Form_Task_AddToHousehold extends CRM_Contact_Form_Task {
             'count' => $valid,
             'plural' => '%count %2 %3 relationships created',
             2 => $relationship,
-            3 => $house
-          ))
+            3 => $house,
+        )),
       );
       if ($duplicate) {
         $status[] = ts('%count was skipped because the contact is already %2 %3', array(
             'count' => $duplicate,
             'plural' => '%count were skipped because the contacts are already %2 %3',
             2 => $relationship,
-            3 => $house
+            3 => $house,
           ));
       }
       if ($invalid) {
         $status[] = ts('%count relationship was not created because the contact is not of the right type for this relationship', array(
             'count' => $invalid,
-            'plural' => '%count relationships were not created because the contact is not of the right type for this relationship'
+            'plural' => '%count relationships were not created because the contact is not of the right type for this relationship',
           ));
       }
       $status = '<ul><li>' . implode('</li><li>', $status) . '</li></ul>';
       CRM_Core_Session::setStatus($status, ts('Relationship created.', array(
             'count' => $valid,
-            'plural' => 'Relationships created.'
+            'plural' => 'Relationships created.',
           )), 'success', array('expires' => 0));
     }
   }
@@ -285,8 +285,7 @@ class CRM_Contact_Form_Task_AddToHousehold extends CRM_Contact_Form_Task {
 
         $contact_type = '<img src="' . $config->resourceBase . 'i/contact_';
 
-        $searchRows[$contactID]['type'] = CRM_Contact_BAO_Contact_Utils::getImage($result->contact_sub_type ?
-            $result->contact_sub_type : $result->contact_type
+        $searchRows[$contactID]['type'] = CRM_Contact_BAO_Contact_Utils::getImage($result->contact_sub_type ? $result->contact_sub_type : $result->contact_type
         );
       }
 
index 0b4995e53da58d80090ebe3802826b399cc8513c..270e6aaeb6884e28555ce1df85dab7df8f748979 100644 (file)
@@ -62,7 +62,7 @@ class CRM_Contact_Form_Task_Print extends CRM_Contact_Form_Task {
           '=',
           1,
           0,
-          0
+          0,
         );
       }
     }
index 935409a302ea733425a23ba773f5cbfa0982fee6..44e435576819f800c6b06ced48dc34c531de7b82 100644 (file)
@@ -51,7 +51,7 @@ class CRM_Contact_Form_Task_SaveSearch extends CRM_Contact_Form_Task {
    *
    * @return void
    */
-  function preProcess() {
+  public function preProcess() {
     $this->_id = NULL;
 
     // get the submitted values of the search form
@@ -95,7 +95,6 @@ class CRM_Contact_Form_Task_SaveSearch extends CRM_Contact_Form_Task {
       CRM_Core_DAO::getAttribute('CRM_Contact_DAO_Group', 'title'), TRUE
     );
 
-
     $this->addElement('textarea', 'description', ts('Description'),
       CRM_Core_DAO::getAttribute('CRM_Contact_DAO_Group', 'description')
     );
index e69a98a79f7495af67dce4228c7ae003b5eb9f42..2056235b6d5b1ddfa3766bc216abfcc1499a2e3d 100644 (file)
@@ -48,7 +48,7 @@ class CRM_Contribute_Page_ContributionRecur extends CRM_Core_Page {
    *
    * @return void
    */
-  function view() {
+  public function view() {
     $recur = new CRM_Contribute_DAO_ContributionRecur();
     $recur->id = $this->_id;
     if ($recur->find(TRUE)) {
index a31f1883a96b6943b77e8fd9c814ea20a3ded79b..e8071a7d73504e142b81d248c799f7f7fd7be8b1 100644 (file)
@@ -200,7 +200,7 @@ class CRM_Core_Action {
    * @return string
    *   the html string
    */
-  static function formLink(
+  public static function formLink(
     $links,
     $mask,
     $values,
@@ -321,8 +321,6 @@ class CRM_Core_Action {
   /**
    * Get the mask for a permission (view, edit or null)
    *
-   * @param string the permission
-   *
    * @return int
    *   the mask for the above permission
    */
index fa6d928a00251d3c911e301e9b2d3611dc62532c..eb5f60a02aa5843ea2699d6649105e446292dd71 100644 (file)
@@ -273,7 +273,7 @@ class CRM_Core_InnoDBIndexer {
   }
 
   /**
-   * @return boolean
+   * @return bool
    */
   public function getActive() {
     return $this->isActive;
index 017f2c219de080ba72535f40b789de3732c8b7b6..7d3bccf0e5226316100fb5530ebc016c1b3ba72e 100644 (file)
@@ -71,36 +71,36 @@ class CRM_Core_Page_Angular extends CRM_Core_Page {
     $angularModules = array();
     $angularModules['angularFileUpload'] = array(
       'ext' => 'civicrm',
-      'js' => array('bower_components/angular-file-upload/angular-file-upload.min.js')
+      'js' => array('bower_components/angular-file-upload/angular-file-upload.min.js'),
     );
     $angularModules['crmApp'] = array('ext' => 'civicrm', 'js' => array('js/angular-crmApp.js'));
     $angularModules['crmAttachment'] = array(
       'ext' => 'civicrm',
       'js' => array('js/angular-crmAttachment.js'),
-      'css' => array('css/angular-crmAttachment.css')
+      'css' => array('css/angular-crmAttachment.css'),
     );
     $angularModules['crmUi'] = array(
       'ext' => 'civicrm',
-      'js' => array('js/angular-crm-ui.js', 'packages/ckeditor/ckeditor.js')
+      'js' => array('js/angular-crm-ui.js', 'packages/ckeditor/ckeditor.js'),
     );
     $angularModules['crmUtil'] = array('ext' => 'civicrm', 'js' => array('js/angular-crm-util.js'));
     // https://github.com/jwstadler/angular-jquery-dialog-service
     $angularModules['dialogService'] = array(
       'ext' => 'civicrm',
-      'js' => array('bower_components/angular-jquery-dialog-service/dialog-service.js')
+      'js' => array('bower_components/angular-jquery-dialog-service/dialog-service.js'),
     );
     $angularModules['ngSanitize'] = array('ext' => 'civicrm', 'js' => array('js/angular-sanitize.js'));
     $angularModules['ui.utils'] = array(
       'ext' => 'civicrm',
-      'js' => array('bower_components/angular-ui-utils/ui-utils.min.js')
+      'js' => array('bower_components/angular-ui-utils/ui-utils.min.js'),
     );
     $angularModules['ui.sortable'] = array(
       'ext' => 'civicrm',
-      'js' => array('bower_components/angular-ui-sortable/sortable.min.js')
+      'js' => array('bower_components/angular-ui-sortable/sortable.min.js'),
     );
     $angularModules['unsavedChanges'] = array(
       'ext' => 'civicrm',
-      'js' => array('bower_components/angular-unsavedChanges/dist/unsavedChanges.min.js')
+      'js' => array('bower_components/angular-unsavedChanges/dist/unsavedChanges.min.js'),
     );
 
     foreach (CRM_Core_Component::getEnabledComponents() as $component) {
index 2c5cce2ebefdcfe6dbdf54358ec05cc9318bd18d..62fea0f40c2bc1fcbe1dbfecb2e6c700a6cf2fc9 100644 (file)
@@ -45,8 +45,7 @@ abstract class CRM_Core_Page_Basic extends CRM_Core_Page {
    *
    * @return string
    */
-
-  abstract function getBAOName();
+  abstract protected function getBAOName();
 
   /**
    * An array of action links
@@ -54,21 +53,21 @@ abstract class CRM_Core_Page_Basic extends CRM_Core_Page {
    * @return array
    *   (reference)
    */
-  abstract function &links();
+  abstract protected function &links();
 
   /**
    * Name of the edit form class
    *
    * @return string
    */
-  abstract function editForm();
+  abstract protected function editForm();
 
   /**
    * Name of the form
    *
    * @return string
    */
-  abstract function editName();
+  abstract protected function editName();
 
   /**
    * UserContext to pop back to
@@ -78,7 +77,7 @@ abstract class CRM_Core_Page_Basic extends CRM_Core_Page {
    *
    * @return string
    */
-  abstract function userContext($mode = NULL);
+  abstract protected function userContext($mode = NULL);
 
   /**
    * Get userContext params
@@ -300,7 +299,7 @@ abstract class CRM_Core_Page_Basic extends CRM_Core_Page {
     if (!empty($values['name']) && in_array($values['name'], array(
         'encounter_medium',
         'case_type',
-        'case_status'
+        'case_status',
       ))
     ) {
       static $caseCount = NULL;
index 1ba3d48227609927d3ce036c68609d633de67c33..0587c681d6a6530fed4e257870a273d8c81ccf2b 100644 (file)
@@ -22,7 +22,7 @@
  * WARY of this when coding
  *
  * -----------------------------------------------------------------------------------------------
- **/
+ */
 class CRM_Core_Payment_Elavon extends CRM_Core_Payment {
   // (not used, implicit in the API, might need to convert?)
   const
index 4fb9aa912d0350b0cd024aff3e79a73620bc553b..089838d49e7cea29ad90b0fdf5b83f68a797ad67 100644 (file)
@@ -29,7 +29,7 @@
  * If an result of class error is returned it is treated as a failure
  *
  * -----------------------------------------------------------------------------------------------
- **/
+ */
 
 /**
  * From Payment processor documentation
@@ -63,7 +63,7 @@ class CRM_Core_Payment_FirstData extends CRM_Core_Payment {
    */
   static private $_singleton = NULL;
 
-  /**********************************************************
+  /**
    * Constructor
    *
    * @param string $mode
@@ -79,12 +79,12 @@ class CRM_Core_Payment_FirstData extends CRM_Core_Payment {
     $this->_paymentProcessor = $paymentProcessor;
   }
 
-  /**********************************************************
+  /**
    * This function is set up and put here to make the mapping of fields
    * from the params object  as visually clear as possible for easy editing
    *
    *  Comment out irrelevant fields
-   **********************************************************/
+   */
   public function mapProcessorFieldstoParams($params) {
     /*concatenate full customer name first  - code from EWAY gateway
          */
@@ -139,10 +139,10 @@ class CRM_Core_Payment_FirstData extends CRM_Core_Payment {
     return $requestFields;
   }
 
-  /**********************************************************
+  /**
    * This function sends request and receives response from
    * the processor
-   **********************************************************/
+   */
   public function doDirectPayment(&$params) {
     if ($params['is_recur'] == TRUE) {
       CRM_Core_Error::fatal(ts('%1 - recurring payments not implemented', array(1 => $paymentProcessor)));
@@ -315,9 +315,9 @@ class CRM_Core_Payment_FirstData extends CRM_Core_Payment {
     return $contribution->find();
   }
 
-  /**************************************************
+  /**
    * Produces error message and returns from class
-   **************************************************/
+   */
   public function &errorExit($errorCode = NULL, $errorMessage = NULL) {
     $e = CRM_Core_Error::singleton();
 
@@ -330,14 +330,14 @@ class CRM_Core_Payment_FirstData extends CRM_Core_Payment {
     return $e;
   }
 
-  /**************************************************
+  /**
    * NOTE: 'doTransferCheckout' not implemented
-   **************************************************/
+   */
   public function doTransferCheckout(&$params, $component) {
     CRM_Core_Error::fatal(ts('This function is not implemented'));
   }
 
-  /********************************************************************************************
+  /**
    * This public function checks to see if we have the right processor config values set
    *
    * NOTE: Called by Events and Contribute to check config params are set prior to trying
@@ -348,10 +348,9 @@ class CRM_Core_Payment_FirstData extends CRM_Core_Payment {
    *
    * returns string $errorMsg if any errors found - null if OK
    *
-   ******************************************************************************************
+   *  function checkConfig( $mode )           CiviCRM V1.9 Declaration
+   * CiviCRM V2.0 Declaration
    */
-  //  function checkConfig( $mode )          // CiviCRM V1.9 Declaration
-  // CiviCRM V2.0 Declaration
   public function checkConfig() {
     $errorMsg = array();
 
index 96d45ccf23996cc93d4b4d3145343394d1af2464..b00c4fbe5d12d1068f65a001a3f53d37d7ba640b 100644 (file)
@@ -163,7 +163,7 @@ class CRM_Core_Payment_PayPalProIPN extends CRM_Core_Payment_BaseIPN {
    * @param array $ids
    * @param array $objects
    * @param bool $first
-   * @return void|boolean
+   * @return void|bool
    */
   public function recur(&$input, &$ids, &$objects, $first) {
     if (!isset($input['txnType'])) {
@@ -227,7 +227,7 @@ class CRM_Core_Payment_PayPalProIPN extends CRM_Core_Payment_BaseIPN {
       case 'recurring_payment_profile_created':
         if (in_array($recur->contribution_status_id, array(
               array_search('Pending', $contributionStatuses),
-              array_search('In Progress', $contributionStatuses)
+              array_search('In Progress', $contributionStatuses),
             ))
           && !empty($recur->processor_id)
         ) {
@@ -389,14 +389,14 @@ class CRM_Core_Payment_PayPalProIPN extends CRM_Core_Payment_BaseIPN {
    * (with the input parameters) & call this & all will be done
    *
    * @todo the references to POST throughout this class need to be removed
-   * @return void|boolean
+   * @return void|bool
    */
   public function main() {
     CRM_Core_Error::debug_var('GET', $_GET, TRUE, TRUE);
     CRM_Core_Error::debug_var('POST', $_POST, TRUE, TRUE);
     if ($this->_isPaymentExpress) {
       $this->handlePaymentExpress();
-      return;
+      return NULL;
     }
     $objects = $ids = $input = array();
     $this->_component = $input['component'] = self::getValue('m');
@@ -542,7 +542,7 @@ INNER JOIN civicrm_membership_payment mp ON m.id = mp.membership_id AND mp.contr
 
     $contributionRecur = civicrm_api3('contribution_recur', 'getsingle', array(
         'return' => 'contact_id, id',
-        'invoice_id' => $input['invoice']
+        'invoice_id' => $input['invoice'],
       ));
     $ids['contact'] = $contributionRecur['contact_id'];
     $ids['contributionRecur'] = $contributionRecur['id'];
@@ -557,8 +557,8 @@ INNER JOIN civicrm_membership_payment mp ON m.id = mp.membership_id AND mp.contr
     $ids['contributionPage'] = CRM_Core_DAO::singleValueQuery("SELECT contribution_page_id FROM civicrm_contribution WHERE invoice_id = %1", array(
         1 => array(
           $ids['contribution'],
-          'Integer'
-        )
+          'Integer',
+        ),
       ));
     // only handle component at this stage - not terribly sure how a recurring event payment would arise
     // & suspec main function may be a victom of copy & paste
index 86439c42a723a15daed575e10897c65cdea32870..00b5db2ac66b1f344b40a39ab9b8e75be834df0b 100644 (file)
@@ -36,13 +36,13 @@ class CRM_Event_BAO_ParticipantStatusType extends CRM_Event_DAO_ParticipantStatu
   /**
    * @param array $params
    *
-   * @return $this|null
+   * @return this|null
    */
   public static function add(&$params) {
     if (empty($params)) {
       return NULL;
     }
-    $dao = new CRM_Event_DAO_ParticipantStatusType;
+    $dao = new CRM_Event_DAO_ParticipantStatusType();
     $dao->copyValues($params);
     return $dao->save();
   }
@@ -50,7 +50,7 @@ class CRM_Event_BAO_ParticipantStatusType extends CRM_Event_DAO_ParticipantStatu
   /**
    * @param array $params
    *
-   * @return $this|null
+   * @return this|null
    */
   public static function &create(&$params) {
     $transaction = new CRM_Core_Transaction();
@@ -70,7 +70,7 @@ class CRM_Event_BAO_ParticipantStatusType extends CRM_Event_DAO_ParticipantStatu
    */
   public static function deleteParticipantStatusType($id) {
     // return early if there are participants with this status
-    $participant = new CRM_Event_DAO_Participant;
+    $participant = new CRM_Event_DAO_Participant();
     $participant->status_id = $id;
     if ($participant->find()) {
       return FALSE;
@@ -78,7 +78,7 @@ class CRM_Event_BAO_ParticipantStatusType extends CRM_Event_DAO_ParticipantStatu
 
     CRM_Utils_Weight::delWeight('CRM_Event_DAO_ParticipantStatusType', $id);
 
-    $dao = new CRM_Event_DAO_ParticipantStatusType;
+    $dao = new CRM_Event_DAO_ParticipantStatusType();
     $dao->id = $id;
     $dao->find(TRUE);
     $dao->delete();
@@ -94,7 +94,7 @@ class CRM_Event_BAO_ParticipantStatusType extends CRM_Event_DAO_ParticipantStatu
   public static function retrieve(&$params, &$defaults) {
     $result = NULL;
 
-    $dao = new CRM_Event_DAO_ParticipantStatusType;
+    $dao = new CRM_Event_DAO_ParticipantStatusType();
     $dao->copyValues($params);
     if ($dao->find(TRUE)) {
       CRM_Core_DAO::storeValues($dao, $defaults);
index f5e59dad35d39d94cf9814cb2e359dd55fb3f2fb..cb77e25ed0d873c85af4f196b368008d5bb28b34 100644 (file)
@@ -51,7 +51,7 @@ class CRM_Event_Badge {
       'cap' => 'round',
       'join' => 'round',
       'dash' => '2,2',
-      'color' => array(0, 0, 200)
+      'color' => array(0, 0, 200),
     );
     $this->format = '5160';
     $this->imgExtension = 'png';
@@ -80,7 +80,7 @@ class CRM_Event_Badge {
    *
    * @param array $participants
    *
-   * @return null
+   * @return;
    */
   public function run(&$participants) {
     // fetch the 1st participant, and take her event to retrieve its attributes
@@ -154,8 +154,8 @@ class CRM_Event_Badge {
             'cap' => 'round',
             'join' => 'round',
             'dash' => '2,10',
-            'color' => array(255, 0, 0)
-          )
+            'color' => array(255, 0, 0),
+          ),
         ));
     }
     $img = $this->getImageFileName($this->event->id, $img);
@@ -172,7 +172,7 @@ class CRM_Event_Badge {
 
   /**
    * This is supposed to be overrided
-   **/
+   */
   public function generateLabel($participant) {
     $txt = "{$this->event['title']}
 {$participant['display_name']}
@@ -189,7 +189,7 @@ class CRM_Event_Badge {
    *
    * @param array $participants
    *
-   * @return null
+   * @return;
    */
   public function createLabels(&$participants) {
 
index 3ba7db4b3a2a295d1bc35706990e86d08f223445..72640c0a1b1d42b1c48e6c03dc66336103e78611 100644 (file)
@@ -248,12 +248,12 @@ class CRM_Event_Form_ManageEvent_Registration extends CRM_Event_Form_ManageEvent
       ts('Allow Online Registration'),
       NULL,
       array(
-        'onclick' => "return showHideByValue('is_online_registration'," . 
-          "''," .
-          "'registration_blocks'," .
-          "'block'," .
-          "'radio'," .
-          "false );",
+        'onclick' => "return showHideByValue('is_online_registration'," .
+        "''," .
+        "'registration_blocks'," .
+        "'block'," .
+        "'radio'," .
+        "false );",
       )
     );
 
index 8cb666679a953ee44e1b8a5429d6f8317e6f7a6c..85b7693570f3527d3a97fb883e437f4955efc454 100644 (file)
@@ -104,7 +104,7 @@ class CRM_Event_Import_Form_Summary extends CRM_Import_Form_Summary {
       'downloadDuplicateRecordsUrl',
       'downloadMismatchRecordsUrl',
       'groupAdditions',
-      'unMatchCount'
+      'unMatchCount',
     );
     foreach ($properties as $property) {
       $this->assign($property, $this->get($property));
index 71d0704f556b5a18b082c6acb0ef2fb8c89c537a..2a0cf10f82f1289d2d38b62003b00debbb7df10a 100644 (file)
@@ -63,7 +63,7 @@ WHERE  mailing_id = %1
    *
    * @return Object
    */
-  static function mailingQuery(
+  public static function mailingQuery(
     $mailingID,
     $offset = NULL, $limit = NULL
   ) {
index fe2606c2267ae1a4ad6f8c91199e1d9543472c3e..2eb5e68da33248546e5e173a59c26a49258ca5bd 100755 (executable)
@@ -280,7 +280,7 @@ class CRM_Mailing_Event_BAO_Delivered extends CRM_Mailing_Event_DAO_Delivered {
    *   Consider mailings that were completed not more than $maxDays ago.
    *
    * @return void
-   **/
+   */
   public static function updateEmailResetDate($minDays = 3, $maxDays = 7) {
     $dao = new CRM_Core_Dao();
 
index 315e0506d2a426f3528271c894c95c26cf9b2f5c..aa7f6290ca14bc53c8a2ee66c487eef19605fb22 100644 (file)
@@ -112,7 +112,7 @@ class CRM_Member_Import_Parser_Membership extends CRM_Member_Import_Parser {
    * @param array $values
    *   The array of values belonging to this line.
    *
-   * @return boolean
+   * @return bool
    */
   public function mapField(&$values) {
     return CRM_Import_Parser::VALID;
@@ -124,7 +124,7 @@ class CRM_Member_Import_Parser_Membership extends CRM_Member_Import_Parser {
    * @param array $values
    *   The array of values belonging to this line.
    *
-   * @return boolean
+   * @return bool
    *   the result of this processing
    */
   public function preview(&$values) {
@@ -137,7 +137,7 @@ class CRM_Member_Import_Parser_Membership extends CRM_Member_Import_Parser {
    * @param array $values
    *   The array of values belonging to this line.
    *
-   * @return boolean
+   * @return bool
    *   the result of this processing
    */
   public function summary(&$values) {
@@ -254,7 +254,7 @@ class CRM_Member_Import_Parser_Membership extends CRM_Member_Import_Parser {
    * @param array $values
    *   The array of values belonging to this line.
    *
-   * @return boolean
+   * @return bool
    *   the result of this processing
    */
   public function import($onDuplicate, &$values) {
@@ -594,7 +594,7 @@ class CRM_Member_Import_Parser_Membership extends CRM_Member_Import_Parser {
    *
    * @param $formatted
    *
-   * @return Array
+   * @return;
    *   formatted containing date values
    */
   public function formattedDates($calcDates, &$formatted) {
index f602508865bd0a92f1cfa4f5d77a7a98e74d446a..b2bf44f66528b965590444ded002b59826920139 100644 (file)
@@ -350,7 +350,7 @@ WHERE pcp.id = %1 AND cc.contribution_status_id =1 AND cc.is_test = 0";
    * @param int $id
    *   Campaign page id.
    *
-   * @return null
+   * @return;
    */
   public static function deleteById($id) {
     CRM_Utils_Hook::pre('delete', 'Campaign', $id, CRM_Core_DAO::$_nullArray);
@@ -582,7 +582,7 @@ WHERE pcp.id = %1 AND cc.contribution_status_id =1 AND cc.is_test = 0";
    *
    * @param $is_active
    *
-   * @return null
+   * @return;
    */
   public static function setIsActive($id, $is_active) {
     switch ($is_active) {
@@ -778,7 +778,7 @@ WHERE pcp.id = %1 AND cc.contribution_status_id =1 AND cc.is_test = 0";
    * @param int $id
    *   Pcp block id.
    *
-   * @return Boolean
+   * @return Bool
    */
   public static function getPcpBlockInUse($id) {
     $query = "
@@ -797,7 +797,7 @@ WHERE pcp.id = %1 AND cc.contribution_status_id =1 AND cc.is_test = 0";
    * @param int $profileId
    *   Supporter's profile id.
    *
-   * @return boolean
+   * @return bool
    */
   public static function checkEmailProfile($profileId) {
     $query = "
@@ -847,7 +847,7 @@ WHERE field_name like 'email%' And is_active = 1 And uf_group_id = %1";
    * @param int $pcpId
    * @param $component
    *
-   * @return String
+   * @return string
    */
   public static function getPcpBlockEntityId($pcpId, $component) {
     $entity_table = self::getPcpEntityTable($component);
@@ -872,7 +872,7 @@ WHERE pcp.id = %1";
    *
    * @param $component
    *
-   * @return String
+   * @return string
    */
   public static function getPcpEntityTable($component) {
     $entity_table_map = array(
index 8024e067faa6ec38aa59b48613c323359c2df089..dcd130087786356b31b0cf6138faf3e3d9729465 100644 (file)
@@ -106,7 +106,7 @@ class CRM_PCP_Form_Event extends CRM_Event_Form_ManageEvent {
     $this->add('select', 'target_entity_id',
       ts('Online Contribution Page'),
       array(
-        '' => ts('- select -')
+        '' => ts('- select -'),
       ) +
       CRM_Contribute_PseudoConstant::contributionPage()
     );
@@ -122,7 +122,7 @@ class CRM_PCP_Form_Event extends CRM_Event_Form_ManageEvent {
     if (!empty($pcpBlock->id) && CRM_PCP_BAO_PCP::getPcpBlockInUse($pcpBlock->id)) {
       foreach (array(
                  'target_entity_type',
-                 'target_entity_id'
+                 'target_entity_id',
                ) as $element_name) {
         $element = $this->getElement($element_name);
         $element->freeze();
index 6e33a0b1da68ebd613acc5702c182deafa68086a..859063d959e311965923e4d722a514e5a48cc927 100644 (file)
@@ -201,7 +201,7 @@ class CRM_Price_BAO_PriceField extends CRM_Price_DAO_PriceField {
     if (!empty($fieldOptions['is_full'])) {
       $element->freeze();
     }
-    return;
+    return NULL;
   }
 
   /**
@@ -309,7 +309,7 @@ class CRM_Price_BAO_PriceField extends CRM_Price_DAO_PriceField {
         $priceVal = implode($seperator, array(
             $customOption[$optionKey][$valueFieldName] + $taxAmount,
             $count,
-            $max_value
+            $max_value,
           ));
 
         $extra = array();
@@ -326,7 +326,8 @@ class CRM_Price_BAO_PriceField extends CRM_Price_DAO_PriceField {
         if (!empty($qf->_membershipBlock) && !empty($qf->_quickConfig) && $field->name == 'other_amount' && empty($qf->_contributionAmount)) {
           $useRequired = 0;
         }
-        elseif (!empty($fieldOptions[$optionKey]['label'])) {      //check for label.
+        elseif (!empty($fieldOptions[$optionKey]['label'])) {
+          //check for label.
           $label = $fieldOptions[$optionKey]['label'];
         }
 
@@ -500,7 +501,7 @@ class CRM_Price_BAO_PriceField extends CRM_Price_DAO_PriceField {
 
         $element = &$qf->add('select', $elementName, $label,
           array(
-            '' => ts('- select -')
+            '' => ts('- select -'),
           ) + $selectOption,
           $useRequired && $field->is_required,
           array('price' => json_encode($priceVal))
@@ -622,7 +623,7 @@ WHERE
 
     $dao = CRM_Core_DAO::executeQuery($query, array(
         1 => array($optionGroupName, 'String'),
-        2 => array($optionLabel, 'String')
+        2 => array($optionLabel, 'String'),
       ));
 
     while ($dao->fetch()) {
@@ -636,7 +637,7 @@ WHERE
    * @param int $id
    *   Field Id.
    *
-   * @return boolean
+   * @return bool
    *
    */
   public static function deleteField($id) {
@@ -688,7 +689,6 @@ WHERE
    * @param bool $allowNoneSelection
    *
    */
-
   public static function priceSetValidation($priceSetId, $fields, &$error, $allowNoneSelection = FALSE) {
     // check for at least one positive
     // amount price field should be selected.
index 8d4303359da287ca77bd726280eeacbe7d5792a9..295b5043c1f3c3a9093ab191a7c092a76de34e85 100644 (file)
@@ -361,7 +361,7 @@ class CRM_Profile_Form extends CRM_Core_Form {
         elseif (!empty($this->_multiRecordFields)
           && (!$this->_multiRecord || !in_array($this->_multiRecord, array(
                 CRM_Core_Action::DELETE,
-                CRM_Core_Action::UPDATE
+                CRM_Core_Action::UPDATE,
               )))
         ) {
           CRM_Core_Resources::singleton()->addScriptFile('civicrm', 'js/crm.livePage.js', 1, 'html-header');
@@ -667,8 +667,7 @@ class CRM_Profile_Form extends CRM_Core_Form {
         ) {
           $return = TRUE;
           if (!$statusMessage) {
-            $statusMessage =
-              ts("This profile is configured for contact type '%1'. It cannot be used to edit contacts of other types.",
+            $statusMessage = ts("This profile is configured for contact type '%1'. It cannot be used to edit contacts of other types.",
                 array(1 => $profileSubType ? $profileSubType : $profileType));
           }
         }
@@ -965,24 +964,24 @@ class CRM_Profile_Form extends CRM_Core_Form {
             $duplicateContactsLinks = '<div class="matching-contacts-found">';
             $duplicateContactsLinks .= ts('One matching contact was found. ', array(
                 'count' => count($contactLinks['rows']),
-                'plural' => '%count matching contacts were found.<br />'
+                'plural' => '%count matching contacts were found.<br />',
               ));
             if ($contactLinks['msg'] == 'view') {
               $duplicateContactsLinks .= ts('You can View the existing contact.', array(
                   'count' => count($contactLinks['rows']),
-                  'plural' => 'You can View the existing contacts.'
+                  'plural' => 'You can View the existing contacts.',
                 ));
             }
             else {
               $duplicateContactsLinks .= ts('You can View or Edit the existing contact.', array(
                   'count' => count($contactLinks['rows']),
-                  'plural' => 'You can View or Edit the existing contacts.'
+                  'plural' => 'You can View or Edit the existing contacts.',
                 ));
             }
             $duplicateContactsLinks .= '</div>';
             $duplicateContactsLinks .= '<table class="matching-contacts-actions">';
             $row = '';
-            for ($i = 0; $i < sizeof($contactLinks['rows']); $i++) {
+            for ($i = 0; $i < count($contactLinks['rows']); $i++) {
               $row .= '  <tr>   ';
               $row .= '    <td class="matching-contacts-name"> ';
               $row .= $contactLinks['rows'][$i]['display_name'];
index c1f94073b9cff90dff145f6f512ee49fa7ee582f..e7b5320b467b6fb3d9f37f3f8de94edc43f6a2c1 100644 (file)
@@ -36,7 +36,7 @@ class CRM_Report_Form_Contribute_LoggingDetail extends CRM_Logging_ReportDetail
   /**
    */
   public function __construct() {
-    $logging = new CRM_Logging_Schema;
+    $logging = new CRM_Logging_Schema();
     $this->tables[] = 'civicrm_contribution';
     $this->tables = array_merge($this->tables, array_keys($logging->customDataLogTables()));
 
index f6f2227f54ee925e22ef618a12fe8abd3f94199c..9fcd9dd1e16092087c389ba4be92b0fd66cad11d 100644 (file)
@@ -250,12 +250,12 @@ class CRM_Report_Form_Extended extends CRM_Report_Form {
         // are grouped together across price sets but there may be a separate need to group
         // by id so that entries in one price set are distinct from others. Not quite sure what
         // to call the distinction for end users benefit
-          array(
-            'price_field_value_label' => array(
-              'title' => ts('Price Field Value Label'),
-              'name' => 'label',
-            ),
+        array(
+          'price_field_value_label' => array(
+            'title' => ts('Price Field Value Label'),
+            'name' => 'label',
           ),
+        ),
       ),
     );
   }
@@ -596,8 +596,7 @@ class CRM_Report_Form_Extended extends CRM_Report_Form {
         'filters' => array(
           'id' => array(
             'title' => ts('Contact ID'),
-          )
-        ,
+          ),
           'sort_name' => array(
             'title' => ts('Contact Name'),
           ),
@@ -1222,7 +1221,7 @@ WHERE   line_item_civireport.id IS NOT NULL
    */
   public function alterNickName($value, &$row) {
     if (empty($row['civicrm_contact_id'])) {
-      return;
+      return NULL;
     }
     $contactID = $row['civicrm_contact_id'];
     return "<div id=contact-{$contactID} class='crm-entity'>
@@ -1374,7 +1373,7 @@ WHERE   line_item_civireport.id IS NOT NULL
    */
   public function alterParticipantStatus($value) {
     if (empty($value)) {
-      return;
+      return NULL;
     }
     return CRM_Event_PseudoConstant::participantStatus($value, FALSE, 'label');
   }
@@ -1386,7 +1385,7 @@ WHERE   line_item_civireport.id IS NOT NULL
    */
   public function alterParticipantRole($value) {
     if (empty($value)) {
-      return;
+      return NULL;
     }
     $roles = explode(CRM_Core_DAO::VALUE_SEPARATOR, $value);
     $value = array();
index 0468f4d447033729fb848d8cc0befda759a0e865..11c169f204ec90158f8cc82fdd109e6b1b01b56b 100644 (file)
@@ -440,7 +440,7 @@ class CRM_Utils_Array {
    * @param array $array
    *   (optional) Array to be checked for emptiness.
    *
-   * @return boolean
+   * @return bool
    *   True if the array is empty.
    */
   public static function crmIsEmptyArray($array = array()) {
@@ -744,18 +744,18 @@ class CRM_Utils_Array {
    * @return array
    *   Each item is a distinct combination of values from $dimensions.
    *
-   * For example, the product of
-   * {
+   *   For example, the product of
+   *   {
    *   fg => {red, blue},
    *   bg => {white, black}
-   * }
-   * would be
-   * {
+   *   }
+   *   would be
+   *   {
    *   {fg => red, bg => white},
    *   {fg => red, bg => black},
    *   {fg => blue, bg => white},
    *   {fg => blue, bg => black}
-   * }
+   *   }
    */
   public static function product($dimensions, $template = array()) {
     if (empty($dimensions)) {
index 93c301e483884f554988a9d68c66f657be8faea0..a456f4538686f13821092aff335b666a2500d75d 100644 (file)
@@ -719,7 +719,7 @@ class CRM_Utils_Date {
    * 'day' => '25', 'month' => '10',
    *                              'year' => '2007' );
    *
-   * @param array $dayParamsArray of the day, month, year.
+   * @param array $dayParams of the day, month, year.
    *   Array of the day, month, year.
    *                             values.
    * @param string $format
@@ -1437,7 +1437,7 @@ class CRM_Utils_Date {
             $from['Y'] = $now['year'];
             $from['H'] = 00;
             $from['i'] = $to['s'] = 00;
-            $to = self::intervalAdd('month', +1, $from);
+            $to = self::intervalAdd('month', 1, $from);
             $to = self::intervalAdd('second', -1, $to);
             break;
         }
@@ -1537,7 +1537,7 @@ class CRM_Utils_Date {
             $from['M'] = $now['mon'];
             $from['Y'] = $now['year'];
             $from = self::intervalAdd('day', -1 * ($now['wday']) + 7, $from);
-            $to = self::intervalAdd('day', +6, $from);
+            $to = self::intervalAdd('day', 6, $from);
             break;
 
           case 'starting':
@@ -1546,7 +1546,7 @@ class CRM_Utils_Date {
             $from['Y'] = $now['year'];
             $from['H'] = 00;
             $from['i'] = $to['s'] = 00;
-            $to = self::intervalAdd('day', +7, $from);
+            $to = self::intervalAdd('day', 7, $from);
             $to = self::intervalAdd('second', -1, $to);
             break;
         }
@@ -1606,7 +1606,7 @@ class CRM_Utils_Date {
             $to['d'] = $now['mday'];
             $to['M'] = $now['mon'];
             $to['Y'] = $now['year'];
-            $to = self::intervalAdd('day', +1, $to);
+            $to = self::intervalAdd('day', 1, $to);
             $from['d'] = $to['d'];
             $from['M'] = $to['M'];
             $from['Y'] = $to['Y'];
@@ -1618,7 +1618,7 @@ class CRM_Utils_Date {
 
     foreach (array(
                'from',
-               'to'
+               'to',
              ) as $item) {
       if (!empty($$item)) {
         $dateRange[$item] = self::format($$item);
index 2694c092f2e7fb18bc9e0e1b6d12ed3104f30f01..1f416f6a024bc1a2b1a32706521c09fb7a84eb6a 100644 (file)
@@ -484,7 +484,7 @@ class CRM_Utils_Migrate_Export {
               'Individual',
               'Organization',
               'Household',
-              'Case'
+              'Case',
             ))) {
             if ($object->extends == 'Event') {
               $key = 'event_type';
index 658a96a202c25a7186fd724156b4c53fc7d83af9..dbe4740e3a721336fed4a0b6f26ebe63b05c8e84 100644 (file)
@@ -468,8 +468,10 @@ class CRM_Utils_REST {
     return CRM_Utils_System::redirect(CRM_Utils_System::url('civicrm/api/explorer'));
   }
 
-  /** used to load a template "inline", eg. for ajax, without having to build a menu for each template */
-  static function loadTemplate() {
+  /**
+   * used to load a template "inline", eg. for ajax, without having to build a menu for each template
+   */
+  public static function loadTemplate() {
     $request = CRM_Utils_Request::retrieve('q', 'String');
     if (FALSE !== strpos($request, '..')) {
       die ("SECURITY FATAL: the url can't contain '..'. Please report the issue on the forum at civicrm.org");
@@ -522,10 +524,11 @@ class CRM_Utils_REST {
     }
   }
 
-  /** This is a wrapper so you can call an api via json (it returns json too)
+  /**
+   * This is a wrapper so you can call an api via json (it returns json too)
    * http://example.org/civicrm/api/json?entity=Contact&action=Get"&json={"contact_type":"Individual","email.get.email":{}} to take all the emails from individuals
    * works for POST & GET (POST recommended)
-   **/
+   */
   public static function ajaxJson() {
     $requestParams = CRM_Utils_Request::exportValues();
 
@@ -559,8 +562,7 @@ class CRM_Utils_REST {
     if (!is_array($params)) {
       CRM_Utils_JSON::output(array(
           'is_error' => 1,
-          'error_message',
-          'invalid json format: ?{"param_with_double_quote":"value"}'
+          'error_message' => 'invalid json format: ?{"param_with_double_quote":"value"}',
         ));
     }
 
index 70c95607b6b17fa463a16251a5274c58446c319c..3ec8a101edc9afba3703b7db178a2bef7a8ec00c 100644 (file)
@@ -142,7 +142,7 @@ abstract class CRM_Utils_System_Base {
    * @return string
    *   Formatted url.
    */
-  function languageNegotiationURL(
+  public function languageNegotiationURL(
     $url,
     $addLanguagePart = TRUE,
     $removeLanguagePart = FALSE
@@ -177,7 +177,7 @@ abstract class CRM_Utils_System_Base {
    * @param string $username
    *
    * @throws CRM_Core_Exception
-   * @return int|NULL
+   * @return;
    */
   public function getUfId($username) {
     $className = get_class($this);
@@ -218,9 +218,9 @@ abstract class CRM_Utils_System_Base {
    * @param string $dir
    *
    * @return array
-   * - $url, (Joomla - non admin url)
-   * - $siteName,
-   * - $siteRoot
+   *   - $url, (Joomla - non admin url)
+   *   - $siteName,
+   *   - $siteRoot
    */
   public function getDefaultSiteSettings($dir) {
     $config = CRM_Core_Config::singleton();
@@ -311,7 +311,7 @@ abstract class CRM_Utils_System_Base {
   /**
    * Get currently logged in user uf id.
    *
-   * @return int
+   * @return;
    *   $userID logged in user uf id.
    */
   public function getLoggedInUfID() {
@@ -320,7 +320,7 @@ abstract class CRM_Utils_System_Base {
   /**
    * Get currently logged in user unique identifier - this tends to be the email address or user name.
    *
-   * @return string
+   * @return;
    *   logged in user unique identifier
    */
   public function getLoggedInUniqueIdentifier() {
index 525aa5d9743219b8b228f36a61fc1195ee650ca1..c8aa0e2408eee6b60958107547e8c9532e96fb4a 100644 (file)
@@ -56,7 +56,7 @@ class CRM_Utils_System_Soap extends CRM_Utils_System_Base {
    * @return void
    */
   public function setTitle($title, $pageTitle) {
-    return;
+    return NULL;
   }
 
   /**
@@ -65,7 +65,7 @@ class CRM_Utils_System_Soap extends CRM_Utils_System_Base {
    * @param string $str
    *   The permission to check.
    *
-   * @return boolean
+   * @return bool
    *   true if yes, else false
    */
   public function checkPermission($str) {
@@ -81,7 +81,7 @@ class CRM_Utils_System_Soap extends CRM_Utils_System_Base {
    * @return void
    */
   public function appendBreadCrumb($title, $url) {
-    return;
+    return NULL;
   }
 
   /**
@@ -93,7 +93,7 @@ class CRM_Utils_System_Soap extends CRM_Utils_System_Base {
    * @return void
    */
   public function addHTMLHead($head) {
-    return;
+    return NULL;
   }
 
   /**
@@ -127,8 +127,6 @@ class CRM_Utils_System_Soap extends CRM_Utils_System_Base {
   /**
    * Figure out the post url for the form
    *
-   * @param the default action if one is pre-specified
-   *
    * @return string
    *   the url to post the form
    */
@@ -199,7 +197,7 @@ class CRM_Utils_System_Soap extends CRM_Utils_System_Base {
    *   If present, add destination to querystring (works for Drupal only).
    *
    * @throws Exception
-   * @return string
+   * @return;
    *   loginURL for the current CMS
    */
   public function getLoginURL($destination = '') {
index 7b2ca2dc64984d12059d965409a1eeaaa9047f98..8e38800620ec14d440b04ae62eb5b4b4c4dd4a44 100644 (file)
@@ -153,7 +153,9 @@ class Manager {
    * @return \Civi\Core\Transaction\Frame
    */
   public function getBaseFrame() {
-    if (empty($this->frames)) return NULL;
+    if (empty($this->frames)) {
+      return NULL;
+    }
     return $this->frames[count($this->frames) - 1];
   }
 
index 4a23bdd0b49182102b2562cccb69226c41e02687..bb31c023a8cc7f121cb73d16eab93ce823f61446 100644 (file)
@@ -69,7 +69,7 @@ function civicrm_api3_contact_create($params) {
       $defLocType = CRM_Core_BAO_LocationType::getDefault();
       $params['email'] = array(
         1 => array(
-      'email' => $email,
+          'email' => $email,
           'is_primary' => 1,
           'location_type_id' => ($defLocType->id) ? $defLocType->id : 1,
         ),
@@ -80,8 +80,8 @@ function civicrm_api3_contact_create($params) {
   if (!empty($params['home_url'])) {
     $websiteTypes = CRM_Core_PseudoConstant::get('CRM_Core_DAO_Website', 'website_type_id');
     $params['website'] = array(
-    1 => array(
-    'website_type_id' => key($websiteTypes),
+      1 => array(
+        'website_type_id' => key($websiteTypes),
         'url' => $params['home_url'],
       ),
     );
@@ -141,8 +141,6 @@ function _civicrm_api3_contact_create_spec(&$params) {
 /**
  * Retrieve one or more contacts, given a set of search params
  *
- * @param array input parameters
- *
  * @return array
  *   API Result Array
  */
@@ -819,7 +817,7 @@ LIMIT    0, {$limit}
       if ($listCurrentEmployer && !empty($currEmpDetails)) {
         $contactList = array(
           array(
-          'data' => $currEmpDetails['data'],
+            'data' => $currEmpDetails['data'],
             'id'   => $currEmpDetails['id'],
           ),
         );
index d57dbb770baf88c07d5fa25e1364a5bd1d2f5caa..77359bc33288e703540556079996b84636a4ce1b 100644 (file)
@@ -57,7 +57,7 @@ function civicrm_api3_uf_match_get($params) {
  *
  * @return array
  *   api result array
- * {@getfields uf_match_create}
+ *   {@getfields uf_match_create}
  */
 function civicrm_api3_uf_match_create($params) {
   return _civicrm_api3_basic_create(_civicrm_api3_get_BAO(__FUNCTION__), $params);
@@ -86,7 +86,7 @@ function _civicrm_api3_uf_match_create_spec(&$params) {
  *
  * @return array
  *   api result array
- * {@getfields uf_match_create}
+ *   {@getfields uf_match_create}
  */
 function civicrm_api3_uf_match_delete($params) {
   return _civicrm_api3_basic_delete(_civicrm_api3_get_BAO(__FUNCTION__), $params);
index 29f4df4ecd97402ebb505bd54318b54b1341ec26..9f30e106ef7012ac0f5a54fe2196c7c9362de9da 100644 (file)
@@ -47,7 +47,7 @@ class AllTests extends CiviTestSuite {
    */
   private static function getInstance() {
     if (is_null(self::$instance)) {
-      self::$instance = new self;
+      self::$instance = new self();
     }
     return self::$instance;
   }
index 09f083bd1b619e24833d15cd2940cecb32c77ca3..1705f48b5e570b36f6ba2626aea71e354a7f9125 100644 (file)
@@ -40,7 +40,7 @@ require_once 'api/api.php';
  * @package CiviCRM
  */
 class CRM_Bridge_OG_DrupalTest extends CiviUnitTestCase {
-  /*
+  /**
    * Test that one (ane only one) role (option value) is deleted by the updateCiviACLRole function
    */
   public function testACLRoleDeleteFunctionality() {
index 644e606cb5bc6a32b6fe1ec6e93fb61fc49cdf66..56fe2c620d1fd661d28f468d99e2b25651ea5cc5 100644 (file)
@@ -313,7 +313,7 @@ class WebTest_Member_OfflineMembershipAddPricesetTest extends CiviSeleniumTestCa
     foreach (array(
                'joinDate',
                'startDate',
-               'endDate'
+               'endDate',
              ) as $date) {
       $$date = CRM_Utils_Date::customFormat($$date, $configVars->dateformatFull);
     }
@@ -403,7 +403,7 @@ class WebTest_Member_OfflineMembershipAddPricesetTest extends CiviSeleniumTestCa
     foreach (array(
                'joinDate',
                'startDate',
-               'endDate'
+               'endDate',
              ) as $date) {
       $$date = CRM_Utils_Date::customFormat($$date, $configVars->dateformatFull);
     }
index ec624c6b269bf48006236d2fb5ea570376de52ea..e1194fc14873a3d4d50f4c46888edc9158d8fe01 100644 (file)
@@ -250,7 +250,7 @@ class WebTest_Member_StandaloneAddTest extends CiviSeleniumTestCase {
     $test = $this;
     $this->customFieldSetLoadOnTheFlyCheck($customSets, $pageUrl,
       function () use ($test) {
-        $test->select('membership_type_id_0', 'value=1');
+      $test->select('membership_type_id_0', 'value=1');
       }
     );
   }
index 50ea90226281d05909ad01f037bab5e8a20dbd07..fa401b3d622d5a740803a6e66db3271de6935f54 100644 (file)
@@ -54,7 +54,7 @@ class api_v3_UFJoinTest extends CiviUnitTestCase {
       )
     );
     $this->_apiversion = 3;
-    $op = new PHPUnit_Extensions_Database_Operation_Insert;
+    $op = new PHPUnit_Extensions_Database_Operation_Insert();
     $op->execute(
       $this->_dbconn,
       $this->createFlatXMLDataSet(dirname(__FILE__) . '/dataset/uf_group_test.xml')