Code cleanup - Batch #9
authorWeb Access <rohan.katkar@webaccessglobal.com>
Thu, 15 Jan 2015 07:02:44 +0000 (12:32 +0530)
committerWeb Access <rohan.katkar@webaccessglobal.com>
Thu, 15 Jan 2015 07:02:44 +0000 (12:32 +0530)
40 files changed:
CRM/Admin/Form/Preferences/Address.php
CRM/Admin/Form/Setting/Mapping.php
CRM/Admin/Page/AJAX.php
CRM/Contact/Form/Edit/Address.php
CRM/Contact/Form/Search/Custom.php
CRM/Contact/Form/Task/Result.php
CRM/Contact/Import/Parser/Contact.php
CRM/Contribute/BAO/Query.php
CRM/Contribute/Form/CancelSubscription.php
CRM/Core/BAO/Email.php
CRM/Core/BAO/LabelFormat.php
CRM/Core/BAO/Mapping.php
CRM/Core/BAO/Note.php
CRM/Core/Controller.php
CRM/Core/LegacyErrorHandler.php
CRM/Core/Page/AJAX.php
CRM/Core/Payment.php
CRM/Core/Payment/AuthorizeNet.php
CRM/Core/Permission/Base.php
CRM/Custom/Form/Field.php
CRM/Custom/Form/Option.php
CRM/Event/Cart/Page/CheckoutAJAX.php
CRM/Event/Form/Registration.php
CRM/Event/Form/Task/ParticipantStatus.php
CRM/Group/Page/Group.php
CRM/Mailing/Event/BAO/Queue.php
CRM/Report/Form/Contact/LoggingDetail.php
CRM/Report/Form/Contact/Relationship.php
CRM/Report/Form/Contact/Summary.php
CRM/Upgrade/Incremental/php/FourTwo.php
CRM/Utils/Migrate/ExportJSON.php
CRM/Utils/Money.php
Civi/Core/Container.php
api/v3/Mailing.php
api/v3/Membership.php
settings/Developer.setting.php
tests/phpunit/CRM/AllTests.php
tests/phpunit/CiviTest/CiviDBAssert.php
tests/phpunit/WebTest/Profile/DedupeTest.php
tests/phpunit/api/v3/PledgeTest.php

