clean up changes for batch 18
authoryashodha <yashodha.chaku@webaccessglobal.com>
Fri, 16 Jan 2015 05:40:54 +0000 (11:10 +0530)
committeryashodha <yashodha.chaku@webaccessglobal.com>
Fri, 16 Jan 2015 05:40:54 +0000 (11:10 +0530)
63 files changed:
CRM/Activity/Form/Task/PickOption.php
CRM/Admin/Form/PdfFormats.php
CRM/Admin/Form/Preferences/Display.php
CRM/Batch/Page/Batch.php
CRM/Contact/Form/Search/Criteria.php
CRM/Contact/Form/Search/Custom/EmployerListing.php
CRM/Contact/Form/Task/Batch.php
CRM/Contact/Form/Task/LabelCommon.php
CRM/Contact/Form/Task/RemoveFromTag.php
CRM/Contact/Import/Form/MapField.php
CRM/Contact/Page/View/GroupContact.php
CRM/Contact/Page/View/Relationship.php
CRM/Contribute/Form/Contribution/Main.php
CRM/Contribute/Form/ManagePremiums.php
CRM/Contribute/Form/UpdateSubscription.php
CRM/Contribute/Import/Parser.php
CRM/Core/BAO/Block.php
CRM/Core/BAO/CustomOption.php
CRM/Core/BAO/Website.php
CRM/Core/Component.php
CRM/Core/Form/Renderer.php
CRM/Core/I18n/SchemaStructure_3_3_beta1.php
CRM/Core/Payment/IATS.php
CRM/Core/Permission/Drupal6.php
CRM/Core/QuickForm/GroupMultiSelect.php
CRM/Core/Selector/Base.php
CRM/Core/Selector/Controller.php
CRM/Core/Session.php
CRM/Custom/Form/DeleteField.php
CRM/Custom/Form/Preview.php
CRM/Event/Form/ManageEvent/ScheduleReminders.php
CRM/Event/Form/Registration/Confirm.php
CRM/Event/Form/Task/Delete.php
CRM/Export/BAO/Export.php
CRM/Extension/Manager/Payment.php
CRM/Financial/Form/FinancialBatch.php
CRM/Friend/Form.php
CRM/Grant/BAO/Query.php
CRM/Mailing/Form/Test.php
CRM/Mailing/Info.php
CRM/Mailing/MailStore.php
CRM/Mailing/MailStore/Localdir.php
CRM/Member/BAO/MembershipPayment.php
CRM/Member/Import/Form/MapField.php
CRM/Member/Page/UserDashboard.php
CRM/Member/Selector/Search.php
CRM/Pledge/BAO/Pledge.php
CRM/Pledge/BAO/PledgePayment.php
CRM/Pledge/Page/UserDashboard.php
CRM/Profile/Form/Edit.php
CRM/Report/Utils/Get.php
CRM/Utils/OpenFlashChart.php
CRM/Utils/SystemLogger.php
api/v3/MembershipStatus.php
api/v3/OptionValue.php
api/v3/Participant.php
api/v3/ParticipantStatusType.php
api/v3/PriceField.php
api/v3/Website.php
bin/deprecated/ParticipantProcessor.php
tests/phpunit/CiviTest/ContributionPage.php
tests/phpunit/WebTest/AllTests.php
tests/phpunit/WebTest/Contact/ProfileChecksumTest.php