index 1b2eac7a0b5d306e49386f53cee412f5b12f67ee..57be327a5967af2fce4063b56e41e05b6ec080a2 100644 (file)
@@ -43,8 +43,8 @@ class CRM_Admin_Form_Preferences_Address extends CRM_Admin_Form_Preferences {
 
     // Address Standardization
     $addrProviders = array(
-        '' => '- select -'
-      ) + CRM_Core_SelectValues::addressProvider();
+      '' => '- select -',
+    ) + CRM_Core_SelectValues::addressProvider();
 
     $this->_varNames = array(
       CRM_Core_BAO_Setting::SYSTEM_PREFERENCES_NAME => array(
index a8a052b75aef6709b511ff48174d172a660a4a9c..ee0acdd7f6139e69e9c485521f7ea4d9d4b22907 100644 (file)
@@ -88,8 +88,6 @@ class CRM_Admin_Form_Setting_Mapping extends CRM_Admin_Form_Setting {
    * add the rules (mainly global rules) for form.
    * All local rules are added near the element
    *
-   * @param null
-   *
    * @return void
    */
   public function addRules() {
index 35d7aa9e2c8f1e5b59a397b6428eb43d8232f4a5..a95514675eea54a5cc6eca43f5b8aee4ebf06a88 100644 (file)
@@ -115,7 +115,7 @@ class CRM_Admin_Page_AJAX {
             $ret['illegal'] = TRUE;
             $table = $template->fetch('CRM/Price/Page/table.tpl');
             $ret['content'] = ts('Unable to disable the \'%1\' price set - it is currently in use by one or more active events, contribution pages or contributions.', array(
-                1 => $priceSet
+                1 => $priceSet,
               )) . "<br/> $table";
           }
           else {
index aff4eb7d354b229a652ba0f267e09f5b5bdbb939..9e62aaee70c947b3f12f8ce74ce8849afd234b0f 100644 (file)
@@ -74,7 +74,7 @@ class CRM_Contact_Form_Edit_Address {
     //make location type required for inline edit
     $form->addSelect("address[$blockId][location_type_id]", array(
         'entity' => 'address',
-        'class' => 'eight'
+        'class' => 'eight',
       ) + $js, $inlineEdit);
 
     if (!$inlineEdit) {
@@ -118,12 +118,12 @@ class CRM_Contact_Form_Edit_Address {
       'postal_code' => array(
         ts('Zip / Postal Code'),
         array_merge($attributes['postal_code'], array('class' => 'crm_postal_code')),
-        NULL
+        NULL,
       ),
       'postal_code_suffix' => array(
         ts('Postal Code Suffix'),
         array('size' => 4, 'maxlength' => 12, 'class' => 'crm_postal_code_suffix'),
-        NULL
+        NULL,
       ),
       'country_id' => array(ts('Country'), $attributes['country_id'], 'country'),
       'state_province_id' => array(ts('State/Province'), $attributes['state_province_id'], NULL),
@@ -144,7 +144,7 @@ class CRM_Contact_Form_Edit_Address {
         if (in_array($nameWithoutID, array(
             'street_number',
             'street_name',
-            'street_unit'
+            'street_unit',
           )) && !empty($addressOptions['street_address_parsing'])
         ) {
           $continue = FALSE;
@@ -265,7 +265,7 @@ class CRM_Contact_Form_Edit_Address {
       $profileLinks = CRM_Core_BAO_UFGroup::getCreateLinks(array(
           'new_individual',
           'new_organization',
-          'new_household'
+          'new_household',
         ), 'shared_address');
       $form->addEntityRef("address[$blockId][master_contact_id]", ts('Share With'), array('create' => $profileLinks));
     }
@@ -359,7 +359,7 @@ class CRM_Contact_Form_Edit_Address {
             'name' => $shareAddressContactNames[$addressValue['master_id']]['name'],
             'options' => CRM_Core_BAO_Address::getValues(array(
                 'entity_id' => $master_cid,
-                'contact_id' => $master_cid
+                'contact_id' => $master_cid,
               )),
             'master_id' => $addressValue['master_id'],
           );
@@ -387,11 +387,11 @@ class CRM_Contact_Form_Edit_Address {
                      'street_number',
                      'street_number_suffix',
                      'street_name',
-                     'street_unit'
+                     'street_unit',
                    ) as $fld) {
             if (in_array($fld, array(
               'street_name',
-              'street_unit'
+              'street_unit',
             ))) {
               $streetAddress .= ' ';
             }
index 7c4edc51724dd59638dc09d683c24b210404bf3a..ad8e178bb483f864a232992b6fe0ae3728e94de5 100644 (file)
@@ -70,7 +70,7 @@ class CRM_Contact_Form_Search_Custom extends CRM_Contact_Form_Search {
         'url' => CRM_Utils_System::url('civicrm/contact/search/custom/list',
           'reset=1'
         ),
-      )
+      ),
     );
     CRM_Utils_System::appendBreadCrumb($breadCrumb);
 
index 3696721a679e67de78956c4d56921c874a224b8c..f8e04f498195ce39171987b1e9b24df7f463f943 100644 (file)
@@ -54,7 +54,7 @@ class CRM_Contact_Form_Task_Result extends CRM_Contact_Form_Task {
     $context = $this->get('context');
     if (in_array($context, array(
       'smog',
-      'amtg'
+      'amtg',
     ))) {
       $urlParams = 'reset=1&force=1&context=smog&gid=';
       $urlParams .= ($context == 'smog') ? $this->get('gid') : $this->get('amtgID');
@@ -94,7 +94,6 @@ class CRM_Contact_Form_Task_Result extends CRM_Contact_Form_Task {
 
     $url = CRM_Utils_System::url('civicrm/contact/' . $fragment, $path);
     $session->replaceUserContext($url);
-    return;
   }
 
   /**
index 0e00b30a59c0b6cdb91768a9e464640f920a2d94..72393cfb39f068c14514d232845d595e2ddce5c6 100644 (file)
@@ -102,7 +102,7 @@ class CRM_Contact_Import_Parser_Contact extends CRM_Contact_Import_Parser {
   /**
    * Class constructor
    */
-  function __construct(
+  public function __construct(
     &$mapperKeys, $mapperLocType = NULL, $mapperPhoneType = NULL, $mapperImProvider = NULL, $mapperRelated = NULL, $mapperRelatedContactType = NULL, $mapperRelatedContactDetails = NULL, $mapperRelatedContactLocType = NULL, $mapperRelatedContactPhoneType = NULL, $mapperRelatedContactImProvider = NULL,
     $mapperWebsiteType = NULL, $mapperRelatedContactWebsiteType = NULL
   ) {
@@ -148,8 +148,8 @@ class CRM_Contact_Import_Parser_Contact extends CRM_Contact_Import_Parser {
     }
 
     //Relationship importables
-    $this->_relationships = $relations =
-      CRM_Contact_BAO_Relationship::getContactRelationshipType(
+    $this->_relationships = $relations
+      CRM_Contact_BAO_Relationship::getContactRelationshipType(
         NULL, NULL, NULL, $this->_contactType,
         FALSE, 'label', TRUE, $this->_contactSubType
       );
@@ -250,7 +250,7 @@ class CRM_Contact_Import_Parser_Contact extends CRM_Contact_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;
@@ -262,7 +262,7 @@ class CRM_Contact_Import_Parser_Contact extends CRM_Contact_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) {
@@ -275,7 +275,7 @@ class CRM_Contact_Import_Parser_Contact extends CRM_Contact_Import_Parser {
    * @param array $values
    *   The array of values belonging to this line.
    *
-   * @return boolean
+   * @return boo
    *   the result of this processing
    */
   public function summary(&$values) {
@@ -449,7 +449,7 @@ class CRM_Contact_Import_Parser_Contact extends CRM_Contact_Import_Parser {
    *
    * @param bool $doGeocodeAddress
    *
-   * @return boolean
+   * @return bool
    *   the result of this processing
    */
   public function import($onDuplicate, &$values, $doGeocodeAddress = FALSE) {
@@ -1013,8 +1013,6 @@ class CRM_Contact_Import_Parser_Contact extends CRM_Contact_Import_Parser {
           $newContact = $this->createContact($formatted, $contactFields, $onDuplicate, $contactId);
         }
         // else skip does nothing and just returns an error code.
-
-
         if ($cid) {
           $contact = array(
             'contact_id' => $cid,
index 88e97572a1074d7c40cdde2bea431c0d5b851e91..c0d5791315b530c3a5c185846bf984fc7eba3e09 100644 (file)
@@ -138,7 +138,7 @@ class CRM_Contribute_BAO_Query {
 
     // get payment instrument id
     if (!empty($query->_returnProperties['payment_instrument_id'])) {
-      $query->_select['instrument_id'] =  "contribution_payment_instrument.value as instrument_id";
+      $query->_select['instrument_id'] = "contribution_payment_instrument.value as instrument_id";
       $query->_select['payment_instrument_id'] = "contribution_payment_instrument.value as payment_instrument_id";
       $query->_element['instrument_id'] = $query->_element['payment_instrument_id'] = 1;
       $query->_tables['civicrm_contribution'] = 1;
@@ -362,7 +362,7 @@ class CRM_Contribute_BAO_Query {
 
       case 'financial_type':
         $query->_where[$grouping][] = CRM_Contact_BAO_Query::buildClause($fields[$name]['where'], $op, $value, 'String');
-        CRM_Core_Error::debug( '$query', $query->_where[$grouping] );
+        CRM_Core_Error::debug('$query', $query->_where[$grouping]);
         list($op, $value) = CRM_Contact_BAO_Query::buildQillForFieldValue('CRM_Contribute_DAO_Contribution', $name, $value, $op);
         $query->_qill[$grouping][] = ts('%1 %2 %3', array(1 => $fields[$name]['title'], 2 => $op, 3 => $value));
         $query->_tables['civicrm_contribution'] = $query->_whereTables['civicrm_contribution'] = 1;
@@ -876,7 +876,7 @@ class CRM_Contribute_BAO_Query {
     $form->add('select', 'contribution_currency_type',
       ts('Currency Type'),
       array(
-        '' => ts('- any -')
+        '' => ts('- any -'),
       ) +
       CRM_Core_PseudoConstant::get('CRM_Contribute_DAO_Contribution', 'currency', array('labelColumn' => 'name')),
       FALSE, array('class' => 'crm-select2')
@@ -890,7 +890,7 @@ class CRM_Contribute_BAO_Query {
     $form->add('select', 'contribution_page_id',
       ts('Contribution Page'),
       array(
-        '' => ts('- any -')
+        '' => ts('- any -'),
       ) +
       CRM_Contribute_PseudoConstant::contributionPage(),
       FALSE, array('class' => 'crm-select2')
@@ -903,7 +903,7 @@ class CRM_Contribute_BAO_Query {
     $form->add('select', 'contribution_pcp_made_through_id',
       ts('Personal Campaign Page'),
       array(
-        '' => ts('- any -')
+        '' => ts('- any -'),
       ) +
       CRM_Contribute_PseudoConstant::pcPage(),
       FALSE, array('class' => 'crm-select2')
@@ -1038,7 +1038,7 @@ class CRM_Contribute_BAO_Query {
   public static function buildDateWhere(&$values, $query, $name, $field, $title) {
     $fieldPart = strpos($name, $field);
     if ($fieldPart === FALSE) {
-      return;
+      return NULL;
     }
     // we only have recurring dates using this ATM so lets' short cut to find the table name
     $table = 'contribution_recur';
index 99553db3cbf26047cd6ac1499ca4cfcfc29c935b..9f06661bd9d72487affb81def17f05dc0399d5c8 100644 (file)
@@ -125,8 +125,8 @@ class CRM_Contribute_Form_CancelSubscription extends CRM_Core_Form {
     $this->assign('mode', $this->_mode);
 
     if ($this->_subscriptionDetails->contact_id) {
-      list($this->_donorDisplayName, $this->_donorEmail) =
-        CRM_Contact_BAO_Contact::getContactDetails($this->_subscriptionDetails->contact_id);
+      list($this->_donorDisplayName, $this->_donorEmail)
+        CRM_Contact_BAO_Contact::getContactDetails($this->_subscriptionDetails->contact_id);
     }
   }
 
@@ -186,8 +186,6 @@ class CRM_Contribute_Form_CancelSubscription extends CRM_Core_Form {
    * 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
    */
@@ -227,8 +225,8 @@ class CRM_Contribute_Form_CancelSubscription extends CRM_Core_Form {
       CRM_Core_Error::displaySessionError($cancelSubscription);
     }
     elseif ($cancelSubscription) {
-      $activityParams =
-        array(
+      $activityParams
+        array(
           'subject' => $this->_mid ? ts('Auto-renewal membership cancelled') : ts('Recurring contribution cancelled'),
           'details' => $message,
         );
@@ -244,10 +242,10 @@ class CRM_Contribute_Form_CancelSubscription extends CRM_Core_Form {
           $inputParams = array('id' => $this->_mid);
           CRM_Member_BAO_Membership::getValues($inputParams, $tplParams);
           $tplParams = $tplParams[$this->_mid];
-          $tplParams['membership_status'] =
-            CRM_Core_DAO::getFieldValue('CRM_Member_DAO_MembershipStatus', $tplParams['status_id']);
-          $tplParams['membershipType'] =
-            CRM_Core_DAO::getFieldValue('CRM_Member_DAO_MembershipType', $tplParams['membership_type_id']);
+          $tplParams['membership_status']
+            CRM_Core_DAO::getFieldValue('CRM_Member_DAO_MembershipStatus', $tplParams['status_id']);
+          $tplParams['membershipType']
+            CRM_Core_DAO::getFieldValue('CRM_Member_DAO_MembershipType', $tplParams['membership_type_id']);
           $status = ts('The automatic renewal of your %1 membership has been cancelled as requested. This does not affect the status of your membership - you will receive a separate notification when your membership is up for renewal.', array(1 => $tplParams['membershipType']));
           $msgTitle = 'Membership Renewal Cancelled';
           $msgType = 'info';
@@ -277,9 +275,8 @@ class CRM_Contribute_Form_CancelSubscription extends CRM_Core_Form {
               $value,
               array('title', 'receipt_from_name', 'receipt_from_email')
             );
-            $receiptFrom =
-              '"' .
-              CRM_Utils_Array::value('receipt_from_name', $value[$this->_subscriptionDetails->contribution_page_id]) .
+            $receiptFrom
+              = '"' . CRM_Utils_Array::value('receipt_from_name', $value[$this->_subscriptionDetails->contribution_page_id]) .
               '" <' .
               $value[$this->_subscriptionDetails->contribution_page_id]['receipt_from_email'] .
               '>';
@@ -290,8 +287,8 @@ class CRM_Contribute_Form_CancelSubscription extends CRM_Core_Form {
           }
 
           // send notification
-          $sendTemplateParams =
-            array(
+          $sendTemplateParams
+            array(
               'groupName' => $this->_mode == 'auto_renew' ? 'msg_tpl_workflow_membership' : 'msg_tpl_workflow_contribution',
               'valueName' => $this->_mode == 'auto_renew' ? 'membership_autorenew_cancelled' : 'contribution_recurring_cancelled',
               'contactId' => $this->_subscriptionDetails->contact_id,
@@ -328,11 +325,12 @@ class CRM_Contribute_Form_CancelSubscription extends CRM_Core_Form {
       $session->setStatus($status, $msgTitle, $msgType);
     }
     elseif (!$userID) {
-      if ($status)
+      if ($status) {
         CRM_Utils_System::setUFMessage($status);
-      // keep result as 1, since we not displaying anything on the redirected page anyway
-      return CRM_Utils_System::redirect(CRM_Utils_System::url('civicrm/contribute/subscriptionstatus',
+        // keep result as 1, since we not displaying anything on the redirected page anyway
+        return CRM_Utils_System::redirect(CRM_Utils_System::url('civicrm/contribute/subscriptionstatus',
         "reset=1&task=cancel&result=1"));
+      }
     }
   }
 }
index 33ff4babd78905d09dd3dcbf74468e6fc7a6d846..87649e5d9fbc599ed9c8db72ee214f81ab024b38 100644 (file)
@@ -117,7 +117,7 @@ WHERE  contact_id = {$params['contact_id']}
    * @param array $entityBlock
    *   Input parameters to find object.
    *
-   * @return boolean
+   * @return bool
    */
   public static function &getValues($entityBlock) {
     return CRM_Core_BAO_Block::getValues('email', $entityBlock);
index 0de74960ee72a88aff6f1d94da1c8b8f597a67a8..089a23a2cf7dbc587446be87bd24948a57408d54 100644 (file)
@@ -157,8 +157,6 @@ class CRM_Core_BAO_LabelFormat extends CRM_Core_DAO_OptionValue {
   /**
    * Get page orientations recognized by the DOMPDF package used to create PDF letters.
    *
-   * @param void
-   *
    * @return array
    *   array of page orientations
    */
@@ -186,8 +184,6 @@ class CRM_Core_BAO_LabelFormat extends CRM_Core_DAO_OptionValue {
   /**
    * Get font sizes supported by the TCPDF package used to create PDF labels.
    *
-   * @param void
-   *
    * @return array
    *   array of font sizes
    */
@@ -203,8 +199,6 @@ class CRM_Core_BAO_LabelFormat extends CRM_Core_DAO_OptionValue {
   /**
    * Get measurement units recognized by the TCPDF package used to create PDF labels.
    *
-   * @param void
-   *
    * @return array
    *   array of measurement units
    */
@@ -220,8 +214,6 @@ class CRM_Core_BAO_LabelFormat extends CRM_Core_DAO_OptionValue {
   /**
    * Get text alignment recognized by the TCPDF package used to create PDF labels.
    *
-   * @param void
-   *
    * @return array
    *   array of alignments
    */
@@ -236,8 +228,6 @@ class CRM_Core_BAO_LabelFormat extends CRM_Core_DAO_OptionValue {
   /**
    * Get text alignment recognized by the TCPDF package used to create PDF labels.
    *
-   * @param void
-   *
    * @return array
    *   array of alignments
    */
@@ -280,6 +270,7 @@ class CRM_Core_BAO_LabelFormat extends CRM_Core_DAO_OptionValue {
   public static function addOrder(&$list, $returnURL) {
     $filter = "option_group_id = " . self::_getGid();
     CRM_Utils_Weight::addOrder($list, 'CRM_Core_DAO_OptionValue', 'id', $returnURL, $filter);
+    return $list;
   }
 
   /**
@@ -462,8 +453,6 @@ class CRM_Core_BAO_LabelFormat extends CRM_Core_DAO_OptionValue {
   /**
    * Return the name of the group for customized labels
    *
-   * @param void
-   *
    * @return void
    */
   public static function customGroupName() {
index 243e496746d5f25cdb538d36c85c2735e76e9947..1c5b94a933b4a362b9a0c9ae2b6417771914cbfc 100644 (file)
@@ -68,7 +68,7 @@ class CRM_Core_BAO_Mapping extends CRM_Core_DAO_Mapping {
    * @param int $id
    *   Mapping id.
    *
-   * @return boolean
+   * @return boo
    */
   public static function del($id) {
     // delete from mapping_field table
@@ -204,7 +204,7 @@ class CRM_Core_BAO_Mapping extends CRM_Core_DAO_Mapping {
    * @param string $mapTypeId
    *   mapping Type.
    *
-   * @return boolean
+   * @return boo
    */
   public static function checkMapping($nameField, $mapTypeId) {
     $mapping = new CRM_Core_DAO_Mapping();
@@ -363,7 +363,7 @@ class CRM_Core_BAO_Mapping extends CRM_Core_DAO_Mapping {
       foreach (array(
                  'groups',
                  'tags',
-                 'notes'
+                 'notes',
                ) as $value) {
         unset($fields['Individual'][$value]);
         unset($fields['Household'][$value]);
@@ -388,7 +388,7 @@ class CRM_Core_BAO_Mapping extends CRM_Core_DAO_Mapping {
         $fields['Contact']['note_body'] = array('title' => $noteTitle . ': ' . ts('Body Only'), 'name' => 'note_body');
         $fields['Contact']['note_subject'] = array(
           'title' => $noteTitle . ': ' . ts('Subject Only'),
-          'name' => 'note_subject'
+          'name' => 'note_subject',
         );
       }
     }
@@ -472,8 +472,8 @@ class CRM_Core_BAO_Mapping extends CRM_Core_DAO_Mapping {
       $csRelationships = array();
 
       if ($mappingType == 'Export') {
-        $subTypeRelationshipTypes =
-          CRM_Contact_BAO_Relationship::getContactRelationshipType(NULL, NULL, NULL, $val['parent'],
+        $subTypeRelationshipTypes
+          CRM_Contact_BAO_Relationship::getContactRelationshipType(NULL, NULL, NULL, $val['parent'],
             FALSE, 'label', TRUE, $subType);
 
         foreach ($subTypeRelationshipTypes as $key => $var) {
@@ -1109,8 +1109,6 @@ class CRM_Core_BAO_Mapping extends CRM_Core_DAO_Mapping {
         0,
       );
     }
-
-
     return $fields;
   }
 
@@ -1190,7 +1188,7 @@ class CRM_Core_BAO_Mapping extends CRM_Core_DAO_Mapping {
     $mappingFields->delete();
 
     if (empty($params['mapper'])) {
-      return;
+      return NULL;
     }
 
     //save record in mapping field table
index bc8fd6c7a708947b561954b514fa662d063c61af..59307f1108e6971c97f6b6b4e8f4503366431dde 100644 (file)
@@ -78,7 +78,7 @@ class CRM_Core_BAO_Note extends CRM_Core_DAO_Note {
    * @param object $note
    *   Either the id of the note to retrieve, or the CRM_Core_DAO_Note object itself.
    *
-   * @return boolean
+   * @return bool
    *   TRUE if the note should be displayed, otherwise FALSE
    *
    */
@@ -223,7 +223,7 @@ class CRM_Core_BAO_Note extends CRM_Core_DAO_Note {
    * @param array $params
    *   (reference ) an assoc array of name/value pairs.
    *
-   * @return boolean
+   * @return bool
    */
   public static function dataExists(&$params) {
     // return if no data present
index cd62a31fe7f1f1ff3e3aba2a5063db8555605c57..34c3cf39c11f24e4060acde19e509bc0cc77c074 100644 (file)
@@ -173,7 +173,7 @@ class CRM_Core_Controller extends HTML_QuickForm_Controller {
    * @param bool $ignoreKey
    *   Should we not set a qfKey for this controller (for standalone forms).
    */
-  function __construct(
+  public function __construct(
     $title = NULL,
     $modal = TRUE,
     $mode = NULL,
@@ -193,8 +193,8 @@ class CRM_Core_Controller extends HTML_QuickForm_Controller {
     // lets try to get it from the session and/or the request vars
     // we do this early on in case there is a fatal error in retrieving the
     // key and/or session
-    $this->_entryURL =
-      CRM_Utils_Request::retrieve('entryURL', 'String', $this);
+    $this->_entryURL
+      CRM_Utils_Request::retrieve('entryURL', 'String', $this);
 
     // add a unique validable key to the name
     $name = CRM_Utils_System::getClassName($this);
@@ -379,8 +379,8 @@ class CRM_Core_Controller extends HTML_QuickForm_Controller {
    * Helper function to add all the needed default actions. Note that the framework
    * redefines all of the default QFC actions
    *
-   * @param string directory to store all the uploaded files
-   * @param array names for the various upload buttons (note u can have more than 1 upload)
+   * @param string $uploadDirectory to store all the uploaded files
+   * @param array $uploadNames for the various upload buttons (note u can have more than 1 upload)
    *
    *
    * @return void
@@ -661,7 +661,7 @@ class CRM_Core_Controller extends HTML_QuickForm_Controller {
   /**
    * Getter for embedded
    *
-   * @return boolean
+   * @return bool
    *   return the embedded value
    */
   public function getEmbedded() {
@@ -682,7 +682,7 @@ class CRM_Core_Controller extends HTML_QuickForm_Controller {
   /**
    * Getter for skipRedirection
    *
-   * @return boolean
+   * @return bool
    *   return the skipRedirection value
    */
   public function getSkipRedirection() {
@@ -738,7 +738,7 @@ class CRM_Core_Controller extends HTML_QuickForm_Controller {
   /**
    * Getter for print
    *
-   * @return boolean
+   * @return bool
    *   return the print value
    */
   public function getPrint() {
index b10026ac6b908fb5784282e288c9e450a644aaf9..57b8c80e6852f443cd54468c5ed3dda5f9ca320f 100644 (file)
@@ -5,7 +5,7 @@ class CRM_Core_LegacyErrorHandler {
    * @param \Civi\Core\Event\UnhandledExceptionEvent $event
    * @throws Exception
    */
-  static function handleException($event) {
+  public static function handleException($event) {
     $e = $event->exception;
     if ($e instanceof CRM_Core_Exception) {
       $params = $e->getErrorData();
index 74fbc48a3bc3dc5688a91035dc355a6eca59fd59..af7ae6613a403866019baf89d540ae0ae45fa45d 100644 (file)
@@ -73,7 +73,7 @@ class CRM_Core_Page_AJAX {
         // FIXME: This is done to maintain current wire protocol, but it might be
         // simpler to just require different 'types' for pages and forms
         if (preg_match('/^CRM_[a-zA-Z0-9]+_Page_Inline_/', $className)) {
-          $page = new $className;
+          $page = new $className();
           $page->run();
         }
         else {
index ab50f68298fd1b6aa750e376d912f2044caf32cb..97f9417ae3d9fbc9c1b4829584808541f6c94a95 100644 (file)
@@ -303,7 +303,7 @@ abstract class CRM_Core_Payment {
             'rule_message' => ts('Please enter a valid value for your card security code. This is usually the last 3-4 digits on the card\'s signature panel.'),
             'rule_name' => 'integer',
             'rule_parameters' => NULL,
-          )
+          ),
         ),
       ),
       'credit_card_exp_date' => array(
@@ -318,7 +318,7 @@ abstract class CRM_Core_Payment {
             'rule_message' => ts('Card expiration date cannot be a past date.'),
             'rule_name' => 'currentDate',
             'rule_parameters' => TRUE,
-          )
+          ),
         ),
       ),
       'credit_card_type' => array(
@@ -357,7 +357,7 @@ abstract class CRM_Core_Payment {
             'rule_message' => ts('Please enter a valid Bank Identification Number (value must not contain punctuation characters).'),
             'rule_name' => 'nopunctuation',
             'rule_parameters' => NULL,
-          )
+          ),
         ),
         'is_required' => TRUE,
       ),
@@ -378,7 +378,7 @@ abstract class CRM_Core_Payment {
             'rule_message' => ts('Please enter a valid Bank Identification Number (value must not contain punctuation characters).'),
             'rule_name' => 'nopunctuation',
             'rule_parameters' => NULL,
-          )
+          ),
         ),
       ),
       'bank_name' => array(
@@ -408,7 +408,7 @@ abstract class CRM_Core_Payment {
    *   the result in an nice formatted array (or an error object)
    * @abstract
    */
-  abstract function doDirectPayment(&$params);
+  abstract protected function doDirectPayment(&$params);
 
   /**
    * Process payment - this function wraps around both doTransferPayment and doDirectPayment
@@ -442,7 +442,7 @@ abstract class CRM_Core_Payment {
    * @return string
    *   the error message if any
    */
-  abstract function checkConfig();
+  abstract protected function checkConfig();
 
   /**
    * @param $paymentProcessor
@@ -589,7 +589,7 @@ abstract class CRM_Core_Payment {
    * @param string $method
    *   Method to check for.
    *
-   * @return boolean
+   * @return bool
    */
   public function isSupported($method = 'cancelSubscription') {
     return method_exists(CRM_Utils_System::getClassName($this), $method);
index a62030a1852f94887a2a7a321bb95e4bae88cc8d..da729de70aa9181feed267aeae18709bd3b34b3e 100644 (file)
@@ -396,7 +396,7 @@ class CRM_Core_Payment_AuthorizeNet extends CRM_Core_Payment {
    *
    * @return string
    *   the HMAC_MD5 encoding string
-   **/
+   */
   public function hmac($key, $data) {
     if (function_exists('mhash')) {
       // Use PHP mhash extension
@@ -459,7 +459,7 @@ class CRM_Core_Payment_AuthorizeNet extends CRM_Core_Payment {
    *
    * @return string
    *   fingerprint
-   **/
+   */
   public function CalculateFP() {
     $x_tran_key = $this->_getParam('paymentKey');
     $loginid = $this->_getParam('apiLogin');
index 5e4f239161dc70a9b7d04cfdeeeefff719af6062..be049664f2482ddc3d0055a4e176630e73445c8e 100644 (file)
@@ -156,10 +156,7 @@ class CRM_Core_Permission_Base {
    * @param string $str
    *   The permission to check.
    *
-   * @return boolean
-   *   true if yes, else false
    */
-
   public function check($str) {
     //no default behaviour
   }
@@ -170,10 +167,9 @@ class CRM_Core_Permission_Base {
    * @param array $array
    *   The roles to check.
    *
-   * @return boolean
+   * @return bool
    *   true if yes, else false
    */
-
   public function checkGroupRole($array) {
     return FALSE;
   }
@@ -184,8 +180,6 @@ class CRM_Core_Permission_Base {
    * @param string $permissionName
    *   Name of the permission we are interested in.
    *
-   * @return string
-   *   a comma separated list of email addresses
    */
   public function permissionEmails($permissionName) {
     CRM_Core_Error::fatal("this function only works in Drupal 6 at the moment");
@@ -197,8 +191,6 @@ class CRM_Core_Permission_Base {
    * @param string $roleName
    *   Name of the role we are interested in.
    *
-   * @return string
-   *   a comma separated list of email addresses
    */
   public function roleEmails($roleName) {
     CRM_Core_Error::fatal("this function only works in Drupal 6 at the moment");
index 9bffda7c665e726e63ba6ecf434d58789c7b74e1..edbcbc8543bae01eb675015dd6bb066c7b205a24 100644 (file)
@@ -84,8 +84,6 @@ class CRM_Custom_Form_Field extends CRM_Core_Form {
   /**
    * Set variables up before form is built
    *
-   * @param null
-   *
    * @return void
    */
   public function preProcess() {
@@ -171,8 +169,6 @@ class CRM_Custom_Form_Field extends CRM_Core_Form {
    * 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
    */
@@ -278,8 +274,6 @@ class CRM_Custom_Form_Field extends CRM_Core_Form {
   /**
    * Build the form object
    *
-   * @param null
-   *
    * @return void
    */
   public function buildQuickForm() {
@@ -342,7 +336,7 @@ class CRM_Custom_Form_Field extends CRM_Core_Form {
         'option_group_id',
         ts('Multiple Choice Option Sets'),
         array(
-          '' => ts('- select -')
+          '' => ts('- select -'),
         ) + $optionGroups
       );
     }
@@ -351,7 +345,7 @@ class CRM_Custom_Form_Field extends CRM_Core_Form {
       ts('Option Type'),
       $optionTypes,
       array(
-        'onclick' => "showOptionSelect();"
+        'onclick' => "showOptionSelect();",
       ), '<br/>'
     );
 
@@ -901,8 +895,6 @@ AND    option_group_id = %2";
   /**
    * Process the form
    *
-   * @param null
-   *
    * @return void
    */
   public function postProcess() {
@@ -924,7 +916,7 @@ AND    option_group_id = %2";
       1,
       2,
       3,
-      5
+      5,
     ))) {
       if (empty($params['is_searchable'])) {
         $params['is_search_range'] = 0;
index ccb0e94cfd96abd989becb55c2655b80e219fb33..e91b96e535d4207e0c12ab3e21011d737b37cdf7 100644 (file)
@@ -67,8 +67,6 @@ class CRM_Custom_Form_Option extends CRM_Core_Form {
   /**
    * Set variables up before form is built
    *
-   * @param null
-   *
    * @return void
    */
   public function preProcess() {
@@ -103,8 +101,6 @@ class CRM_Custom_Form_Option extends CRM_Core_Form {
    * 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
    */
@@ -151,8 +147,6 @@ class CRM_Custom_Form_Option extends CRM_Core_Form {
   /**
    * Build the form object
    *
-   * @param null
-   *
    * @return void
    */
   public function buildQuickForm() {
@@ -392,8 +386,6 @@ SELECT count(*)
   /**
    * Process the form
    *
-   * @param null
-   *
    * @return void
    */
   public function postProcess() {
@@ -425,8 +417,8 @@ SELECT count(*)
     }
 
     $fieldValues = array('option_group_id' => $this->_optionGroupID);
-    $customOption->weight =
-      CRM_Utils_Weight::updateOtherWeights(
+    $customOption->weight
+      CRM_Utils_Weight::updateOtherWeights(
         'CRM_Core_DAO_OptionValue',
         $oldWeight,
         $params['weight'],
@@ -456,8 +448,8 @@ SELECT count(*)
           else {
             $defVal[] = $customOption->value;
           }
-          $customField->default_value =
-            CRM_Core_DAO::VALUE_SEPARATOR .
+          $customField->default_value
+            CRM_Core_DAO::VALUE_SEPARATOR .
             implode(CRM_Core_DAO::VALUE_SEPARATOR, $defVal) .
             CRM_Core_DAO::VALUE_SEPARATOR;
           $customField->save();
@@ -471,8 +463,8 @@ SELECT count(*)
           }
         }
 
-        $customField->default_value =
-          CRM_Core_DAO::VALUE_SEPARATOR .
+        $customField->default_value
+          CRM_Core_DAO::VALUE_SEPARATOR .
           implode(CRM_Core_DAO::VALUE_SEPARATOR, $tempVal) .
           CRM_Core_DAO::VALUE_SEPARATOR;
         $customField->save();
index 194f8b50fb63ae8882ab5e958d3cc73f87f6b3fd..da1a2395ab39cf02a93c118140a5624cd2499f6a 100644 (file)
@@ -14,7 +14,7 @@ class CRM_Event_Cart_Page_CheckoutAJAX {
     $params_array = array(
       'cart_id' => $cart->id,
       'contact_id' => CRM_Event_Cart_Form_Cart::find_or_create_contact(),
-      'event_id' => $event_id
+      'event_id' => $event_id,
     );
 
     //XXX security?
index d5afd4e341737fa46de05ae4f1de47065bbbd969..cfe4db0805fb559661a7b49d83c5b1a39116286f 100644 (file)
@@ -393,8 +393,8 @@ class CRM_Event_Form_Registration extends CRM_Core_Form {
       $this->set('values', $this->_values);
       $this->set('fields', $this->_fields);
 
-      $this->_availableRegistrations =
-        CRM_Event_BAO_Participant::eventFull(
+      $this->_availableRegistrations
+        CRM_Event_BAO_Participant::eventFull(
           $this->_values['event']['id'], TRUE,
           CRM_Utils_Array::value('has_waitlist', $this->_values['event'])
         );
@@ -832,7 +832,7 @@ class CRM_Event_Form_Registration extends CRM_Core_Form {
    */
   public static function addParticipant(&$form, $contactID) {
     if (empty($form->_params)) {
-      return;
+      return NULL;
     }
     $params = $form->_params;
     $transaction = new CRM_Core_Transaction();
@@ -879,8 +879,7 @@ WHERE  v.option_group_id = g.id
       ),
       'register_date' => ($registerDate) ? $registerDate : date('YmdHis'),
       'source' => CRM_Utils_String::ellipsify(
-        isset($params['participant_source']) ?
-          CRM_Utils_Array::value('participant_source', $params) : CRM_Utils_Array::value('description', $params),
+        isset($params['participant_source']) ? CRM_Utils_Array::value('participant_source', $params) : CRM_Utils_Array::value('description', $params),
         $participantFields['participant_source']['maxlength']
       ),
       'fee_level' => CRM_Utils_Array::value('amount_level', $params),
@@ -1290,8 +1289,8 @@ WHERE  v.option_group_id = g.id
             $optionMaxValues[$priceFieldId][$optId] = $currentMaxValue;
           }
           else {
-            $optionMaxValues[$priceFieldId][$optId] =
-              $currentMaxValue + CRM_Utils_Array::value($optId, CRM_Utils_Array::value($priceFieldId, $optionMaxValues), 0);
+            $optionMaxValues[$priceFieldId][$optId]
+              $currentMaxValue + CRM_Utils_Array::value($optId, CRM_Utils_Array::value($priceFieldId, $optionMaxValues), 0);
           }
 
         }
@@ -1308,16 +1307,16 @@ WHERE  v.option_group_id = g.id
         if ($optMax && $total > $optMax) {
           $errors['soldOutOptions'][] = ts('Option %1 has sold out.', array(1 => $feeBlock[$fieldId]['options'][$optId]['label']));
           if ($opDbCount && ($opDbCount >= $optMax)) {
-            $errors[$currentParticipantNum]["price_{$fieldId}"] =
-              ts('Sorry, this option is currently sold out.');
+            $errors[$currentParticipantNum]["price_{$fieldId}"]
+              ts('Sorry, this option is currently sold out.');
           }
           elseif (($optMax - $opDbCount) == 1) {
-            $errors[$currentParticipantNum]["price_{$fieldId}"] =
-              ts('Sorry, currently only a single seat is available for this option.', array(1 => ($optMax - $opDbCount)));
+            $errors[$currentParticipantNum]["price_{$fieldId}"]
+              ts('Sorry, currently only a single seat is available for this option.', array(1 => ($optMax - $opDbCount)));
           }
           else {
-            $errors[$currentParticipantNum]["price_{$fieldId}"] =
-              ts('Sorry, currently only %1 seats are available for this option.', array(1 => ($optMax - $opDbCount)));
+            $errors[$currentParticipantNum]["price_{$fieldId}"]
+              ts('Sorry, currently only %1 seats are available for this option.', array(1 => ($optMax - $opDbCount)));
           }
         }
       }
index f27f0d818bd52a9b3d409eaf4d74d0da9d2a65b6..046e5f02381bdab6edc62af76b94f449defcef34 100644 (file)
@@ -36,7 +36,7 @@ class CRM_Event_Form_Task_ParticipantStatus extends CRM_Event_Form_Task_Batch {
   public function buildQuickForm() {
     // CRM_Event_Form_Task_Batch::buildQuickForm() gets ufGroupId
     // from the form, so set it here to the id of the reserved profile
-    $dao = new CRM_Core_DAO_UFGroup;
+    $dao = new CRM_Core_DAO_UFGroup();
     $dao->name = 'participant_status';
     $dao->find(TRUE);
     $this->set('ufGroupId', $dao->id);
@@ -45,7 +45,7 @@ class CRM_Event_Form_Task_ParticipantStatus extends CRM_Event_Form_Task_Batch {
     asort($statuses, SORT_STRING);
     $this->add('select', 'status_change', ts('Change All Statuses'),
       array(
-        '' => ts('- select status -')
+        '' => ts('- select status -'),
       ) + $statuses
     );
 
index df6c826064f8593d9e93e2a62cd8b1b9d5674558..e65020f9b0f2f5152bf3fd4c0d65ab770fe08026 100644 (file)
@@ -45,7 +45,6 @@ class CRM_Group_Page_Group extends CRM_Core_Page_Basic {
   /**
    * Define action links
    *
-   * @return array
    *   self::$_links array of action links
    */
   public function &links() {
index 1ed0e5491098c32c2912e2e32221dd48a06dfc1e..1546ac382ffbfe1a4e79a1bcaaa21726775ed57f 100644 (file)
@@ -63,8 +63,6 @@ class CRM_Mailing_Event_BAO_Queue extends CRM_Mailing_Event_DAO_Queue {
   /**
    * Create a security hash from the job, email and contact ids
    *
-   * @param array The ids to be hashed
-   *
    * @return int
    *   The hash
    */
index 3422bdc2c9102153a2800731b41d02b9ceb757f0..ddf5cea27248887b501cbe9cefdcd0cb084662fb 100644 (file)
@@ -36,7 +36,7 @@ class CRM_Report_Form_Contact_LoggingDetail extends CRM_Logging_ReportDetail {
   /**
    */
   public function __construct() {
-    $logging = new CRM_Logging_Schema;
+    $logging = new CRM_Logging_Schema();
     $this->tables[] = 'civicrm_contact';
     $this->tables = array_merge($this->tables, array_keys($logging->customDataLogTables()));
     $this->tables[] = 'civicrm_email';
index 0f67ecbff30762e4d5952b51a17858291e468260..d8b46dc0559902aa53b73dc3951fe10e2d811c71 100644 (file)
@@ -223,9 +223,9 @@ class CRM_Report_Form_Contact_Relationship extends CRM_Report_Form {
             'title' => ts('Relationship'),
             'operatorType' => CRM_Report_Form::OP_SELECT,
             'options' => array(
-                '' => '- any relationship type -',
-              ) +
-              CRM_Contact_BAO_Relationship::getContactRelationshipType(NULL, 'null', NULL, NULL, TRUE),
+              '' => '- any relationship type -',
+            ) +
+            CRM_Contact_BAO_Relationship::getContactRelationshipType(NULL, 'null', NULL, NULL, TRUE),
             'type' => CRM_Utils_Type::T_INT,
           ),
         ),
@@ -587,8 +587,8 @@ class CRM_Report_Form_Contact_Relationship extends CRM_Report_Form {
           'reset=1&force=1&id_op=eq&id_value=' . $row['civicrm_contact_id'],
           $this->_absoluteUrl, $this->_id, $this->_drilldownReport
         );
-        $rows[$rowNum]['civicrm_contact_sort_name_a'] =
-          $rows[$rowNum]['civicrm_contact_sort_name_a'] . ' (' .
+        $rows[$rowNum]['civicrm_contact_sort_name_a']
+          $rows[$rowNum]['civicrm_contact_sort_name_a'] . ' (' .
           $rows[$rowNum]['civicrm_contact_id'] . ')';
         $rows[$rowNum]['civicrm_contact_sort_name_a_link'] = $url;
         $rows[$rowNum]['civicrm_contact_sort_name_a_hover'] = ts("View Contact details for this contact.");
@@ -602,8 +602,8 @@ class CRM_Report_Form_Contact_Relationship extends CRM_Report_Form {
           'reset=1&force=1&id_op=eq&id_value=' . $row['civicrm_contact_b_id'],
           $this->_absoluteUrl, $this->_id, $this->_drilldownReport
         );
-        $rows[$rowNum]['civicrm_contact_b_sort_name_b'] =
-          $rows[$rowNum]['civicrm_contact_b_sort_name_b'] . ' (' .
+        $rows[$rowNum]['civicrm_contact_b_sort_name_b']
+          $rows[$rowNum]['civicrm_contact_b_sort_name_b'] . ' (' .
           $rows[$rowNum]['civicrm_contact_b_id'] . ')';
         $rows[$rowNum]['civicrm_contact_b_sort_name_b_link'] = $url;
         $rows[$rowNum]['civicrm_contact_b_sort_name_b_hover'] = ts("View Contact details for this contact.");
@@ -613,8 +613,7 @@ class CRM_Report_Form_Contact_Relationship extends CRM_Report_Form {
       if (array_key_exists('civicrm_relationship_relationship_id', $row) &&
         array_key_exists('civicrm_contact_id', $row)
       ) {
-        $url =
-          "/civicrm/contact/view/rel?reset=1&action=update&rtype=a_b&cid=" .
+        $url = "/civicrm/contact/view/rel?reset=1&action=update&rtype=a_b&cid=" .
           $row['civicrm_contact_id'] . "&id=" .
           $row['civicrm_relationship_relationship_id'];
         $rows[$rowNum]['civicrm_relationship_relationship_id_link'] = $url;
index 6a3ae459ccc4ff0ed31e97ec80077bf6bc2ab4f2..cb5140fc8397941371d5674ea06c762b95be0c4f 100644 (file)
@@ -54,108 +54,108 @@ class CRM_Report_Form_Contact_Summary extends CRM_Report_Form {
   public function __construct() {
     $this->_autoIncludeIndexedFieldsAsOrderBys = 1;
     $this->_columns = array(
-        'civicrm_contact' => array(
-          'dao' => 'CRM_Contact_DAO_Contact',
-          'fields' => array(
-            'sort_name' => array(
-              'title' => ts('Contact Name'),
-              'required' => TRUE,
-              'no_repeat' => TRUE,
-            ),
-            'first_name' => array(
-              'title' => ts('First Name'),
-            ),
-            'middle_name' => array(
-              'title' => ts('Middle Name'),
-            ),
-            'last_name' => array(
-              'title' => ts('Last Name'),
-            ),
-            'id' => array(
-              'no_display' => TRUE,
-              'required' => TRUE,
-            ),
-            'contact_type' => array(
-              'title' => ts('Contact Type'),
-            ),
-            'contact_sub_type' => array(
-              'title' => ts('Contact Subtype'),
-            ),
-            'gender_id' => array(
-              'title' => ts('Gender'),
-            ),
-            'birth_date' => array(
-              'title' => ts('Birth Date'),
-            ),
-            'age' => array(
-              'title' => ts('Age'),
-              'dbAlias' => 'TIMESTAMPDIFF(YEAR, contact_civireport.birth_date, CURDATE())',
-            ),
+      'civicrm_contact' => array(
+        'dao' => 'CRM_Contact_DAO_Contact',
+        'fields' => array(
+          'sort_name' => array(
+            'title' => ts('Contact Name'),
+            'required' => TRUE,
+            'no_repeat' => TRUE,
           ),
-          'filters' => array(
-            'sort_name' => array('title' => ts('Contact Name')),
-            'source' => array(
-              'title' => ts('Contact Source'),
-              'type' => CRM_Utils_Type::T_STRING,
-            ),
-            'id' => array(
-              'title' => ts('Contact ID'),
-              'no_display' => TRUE,
-            ),
-            'gender_id' => array(
-              'title' => ts('Gender'),
-              'operatorType' => CRM_Report_Form::OP_MULTISELECT,
-              'options' => CRM_Core_PseudoConstant::get('CRM_Contact_DAO_Contact', 'gender_id'),
-            ),
-            'birth_date' => array(
-              'title' => ts('Birth Date'),
-              'operatorType' => CRM_Report_Form::OP_DATE,
-            ),
+          'first_name' => array(
+            'title' => ts('First Name'),
           ),
-          'grouping' => 'contact-fields',
-          'order_bys' => array(
-            'sort_name' => array(
-              'title' => ts('Last Name, First Name'),
-              'default' => '1',
-              'default_weight' => '0',
-              'default_order' => 'ASC',
-            ),
-            'gender_id' => array(
-              'name' => 'gender_id',
-              'title' => ts('Gender'),
-            ),
-            'birth_date' => array(
-              'name' => 'birth_date',
-              'title' => ts('Birth Date'),
-            ),
+          'middle_name' => array(
+            'title' => ts('Middle Name'),
+          ),
+          'last_name' => array(
+            'title' => ts('Last Name'),
+          ),
+          'id' => array(
+            'no_display' => TRUE,
+            'required' => TRUE,
+          ),
+          'contact_type' => array(
+            'title' => ts('Contact Type'),
+          ),
+          'contact_sub_type' => array(
+            'title' => ts('Contact Subtype'),
+          ),
+          'gender_id' => array(
+            'title' => ts('Gender'),
+          ),
+          'birth_date' => array(
+            'title' => ts('Birth Date'),
+          ),
+          'age' => array(
+            'title' => ts('Age'),
+            'dbAlias' => 'TIMESTAMPDIFF(YEAR, contact_civireport.birth_date, CURDATE())',
           ),
         ),
-        'civicrm_email' => array(
-          'dao' => 'CRM_Core_DAO_Email',
-          'fields' => array(
-            'email' => array(
-              'title' => ts('Email'),
-              'no_repeat' => TRUE,
-            ),
+        'filters' => array(
+          'sort_name' => array('title' => ts('Contact Name')),
+          'source' => array(
+            'title' => ts('Contact Source'),
+            'type' => CRM_Utils_Type::T_STRING,
+          ),
+          'id' => array(
+            'title' => ts('Contact ID'),
+            'no_display' => TRUE,
+          ),
+          'gender_id' => array(
+            'title' => ts('Gender'),
+            'operatorType' => CRM_Report_Form::OP_MULTISELECT,
+            'options' => CRM_Core_PseudoConstant::get('CRM_Contact_DAO_Contact', 'gender_id'),
           ),
-          'grouping' => 'contact-fields',
-          'order_bys' => array(
-            'email' => array(
-              'title' => ts('Email'),
-            ),
+          'birth_date' => array(
+            'title' => ts('Birth Date'),
+            'operatorType' => CRM_Report_Form::OP_DATE,
+          ),
+        ),
+        'grouping' => 'contact-fields',
+        'order_bys' => array(
+          'sort_name' => array(
+            'title' => ts('Last Name, First Name'),
+            'default' => '1',
+            'default_weight' => '0',
+            'default_order' => 'ASC',
+          ),
+          'gender_id' => array(
+            'name' => 'gender_id',
+            'title' => ts('Gender'),
+          ),
+          'birth_date' => array(
+            'name' => 'birth_date',
+            'title' => ts('Birth Date'),
+          ),
+        ),
+      ),
+      'civicrm_email' => array(
+        'dao' => 'CRM_Core_DAO_Email',
+        'fields' => array(
+          'email' => array(
+            'title' => ts('Email'),
+            'no_repeat' => TRUE,
+          ),
+        ),
+        'grouping' => 'contact-fields',
+        'order_bys' => array(
+          'email' => array(
+            'title' => ts('Email'),
           ),
         ),
-        'civicrm_phone' => array(
-          'dao' => 'CRM_Core_DAO_Phone',
-          'fields' => array(
-            'phone' => NULL,
-            'phone_ext' => array(
-              'title' => ts('Phone Extension'),
-            ),
+      ),
+      'civicrm_phone' => array(
+        'dao' => 'CRM_Core_DAO_Phone',
+        'fields' => array(
+          'phone' => NULL,
+          'phone_ext' => array(
+            'title' => ts('Phone Extension'),
           ),
-          'grouping' => 'contact-fields',
         ),
-      ) + $this->getAddressColumns(array('group_by' => FALSE));
+        'grouping' => 'contact-fields',
+      ),
+    ) + $this->getAddressColumns(array('group_by' => FALSE));
 
     $this->_groupFilter = TRUE;
     $this->_tagFilter = TRUE;
index b2cf8f1448a011668fc61a514fce34c1fe5203ec..8f91366a4ec0e2d420161002f7695831badb5242 100644 (file)
@@ -188,7 +188,7 @@ INNER JOIN civicrm_price_set cps ON cps.id = cpf.price_set_id AND cps.name <>'de
       }
     }
     // Drop index UI_title for civicrm_price_set
-    $domain = new CRM_Core_DAO_Domain;
+    $domain = new CRM_Core_DAO_Domain();
     $domain->find(TRUE);
     if ($domain->locales) {
       $locales = explode(CRM_Core_DAO::VALUE_SEPARATOR, $domain->locales);
@@ -285,7 +285,7 @@ INNER JOIN civicrm_price_set cps ON cps.id = cpf.price_set_id AND cps.name <>'de
       $dao = CRM_Core_DAO::executeQuery($query);
 
       if (!$dao->N) {
-        $domain = new CRM_Core_DAO_Domain;
+        $domain = new CRM_Core_DAO_Domain();
         $domain->find(TRUE);
 
         if ($domain->locales) {
@@ -433,7 +433,7 @@ WHERE     cpse.price_set_id IS NULL";
       $addTo[2] = $dao->contribution_page_id;
       $options = array(
         'otherAmount' => $dao->is_allow_other_amount,
-        'membership' => $dao->membership_block_id
+        'membership' => $dao->membership_block_id,
       );
       self::createPriceSet($daoName, $addTo, $options);
     }
@@ -655,7 +655,7 @@ WHERE     cpf.price_set_id = %1
           '2' => array('membership_amount', 'String'),
           '3' => array(
             CRM_Core_DAO::getFieldValue('CRM_Member_DAO_Membership', $result->membership_id, 'membership_type_id'),
-            'Integer'
+            'Integer',
           ),
         );
         $res = CRM_Core_DAO::executeQuery($sql, $params);
@@ -721,12 +721,12 @@ WHERE     cpf.price_set_id = %1
           if (!empty($defaults)) {
             $lineParams['price_field_id'] = $defaults['id'];
             $lineParams['label'] = $defaults['label'];
-            $lineParams['price_field_value_id'] =
-              CRM_Core_DAO::getFieldValue('CRM_Upgrade_Snapshot_V4p2_Price_DAO_FieldValue', $defaults['id'], 'id', 'price_field_id');
+            $lineParams['price_field_value_id']
+              CRM_Core_DAO::getFieldValue('CRM_Upgrade_Snapshot_V4p2_Price_DAO_FieldValue', $defaults['id'], 'id', 'price_field_id');
           }
           else {
-            $lineParams['price_field_id'] =
-              CRM_Core_DAO::getFieldValue('CRM_Upgrade_Snapshot_V4p2_Price_DAO_Field', $priceSetId, 'id', 'price_set_id');
+            $lineParams['price_field_id']
+              CRM_Core_DAO::getFieldValue('CRM_Upgrade_Snapshot_V4p2_Price_DAO_Field', $priceSetId, 'id', 'price_set_id');
             $lineParams['label'] = 'Contribution Amount';
           }
           $lineParams['qty'] = 1;
@@ -967,7 +967,7 @@ VALUES
           $dao->start_date,
           $dao->end_date,
           $memStatus,
-          $status
+          $status,
         );
       }
     }
index 4cda5c0e1ce20d879dff84268f807baf61d89c7d..6d7e589a2b6c8cb686c4da7b42cacc80776e1103 100644 (file)
@@ -185,7 +185,7 @@ WHERE      g.name IN ( $nameString )
    * @param $whereField
    * @param null $additionalWhereCond
    */
-  function table(
+  public function table(
     &$ids,
     $tableName,
     &$fields,
@@ -590,7 +590,7 @@ WHERE ac.contact_id IN ( $ids )
    * @param null $lastExportTime
    * @param bool $discoverContacts
    */
-  function run(
+  public function run(
     $fileName,
     $lastExportTime = NULL,
     $discoverContacts = FALSE
index 4bb1672441572554a9a280f5ce82fe12ae53c475..916c892938c710ded1e03456647e06650c1a7247 100644 (file)
@@ -90,7 +90,7 @@ class CRM_Utils_Money {
     if (!self::$_currencySymbols) {
       self::$_currencySymbols = CRM_Core_PseudoConstant::get('CRM_Contribute_DAO_Contribution', 'currency', array(
           'keyColumn' => 'name',
-          'labelColumn' => 'symbol'
+          'labelColumn' => 'symbol',
         ));
     }
 
index 000143abde1a271231f4bea0cbfaf7cb56a3a245..fa56a554c7ff86840275021cc06829b29d44cdb7 100644 (file)
@@ -100,7 +100,7 @@ class Container {
     $dispatcher->addListener('DAO::post-delete', array('\CRM_Core_BAO_RecurringEntity', 'triggerDelete'));
     $dispatcher->addListener('hook_civicrm_unhandled_exception', array(
         'CRM_Core_LegacyErrorHandler',
-        'handleException'
+        'handleException',
       ));
     return $dispatcher;
   }
index 7bc6be0ae629fa72e0174fde468fdc7f5977d55d..f85074236ae22355e6be4971fca5204a2497a774 100755 (executable)
@@ -41,7 +41,7 @@
  *
  * @param array $params
  * @return array API Success Array
- * API Success Array
+ *    API Success Array
  * @throws \API_Exception
  * @throws \Civi\API\Exception\UnauthorizedException
  */
@@ -206,7 +206,7 @@ function civicrm_api3_mailing_event_bounce($params) {
     return civicrm_api3_create_success($params);
   }
   else {
-    throw new API_Exception(ts('Queue event could not be found'),'no_queue_event
+    throw new API_Exception(ts('Queue event could not be found'), 'no_queue_event
       ');
   }
 }
@@ -573,31 +573,31 @@ function civicrm_api3_mailing_stats($params) {
     switch ($detail) {
       case 'Delivered':
         $stats[$params['mailing_id']] += array(
-          $detail =>  CRM_Mailing_Event_BAO_Delivered::getTotalCount($params['mailing_id'], $params['job_id'], FALSE, $params['date']),
+          $detail => CRM_Mailing_Event_BAO_Delivered::getTotalCount($params['mailing_id'], $params['job_id'], FALSE, $params['date']),
         );
         break;
 
       case 'Bounces':
         $stats[$params['mailing_id']] += array(
-          $detail =>  CRM_Mailing_Event_BAO_Bounce::getTotalCount($params['mailing_id'], $params['job_id'], FALSE, $params['date']),
+          $detail => CRM_Mailing_Event_BAO_Bounce::getTotalCount($params['mailing_id'], $params['job_id'], FALSE, $params['date']),
         );
         break;
 
       case 'Unsubscribers':
         $stats[$params['mailing_id']] += array(
-          $detail =>  CRM_Mailing_Event_BAO_Unsubscribe::getTotalCount($params['mailing_id'], $params['job_id'], FALSE, NULL, $params['date']),
+          $detail => CRM_Mailing_Event_BAO_Unsubscribe::getTotalCount($params['mailing_id'], $params['job_id'], FALSE, NULL, $params['date']),
         );
         break;
 
       case 'Unique Clicks':
         $stats[$params['mailing_id']] += array(
-          $detail =>  CRM_Mailing_Event_BAO_TrackableURLOpen::getTotalCount($params['mailing_id'], $params['job_id'], FALSE, NULL, $params['date']),
+          $detail => CRM_Mailing_Event_BAO_TrackableURLOpen::getTotalCount($params['mailing_id'], $params['job_id'], FALSE, NULL, $params['date']),
         );
         break;
 
       case 'Opened':
         $stats[$params['mailing_id']] += array(
-          $detail =>  CRM_Mailing_Event_BAO_Opened::getTotalCount($params['mailing_id'], $params['job_id'], FALSE, $params['date']),
+          $detail => CRM_Mailing_Event_BAO_Opened::getTotalCount($params['mailing_id'], $params['job_id'], FALSE, $params['date']),
         );
         break;
     }
index d16aff5ec8adefe789de517dbdd6ab64c39d06c7..dfc3ca78d710c79c8d2fd393daf6fa312b18899f 100644 (file)
@@ -46,7 +46,7 @@
  *
  * @return array
  *   api result
- * {@getfields membership_delete}
+ *   {@getfields membership_delete}
  */
 function civicrm_api3_membership_delete($params) {
   return _civicrm_api3_basic_delete(_civicrm_api3_get_BAO(__FUNCTION__), $params);
@@ -63,7 +63,7 @@ function civicrm_api3_membership_delete($params) {
  *
  * @return array
  *   Array of newly created membership property values.
- * {@getfields membership_create}
+ *   {@getfields membership_create}
  */
 function civicrm_api3_membership_create($params) {
   // check params for membership id during update
@@ -112,10 +112,10 @@ function civicrm_api3_membership_create($params) {
   // Fixme: This code belongs in the BAO
   $action = CRM_Core_Action::ADD;
   // we need user id during add mode
-    $ids = array();
-    if (!empty($params['contact_id'])) {
-      $ids['userId'] = $params['contact_id'];
-    }
+  $ids = array();
+  if (!empty($params['contact_id'])) {
+    $ids['userId'] = $params['contact_id'];
+  }
   //for edit membership id should be present
   if (!empty($params['id'])) {
     $ids['membership'] = $params['id'];
@@ -202,7 +202,7 @@ function civicrm_api3_membership_get($params) {
     $params['status_id'] = array('IN' => CRM_Member_BAO_MembershipStatus::getMembershipStatusCurrent());
   }
 
-  $options = _civicrm_api3_get_options_from_params($params, TRUE,'membership', 'get');
+  $options = _civicrm_api3_get_options_from_params($params, TRUE, 'membership', 'get');
   if ($options['is_count']) {
     return _civicrm_api3_basic_get(_civicrm_api3_get_BAO(__FUNCTION__), $params);
   }
@@ -218,7 +218,7 @@ function civicrm_api3_membership_get($params) {
     return civicrm_api3_create_success($membershipValues, $params, 'membership', 'get');
   }
 
-  $members =  _civicrm_api3_membership_relationsship_get_customv2behaviour($params, $membershipValues, $contactID);
+  $members = _civicrm_api3_membership_relationsship_get_customv2behaviour($params, $membershipValues, $contactID);
   return civicrm_api3_create_success($members, $params, 'membership', 'get');
 }
 
index c9c31e33e1fa0992dc98441e3bfc7dfbcf03a3e6..132090d2dc0f0520c4b91f875ea5b5afd16e0e16 100644 (file)
@@ -66,8 +66,7 @@ return array(
     'is_contact' => 0,
     'description' => "Set this value to Yes if you want CiviCRM error/debugging messages to also appear in Drupal error logs",
     'prefetch' => 1,
-    'help_text' => "Set this value to Yes if you want CiviCRM error/debugging messages the appear in your CMS' error log.
-In the case of Drupal, this will cause all CiviCRM error messages to appear in the watchdog (assuming you have Drupal's watchdog enabled)",
+    'help_text' => "Set this value to Yes if you want CiviCRM error/debugging messages the appear in your CMS' error log. In the case of Drupal, this will cause all CiviCRM error messages to appear in the watchdog (assuming you have Drupal's watchdog enabled)",
   ),
   'backtrace' => array(
     'group_name' => 'Developer Preferences',
index c63f8c95c08b4f1a1570ce4a1a808e449caaecec..f910defe84efe7fce7026d8be5b3d70ddddcba05 100644 (file)
@@ -47,7 +47,7 @@ class CRM_AllTests extends CiviTestSuite {
    */
   private static function getInstance() {
     if (is_null(self::$instance)) {
-      self::$instance = new self;
+      self::$instance = new self();
     }
     return self::$instance;
   }
index ff12960746175a26e09d70927c411ec8a3ea75ca..dc056dbc1c14cadcba2658e912cce789b2a0afb8 100644 (file)
@@ -150,7 +150,7 @@ class CiviDBAssert {
    * @param $expectedValue
    * @param string $message
    */
-  function assertDBCompareValue(
+  public function assertDBCompareValue(
     &$testCase, $daoName, $searchValue, $returnColumn, $searchColumn,
     $expectedValue, $message
   ) {
index 00e69530be6bee90cbb6ebfb44d7b830e4b40186..accd8a4faa709b6d2616d1cd32b83a4e4d94c8f4 100644 (file)
@@ -41,7 +41,7 @@ class WebTest_Profile_DedupeTest extends CiviSeleniumTestCase {
       'edit-1-profile-create',
       'edit-1-profile-edit',
       'edit-1-profile-listings',
-      'edit-1-profile-view'
+      'edit-1-profile-view',
     );
     $this->changePermissions($permission);
 
index 55c452bcad62a510465993cdaa0a7290f1383b57..d1873c6b120436aca98f6b0462522c0a1637cfda 100644 (file)
@@ -101,8 +101,6 @@ class api_v3_PledgeTest extends CiviUnitTestCase {
     $this->customGroupDelete($ids['custom_group_id']);
   }
 
-  /*
-   */
   public function testgetfieldspledge() {
     $result = $this->callAPISuccess('pledge', 'getfields', array('action' => 'get'));
     $this->assertEquals(1, $result['values']['next_pay_date']['api.return']);
@@ -184,7 +182,7 @@ class api_v3_PledgeTest extends CiviUnitTestCase {
     $this->assertEquals($oldPledge['id'], $earlyPledge['id'], ' check correct pledge returned ' . __LINE__);
   }
 
-  /*
+  /**
    * create 2 pledges - see if we can get by status id
    */
   public function testGetOverduePledge() {
@@ -207,7 +205,7 @@ class api_v3_PledgeTest extends CiviUnitTestCase {
   }
 
 
-  /*
+  /**
    * create 2 pledges - see if we can get by status id
    */
   public function testSortParamPledge() {
@@ -264,7 +262,7 @@ class api_v3_PledgeTest extends CiviUnitTestCase {
     $pledge = $this->callAPISuccess('pledge', 'delete', $pledgeID);
   }
 
-  /*
+  /**
    * Test that pledge with weekly schedule calculates dates correctly
    */
   public function testCreatePledgeWeeklySchedule() {
@@ -279,7 +277,7 @@ class api_v3_PledgeTest extends CiviUnitTestCase {
     //ensure that correct number of payments created & last payment has the right date
     $payments = $this->callAPISuccess('PledgePayment', 'Get', array(
       'pledge_id' => $pledge['id'],
-      'sequential' => 1
+      'sequential' => 1,
     ));
     $this->assertEquals($payments['count'], 5, 'In line ' . __LINE__);
     $this->assertEquals('2011-07-06 00:00:00', $payments['values'][4]['scheduled_date'], 'In line ' . __LINE__);
@@ -331,10 +329,10 @@ class api_v3_PledgeTest extends CiviUnitTestCase {
     $pledge = $this->callAPISuccess('pledge', 'delete', $pledgeID);
   }
 
-  /*
-  * test that using original_installment_amount rather than pledge_original_installment_amount works
-  * Pledge field behaviour is a bit random & so pledge has come to try to handle both unique & non -unique fields
-  */
+  /**
+   * test that using original_installment_amount rather than pledge_original_installment_amount works
+   * Pledge field behaviour is a bit random & so pledge has come to try to handle both unique & non -unique fields
+   */
   public function testCreatePledgeWithNonUnique() {
     $params = $this->_params;
     $params['original_installment_amount'] = $params['pledge_original_installment_amount'];
@@ -468,7 +466,6 @@ class api_v3_PledgeTest extends CiviUnitTestCase {
     $pledge = $this->callAPISuccess('pledge', 'delete', $new_params);
   }
 
-  ///////////////// civicrm_pledge_delete methods
   public function testDeleteEmptyParamsPledge() {
     $pledge = $this->callAPIFailure('pledge', 'delete', array(), 'Mandatory key(s) missing from params array: id');
   }