index 7f138e90e664b9028ba4bbeb6552ad5db30a8502..c63c17493c2ce603131af0216907b3ad198e55ab 100644 (file)
@@ -81,7 +81,7 @@ class CRM_Activity_Form_Task_PickOption extends CRM_Activity_Form_Task {
     if (count($this->_activityHolderIds) > $this->_maxActivities) {
       CRM_Core_Session::setStatus(ts("The maximum number of Activities you can select to send an email is %1. You have selected %2. Please select fewer Activities from your search results and try again.", array(
             1 => $this->_maxActivities,
-            2 => count($this->_activityHolderIds)
+            2 => count($this->_activityHolderIds),
           )), ts("Maximum Exceeded"), "error");
       $validate = TRUE;
     }
index 5daa96f03c976eff82ee3ea54f0a64239bcd781b..b22c7406c907ef084197d6b1ba548d8985933790 100644 (file)
@@ -66,7 +66,7 @@ class CRM_Admin_Form_PdfFormats extends CRM_Admin_Form {
 
     $this->add('select', 'paper_size', ts('Paper Size'),
       array(
-        0 => ts('- default -')
+        0 => ts('- default -'),
       ) + CRM_Core_BAO_PaperSize::getList(TRUE), FALSE,
       array('onChange' => "selectPaper( this.value );")
     );
@@ -86,7 +86,7 @@ class CRM_Admin_Form_PdfFormats extends CRM_Admin_Form {
 
     $this->addRule('name', ts('Name already exists in Database.'), 'objectExists', array(
         'CRM_Core_BAO_PdfFormat',
-        $this->_id
+        $this->_id,
       ));
     $this->addRule('margin_left', ts('Margin must be numeric'), 'numeric');
     $this->addRule('margin_right', ts('Margin must be numeric'), 'numeric');
index fcfa3b6292420200a1926759660bf175424cee5d..3ba0ddffd4d9e52d4e13d8eed3dccfab9cd0a2f4 100644 (file)
@@ -118,8 +118,7 @@ class CRM_Admin_Form_Preferences_Display extends CRM_Admin_Form_Preferences {
       $defaults['editor_id'] = $this->_config->editor_id;
     }
     if (empty($this->_config->display_name_format)) {
-      $defaults['display_name_format'] =
-        "{contact.individual_prefix}{ }{contact.first_name}{ }{contact.last_name}{ }{contact.individual_suffix}";
+      $defaults['display_name_format'] = "{contact.individual_prefix}{ }{contact.first_name}{ }{contact.last_name}{ }{contact.individual_suffix}";
     }
     else {
       $defaults['display_name_format'] = $this->_config->display_name_format;
index 98580cd708dfe30751afe6e7c9ae3530f7338e68..3e326a1cfae9a425732f609b06f4155490e83c9e 100644 (file)
@@ -58,8 +58,6 @@ class CRM_Batch_Page_Batch extends CRM_Core_Page_Basic {
   /**
    * Get action Links
    *
-   * @return array
-   *   (reference) of action links
    */
   public function &links() {
   }
index 00d7caa030a5364451437a7aebbf4baf42797725..bf0f9e74e4151cb7d2ab8dee61e49d24ba82ec0a 100644 (file)
@@ -116,7 +116,6 @@ class CRM_Contact_Form_Search_Criteria {
     //added job title
     $form->addElement('text', 'job_title', ts('Job Title'), CRM_Core_DAO::getAttribute('CRM_Contact_DAO_Contact', 'job_title'));
 
-
     //added internal ID
     $form->addElement('text', 'contact_id', ts('Contact ID'), CRM_Core_DAO::getAttribute('CRM_Contact_DAO_Contact', 'id'));
     $form->addRule('contact_id', ts('Please enter valid Contact ID'), 'positiveInteger');
@@ -158,7 +157,7 @@ class CRM_Contact_Form_Search_Criteria {
       'uf_group_id',
       ts('Search Views'),
       array(
-        '0' => ts('- default view -')
+        '0' => ts('- default view -'),
       ) + $searchProfiles
     );
 
@@ -488,7 +487,6 @@ class CRM_Contact_Form_Search_Criteria {
 
     CRM_Core_Form_Date::buildDateRange($form, 'deceased_date', 1, '_low', '_high', ts('From'), FALSE, FALSE, 'birth');
 
-
     // radio button for is_deceased
     $form->addYesNo('is_deceased', ts('Deceased'), TRUE);
   }
index e3545d2e4c6bab448c40a57e72787a61c2dc8d8d..bd6f4d73151210dd137055a30e16dfa2f445f71e 100644 (file)
@@ -104,7 +104,7 @@ class CRM_Contact_Form_Search_Custom_EmployerListing implements CRM_Contact_Form
   /**
    * Construct the search query
    */
-  function all(
+  public function all(
     $offset = 0, $rowcount = 0, $sort = NULL,
     $includeContactIDs = FALSE, $justIDs = FALSE
   ) {
index 9b9dc78c9cf6887ba1b34aa9b0f421998f355e39..c768b4fc6a29f6fb4dd593c0f0997f96155c3691 100644 (file)
@@ -121,7 +121,6 @@ class CRM_Contact_Form_Task_Batch extends CRM_Contact_Form_Task {
       )
     );
 
-
     $this->assign('profileTitle', $this->_title);
     $this->assign('componentIds', $this->_contactIds);
 
@@ -170,7 +169,7 @@ class CRM_Contact_Form_Task_Batch extends CRM_Contact_Form_Task {
    */
   public function setDefaultValues() {
     if (empty($this->_fields)) {
-      return;
+      return NULL;
     }
 
     $defaults = $sortName = array();
@@ -258,7 +257,7 @@ class CRM_Contact_Form_Task_Batch extends CRM_Contact_Form_Task {
     if ($inValidSubtypeCnt) {
       CRM_Core_Session::setStatus(ts('Contact Subtype field of 1 contact has not been updated.', array(
             'plural' => 'Contact Subtype field of %count contacts has not been updated.',
-            'count' => $inValidSubtypeCnt
+            'count' => $inValidSubtypeCnt,
           )), ts('Invalid Subtype'));
     }
   }
index 70d787e59d47e41c4d02c29bfa635c5b1008de6e..6fc52e7510519c03a9a81b5ec5124e97f9be8560 100644 (file)
@@ -50,7 +50,6 @@ class CRM_Contact_Form_Task_LabelCommon {
    * @param string $fileName
    *   The name of the file to save the label in.
    *
-   * @return null
    */
   public static function createLabel(&$contactRows, &$format, $fileName = 'MailingLabels_CiviCRM.pdf') {
     $pdf = new CRM_Utils_PDF_Label($format, 'mm');
@@ -84,7 +83,6 @@ class CRM_Contact_Form_Task_LabelCommon {
    * @return array
    *   Array of rows for labels
    */
-
   public static function getRows($contactIDs, $locationTypeID, $respectDoNotMail, $mergeSameAddress, $mergeSameHousehold) {
     $locName = NULL;
     //get the address format sequence from the config file
@@ -214,7 +212,7 @@ class CRM_Contact_Form_Task_LabelCommon {
                   'email',
                   'phone',
                   'im',
-                  'openid'
+                  'openid',
                 ))) {
                   if ($k == 'im') {
                     $rows[$value][$k] = $v['1']['name'];
index 0c19f2109d9f6e4c4e1b2f292e160d2e068a5fdf..95a2ec6d4ad37476c3ec7455284bab381fc92949 100644 (file)
@@ -137,13 +137,13 @@ class CRM_Contact_Form_Task_RemoveFromTag extends CRM_Contact_Form_Task {
       $status = array(
         ts('%count contact un-tagged', array(
             'count' => $removed,
-            'plural' => '%count contacts un-tagged'
-          ))
+            'plural' => '%count contacts un-tagged',
+           )),
       );
       if ($notRemoved) {
         $status[] = ts('1 contact already did not have this tag', array(
             'count' => $notRemoved,
-            'plural' => '%count contacts already did not have this tag'
+            'plural' => '%count contacts already did not have this tag',
           ));
       }
       $status = '<ul><li>' . implode('</li><li>', $status) . '</li></ul>';
index 92535b3d26f10529c23ee341d0006f25f5b46492..2f4491e642e3b0b5b216f2bfbfe3edd24956aa32 100644 (file)
@@ -62,8 +62,8 @@ class CRM_Contact_Import_Form_MapField extends CRM_Import_Form_MapField {
    * Attempt to match header labels with our mapper fields
    * FIXME: This is essentially the same function as parent::defaultFromHeader
    *
-   * @param header
-   * @param mapperFields
+   * @param $columnName name of column header
+   * @param $patterns pattern to match for the column
    *
    * @return string
    */
@@ -135,7 +135,7 @@ class CRM_Contact_Import_Form_MapField extends CRM_Import_Form_MapField {
       foreach (array(
                  'Individual',
                  'Household',
-                 'Organization'
+                 'Organization',
                ) as $cType) {
         $ruleParams = array(
           'contact_type' => $cType,
index 10251a07e689dea46d5c91d76fd3d22110a6dcd4..7df74a3ba9ed8c13d9ca9e5e7038bf9d2a54f5a2 100644 (file)
@@ -37,7 +37,6 @@ class CRM_Contact_Page_View_GroupContact extends CRM_Core_Page {
   /**
    * called when action is browse
    *
-   * @return null
    */
   public function browse() {
 
@@ -74,7 +73,6 @@ class CRM_Contact_Page_View_GroupContact extends CRM_Core_Page {
    *
    * @param int $groupId
    *
-   * @return null
    */
   public function edit($groupId = NULL) {
     $controller = new CRM_Core_Controller_Simple(
@@ -176,8 +174,7 @@ class CRM_Contact_Page_View_GroupContact extends CRM_Core_Page {
         break;
     }
 
-    $groupNum =
-      CRM_Contact_BAO_GroupContact::getContactGroup($contactID, 'Added', NULL, TRUE, TRUE);
+    $groupNum = CRM_Contact_BAO_GroupContact::getContactGroup($contactID, 'Added', NULL, TRUE, TRUE);
     if ($groupNum == 1 &&
       $groupStatus == 'Removed' &&
       CRM_Core_BAO_Setting::getItem(CRM_Core_BAO_Setting::MULTISITE_PREFERENCES_NAME,
index 0e20b552f53ef640fc657ff9243c2785041807d8..97b8115490b0838969b93c1a190f27e8c84b565a 100644 (file)
@@ -135,7 +135,6 @@ class CRM_Contact_Page_View_Relationship extends CRM_Core_Page {
   /**
    * called when action is browse
    *
-   * @return null
    */
   public function browse() {
     // do nothing :) we are using datatable for rendering relationship selectors
@@ -144,7 +143,6 @@ class CRM_Contact_Page_View_Relationship extends CRM_Core_Page {
   /**
    * called when action is update or new
    *
-   * @return null
    */
   public function edit() {
     $controller = new CRM_Core_Controller_Simple('CRM_Contact_Form_Relationship', ts('Contact Relationships'), $this->_action);
@@ -260,7 +258,6 @@ class CRM_Contact_Page_View_Relationship extends CRM_Core_Page {
   /**
    * called to delete the relationship of a contact
    *
-   * @return null
    */
   public function delete() {
     // calls a function to delete relationship
index 4ddf9a9e5808159ef7f6b76d2c7b924f8ba43fcb..9741a5e8783dfb22b04b629cea014062aaae4202 100644 (file)
@@ -159,7 +159,7 @@ class CRM_Contribute_Form_Contribution_Main extends CRM_Contribute_Form_Contribu
         foreach (array(
                    'Individual',
                    'Organization',
-                   'Household'
+                   'Household',
                  ) as $contactType) {
           if (in_array($contactType, $onBehalfProfile) &&
             (in_array('Membership', $onBehalfProfile) ||
@@ -176,7 +176,7 @@ class CRM_Contribute_Form_Contribution_Main extends CRM_Contribute_Form_Contribu
         foreach (array(
                    'Individual',
                    'Organization',
-                   'Household'
+                   'Household',
                  ) as $contactType) {
           if (in_array($contactType, $postProfile) &&
             (in_array('Membership', $postProfile) ||
@@ -687,7 +687,7 @@ class CRM_Contribute_Form_Contribution_Main extends CRM_Contribute_Form_Contribu
     $unitVals = explode(CRM_Core_DAO::VALUE_SEPARATOR, $frUnits);
 
     // CRM 10860, display text instead of a dropdown if there's only 1 frequency unit
-    if (sizeof($unitVals) == 1) {
+    if (count($unitVals) == 1) {
       $form->assign('one_frequency_unit', TRUE);
       $unit = $unitVals[0];
       $form->add('hidden', 'frequency_unit', $unit);
index 7662738b93c2d9800ac8befa4ab9cd9b87b0217d..4b32e02743c20a4d2ad27bc1f2119697959aac50 100644 (file)
@@ -106,7 +106,7 @@ class CRM_Contribute_Form_ManagePremiums extends CRM_Contribute_Form {
     $this->add('text', 'name', ts('Name'), CRM_Core_DAO::getAttribute('CRM_Contribute_DAO_Product', 'name'), TRUE);
     $this->addRule('name', ts('A product with this name already exists. Please select another name.'), 'objectExists', array(
         'CRM_Contribute_DAO_Product',
-        $this->_id
+        $this->_id,
       ));
     $this->add('text', 'sku', ts('SKU'), CRM_Core_DAO::getAttribute('CRM_Contribute_DAO_Product', 'sku'));
 
@@ -139,7 +139,7 @@ class CRM_Contribute_Form_ManagePremiums extends CRM_Contribute_Form {
     $this->add('select', 'period_type', ts('Period Type'), array(
         '' => '- select -',
         'rolling' => 'Rolling',
-        'fixed' => 'Fixed'
+        'fixed' => 'Fixed',
       ));
 
     $this->add('text', 'fixed_period_start_day', ts('Fixed Period Start Day'), CRM_Core_DAO::getAttribute('CRM_Contribute_DAO_Product', 'fixed_period_start_day'));
@@ -344,7 +344,7 @@ class CRM_Contribute_Form_ManagePremiums extends CRM_Contribute_Form {
       foreach (array(
                  'cost',
                  'price',
-                 'min_contribution'
+                 'min_contribution',
                ) as $f) {
         $params[$f] = CRM_Utils_Rule::cleanMoney($params[$f]);
       }
index 7f86dcd1cf50a5c35b94c42b5185febc02edc643..8ba973c2eb0c13234127e648122d22ffd8c07a70 100644 (file)
@@ -149,7 +149,7 @@ class CRM_Contribute_Form_UpdateSubscription extends CRM_Core_Form {
     // define the fields
     $this->addMoney('amount', ts('Recurring Contribution Amount'), TRUE,
       array(
-        'size' => 20
+        'size' => 20,
       ), TRUE,
       'currency', NULL, TRUE
     );
@@ -238,7 +238,7 @@ class CRM_Contribute_Form_UpdateSubscription extends CRM_Core_Form {
       if ($this->_subscriptionDetails->installments != $params['installments']) {
         $message .= "<br /> " . ts("Recurring contribution installments have been updated from %1 to %2 for this subscription.", array(
               1 => $this->_subscriptionDetails->installments,
-              2 => $params['installments']
+              2 => $params['installments'],
             )) . ' ';
       }
 
@@ -315,8 +315,9 @@ class CRM_Contribute_Form_UpdateSubscription extends CRM_Core_Form {
       CRM_Core_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',
         "reset=1&task=update&result=1"));
index cba249edf45911c8ae97aa90d51781b3ac63465d..13c912e28c8f06a5a77fc500f96cceba62fef6e7 100644 (file)
@@ -120,7 +120,7 @@ abstract class CRM_Contribute_Import_Parser extends CRM_Import_Parser {
    * @return mixed
    * @throws Exception
    */
-  function run(
+  public function run(
     $fileName,
     $seperator = ',',
     &$mapper,
@@ -407,7 +407,7 @@ abstract class CRM_Contribute_Import_Parser extends CRM_Import_Parser {
    * Given a list of the importable field keys that the user has selected
    * set the active fields array to this list
    *
-   * @param array mapped array of values
+   * @param array $fieldKeys mapped array of values
    *
    * @return void
    */
index c3478a8be33dcc9fdff8472f08ec5844cdba1978..dc985d5049c1fb05134cf355dd9b14e39445778a 100644 (file)
@@ -133,7 +133,7 @@ class CRM_Core_BAO_Block {
    * @param array $params
    *   Associated array of submitted fields.
    *
-   * @return boolean
+   * @return bool
    *   true if the block has data, otherwise false
    */
   public static function dataExists($blockFields, &$params) {
@@ -153,7 +153,7 @@ class CRM_Core_BAO_Block {
    * @param array $params
    *   Associated array of submitted fields.
    *
-   * @return boolean
+   * @return bool
    *   true if the block exits, otherwise false
    */
   public static function blockExists($blockName, &$params) {
@@ -518,7 +518,7 @@ class CRM_Core_BAO_Block {
    * (sort function for sortPrimaryFirst)
    * @param array $location1
    * @param array $location2
-   * @return number
+   * @return int
    */
   public static function primaryComparison($location1, $location2) {
     $l1 = CRM_Utils_Array::value('is_primary', $location1);
index 54bcf9dc11568f77089f6df00517b261093bb821..a8a000495fa34c4311d2fd61184e6e529d81dda6 100644 (file)
@@ -64,13 +64,13 @@ class CRM_Core_BAO_CustomOption {
    *
    * @param int $fieldID
    *   Field whose options are needed.
-   * @param bool $inactiveNeededDo we need inactive options ?.
+   * @param bool $inactiveNeeded Do we need inactive options ?.
    *   Do we need inactive options ?.
    *
    * @return array
    *   all active options for fieldId
    */
-  static function getCustomOption(
+  public static function getCustomOption(
     $fieldID,
     $inactiveNeeded = FALSE
   ) {
@@ -153,7 +153,7 @@ WHERE  id = %1
       case 'Autocomplete-Select':
         if (!in_array($dataType, array(
           'Boolean',
-          'ContactReference'
+          'ContactReference',
         ))
         ) {
           $options = self::valuesByID($fieldId);
index c47b4aa3ef800c18c173a66d00cfc382cc59fa32..28a93932e4c97d046cfa87d5944ff0f4f0208317 100644 (file)
@@ -131,7 +131,7 @@ class CRM_Core_BAO_Website extends CRM_Core_DAO_Website {
    * @param array $params
    * @param $values
    *
-   * @return boolean
+   * @return bool
    */
   public static function &getValues(&$params, &$values) {
     $websites = array();
index aab1d998947a4cfc6f8eab4db9706458e5733a74..149b1f13d007de05ee1d92ab9df575cf584c83b2 100644 (file)
@@ -230,7 +230,7 @@ class CRM_Core_Component {
       $cfg = $comp->getConfigObject();
       $cfg->add($config, $oldMode);
     }
-    return;
+    return NULL;
   }
 
   /**
@@ -327,7 +327,7 @@ class CRM_Core_Component {
    *
    * @return null
    */
-  static function &defaultReturnProperties(
+  public static function &defaultReturnProperties(
     $mode,
     $includeCustomFields = TRUE
   ) {
@@ -423,7 +423,6 @@ class CRM_Core_Component {
    * @param array $tables
    *   Array of tables.
    *
-   * @return null
    */
   public static function tableNames(&$tables) {
     $info = self::_info();
index 992a03e5b3c8714063880582fa03c99dfdc98780..1b4d6f86fb271f17617c635f05e80aad7da64e4c 100644 (file)
@@ -67,7 +67,7 @@ class CRM_Core_Form_Renderer extends HTML_QuickForm_Renderer_ArraySmarty {
   /**
    * Constructor
    */
-  function __construct() {
+  public function __construct() {
     $template = CRM_Core_Smarty::singleton();
     parent::__construct($template);
   }
@@ -147,7 +147,6 @@ class CRM_Core_Form_Renderer extends HTML_QuickForm_Renderer_ArraySmarty {
    * @param string $error
    *   Error associated with the element.
    *
-   * @return array
    */
   public static function updateAttributes(&$element, $required, $error) {
     // lets create an id for all input elements, so we can generate nice label tags
@@ -289,7 +288,7 @@ class CRM_Core_Form_Renderer extends HTML_QuickForm_Renderer_ArraySmarty {
         }
         $field->setAttribute('data-entity-value', json_encode(array(
               'id' => $contact['id'],
-              'text' => implode(' :: ', $view)
+              'text' => implode(' :: ', $view),
             )));
       }
     }
index 106d00efa61caac555caf99bb670acd3ad4ce358..02b88a9b20af8fe4b1a5a3748e2ea48814dae60f 100644 (file)
@@ -36,8 +36,7 @@ class CRM_Core_I18n_SchemaStructure_3_3_beta1 {
   /**
    * @return array
    */
-  static
-  public function &columns() {
+  static public function &columns() {
     static $result = NULL;
     if (!$result) {
       $result = array(
@@ -205,8 +204,7 @@ class CRM_Core_I18n_SchemaStructure_3_3_beta1 {
   /**
    * @return array
    */
-  static
-  public function &indices() {
+  static public function &indices() {
     static $result = NULL;
     if (!$result) {
       $result = array(
@@ -287,8 +285,7 @@ class CRM_Core_I18n_SchemaStructure_3_3_beta1 {
   /**
    * @return array
    */
-  static
-  public function &tables() {
+  static public function &tables() {
     static $result = NULL;
     if (!$result) {
       $result = array_keys(self::columns());
index 88fa623cc5e7abb0e3d08f81c9b621864427919a..3336c2035cba3e7f89179f2a2608f09b64c2998e 100644 (file)
@@ -106,11 +106,11 @@ class CRM_Core_Payment_IATS extends CRM_Core_Payment {
 
     if ($isRecur) {
       include_once 'Services/IATS/iats_reoccur.php';
-      $iatslink1 = new iatslinkReoccur;
+      $iatslink1 = new iatslinkReoccur();
     }
     else {
       include_once 'Services/IATS/iatslink.php';
-      $iatslink1 = new iatslink;
+      $iatslink1 = new iatslink();
     }
 
     $iatslink1->setTestMode($this->_profile['mode'] != 'live');
@@ -204,7 +204,7 @@ class CRM_Core_Payment_IATS extends CRM_Core_Payment {
 
         default:
           die('Invalid frequency unit!');
-          break;
+        break;
       }
       $endDate = date('Y-m-d', $endTime);
       $startDate = date('Y-m-d', $startTime);
index c9569fc5a06d80703028057745350ecbfedc96ad..6e8760f04b8461351ff20314c0d29c534a602a65 100644 (file)
@@ -69,7 +69,7 @@ class CRM_Core_Permission_Drupal6 extends CRM_Core_Permission_DrupalBase {
    *
    * @param int $contactID
    *
-   * @return boolean
+   * @return bool
    *   true if yes, else false
    */
   public function check($str, $contactID = NULL) {
@@ -95,10 +95,9 @@ class CRM_Core_Permission_Drupal6 extends CRM_Core_Permission_DrupalBase {
    * @param array $array
    *   The roles to check.
    *
-   * @return boolean
+   * @return bool
    *   true if yes, else false
    */
-
   public function checkGroupRole($array) {
     if (function_exists('user_load') && isset($array)) {
       $user = user_load(array('uid' => $GLOBALS['user']->uid));
index c2e7193063cd77b2082b3101b363d3608ebf8140..7365321b9149b29a9c98325a2f843fa59a3382db 100644 (file)
@@ -118,7 +118,7 @@ class CRM_Core_QuickForm_GroupMultiSelect extends CRM_Core_QuickForm_NestedAdvMu
       // set name of Select From Box
       $this->_attributesUnselected = array(
         'name' => '__' . $selectName,
-        'ondblclick' => "{$this->_jsPrefix}{$this->_jsPostfix}(this.form.elements['__" . $selectName . "'], this.form.elements['_" . $selectName . "'], this.form.elements['" . $selectName . "'], 'add')"
+        'ondblclick' => "{$this->_jsPrefix}{$this->_jsPostfix}(this.form.elements['__" . $selectName . "'], this.form.elements['_" . $selectName . "'], this.form.elements['" . $selectName . "'], 'add')",
       );
       $this->_attributesUnselected = array_merge($this->_attributes, $this->_attributesUnselected);
       $attrUnselected = $this->_getAttrString($this->_attributesUnselected);
@@ -126,7 +126,7 @@ class CRM_Core_QuickForm_GroupMultiSelect extends CRM_Core_QuickForm_NestedAdvMu
       // set name of Select To Box
       $this->_attributesSelected = array(
         'name' => '_' . $selectName,
-        'ondblclick' => "{$this->_jsPrefix}{$this->_jsPostfix}(this.form.elements['__" . $selectName . "'], this.form.elements['_" . $selectName . "'], this.form.elements['" . $selectName . "'], 'remove')"
+        'ondblclick' => "{$this->_jsPrefix}{$this->_jsPostfix}(this.form.elements['__" . $selectName . "'], this.form.elements['_" . $selectName . "'], this.form.elements['" . $selectName . "'], 'remove')",
       );
       $this->_attributesSelected = array_merge($this->_attributes, $this->_attributesSelected);
       $attrSelected = $this->_getAttrString($this->_attributesSelected);
@@ -134,7 +134,7 @@ class CRM_Core_QuickForm_GroupMultiSelect extends CRM_Core_QuickForm_NestedAdvMu
       // set name of Select hidden Box
       $this->_attributesHidden = array(
         'name' => $selectName,
-        'style' => 'overflow: hidden; visibility: hidden; width: 1px; height: 0;'
+        'style' => 'overflow: hidden; visibility: hidden; width: 1px; height: 0;',
       );
       $this->_attributesHidden = array_merge($this->_attributes, $this->_attributesHidden);
       $attrHidden = $this->_getAttrString($this->_attributesHidden);
index 1674ec58734cff762fe73c0ef4c92a2db316e116..f9c6162d2c1f75bd61070da65780165d44f306f5 100644 (file)
@@ -63,8 +63,8 @@ class CRM_Core_Selector_Base {
    * This function gets the attribute for the action that
    * it matches.
    *
-   * @param string match the action to match against
-   * @param string attribute the attribute to return ( name, link, title )
+   * @param string $match the action to match against
+   * @param string $attribute the attribute to return ( name, link, title )
    *
    * @return string
    *   the attribute that matches the action if any
@@ -112,7 +112,7 @@ class CRM_Core_Selector_Base {
   /**
    * Getter for the sorting direction for the fields which will be displayed on the form.
    *
-   * @param string action the action being performed
+   * @param string $action the action being performed
    *
    * @return array
    *   the elements that can be sorted along with their properties
index 157a25d763ba8f02467d6f5f285876713603b25e..046eb4eb2beb19b0ecf74bc8e331e561dc9b832b 100644 (file)
@@ -119,7 +119,7 @@ class CRM_Core_Selector_Controller {
    * so the display routine needs to not do any work. (The
    * parent object takes care of the display)
    *
-   * @var boolean
+   * @var bool
    */
   protected $_embedded = FALSE;
 
@@ -127,7 +127,7 @@ class CRM_Core_Selector_Controller {
    * Are we in print mode? if so we need to modify the display
    * functionality to do a minimal display :)
    *
-   * @var boolean
+   * @var bool
    */
   protected $_print = FALSE;
 
@@ -169,7 +169,7 @@ class CRM_Core_Selector_Controller {
   /**
    * Should we compute actions dynamically (since they are quite verbose)
    *
-   * @var boolean
+   * @var bool
    */
   protected $_dynamicAction = FALSE;
 
@@ -258,7 +258,7 @@ class CRM_Core_Selector_Controller {
    * @param int $reset
    *   Are we being reset.
    *
-   * @return boolean
+   * @return bool
    *   if the GET params are different from the session params
    */
   public function hasChanged($reset) {
@@ -496,7 +496,7 @@ class CRM_Core_Selector_Controller {
   /**
    * Getter for embedded
    *
-   * @return boolean
+   * @return bool
    *   return the embedded value
    */
   public function getEmbedded() {
@@ -517,7 +517,7 @@ class CRM_Core_Selector_Controller {
   /**
    * Getter for print
    *
-   * @return boolean
+   * @return bool
    *   return the print value
    */
   public function getPrint() {
index df162f25ad57855c15dad66d404876ff9d52535c..88e539c5b78a1b06515878b8a994c3b3e889c857 100644 (file)
@@ -95,7 +95,7 @@ class CRM_Core_Session {
    */
   public static function &singleton() {
     if (self::$_singleton === NULL) {
-      self::$_singleton = new CRM_Core_Session;
+      self::$_singleton = new CRM_Core_Session();
     }
     return self::$_singleton;
   }
@@ -144,7 +144,7 @@ class CRM_Core_Session {
     ) {
       $this->_session[$this->_key] = array();
     }
-    return;
+    return NULL;
   }
 
   /**
@@ -167,7 +167,7 @@ class CRM_Core_Session {
       $this->_session = array();
     }
 
-    return;
+    return NULL;
   }
 
   /**
index b531dc0ddaeaf02910d0bd1344294f1b60db227d..7b82e0b170a8627a7f5bea5635f7d4a856027371 100644 (file)
@@ -55,8 +55,6 @@ class CRM_Custom_Form_DeleteField extends CRM_Core_Form {
   /**
    * Set up variables to build the form
    *
-   * @param null
-   *
    * @return void
    * @acess protected
    */
@@ -75,8 +73,6 @@ class CRM_Custom_Form_DeleteField extends CRM_Core_Form {
   /**
    * Build the form object
    *
-   * @param null
-   *
    * @return void
    */
   public function buildQuickForm() {
@@ -98,8 +94,6 @@ class CRM_Custom_Form_DeleteField extends CRM_Core_Form {
   /**
    * Process the form when submitted
    *
-   * @param null
-   *
    * @return void
    */
   public function postProcess() {
index d5347b3b4a843690125765c155d354fd56c878af..92c867b7b585d39afc0d096fa568d9545e181ed6 100644 (file)
@@ -55,8 +55,6 @@ class CRM_Custom_Form_Preview extends CRM_Core_Form {
    *
    * gets session variables for group or field id
    *
-   * @param null
-   *
    * @return void
    */
   public function preProcess() {
@@ -94,8 +92,6 @@ class CRM_Custom_Form_Preview extends CRM_Core_Form {
   /**
    * Set the default form values
    *
-   * @param null
-   *
    * @return array
    *   the default array reference
    */
@@ -110,8 +106,6 @@ class CRM_Custom_Form_Preview extends CRM_Core_Form {
   /**
    * Build the form object
    *
-   * @param null
-   *
    * @return void
    */
   public function buildQuickForm() {
index c8ada7f0a654fd0fc485573da8894caa7400c6e0..45d2f7199cb053f218eaf2adede95f130d6ec77b 100644 (file)
@@ -64,7 +64,7 @@ class CRM_Event_Form_ManageEvent_ScheduleReminders extends CRM_Event_Form_Manage
         else {
           $action += CRM_Core_Action::ENABLE;
         }
-        $scheduleReminder = new CRM_Admin_Page_ScheduleReminders;
+        $scheduleReminder = new CRM_Admin_Page_ScheduleReminders();
         $links = $scheduleReminder->links();
         $links[CRM_Core_Action::DELETE]['qs'] .= "&context=event&compId={$this->_id}";
         $links[CRM_Core_Action::UPDATE]['qs'] .= "&context=event&compId={$this->_id}";
index b901947c09d7b8e1aeda354888c3a184ff0fc0bd..4fdd2cd163f627ed966bf17dac1e4aae019cedd4 100644 (file)
@@ -239,7 +239,7 @@ class CRM_Event_Form_Registration_Confirm extends CRM_Event_Form_Registration {
         if (is_array($v)) {
           foreach (array(
                      'first_name',
-                     'last_name'
+                     'last_name',
                    ) as $name) {
             if (isset($v['billing_' . $name]) &&
               !isset($v[$name])
@@ -640,8 +640,7 @@ class CRM_Event_Form_Registration_Confirm extends CRM_Event_Form_Registration {
           }
 
           //passing contribution id is already registered.
-          $contribution =
-            self::processContribution($this, $value, $result, $contactID, $pending, $isAdditionalAmount);
+          $contribution = self::processContribution($this, $value, $result, $contactID, $pending, $isAdditionalAmount);
           $value['contributionID'] = $contribution->id;
           $value['contributionTypeID'] = $contribution->financial_type_id;
           $value['receive_date'] = $contribution->receive_date;
@@ -958,7 +957,7 @@ class CRM_Event_Form_Registration_Confirm extends CRM_Event_Form_Registration {
    *
    * @return void
    */
-  static function processContribution(
+  public static function processContribution(
     &$form, $params, $result, $contactID,
     $pending = FALSE, $isAdditionalAmount = FALSE
   ) {
index 7269f6665fda8b0064833419427eeef433098558..6f23ff5ab5d7abbbd3c9b5f1590cc10932733781 100644 (file)
@@ -79,7 +79,6 @@ class CRM_Event_Form_Task_Delete extends CRM_Event_Form_Task {
       2 => ts('Delete only this participant record.'),
     );
 
-
     $this->addRadio('delete_participant', NULL, $deleteParticipants, NULL, '<br />');
     $this->setDefaults(array('delete_participant' => 1));
 
@@ -127,8 +126,8 @@ class CRM_Event_Form_Task_Delete extends CRM_Event_Form_Task {
     $status = array(
       ts('Participant(s) Deleted: %1 (Total Selected: %2)', array(
           1 => $deletedParticipants,
-          2 => count($this->_participantIds)
-        ))
+          2 => count($this->_participantIds),
+        )),
     );
     if (!empty($participantLinks)) {
       $status[] = ts('The following participants no longer have an event fee recorded. You can edit their registration and record a replacement contribution by clicking the links below:') . '<br/>' . $participantLinks;
index 152fc2bd2ec4573a55d0279f8b2627ce6567fdee..1227a184ea16c58cfc41cf67a19b196f6f4f2ed1 100644 (file)
@@ -73,7 +73,7 @@ class CRM_Export_BAO_Export {
    * @param string $queryOperator
    *
    */
-  static function exportComponents(
+  public static function exportComponents(
     $selectAll,
     $ids,
     $params,
@@ -322,8 +322,7 @@ class CRM_Export_BAO_Export {
         $componentReturnProperties = CRM_Contact_BAO_Query::defaultReturnProperties($queryMode);
         if ($queryMode == CRM_Contact_BAO_Query::MODE_CONTRIBUTE) {
           // soft credit columns are not automatically populated, because contribution search doesn't require them by default
-          $componentReturnProperties =
-            array_merge(
+          $componentReturnProperties = array_merge(
               $componentReturnProperties,
               CRM_Contribute_BAO_Query::softCreditReturnProperties(TRUE));
         }
@@ -1692,7 +1691,7 @@ WHERE  id IN ( $deleteIDString )
    * Example: 'Hello {contact.first_name}' => converted to => '{contact.first_name}'
    * i.e 'Hello Alan' => converted to => 'Alan'
    */
-  static function _trimNonTokens(
+  public static function _trimNonTokens(
     &$parsedString, $defaultGreeting,
     $addressMergeGreetings, $greetingType = 'postal_greeting'
   ) {
index e48267dd10005b3c54490a8acf93cb5213e1d68d..8e031c0df2c3332f40104cd973b8cead344c19ef 100644 (file)
@@ -197,7 +197,7 @@ class CRM_Extension_Manager_Payment extends CRM_Extension_Manager_Base {
         CRM_Core_Session::setStatus(ts('Failed to load file (%3) for payment processor (%1) while running "%2"', array(
               1 => $info->key,
               2 => $method,
-              3 => $file
+              3 => $file,
             )), '', 'error');
         return;
       }
@@ -208,7 +208,7 @@ class CRM_Extension_Manager_Payment extends CRM_Extension_Manager_Base {
     catch (CRM_Extension_Exception $e) {
       CRM_Core_Session::setStatus(ts('Failed to determine file path for payment processor (%1) while running "%2"', array(
             1 => $info->key,
-            2 => $method
+            2 => $method,
           )), '', 'error');
       return;
     }
@@ -288,7 +288,7 @@ class CRM_Extension_Manager_Payment extends CRM_Extension_Manager_Base {
         // Does PP implement this method, and can we call it?
         if (method_exists($processorInstance, $method) && is_callable(array(
             $processorInstance,
-            $method
+            $method,
           ))
         ) {
           // If so, call it ...
index 406e862534347d20e55687a357527d57b6548437..eeea3031082d1b41274eace803e198feab138da5 100644 (file)
@@ -64,14 +64,14 @@ class CRM_Financial_Form_FinancialBatch extends CRM_Contribute_Form {
             'edit own manual batches',
             'edit all manual batches',
           ),
-          'actionName' => 'edit'
+          'actionName' => 'edit',
         ),
         CRM_Core_Action::DELETE => array(
           'permission' => array(
             'delete own manual batches',
             'delete all manual batches',
           ),
-          'actionName' => 'delete'
+          'actionName' => 'delete',
         ),
       );
 
@@ -296,7 +296,6 @@ class CRM_Financial_Form_FinancialBatch extends CRM_Contribute_Form {
    * @param int $userContactID
    * @param string $actionName
    *
-   * @return array
    *   list of errors to be posted back to the form
    */
   public function checkPermissions($action, $permissions, $createdID, $userContactID, $actionName) {
index 9607a3f0468a6ea719c1124dcf1ebf76582d04bb..950d87bd67429c1468f40db5dcf68fa5e20caa96 100644 (file)
@@ -75,7 +75,7 @@ class CRM_Friend_Form extends CRM_Core_Form {
 
     if (in_array($pcomponent, array(
       'contribute',
-      'event'
+      'event',
     ))) {
       $values = array();
       $params = array('id' => $this->_entityId);
index 66f358afeb103f5f7f9ca81cbbee6e265b5e6506..b7a9d6bb2fa8043f6e757df003b6ffff97f0e33e 100644 (file)
@@ -212,7 +212,7 @@ class CRM_Grant_BAO_Query {
 
         $query->_qill[$grouping][] = ts('Grant Type %2 %1', array(
             1 => implode(' ' . ts('or') . ' ', $names),
-            2 => $op
+            2 => $op,
           ));
         $query->_tables['civicrm_grant'] = $query->_whereTables['civicrm_grant'] = 1;
 
@@ -248,7 +248,7 @@ class CRM_Grant_BAO_Query {
 
         $query->_qill[$grouping][] = ts('Grant Status %2 %1', array(
             1 => implode(' ' . ts('or') . ' ', $names),
-            2 => $op
+            2 => $op,
           ));
         $query->_tables['civicrm_grant'] = $query->_whereTables['civicrm_grant'] = 1;
 
@@ -331,7 +331,7 @@ class CRM_Grant_BAO_Query {
    *
    * @return array|null
    */
-  static function defaultReturnProperties(
+  public static function defaultReturnProperties(
     $mode,
     $includeCustomFields = TRUE
   ) {
index c1282d9a0f2ed196f86b7cb535e77cd907ba5d65..bcdb26f9d5f6c4cccc9cfcde6caadf2f31ab7fef 100644 (file)
@@ -177,7 +177,7 @@ class CRM_Mailing_Form_Test extends CRM_Core_Form {
    * @param array $self
    *   An current this object.
    *
-   * @return boolean
+   * @return bool
    *   true on successful SMTP handoff
    */
   public static function testMail($testParams, $files, $self) {
@@ -266,7 +266,7 @@ class CRM_Mailing_Form_Test extends CRM_Core_Form {
     ) {
       return array(
         '_qf_default' =>
-          ts("You can not schedule or send this mailing because there are currently no recipients selected. Click 'Previous' to return to the Select Recipients step, OR click 'Save & Continue Later'."),
+        ts("You can not schedule or send this mailing because there are currently no recipients selected. Click 'Previous' to return to the Select Recipients step, OR click 'Save & Continue Later'."),
       );
     }
 
@@ -326,7 +326,7 @@ ORDER BY   e.is_bulkmail DESC, e.is_primary DESC
                 'email' => $email,
                 'is_primary' => 1,
                 'location_type_id' => 1,
-              )
+              ),
             ),
           );
           $contact = CRM_Contact_BAO_Contact::create($params);
index 2c3384e309c886590a1842d8057f11e38c76bba8..38e64683bd46d75877e292860da31f820f16d034 100644 (file)
@@ -64,7 +64,7 @@ class CRM_Mailing_Info extends CRM_Core_Component_Info {
       'js' => array(
         'js/angular-crmMailing.js',
         'js/angular-crmMailing/services.js',
-        'js/angular-crmMailing/directives.js'
+        'js/angular-crmMailing/directives.js',
       ),
       'css' => array('css/angular-crmMailing.css'),
     );
@@ -73,7 +73,7 @@ class CRM_Mailing_Info extends CRM_Core_Component_Info {
       'js' => array(
         'js/angular-crmMailingAB.js',
         'js/angular-crmMailingAB/services.js',
-        'js/angular-crmMailingAB/directives.js'
+        'js/angular-crmMailingAB/directives.js',
       ),
       'css' => array('css/angular-crmMailingAB.css'),
     );
@@ -139,7 +139,7 @@ class CRM_Mailing_Info extends CRM_Core_Component_Info {
         'fromAddress' => array_values($fromAddress['values'][0]['api.OptionValue.get']['values']),
         'defaultTestEmail' => civicrm_api3('Contact', 'getvalue', array(
             'id' => 'user_contact_id',
-            'return' => 'email'
+            'return' => 'email',
           )),
         'visibility' => array(
           array('value' => 'Public Pages', 'label' => ts('Public Pages')),
index f11c413f224aad4c0ffe3d13ecd1c537cdc0df6e..51259f7e364d3af21406977e4618d2741f6dad1a 100644 (file)
@@ -47,7 +47,7 @@ class CRM_Mailing_MailStore {
    *   mail store implementation for processing CiviMail-bound emails
    */
   public static function getStore($name = NULL) {
-    $dao = new CRM_Core_DAO_MailSettings;
+    $dao = new CRM_Core_DAO_MailSettings();
     $dao->domain_id = CRM_Core_Config::domainID();
     $name ? $dao->name = $name : $dao->is_default = 1;
     if (!$dao->find(TRUE)) {
@@ -128,7 +128,7 @@ class CRM_Mailing_MailStore {
       return array();
     }
     $mails = array();
-    $parser = new ezcMailParser;
+    $parser = new ezcMailParser();
     //set property text attachment as file CRM-5408
     $parser->options->parseTextAttachmentsAsFiles = TRUE;
 
@@ -158,7 +158,7 @@ class CRM_Mailing_MailStore {
     foreach (array(
                'cur',
                'new',
-               'tmp'
+               'tmp',
              ) as $sub) {
       if (!file_exists($dir . DIRECTORY_SEPARATOR . $sub)) {
         if ($this->_debug) {
index 27487fd1a875138774ca760333f8b4fd8e34f242..4c8cfdd9b9c81d155bc649ee09e3bcb46cb6a1f9 100644 (file)
@@ -56,13 +56,13 @@ class CRM_Mailing_MailStore_Localdir extends CRM_Mailing_MailStore {
           'CiviMail.ignored',
           date('Y'),
           date('m'),
-          date('d')
+          date('d'),
         )));
     $this->_processed = $this->maildir(implode(DIRECTORY_SEPARATOR, array(
           'CiviMail.processed',
           date('Y'),
           date('m'),
-          date('d')
+          date('d'),
         )));
   }
 
@@ -101,7 +101,7 @@ class CRM_Mailing_MailStore_Localdir extends CRM_Mailing_MailStore {
       }
 
       $set = new ezcMailFileSet(array($file));
-      $parser = new ezcMailParser;
+      $parser = new ezcMailParser();
       //set property text attachment as file CRM-5408
       $parser->options->parseTextAttachmentsAsFiles = TRUE;
 
index 1e7e31c7fd065b04ba5d80147965de5ac5650a9f..aeef774c4a53588f6911258483fbad4089121cde 100644 (file)
@@ -69,7 +69,7 @@ class CRM_Member_BAO_MembershipPayment extends CRM_Member_DAO_MembershipPayment
     // OR the caller will have taken responsibility for updating the line items themselves so we will update using SQL here
     $membershipTypeID = civicrm_api3('membership', 'getvalue', array(
         'id' => $dao->membership_id,
-        'return' => 'membership_type_id'
+        'return' => 'membership_type_id',
       ));
     $sql = "UPDATE civicrm_line_item li
       LEFT JOIN civicrm_price_field_value pv ON pv.id = li.price_field_value_id
@@ -79,7 +79,7 @@ class CRM_Member_BAO_MembershipPayment extends CRM_Member_DAO_MembershipPayment
     CRM_Core_DAO::executeQuery($sql, array(
         1 => array($dao->membership_id, 'Integer'),
         2 => array($membershipTypeID, 'Integer'),
-        3 => array($dao->contribution_id, 'Integer')
+        3 => array($dao->contribution_id, 'Integer'),
       ));
     return $dao;
   }
index c4818fb97be02cff122261c523ebcc1ecfc7e5f0..6f830f517e9b0a9b6da01badccf9536bbcae769b 100644 (file)
@@ -95,7 +95,7 @@ class CRM_Member_Import_Form_MapField extends CRM_Import_Form_MapField {
         'email',
         'external_identifier',
         'membership_start_date',
-        'membership_type_id'
+        'membership_type_id',
       );
       foreach ($highlightedFieldsArray as $name) {
         $highlightedFields[] = $name;
@@ -362,7 +362,7 @@ class CRM_Member_Import_Form_MapField extends CRM_Import_Form_MapField {
                 $errors['_qf_default'] = '';
               }
               $errors['_qf_default'] .= ts('Missing required contact matching fields.') . " $fieldMessage " . ts('(Sum of all weights should be greater than or equal to threshold: %1).', array(
-                  1 => $threshold
+                1 => $threshold,
                 )) . ' ' . ts('(OR Membership ID if update mode.)') . '<br />';
             }
           }
@@ -371,7 +371,7 @@ class CRM_Member_Import_Form_MapField extends CRM_Import_Form_MapField {
               $errors['_qf_default'] = '';
             }
             $errors['_qf_default'] .= ts('Missing required field: %1', array(
-                1 => $title
+              1 => $title,
               )) . '<br />';
           }
         }
index a53740f9942f47aef5e003ca635192dbbe0cb690..3d73bc8041e3321ad90f8d087deeddf97f66c4f5 100644 (file)
@@ -41,7 +41,6 @@ class CRM_Member_Page_UserDashboard extends CRM_Contact_Page_View_UserDashBoard
   /**
    * List memberships for the UF user
    *
-   * @return null
    */
   public function listMemberships() {
     $membership = array();
@@ -58,7 +57,7 @@ class CRM_Member_Page_UserDashboard extends CRM_Contact_Page_View_UserDashBoard
       $statusANDType = CRM_Member_BAO_Membership::getStatusANDTypeValues($dao->id);
       foreach (array(
                  'status',
-                 'membership_type'
+                 'membership_type',
                ) as $fld) {
         $membership[$dao->id][$fld] = CRM_Utils_Array::value($fld, $statusANDType[$dao->id]);
       }
@@ -96,7 +95,6 @@ class CRM_Member_Page_UserDashboard extends CRM_Contact_Page_View_UserDashBoard
    * the main function that is called when the page
    * loads, it decides the which action has to be taken for the page.
    *
-   * @return null
    */
   public function run() {
     parent::preProcess();
index e7c9a1e4d3adb2eff9f87a8da4213a98868a6073..e7cb069028c369d7070e0f430e25b3cff09dc94f 100644 (file)
@@ -143,7 +143,7 @@ class CRM_Member_Selector_Search extends CRM_Core_Selector_Base implements CRM_C
    *
    * @return \CRM_Member_Selector_Search
    */
-  function __construct(
+  public function __construct(
     &$queryParams,
     $action = CRM_Core_Action::NONE,
     $memberClause = NULL,
@@ -190,8 +190,7 @@ class CRM_Member_Selector_Search extends CRM_Core_Selector_Base implements CRM_C
    *
    * @return array
    */
-  static
-  function &links(
+  public static function &links(
     $status = 'all',
     $isPaymentProcessor = NULL,
     $accessContribution = NULL,
index c60fa7fc91c69f64b4763f3aeabe039eb79d3e2b..95f110ed6eac87c9bbe45c488b18c4ca7959ffe2 100644 (file)
@@ -780,7 +780,7 @@ GROUP BY  currency
     foreach (array(
                'Pending',
                'In Progress',
-               'Overdue'
+               'Overdue',
              ) as $name) {
       if ($statusId = array_search($name, $pledgeStatuses)) {
         $status[] = $statusId;
@@ -839,7 +839,7 @@ GROUP BY  currency
     foreach (array(
                'Completed',
                'Cancelled',
-               'Failed'
+               'Failed',
              ) as $statusKey) {
       if ($key = CRM_Utils_Array::key($statusKey, $allStatus)) {
         unset($allStatus[$key]);
@@ -1125,7 +1125,7 @@ SELECT  pledge.contact_id              as contact_id,
 
     return civicrm_api3('pledge_payment', 'getcount', array(
         'pledge_id' => $pledgeID,
-        'status_id' => array('IN' => self::getTransactionalStatus())
+        'status_id' => array('IN' => self::getTransactionalStatus()),
       ));
   }
 
index 7deb2ac27bf9fc5423d3933e749ce1def2bf525d..2de607bd6371a0ff8f0e806a1d7ff5b22a5d5f2f 100644 (file)
@@ -213,7 +213,7 @@ WHERE     pledge_id = %1
    * @return CRM_Pledge_BAO_PledgePayment
    */
   public static function retrieve(&$params, &$defaults) {
-    $payment = new CRM_Pledge_BAO_PledgePayment;
+    $payment = new CRM_Pledge_BAO_PledgePayment();
     $payment->copyValues($params);
     if ($payment->find(TRUE)) {
       CRM_Core_DAO::storeValues($payment, $defaults);
@@ -339,7 +339,7 @@ WHERE     pledge_id = %1
    * @return int
    *   $newStatus, updated status id (or 0)
    */
-  static function updatePledgePaymentStatus(
+  public static function updatePledgePaymentStatus(
     $pledgeID,
     $paymentIDs = NULL,
     $paymentStatusID = NULL,
@@ -631,7 +631,7 @@ WHERE  civicrm_pledge.id = %2
    *   , is function being called from bin script?.
    *
    */
-  static function updatePledgePayments(
+  public static function updatePledgePayments(
     $pledgeId,
     $paymentStatusId,
     $paymentIds = NULL,
index e26b2ad9d7dd0788ee4e07173e7314969574d56d..98eb57866e248b70cb823f465671e67f07074e5e 100644 (file)
@@ -37,7 +37,6 @@ class CRM_Pledge_Page_UserDashboard extends CRM_Contact_Page_View_UserDashBoard
   /**
    * called when action is browse
    *
-   * @return null
    */
   public function listPledges() {
     $controller = new CRM_Core_Controller_Simple(
@@ -72,7 +71,6 @@ class CRM_Pledge_Page_UserDashboard extends CRM_Contact_Page_View_UserDashBoard
    * the main function that is called when the page
    * loads, it decides the which action has to be taken for the page.
    *
-   * @return null
    */
   public function run() {
     parent::preProcess();
index a570ff68b7eef046f73502ba79b68cbc34028f68..23bd609dbaadde299fe7b2f920c928a998276cda 100644 (file)
@@ -311,7 +311,7 @@ SELECT module,is_reserved
    * This is a first version and will be tweaked over a period of time
    *
    *
-   * @return boolean
+   * @return bool
    *   true if no error found
    */
   public function validate() {
index 4a7fbba404d4a0ccd6093c5236386394fba21b62..9ce1496e31405fa45f2bdba4e59bcd1faa8d6fcc 100644 (file)
@@ -196,7 +196,7 @@ class CRM_Report_Utils_Get {
     $chartType = CRM_Utils_Array::value("charts", $_GET);
     if (in_array($chartType, array(
       'barChart',
-      'pieChart'
+      'pieChart',
     ))) {
       $defaults["charts"] = $chartType;
     }
index 5ca9f9f3af20a33dfce8dd82de6e4e590396dd3d..21f140d9ab6e562b2ba0030920c9864870cd0912 100644 (file)
@@ -452,13 +452,12 @@ class CRM_Utils_OpenFlashChart {
       foreach ($rows['multiValue'] as $key => $val) {
         $graph[$key] = array_combine($dateKeys, $rows['multiValue'][$key]);
       }
-      $chartData =
-        array(
-          'legend' => "$legend " . CRM_Utils_Array::value('legend', $rows, ts('Contribution')) . ' ' . ts('Summary'),
-          'values' => $graph[0],
-          'multiValues' => $graph,
-          'barKeys' => CRM_Utils_Array::value('barKeys', $rows, array()),
-        );
+      $chartData = array(
+        'legend' => "$legend " . CRM_Utils_Array::value('legend', $rows, ts('Contribution')) . ' ' . ts('Summary'),
+        'values' => $graph[0],
+        'multiValues' => $graph,
+        'barKeys' => CRM_Utils_Array::value('barKeys', $rows, array()),
+      );
     }
 
     // rotate the x labels.
@@ -475,7 +474,7 @@ class CRM_Utils_OpenFlashChart {
     foreach (array(
                'xSize',
                'ySize',
-               'divName'
+               'divName',
              ) as $f) {
       if (!empty($rows[$f])) {
         $chartData[$f] = $rows[$f];
@@ -515,7 +514,7 @@ class CRM_Utils_OpenFlashChart {
     foreach (array(
                'xSize',
                'ySize',
-               'divName'
+               'divName',
              ) as $f) {
       if (!empty($rows[$f])) {
         $chartData[$f] = $rows[$f];
index 6b1222ced92cf7cfdc35ca8a66fa90125e1c382a..ee18da9ea10f741fb221f8df6a3f186caaf59cf7 100644 (file)
@@ -36,17 +36,10 @@ class CRM_Utils_SystemLogger extends Psr\Log\AbstractLogger implements \Psr\Log\
   /**
    * Logs with an arbitrary level.
    *
-   * @param mixed $level
-   * @param string $message
-   * @param array $context
-   * @return null
-   */
-  /**
    * @param mixed $level
    * @param string $message
    * @param array $context
    *
-   * @return null
    */
   public function log($level, $message, array $context = array()) {
     if (!isset($context['hostname'])) {
index da355eef6c3e0ce9d69b5d964e7cb4f52034677d..9be467237db26d8f81ef59e26a935d2cb342849a 100644 (file)
@@ -46,7 +46,7 @@
  *
  * @return array
  *   Array of newly created membership status property values.
- * {@getfields MembershipStatus_create}
+ *   {@getfields MembershipStatus_create}
  */
 function civicrm_api3_membership_status_create($params) {
   return _civicrm_api3_basic_create(_civicrm_api3_get_BAO(__FUNCTION__), $params);
@@ -62,7 +62,7 @@ function civicrm_api3_membership_status_create($params) {
  *
  * @return array
  *   Array of all found membership status property values.
- * {@getfields MembershipStatus_get}
+ *   {@getfields MembershipStatus_get}
  */
 function civicrm_api3_membership_status_get($params) {
   return _civicrm_api3_basic_get('CRM_Member_BAO_MembershipStatus', $params);
@@ -135,7 +135,7 @@ function civicrm_api3_membership_status_delete($params) {
  *
  * @throws API_Exception
  *
- * @return Array
+ * @return array
  *   Array of status id and status name
  * @public
  */
index b552e303143a8e69eee0c21fdb383491a0e0bdf8..f693c9c0e8ef1a5b6e925933fa6ace7bf016279d 100644 (file)
@@ -10,7 +10,7 @@
  *
  * @return array
  *   details of found Option Values
- * {@getfields OptionValue_get}
+ *   {@getfields OptionValue_get}
  */
 function civicrm_api3_option_value_get($params) {
 
@@ -38,7 +38,7 @@ function civicrm_api3_option_value_get($params) {
  * @throws API_Exception
  * @return array
  *   Array of newly created option_value property values.
- * {@getfields OptionValue_create}
+ *   {@getfields OptionValue_create}
  */
 function civicrm_api3_option_value_create($params) {
   $result = _civicrm_api3_basic_create(_civicrm_api3_get_BAO(__FUNCTION__), $params);
@@ -79,7 +79,7 @@ function _civicrm_api3_option_value_create_spec(&$params) {
  *
  * @return array
  *   Api result
- * {@getfields OptionValue_create}
+ *   {@getfields OptionValue_create}
  */
 function civicrm_api3_option_value_delete($params) {
   // we will get the option group id before deleting so we can flush pseudoconstants
index 9ce240473dcb0cef0de701f5ccd4a76a2cbe6622..2d4742bddede9071969218929d1cb53b3e35039c 100644 (file)
@@ -48,7 +48,7 @@
  *
  * @return array
  *   apiresult
- * {@getfields participant_create}
+ *   {@getfields participant_create}
  */
 function civicrm_api3_participant_create($params) {
   //check that event id is not an template - should be done @ BAO layer
@@ -163,7 +163,7 @@ function _civicrm_api3_participant_create_spec(&$params) {
  *
  * @return array
  *   (reference )        array of properties, if error an array with an error id and error message
- * {@getfields participant_get}
+ *   {@getfields participant_get}
  */
 function civicrm_api3_participant_get($params) {
   $mode = CRM_Contact_BAO_Query::MODE_EVENT;
index e789623e0f0a436773a873f5a7b0e66cfb6c05f6..9b8870f87641aaa4f46b35b938c0fcffd01ad6af 100644 (file)
@@ -46,7 +46,7 @@
  *
  * @return array
  *   participant_status array
- * {@getfields ParticipantStatusType_create}
+ *   {@getfields ParticipantStatusType_create}
  * @example ParticipantStatusTypeCreate.php
  */
 function civicrm_api3_participant_status_type_create($params) {
@@ -63,7 +63,7 @@ function civicrm_api3_participant_status_type_create($params) {
  *
  * @return array
  *   Array of matching participant_statuses
- * {@getfields ParticipantStatusType_get}
+ *   {@getfields ParticipantStatusType_get}
  * @example ParticipantStatusTypeGet.php
  */
 function civicrm_api3_participant_status_type_get($params) {
@@ -82,7 +82,7 @@ function civicrm_api3_participant_status_type_get($params) {
  *
  * @return array
  *   api result array
- * {@getfields ParticipantStatusType_delete}
+ *   {@getfields ParticipantStatusType_delete}
  * @example ParticipantStatusTypeDelete.php
  */
 function civicrm_api3_participant_status_type_delete($params) {
index bfe932940d7f1cfcf5b485fd00c06d72587b1cc0..0fcc744ddcd9f970f874d892b78804583903dbd7 100644 (file)
@@ -44,7 +44,7 @@
  *
  * @return array
  *   api result array
- * {@getfields price_field_create}
+ *   {@getfields price_field_create}
  */
 function civicrm_api3_price_field_create($params) {
   return _civicrm_api3_basic_create(_civicrm_api3_get_BAO(__FUNCTION__), $params);
@@ -70,7 +70,7 @@ function _civicrm_api3_price_field_create_spec(&$params) {
  *
  * @return array
  *   Array of matching price_fields
- * {@getfields price_field_get}
+ *   {@getfields price_field_get}
  */
 function civicrm_api3_price_field_get($params) {
   return _civicrm_api3_basic_get(_civicrm_api3_get_BAO(__FUNCTION__), $params);
@@ -88,7 +88,7 @@ function civicrm_api3_price_field_get($params) {
  *
  * @return array
  *   API result array
- * {@getfields price_field_delete}
+ *   {@getfields price_field_delete}
  */
 function civicrm_api3_price_field_delete($params) {
   return _civicrm_api3_basic_delete(_civicrm_api3_get_BAO(__FUNCTION__), $params);
index c8df00738f8beb65ad368bbb2eeaeb88b3153834..6811f0874f707f54f7e92ce579d96feecab5c386 100644 (file)
@@ -98,12 +98,9 @@ function civicrm_api3_website_delete($params) {
 /**
  * Retrieve one or more websites
  *
- * @param mixed[] (reference ) input parameters
- * {@getfields website_get}
+ * @param array $params
  * {@example WebsiteGet.php 0}
  * @example WebsiteGet.php
- * @param array $params
- *   An associative array of name/value pairs.
  *
  * @return array
  *   details of found websites
index 518ea881b2be8d7a3a7eba8b83ec87bde7772d23..63999e25bbc722e7483ebae4667b1bb684bc363d 100644 (file)
@@ -41,7 +41,7 @@ require_once 'CRM/Core/Config.php';
 class CRM_ParticipantProcessor {
   /**
    */
-  function __construct() {
+  public function __construct() {
     $config = CRM_Core_Config::singleton();
 
     //this does not return on failure
index a9b1f81a95104345c5a3065ad3d1e7bf6a46213c..4c2154331afb4dc4169d959993a94a3cc0f69eb4 100644 (file)
@@ -45,7 +45,7 @@ class ContributionPage extends PHPUnit_Framework_Testcase {
    * @param int $contributionPageId
    *   Id of the Contribution Page.
    *   to be deleted
-   * @return boolean
+   * @return bool
    *   true if Contribution Page deleted, false otherwise
    */
   public static function delete($contributionPageId) {
index b9942504ebe8a27e20abd887869a07dace835ac2..1f9287e9d7a57ab6b765b559fdeeabaaac3ea7b7 100644 (file)
@@ -42,7 +42,7 @@ class WebTest_AllTests extends CiviTestSuite {
    */
   private static function getInstance() {
     if (is_null(self::$instance)) {
-      self::$instance = new self;
+      self::$instance = new self();
     }
     return self::$instance;
   }
index ed9d24942833f2d64df88dca188748027bceecf0..673991e1c6230849c385ee0dcdffc61e83984226 100644 (file)
@@ -107,7 +107,7 @@ class WebTest_Contact_ProfileChecksumTest extends CiviSeleniumTestCase {
       'edit-1-profile-edit',
       'edit-1-profile-create',
       'edit-1-access-all-custom-data',
-      'edit-1-edit-all-contacts'
+      'edit-1-edit-all-contacts',
     );
     $this->changePermissions($permission);