INFRA-132 - Batch #8
authoratif-shaikh <shaikh388@gmail.com>
Thu, 15 Jan 2015 14:35:41 +0000 (20:05 +0530)
committeratif-shaikh <shaikh388@gmail.com>
Thu, 15 Jan 2015 14:35:41 +0000 (20:05 +0530)
38 files changed:
CRM/Activity/Import/Form/Summary.php
CRM/Admin/Form/RelationshipType.php
CRM/Case/Form/EditClient.php
CRM/Case/Form/Task/Delete.php
CRM/Contact/BAO/GroupNesting.php
CRM/Contact/Form/Task/AddToOrganization.php
CRM/Contact/Form/Task/PickProfile.php
CRM/Contribute/BAO/ManagePremiums.php
CRM/Core/Block.php
CRM/Core/Error.php
CRM/Core/I18n/Schema.php
CRM/Core/I18n/SchemaStructure_3_1_alpha1.php
CRM/Core/IDS.php
CRM/Core/Page/Redirect.php
CRM/Core/Payment/GoogleIPN.php
CRM/Custom/Form/Group.php
CRM/Event/Cart/Page/AddToCart.php
CRM/Event/Page/UserDashboard.php
CRM/Grant/Selector/Search.php
CRM/Mailing/Form/ForwardMailing.php
CRM/Mailing/Form/Schedule.php
CRM/Member/Import/Parser.php
CRM/Pledge/BAO/Query.php
CRM/Report/Form/Campaign/SurveyDetails.php
CRM/Report/Form/Contact/LoggingSummary.php
CRM/Upgrade/Incremental/php/ThreeFour.php
CRM/Utils/OptionBag.php
CRM/Utils/Signer.php
CRM/Utils/System/DrupalBase.php
Civi/CCase/Events.php
api/class.api.php
bin/ContributionProcessor.php
tests/extensions/test.extension.manager.paymenttest/main.php
tests/mailing_test/controllerTest.js
tests/phpunit/CiviTest/CiviSeleniumTestCase.php
tests/phpunit/CiviTest/Contact.php
tests/phpunit/CiviTest/Participant.php
tests/phpunit/WebTest/Case/CaseDashboardTest.php

index 6a33eb5cff67623768ea10f1af0d265417834d91..ba091c358620313efc8aefb440c4ac09acc6a946 100644 (file)
@@ -105,7 +105,7 @@ class CRM_Activity_Import_Form_Summary extends CRM_Import_Form_Summary {
       'downloadDuplicateRecordsUrl',
       'downloadMismatchRecordsUrl',
       'groupAdditions',
-      'unMatchCount'
+      'unMatchCount',
     );
     foreach ($properties as $property) {
       $this->assign($property, $this->get($property));
index d9572b6c31d818d8180c7136c62f98dc675c101b..8129662b8b0df42a185bb8228b653a905e056366 100644 (file)
@@ -78,12 +78,12 @@ class CRM_Admin_Form_RelationshipType extends CRM_Admin_Form {
     // add select for contact type
     $contactTypeA = &$this->add('select', 'contact_types_a', ts('Contact Type A') . ' ',
       array(
-        '' => ts('All Contacts')
+        '' => ts('All Contacts'),
       ) + $contactTypes
     );
     $contactTypeB = &$this->add('select', 'contact_types_b', ts('Contact Type B') . ' ',
       array(
-        '' => ts('All Contacts')
+        '' => ts('All Contacts'),
       ) + $contactTypes
     );
 
index 0253bc94eb0de056ae33976bdc801347a0c21aef..82ea24668688f5650d432fbfea5ab1644d3d5284 100644 (file)
@@ -68,7 +68,7 @@ class CRM_Case_Form_EditClient extends CRM_Core_Form {
     }
     elseif (in_array($context, array(
       'dashlet',
-      'dashletFullscreen'
+      'dashletFullscreen',
     ))) {
       $url = CRM_Utils_System::url('civicrm/dashboard', 'reset=1');
     }
index 274fcdf798f86ae6d2d9c9ba83e09d99e909ed39..661c1e63be08430de6858b4cde306b569ea787ea 100644 (file)
@@ -60,7 +60,7 @@ class CRM_Case_Form_Task_Delete extends CRM_Case_Form_Task {
    *
    * @return void
    */
-  function preProcess() {
+  public function preProcess() {
     //check for delete
     if (!CRM_Core_Permission::checkActionPermission('CiviCase', CRM_Core_Action::DELETE)) {
       CRM_Core_Error::fatal(ts('You do not have permission to access this page.'));
index af8b73c5b68c3cfcf40f083b48ef67afce151119..a9dab201af039e5062cce814277a46bf2e68006b 100644 (file)
@@ -158,7 +158,7 @@ class CRM_Contact_BAO_GroupNesting extends CRM_Contact_DAO_GroupNesting implemen
       $nextGroup = $this->_getNextSiblingGroup($currentGroup);
       if (!$nextGroup) {
         // no sibling, find an ancestor w/ a sibling
-        for (; ;) {
+        for (;;) {
           // since we pop this array everytime, we should be
           // reasonably safe from infinite loops, I think :)
           $ancestor = array_pop($this->_parentStack);
@@ -338,7 +338,7 @@ class CRM_Contact_BAO_GroupNesting extends CRM_Contact_DAO_GroupNesting implemen
    * @param $childID
    *   The child id of the association.
    *
-   * @return boolean
+   * @return bool
    *   True if association is found, false otherwise.
    */
   public static function isParentChild($parentID, $childID) {
@@ -358,7 +358,7 @@ class CRM_Contact_BAO_GroupNesting extends CRM_Contact_DAO_GroupNesting implemen
    * @param $groupId
    *   The id of the group to check for child groups.
    *
-   * @return boolean
+   * @return bool
    *   True if 1 or more child groups are found, false otherwise.
    */
   public static function hasChildGroups($groupId) {
@@ -379,7 +379,7 @@ class CRM_Contact_BAO_GroupNesting extends CRM_Contact_DAO_GroupNesting implemen
    * @param $groupId
    *   The id of the group to check for parent groups.
    *
-   * @return boolean
+   * @return bool
    *   True if 1 or more parent groups are found, false otherwise.
    */
   public static function hasParentGroups($groupId) {
@@ -401,7 +401,7 @@ class CRM_Contact_BAO_GroupNesting extends CRM_Contact_DAO_GroupNesting implemen
    * @param $checkGroupId
    *   The group id to check if it is a parent of the $groupIds group(s).
    *
-   * @return boolean
+   * @return bool
    *   True if $checkGroupId points to a group that is a parent of one of the $groupIds groups, false otherwise.
    */
   public static function isParentGroup($groupIds, $checkGroupId) {
@@ -433,7 +433,7 @@ class CRM_Contact_BAO_GroupNesting extends CRM_Contact_DAO_GroupNesting implemen
    * @param $checkGroupId
    *   The group id to check if it is a child of the $groupIds group(s).
    *
-   * @return boolean
+   * @return bool
    *   True if $checkGroupId points to a group that is a child of one of the $groupIds groups, false otherwise.
    */
   public static function isChildGroup($groupIds, $checkGroupId) {
@@ -467,7 +467,7 @@ class CRM_Contact_BAO_GroupNesting extends CRM_Contact_DAO_GroupNesting implemen
    * @param $checkGroupId
    *   The group id to check if it is an ancestor of the $groupIds group(s).
    *
-   * @return boolean
+   * @return bool
    *   True if $checkGroupId points to a group that is an ancestor of one of the $groupIds groups, false otherwise.
    */
   public static function isAncestorGroup($groupIds, $checkGroupId) {
@@ -508,7 +508,7 @@ class CRM_Contact_BAO_GroupNesting extends CRM_Contact_DAO_GroupNesting implemen
    * @param $checkGroupId
    *   The group id to check if it is a descendent of the $groupIds group(s).
    *
-   * @return boolean
+   * @return bool
    *   True if $checkGroupId points to a group that is a descendent of one of the $groupIds groups, false otherwise.
    */
   public static function isDescendentGroup($groupIds, $checkGroupId) {
index 1a3a3951a4f3337bdfc02c1aa697651d5477d83b..325031c912747849ea6bb7178c0148f45472ecc2 100644 (file)
@@ -63,7 +63,7 @@ class CRM_Contact_Form_Task_AddToOrganization extends CRM_Contact_Form_Task {
       'relationship_type_id',
       ts('Relationship Type'),
       array(
-        '' => ts('- select -')
+        '' => ts('- select -'),
       ) +
       CRM_Contact_BAO_Relationship::getRelationType("Organization"), TRUE
     );
@@ -88,14 +88,12 @@ class CRM_Contact_Form_Task_AddToOrganization extends CRM_Contact_Form_Task {
       $this->assign('searchRows', $searchRows);
     }
 
-
     $this->assign('searchCount', $searchCount);
     $this->assign('searchDone', $this->get('searchDone'));
     $this->assign('contact_type_display', ts('Organization'));
     $this->addElement('submit', $this->getButtonName('refresh'), ts('Search'), array('class' => 'crm-form-submit'));
     $this->addElement('submit', $this->getButtonName('cancel'), ts('Cancel'), array('class' => 'crm-form-submit'));
 
-
     $this->addButtons(array(
         array(
           'type' => 'next',
@@ -166,27 +164,27 @@ class CRM_Contact_Form_Task_AddToOrganization extends CRM_Contact_Form_Task {
             'count' => $valid,
             'plural' => '%count %2 %3 relationships created',
             2 => $relationship,
-            3 => $org
-          ))
+            3 => $org,
+          )),
       );
       if ($duplicate) {
         $status[] = ts('%count was skipped because the contact is already %2 %3', array(
             'count' => $duplicate,
             'plural' => '%count were skipped because the contacts are already %2 %3',
             2 => $relationship,
-            3 => $org
+            3 => $org,
           ));
       }
       if ($invalid) {
         $status[] = ts('%count relationship was not created because the contact is not of the right type for this relationship', array(
             'count' => $invalid,
-            'plural' => '%count relationships were not created because the contact is not of the right type for this relationship'
+            'plural' => '%count relationships were not created because the contact is not of the right type for this relationship',
           ));
       }
       $status = '<ul><li>' . implode('</li><li>', $status) . '</li></ul>';
       CRM_Core_Session::setStatus($status, ts('Relationship created.', array(
             'count' => $valid,
-            'plural' => 'Relationships created.'
+            'plural' => 'Relationships created.',
           )), 'success', array('expires' => 0));
     }
   }
index b2f46d3c5b703a6a415b2ea0aa2008fe62d858e5..05c7774450204b237e602b3ff87f47aeec3709ab 100644 (file)
@@ -77,7 +77,7 @@ class CRM_Contact_Form_Task_PickProfile extends CRM_Contact_Form_Task {
     if (count($this->_contactIds) > $this->_maxContacts) {
       CRM_Core_Session::setStatus(ts("The maximum number of contacts you can select for Batch Update is %1. You have selected %2. Please select fewer contacts from your search results and try again.", array(
             1 => $this->_maxContacts,
-            2 => count($this->_contactIds)
+            2 => count($this->_contactIds),
           )), ts('Maximum Exceeded'), 'error');
       $validate = TRUE;
     }
index 28da593c424c604a1e1c98b7a23f128010529241..a46c380ae8382abe2cb3d3e0a7d4bf297f807423 100644 (file)
@@ -133,7 +133,6 @@ class CRM_Contribute_BAO_ManagePremiums extends CRM_Contribute_DAO_Product {
    *
    * @param int $productID
    */
-
   public static function del($productID) {
     //check dependencies
     $premiumsProduct = new CRM_Contribute_DAO_PremiumsProduct();
index 8f6a9d635f6f0c286379682630f26132f3af527d..2b6366ab09beb0c78a910bbdd3ff92eb42e9da2f 100644 (file)
@@ -241,7 +241,7 @@ class CRM_Core_Block {
       if ($value['active']) {
         if (in_array($id, array(
           self::ADD,
-          self::CREATE_NEW
+          self::CREATE_NEW,
         ))) {
           $hasAccess = TRUE;
           if (!CRM_Core_Permission::check('add contacts') &&
@@ -329,7 +329,7 @@ class CRM_Core_Block {
         $urlArray = array(
           'fullTextSearchID' => CRM_Core_DAO::getFieldValue('CRM_Core_DAO_OptionValue',
             'CRM_Contact_Form_Search_Custom_FullText', 'value', 'name'
-          )
+          ),
         );
         self::setProperty(self::FULLTEXT_SEARCH, 'templateValues', $urlArray);
         break;
@@ -369,7 +369,7 @@ class CRM_Core_Block {
           'query' => 'action=add&reset=1&context=standalone',
           'ref' => 'new-activity',
           'title' => ts('Activity'),
-        )
+        ),
       ));
 
       $components = CRM_Core_Component::getEnabledComponents();
@@ -392,7 +392,7 @@ class CRM_Core_Block {
           'query' => 'atype=3&action=add&reset=1&context=standalone',
           'ref' => 'new-email',
           'title' => ts('Email'),
-        )
+        ),
       ));
 
       if (CRM_Core_Permission::check('edit groups')) {
@@ -402,7 +402,7 @@ class CRM_Core_Block {
             'query' => 'reset=1',
             'ref' => 'new-group',
             'title' => ts('Group'),
-          )
+          ),
         ));
       }
 
@@ -413,7 +413,7 @@ class CRM_Core_Block {
             'query' => 'reset=1&action=add',
             'ref' => 'new-tag',
             'title' => ts('Tag'),
-          )
+          ),
         ));
       }
 
@@ -481,7 +481,7 @@ class CRM_Core_Block {
           'path' => 'civicrm/user',
           'query' => 'reset=1',
           'title' => ts('My Contact Dashboard'),
-        )
+        ),
       );
     }
 
@@ -590,7 +590,7 @@ class CRM_Core_Block {
     // return if upgrade mode
     $config = CRM_Core_Config::singleton();
     if ($config->isUpgradeMode()) {
-      return;
+      return NULL;
     }
 
     if (!self::getProperty($id, 'active')) {
index 8a16fa3652b70674bb2ed818c1d7698351a40ac4..ae0bab6e0de08849c2e03af470dda277e46d5ad7 100644 (file)
@@ -57,7 +57,7 @@ class CRM_Exception extends PEAR_Exception {
    *  - PEAR_Exception(string $message, array $causes);
    *  - PEAR_Exception(string $message, array $causes, int $code);
    *
-   * @param string exception message
+   * @param string $message exception message
    * @param int $code
    * @param Exception $previous
    */
@@ -167,7 +167,7 @@ class CRM_Core_Error extends PEAR_ErrorStack {
    * the errors we expect are from the pear modules DB, DB_DataObject
    * which currently use PEAR::raiseError to notify of error messages.
    *
-   * @param object PEAR_Error
+   * @param object $pearError PEAR_Error
    *
    * @return void
    */
@@ -461,11 +461,11 @@ class CRM_Core_Error extends PEAR_ErrorStack {
    * Outputs pre-formatted debug information. Flushes the buffers
    * so we can interrupt a potential POST/redirect
    *
-   * @param string name of debug section
-   * @param mixed reference to variables that we need a trace of
-   * @param bool should we log or return the output
-   * @param bool whether to generate a HTML-escaped output
-   * @param bool should we check permissions before displaying output
+   * @param string $name name of debug section
+   * @param $variable mixed reference to variables that we need a trace of
+   * @param bool $log should we log or return the output
+   * @param bool $html whether to generate a HTML-escaped output
+   * @param bool $checkPermission should we check permissions before displaying output
    *                useful when we die during initialization and permissioning
    *                subsystem is not initialized - CRM-13765
    *
@@ -526,7 +526,7 @@ class CRM_Core_Error extends PEAR_ErrorStack {
    * @see CRM_Core_Error::debug()
    * @see CRM_Core_Error::debug_log_message()
    */
-  static function debug_var(
+  public static function debug_var(
     $variable_name,
     $variable,
     $print = TRUE,
index 61f5dc211285df1decd5547a1c7bd9af302480bd..f8fb5ed044914ef1fa27382d88e7001a155f1646 100644 (file)
@@ -121,7 +121,7 @@ class CRM_Core_I18n_Schema {
    * @return void
    */
   public static function makeSinglelingual($retain) {
-    $domain = new CRM_Core_DAO_Domain;
+    $domain = new CRM_Core_DAO_Domain();
     $domain->find(TRUE);
     $locales = explode(CRM_Core_DAO::VALUE_SEPARATOR, $domain->locales);
 
@@ -131,7 +131,7 @@ class CRM_Core_I18n_Schema {
     }
 
     // lets drop all triggers first
-    $logging = new CRM_Logging_Schema;
+    $logging = new CRM_Logging_Schema();
     $logging->dropTriggers();
 
     // turn subsequent tables singlelingual
@@ -166,13 +166,13 @@ class CRM_Core_I18n_Schema {
    *
    * @return void
    */
-  static function makeSinglelingualTable(
+  public static function makeSinglelingualTable(
     $retain,
     $table,
     $class = 'CRM_Core_I18n_SchemaStructure',
     $triggers = array()
   ) {
-    $domain = new CRM_Core_DAO_Domain;
+    $domain = new CRM_Core_DAO_Domain();
     $domain->find(TRUE);
     $locales = explode(CRM_Core_DAO::VALUE_SEPARATOR, $domain->locales);
 
@@ -212,7 +212,7 @@ class CRM_Core_I18n_Schema {
     $queries = array_merge($queries, self::createIndexQueries(NULL, $table));
 
     // execute the queries without i18n rewriting
-    $dao = new CRM_Core_DAO;
+    $dao = new CRM_Core_DAO();
     foreach ($queries as $query) {
       $dao->query($query, FALSE);
     }
@@ -318,7 +318,7 @@ class CRM_Core_I18n_Schema {
     $indices =& $class::indices();
     $tables =& $class::tables();
     $queries = array();
-    $dao = new CRM_Core_DAO;
+    $dao = new CRM_Core_DAO();
 
     // get all of the already existing indices
     $existing = array();
@@ -562,7 +562,7 @@ class CRM_Core_I18n_Schema {
         foreach ($locales as $old) {
           $trigger[] = "ELSEIF NEW.{$column}_{$old} IS NOT NULL THEN";
           foreach (array_merge($locales, array(
-            $locale
+            $locale,
           )) as $loc) {
             if ($loc == $old) {
               continue;
@@ -593,7 +593,7 @@ class CRM_Core_I18n_Schema {
         foreach ($locales as $old) {
           $trigger[] = "ELSEIF NEW.{$column}_{$old} IS NOT NULL THEN";
           foreach (array_merge($locales, array(
-            $locale
+            $locale,
           )) as $loc) {
             if ($loc == $old) {
               continue;
index 09ce51c5fd4ef5766ea6656244aaa82c835c6361..3ac06772d4d3a7f9d68bbb2fea1f1b1712d33254 100644 (file)
@@ -36,7 +36,7 @@ class CRM_Core_I18n_SchemaStructure_3_1_alpha1 {
   /**
    * @return array
    */
-  static function &columns() {
+  public static function &columns() {
     static $result = NULL;
     if (!$result) {
       $result = array(
index c65a417d8764fb7c895afbb6c737ac06f7fb537f..5b5bb44b694c19545f7e8a6bddefc44bded31bdb 100644 (file)
@@ -52,16 +52,16 @@ class CRM_Core_IDS {
    * This function includes the IDS vendor parts and runs the
    * detection routines on the request array.
    *
-   * @param object cake controller object
+   * @param object $args cake controller object
    *
-   * @return boolean
+   * @return bool
    */
   public function check(&$args) {
     // lets bypass a few civicrm urls from this check
     static $skip = array('civicrm/admin/setting/updateConfigBackend', 'civicrm/admin/messageTemplates');
     $path = implode('/', $args);
     if (in_array($path, $skip)) {
-      return;
+      return NULL;
     }
 
     #add request url and user agent
@@ -162,7 +162,6 @@ class CRM_Core_IDS {
       CRM_Core_Error::movedSiteError($configFile);
     }
 
-
     // also create the .htaccess file so we prevent the reading of the log and ini files
     // via a browser, CRM-3875
     CRM_Utils_File::restrictAccess($config->configAndLogDir);
@@ -179,7 +178,7 @@ class CRM_Core_IDS {
    *
    * @param IDS_Report $result
    *
-   * @return boolean
+   * @return bool
    */
   private function react(IDS_Report $result) {
 
@@ -210,14 +209,12 @@ class CRM_Core_IDS {
    * @param array $result
    * @param int $reaction
    *
-   * @return boolean
+   * @return bool
    */
   private function log($result, $reaction = 0) {
     $ip = (isset($_SERVER['SERVER_ADDR']) &&
-      $_SERVER['SERVER_ADDR'] != '127.0.0.1'
-    ) ? $_SERVER['SERVER_ADDR'] : (isset($_SERVER['HTTP_X_FORWARDED_FOR']) ?
-      $_SERVER['HTTP_X_FORWARDED_FOR'] :
-      '127.0.0.1'
+      $_SERVER['SERVER_ADDR'] != '127.0.0.1') ? $_SERVER['SERVER_ADDR'] : (
+      isset($_SERVER['HTTP_X_FORWARDED_FOR']) ? $_SERVER['HTTP_X_FORWARDED_FOR'] : '127.0.0.1'
     );
 
     $data = array();
index 31ad0fc831d24dc1fd5511c5beb8d95fe44d0ec6..9883bfcd8504d10c623c4710b1a8a56434796c6b 100644 (file)
@@ -25,8 +25,6 @@ class CRM_Core_Page_Redirect extends CRM_Core_Page {
    *   Any GET arguments.
    * @param array $pageArgs
    *   The page_arguments registered in the router.
-   * @param array $requestArgs
-   *   Any parameters passed as part of this page request.
    * @param bool $absolute
    *   Whether to return an absolute URL.
    * @return string
index 2c2fa54d7f0e98dc8ec4961210ba5de83780c194..16c5d6633cf5e3eafb738dec24a35ddf4dd1b29b 100644 (file)
@@ -303,7 +303,7 @@ class CRM_Core_Payment_GoogleIPN extends CRM_Core_Payment_BaseIPN {
       foreach (array(
                  'membership',
                  'related_contact',
-                 'onbehalf_dupe_alert'
+                 'onbehalf_dupe_alert',
                ) as $fld) {
         if (!is_numeric($ids[$fld])) {
           unset($ids[$fld]);
@@ -491,8 +491,7 @@ WHERE  contribution_recur_id = {$ids['contributionRecur']}
         $ids['event'] = $privateData['eventID'];
       }
       else {
-        list($ids['event'], $ids['participant']) =
-          explode(CRM_Core_DAO::VALUE_SEPARATOR, $contribution->trxn_id);
+        list($ids['event'], $ids['participant']) = explode(CRM_Core_DAO::VALUE_SEPARATOR, $contribution->trxn_id);
       }
     }
 
@@ -585,13 +584,12 @@ WHERE  contribution_recur_id = {$ids['contributionRecur']}
       case "merchant-calculation-callback":
         break;
 
-      case "new-order-notification": {
+      case "new-order-notification":
         $response->SendAck($serial, FALSE);
         $ipn->newOrderNotify($data[$root], $privateData, $module);
         break;
-      }
 
-      case "order-state-change-notification": {
+      case "order-state-change-notification":
         $response->SendAck($serial, FALSE);
         $new_financial_state = $data[$root]['new-financial-order-state']['VALUE'];
         $new_fulfillment_order = $data[$root]['new-fulfillment-order-state']['VALUE'];
@@ -615,9 +613,8 @@ WHERE  contribution_recur_id = {$ids['contributionRecur']}
             break;
         }
         break;
-      }
 
-      case "authorization-amount-notification": {
+      case "authorization-amount-notification":
         $response->SendAck($serial, FALSE);
         $new_financial_state = $data[$root]['order-summary']['financial-order-state']['VALUE'];
         $new_fulfillment_order = $data[$root]['order-summary']['fulfillment-order-state']['VALUE'];
@@ -644,7 +641,6 @@ WHERE  contribution_recur_id = {$ids['contributionRecur']}
             break;
         }
         break;
-      }
 
       case "charge-amount-notification":
       case "chargeback-amount-notification":
index 674610f0f1324a2eb48bc9a36efde899f7732a25..ba2459484d9d1f129b8385040a3ed3b546b4c9f9 100644 (file)
@@ -69,7 +69,6 @@ class CRM_Custom_Form_Group extends CRM_Core_Form {
   /**
    * Set variables up before form is built
    *
-   * @param null
    *
    * @return void
    */
@@ -174,7 +173,6 @@ class CRM_Custom_Form_Group extends CRM_Core_Form {
    * add the rules (mainly global rules) for form.
    * All local rules are added near the element
    *
-   * @param null
    *
    * @return void
    * @see valid_date
@@ -186,7 +184,6 @@ class CRM_Custom_Form_Group extends CRM_Core_Form {
   /**
    * Build the form object
    *
-   * @param null
    *
    * @return void
    */
@@ -257,13 +254,13 @@ class CRM_Custom_Form_Group extends CRM_Core_Form {
         if ($main == 'Relationship') {
           $relName = self::getFormattedList($sel2[$main]);
           $sel2[$main] = array(
-              '' => ts("- Any -")
-            ) + $relName;
+            '' => ts("- Any -"),
+          ) + $relName;
         }
         else {
           $sel2[$main] = array(
-              '' => ts("- Any -")
-            ) + $sel2[$main];
+            '' => ts("- Any -"),
+          ) + $sel2[$main];
         }
       }
     }
@@ -276,8 +273,8 @@ class CRM_Custom_Form_Group extends CRM_Core_Form {
         $contactSubTypes[$key] = $key;
       }
       $sel2['Contact'] = array(
-          "" => "-- Any --"
-        ) + $contactSubTypes;
+        "" => "-- Any --",
+      ) + $contactSubTypes;
     }
     else {
       if (!isset($this->_id)) {
@@ -405,7 +402,6 @@ class CRM_Custom_Form_Group 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
@@ -457,7 +453,6 @@ class CRM_Custom_Form_Group extends CRM_Core_Form {
   /**
    * Process the form
    *
-   * @param null
    *
    * @return void
    */
index 1aabf6919a4ba5fe88210523b31f0c4f09fa6380..d248e6ad95ef210516e17213af30845c5f999829 100644 (file)
@@ -18,7 +18,7 @@ class CRM_Event_Cart_Page_AddToCart extends CRM_Core_Page {
     $url = CRM_Utils_System::url('civicrm/event/view_cart');
     CRM_Utils_System::setUFMessage(ts("<b>%1</b> has been added to your cart. <a href='%2'>View your cart.</a>", array(
           1 => $event_in_cart->event->title,
-          2 => $url
+          2 => $url,
         )));
 
     $transaction->commit();
index b14799a9db7be9a64137db74bd817b585682ed0c..35f7cfb95960b4de51fbdef27f2ae41eb3ea89f2 100644 (file)
@@ -41,7 +41,6 @@ class CRM_Event_Page_UserDashboard extends CRM_Contact_Page_View_UserDashBoard {
   /**
    * List participations for the UF user
    *
-   * @return null
    */
   public function listParticipations() {
     $controller = new CRM_Core_Controller_Simple(
@@ -63,7 +62,6 @@ class CRM_Event_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 7e0d26c15b37e6eb0eac90dad6bbc38733298910..ba089a3cb29e330186b95484c5d634922b566ed6 100644 (file)
@@ -143,7 +143,7 @@ class CRM_Grant_Selector_Search extends CRM_Core_Selector_Base implements CRM_Co
    *
    * @return \CRM_Grant_Selector_Search
    */
-  function __construct(
+  public function __construct(
     &$queryParams,
     $action = CRM_Core_Action::NONE,
     $grantClause = NULL,
index 005bb10edf3ec88a76ee6edd6a5b0a4d3e092fff..29f0ff932b0ee26982651f2426fc964c824e705c 100644 (file)
@@ -147,12 +147,12 @@ class CRM_Mailing_Form_ForwardMailing extends CRM_Core_Form {
 
     $status = ts('Mailing is not forwarded to the given email address.', array(
         'count' => count($emails),
-        'plural' => 'Mailing is not forwarded to the given email addresses.'
+        'plural' => 'Mailing is not forwarded to the given email addresses.',
       ));
     if ($forwarded) {
       $status = ts('Mailing is forwarded successfully to %count email address.', array(
           'count' => $forwarded,
-          'plural' => 'Mailing is forwarded successfully to %count email addresses.'
+          'plural' => 'Mailing is forwarded successfully to %count email addresses.',
         ));
     }
 
index 49028a4cede201d5a9cb4006412aed960a83cc0e..223d8e25c6f09c6dcabda79ab1edf4a720f95051 100644 (file)
@@ -196,7 +196,7 @@ class CRM_Mailing_Form_Schedule extends CRM_Core_Form {
    * @param $files
    * @param $self
    *
-   * @return boolean
+   * @return bool
    *   True if either we deliver immediately, or the
    *                          date is properly set.
    */
index 2dad4b2984d9439baa99e3862ac725f4396bb867..269e8a4f3c85e437996198e46a13c841fa7a86ec 100644 (file)
@@ -74,7 +74,7 @@ abstract class CRM_Member_Import_Parser extends CRM_Import_Parser {
    * @return mixed
    * @throws Exception
    */
-  function run(
+  public function run(
     $fileName,
     $seperator = ',',
     &$mapper,
@@ -283,7 +283,7 @@ abstract class CRM_Member_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 762f26244660106d676e80c8799cd4198ec379fe..129ae568e1897b61a5328d67512b45c6873564a6 100644 (file)
@@ -471,7 +471,7 @@ class CRM_Pledge_BAO_Query {
   /**
    * Ideally this function should include fields that are displayed in the selector
    */
-  static function defaultReturnProperties(
+  public static function defaultReturnProperties(
     $mode,
     $includeCustomFields = TRUE
   ) {
index 6ee444f9b9bd86c8dbdda6c2b7c101fa1627870f..5ba8f798415189f085760a43e5b88385211177b0 100644 (file)
@@ -99,8 +99,8 @@ class CRM_Report_Form_Campaign_SurveyDetails extends CRM_Report_Form {
             'type' => CRM_Utils_Type::T_INT,
             'operatorType' => CRM_Report_Form::OP_SELECT,
             'options' => array(
-                '' => ts('- any interviewer -'),
-              ) + $allSurveyInterviewers,
+              '' => ts('- any interviewer -'),
+            ) + $allSurveyInterviewers,
           ),
         ),
         'grouping' => 'survey-interviewer-fields',
@@ -412,8 +412,7 @@ class CRM_Report_Form_Campaign_SurveyDetails extends CRM_Report_Form {
     //apply survey activity types filter.
     $surveyActivityTypes = CRM_Campaign_BAO_Survey::getSurveyActivityType();
     if (!empty($surveyActivityTypes)) {
-      $clauses[] =
-        "( {$this->_aliases['civicrm_activity']}.activity_type_id IN ( " .
+      $clauses[] = "( {$this->_aliases['civicrm_activity']}.activity_type_id IN ( " .
         implode(' , ', array_keys($surveyActivityTypes)) . ' ) )';
     }
 
@@ -489,8 +488,7 @@ class CRM_Report_Form_Campaign_SurveyDetails extends CRM_Report_Form {
           //get the data per page.
           $dataPerPage[$pageCnt][] = $row;
           $lastStreetName = CRM_Utils_Array::value('civicrm_address_street_name', $row);
-          $lastStreetNum =
-            CRM_Utils_Array::value('civicrm_address_street_number', $row) % 2;
+          $lastStreetNum = CRM_Utils_Array::value('civicrm_address_street_number', $row) % 2;
         }
 
         foreach ($dataPerPage as $page) {
@@ -870,20 +868,19 @@ INNER JOIN  civicrm_custom_group cg ON ( cg.id = cf.custom_group_id )
     foreach ($responseFields as $key => $value) {
       if (substr($key, 0, 5) == 'phone' && !empty($value['location_type_id'])) {
         $fName = str_replace('-', '_', $key);
-        $this->_columns["civicrm_{$fName}"] =
-          array(
-            'dao' => 'CRM_Core_DAO_Phone',
-            'alias' => "phone_civireport_{$fName}",
-            'fields' => array(
-              $fName => array_merge($value, array(
-                  'is_required' => '1',
-                  'alias' => "phone_civireport_{$fName}",
-                  'dbAlias' => "phone_civireport_{$fName}.phone",
-                  'no_display' => TRUE,
-                )
-              ),
+        $this->_columns["civicrm_{$fName}"] = array(
+          'dao' => 'CRM_Core_DAO_Phone',
+          'alias' => "phone_civireport_{$fName}",
+          'fields' => array(
+            $fName => array_merge($value, array(
+                'is_required' => '1',
+                'alias' => "phone_civireport_{$fName}",
+                'dbAlias' => "phone_civireport_{$fName}.phone",
+                'no_display' => TRUE,
+              )
             ),
-          );
+          ),
+        );
         $this->_aliases["civicrm_phone_{$fName}"] = $this->_columns["civicrm_{$fName}"]['alias'];
         $this->_locationBasedPhoneField = TRUE;
       }
@@ -957,9 +954,9 @@ INNER  JOIN  civicrm_custom_field cf ON ( cg.id = cf.custom_group_id )
         'htmlType' => $response->html_type,
         'required' => TRUE,
         'alias' => ($response->data_type == 'ContactReference') ? $this->_columns[$resTable]['alias'] .
-          '_contact' : $this->_columns[$resTable]['alias'],
+        '_contact' : $this->_columns[$resTable]['alias'],
         'dbAlias' => $this->_columns[$resTable]['alias'] . '.' .
-          $response->column_name,
+        $response->column_name,
         'no_display' => TRUE,
         'isSurveyResponseField' => TRUE,
       );
index 9968e950aa33114f94d00ee6c62bbf3f43ccd426..a4438b3775db204f39768274d2ac8e1a41df8174 100644 (file)
@@ -166,15 +166,15 @@ class CRM_Report_Form_Contact_LoggingSummary extends CRM_Logging_ReportSummary {
 
     foreach ($rows as $key => &$row) {
       if (!isset($isDeleted[$row['log_civicrm_entity_altered_contact_id']])) {
-        $isDeleted[$row['log_civicrm_entity_altered_contact_id']] =
-          CRM_Core_DAO::getFieldValue('CRM_Contact_DAO_Contact', $row['log_civicrm_entity_altered_contact_id'], 'is_deleted') !== '0';
+        $isDeleted[$row['log_civicrm_entity_altered_contact_id']] = CRM_Core_DAO::getFieldValue('CRM_Contact_DAO_Contact',
+          $row['log_civicrm_entity_altered_contact_id'], 'is_deleted') !== '0';
       }
 
       if (!empty($row['log_civicrm_entity_altered_contact']) &&
         !$isDeleted[$row['log_civicrm_entity_altered_contact_id']]
       ) {
-        $row['log_civicrm_entity_altered_contact_link'] =
-          CRM_Utils_System::url('civicrm/contact/view', 'reset=1&cid=' . $row['log_civicrm_entity_altered_contact_id']);
+        $row['log_civicrm_entity_altered_contact_link'] = CRM_Utils_System::url('civicrm/contact/view',
+          'reset=1&cid=' . $row['log_civicrm_entity_altered_contact_id']);
         $row['log_civicrm_entity_altered_contact_hover'] = ts("Go to contact summary");
         $entity = $this->getEntityValue($row['log_civicrm_entity_id'], $row['log_civicrm_entity_log_type'], $row['log_civicrm_entity_log_date']);
         if ($entity) {
index 9521d07f7d6ab31722dd6fc5fbf5cdf8e9187dca..d34f811c08a7736f05976a1522a53b5e9cd2bc31 100644 (file)
@@ -72,7 +72,7 @@ class CRM_Upgrade_Incremental_php_ThreeFour {
       'contact/log',
       'activitySummary',
       'case/timespent',
-      'case/summary'
+      'case/summary',
     );
 
     $instances = CRM_Core_DAO::executeQuery("SELECT id, form_values, report_id FROM civicrm_report_instance WHERE report_id IN ('" . implode("','", $modifiedReportIds) . "')");
@@ -149,7 +149,7 @@ class CRM_Upgrade_Incremental_php_ThreeFour {
    */
   public function upgrade_3_4_beta3($rev) {
     // do the regular upgrade
-    $upgrade = new CRM_Upgrade_Form;
+    $upgrade = new CRM_Upgrade_Form();
     $upgrade->processSQL($rev);
 
     if ($upgrade->multilingual) {
@@ -185,7 +185,7 @@ class CRM_Upgrade_Incremental_php_ThreeFour {
       'Contact',
       'Individual',
       'Organization',
-      'Household'
+      'Household',
     );
     while ($ufGroups->fetch()) {
       if (!in_array($ufGroups->group_type, $skipGroupTypes)) {
@@ -219,7 +219,7 @@ INSERT INTO civicrm_location_type ( name, description, is_reserved, is_active )
       CRM_Core_DAO::executeQuery($query);
     }
 
-    $upgrade = new CRM_Upgrade_Form;
+    $upgrade = new CRM_Upgrade_Form();
     $upgrade->processSQL($rev);
   }
 
@@ -299,7 +299,7 @@ INSERT INTO civicrm_location_type ( name, description, is_reserved, is_active )
             'event_start_date_from',
             'event_start_date_to',
             'event_end_date_from',
-            'event_end_date_to'
+            'event_end_date_to',
           );
           foreach ($eventDates as $date) {
             if (isset($formValues[$date]) && $formValues[$date] == ' ') {
@@ -353,7 +353,8 @@ FROM   civicrm_option_value v,
 WHERE  v.option_group_id = g.id
   AND  g.name      = %1
   AND  g.is_active = 1
-  AND  v.name      = %2", array(
+  AND  v.name      = %2",
+    array(
       1 => array('activity_type', 'String'),
       2 => array('Bulk Email', 'String'),
     ));
index 64b317ceda1ae7a153d205bd4b3114a49aeea7dd..e51baf5dd154b186500f5fa00c2214471b72d213 100644 (file)
@@ -62,11 +62,11 @@ class CRM_Utils_OptionBag implements ArrayAccess, IteratorAggregate, Countable {
    *   <p>.
    * An offset to check for.
    * </p>
-   * @return boolean
+   * @return bool
    *   true on success or false on failure.
-   * </p>
-   * <p>
-   * The return value will be casted to boolean if non-boolean was returned.
+   *   </p>
+   *   <p>
+   *   The return value will be casted to boolean if non-boolean was returned.
    */
   public function offsetExists($offset) {
     return array_key_exists($offset, $this->data);
@@ -125,7 +125,7 @@ class CRM_Utils_OptionBag implements ArrayAccess, IteratorAggregate, Countable {
    * @link http://php.net/manual/en/iteratoraggregate.getiterator.php
    * @return Traversable
    *   An instance of an object implementing <b>Iterator</b> or
-   * <b>Traversable</b>
+   *   <b>Traversable</b>
    */
   public function getIterator() {
     return new ArrayIterator($this->data);
@@ -137,9 +137,9 @@ class CRM_Utils_OptionBag implements ArrayAccess, IteratorAggregate, Countable {
    * @link http://php.net/manual/en/countable.count.php
    * @return int
    *   The custom count as an integer.
-   * </p>
-   * <p>
-   * The return value is cast to an integer.
+   *   </p>
+   *   <p>
+   *   The return value is cast to an integer.
    */
   public function count() {
     return count($this->data);
index 5764519258bd02c8ba451e5b9ca9f98ee2421771..1116cfbad181bd03478da873723d00dc4bc86ce3 100644 (file)
@@ -99,7 +99,7 @@ class CRM_Utils_Signer {
           $params[$paramName] = (string) $params[$paramName];
         }
       }
-      else { // $paramName is not included or ===NULL
+      else {// $paramName is not included or ===NULL
         $params[$paramName] = '';
       }
       $message['payload'][$paramName] = $params[$paramName];
index e03ff71448a99ec5eb7194af7b599d404d828976..bcc763040586d80991397c12ff8cbc4dbd2d0cf6 100644 (file)
@@ -59,13 +59,13 @@ abstract class CRM_Utils_System_DrupalBase extends CRM_Utils_System_Base {
   }
 
   /**
-   * @param string dir base civicrm directory
+   * @param string $dir base civicrm directory
    * Return default Site Settings
    * @return array
    *   array
-   * - $url, (Joomla - non admin url)
-   * - $siteName,
-   * - $siteRoot
+   *   - $url, (Joomla - non admin url)
+   *   - $siteName,
+   *   - $siteRoot
    */
   public function getDefaultSiteSettings($dir) {
     $config = CRM_Core_Config::singleton();
@@ -92,7 +92,7 @@ abstract class CRM_Utils_System_DrupalBase extends CRM_Utils_System_Base {
   /**
    * Check if a resource url is within the drupal directory and format appropriately
    *
-   * @param url (reference)
+   * @param $url (reference)
    *
    * @return bool
    *   TRUE for internal paths, FALSE for external. The drupal_add_js fn is able to add js more
@@ -166,7 +166,7 @@ abstract class CRM_Utils_System_DrupalBase extends CRM_Utils_System_Base {
    * @return string
    *   an HTML string containing a link to the given path.
    */
-  function url(
+  public function url(
     $path = NULL, $query = NULL, $absolute = FALSE,
     $fragment = NULL, $htmlize = TRUE,
     $frontend = FALSE, $forceBackend = FALSE
@@ -278,7 +278,7 @@ abstract class CRM_Utils_System_DrupalBase extends CRM_Utils_System_Base {
     if (CRM_Core_Session::singleton()
         ->get('userID') == $contactID || CRM_Core_Permission::checkAnyPerm(array(
           'cms:administer users',
-          'cms:view user account'
+          'cms:view user account',
         ))
     ) {
       return CRM_Utils_System::url('user/' . $uid);
@@ -479,7 +479,7 @@ abstract class CRM_Utils_System_DrupalBase extends CRM_Utils_System_Base {
    * e.g. for drupal: records a watchdog message about the new session, saves the login timestamp,
    * calls hook_user op 'login' and generates a new session.
    *
-   * @param array params
+   * @param array $params
    *
    * FIXME: Document values accepted/required by $params
    */
index 13a42efe33751e8cf49ac04beeadbb7cc5b5c456..88b11c5ff99e9a00fe8a76388398affcafe45233 100644 (file)
@@ -87,7 +87,7 @@ class Events {
   /**
    * Find any extra listeners declared in XML and pass the event along to them
    *
-   * @param Event\CaseChangeEvent $event
+   * @param \Civi\CCase\Event\CaseChangeEvent $event
    */
   public static function delegateToXmlListeners(\Civi\CCase\Event\CaseChangeEvent $event) {
     $p = new \CRM_Case_XMLProcessor_Process();
index f2350ccba0da87a0ba0643970356f389c40fdc12..2b378bc5074d7c62adfa9a75ed84ecc8b96952ff 100644 (file)
@@ -81,7 +81,7 @@ class civicrm_api3 {
   /**
    * @param array $config API configuration.
    */
-  function __construct($config = NULL) {
+  public function __construct($config = NULL) {
     $this->local      = TRUE;
     $this->input      = array();
     $this->lastResult = array();
@@ -165,7 +165,7 @@ class civicrm_api3 {
    * @param array $params
    * @return \stdClass
    */
-  function remoteCall($entity, $action, $params = array()) {
+  public function remoteCall($entity, $action, $params = array()) {
     $fields = "key={$this->key}&api_key={$this->api_key}";
     $query = $this->uri . "&entity=$entity&action=$action";
     foreach ($params as $k => $v) {
@@ -182,7 +182,7 @@ class civicrm_api3 {
       $result = curl_exec($ch);
       // CiviCRM expects to get back a CiviCRM error object.
       if (curl_errno($ch)) {
-        $res = new stdClass;
+        $res = new stdClass();
         $res->is_error = 1;
         $res->error_message = curl_error($ch);
         $res->level = "cURL";
@@ -197,7 +197,7 @@ class civicrm_api3 {
       $result = file_get_contents($query . '&' . $fields);
     }
     if (!$res = json_decode($result)) {
-      $res = new stdClass;
+      $res = new stdClass();
       $res->is_error = 1;
       $res->error_message = 'Unable to parse returned JSON';
       $res->level = 'json_decode';
@@ -214,7 +214,7 @@ class civicrm_api3 {
    *
    * @return bool
    */
-  function call($entity, $action = 'Get', $params = array()) {
+  public function call($entity, $action = 'Get', $params = array()) {
     if (is_int($params)) {
       $params = array('id' => $params);
     }
@@ -248,7 +248,7 @@ class civicrm_api3 {
   /**
    * Helper method for long running programs (eg bots).
    */
-  function ping() {
+  public function ping() {
     global $_DB_DATAOBJECT;
     foreach ($_DB_DATAOBJECT['CONNECTIONS'] as & $c) {
       if (!$c->connection->ping()) {
@@ -264,14 +264,14 @@ class civicrm_api3 {
    * Return the last error message.
    * @return string
    */
-  function errorMsg() {
+  public function errorMsg() {
     return $this->lastResult->error_message;
   }
 
   /**
    * Initialize
    */
-  function init() {
+  public function init() {
     CRM_Core_DAO::init($this->cfg->dsn);
   }
 
index 0e867e266a0856bccb3c9e1a4583abbf9081da14..10b4dbdc3fe84a2649d802ba852c7f51e04bf2b2 100644 (file)
@@ -142,8 +142,7 @@ class CiviContributeProcessor {
    * @param $start
    * @param $end
    */
-  static
-  function paypal($paymentProcessor, $paymentMode, $start, $end) {
+  public static function paypal($paymentProcessor, $paymentMode, $start, $end) {
     $url = "{$paymentProcessor['url_api']}nvp";
 
     $keyArgs = array(
@@ -186,7 +185,7 @@ class CiviContributeProcessor {
           // details about a transaction, let's make sure that it doesn't
           // already exist in the database.
           require_once 'CRM/Contribute/DAO/Contribution.php';
-          $dao = new CRM_Contribute_DAO_Contribution;
+          $dao = new CRM_Contribute_DAO_Contribution();
           $dao->trxn_id = $value;
           if ($dao->find(TRUE)) {
             preg_match('/(\d+)$/', $name, $matches);
@@ -248,8 +247,7 @@ class CiviContributeProcessor {
    * @param $start
    * @param $end
    */
-  static
-  function google($paymentProcessor, $paymentMode, $start, $end) {
+  public static function google($paymentProcessor, $paymentMode, $start, $end) {
     require_once "CRM/Contribute/BAO/Contribution/Utils.php";
     require_once 'CRM/Core/Payment/Google.php';
     $nextPageToken = TRUE;
@@ -282,9 +280,7 @@ class CiviContributeProcessor {
           $response[1][$response[0]]['notifications']['charge-amount-notification'][] = $chrgAmt;
         }
 
-        foreach ($response[1][$response[0]]['notifications']['charge-amount-notification']
-                 as $amtData
-        ) {
+        foreach ($response[1][$response[0]]['notifications']['charge-amount-notification'] as $amtData) {
           $searchParams = array(
             'order-numbers' => array($amtData['google-order-number']['VALUE']),
             'notification-types' => array('risk-information', 'new-order', 'charge-amount'),
@@ -321,8 +317,7 @@ class CiviContributeProcessor {
     }
   }
 
-  static
-  function csv() {
+  public static function csv() {
     $csvFile = '/home/deepak/Desktop/crm-4247.csv';
     $delimiter = ";";
     $row = 1;
@@ -364,8 +359,7 @@ class CiviContributeProcessor {
     fclose($handle);
   }
 
-  static
-  function process() {
+  public static function process() {
     require_once 'CRM/Utils/Request.php';
 
     $type = CRM_Utils_Request::retrieve('type', 'String', CRM_Core_DAO::$_nullObject, FALSE, 'csv', 'REQUEST');
index 567beba88457583ce39e92a0cbd33145d3ff5e41..a2cd9d787f654d25dca286a328e41dad9cbeca94 100644 (file)
@@ -20,7 +20,7 @@ class test_extension_manager_paymenttest extends CRM_Core_Payment {
    *
    * @return object
    */
-  static function &singleton($mode = 'test', &$paymentProcessor, &$paymentForm = NULL, $force = FALSE) {
+  public static function &singleton($mode = 'test', &$paymentProcessor, &$paymentForm = NULL, $force = FALSE) {
     $processorName = $paymentProcessor['name'];
     if (self::$_singleton[$processorName] === NULL) {
       self::$_singleton[$processorName] = new test_extension_manager_paymenttest();
@@ -30,19 +30,19 @@ class test_extension_manager_paymenttest extends CRM_Core_Payment {
 
   static $counts = array();
 
-  function install() {
+  public function install() {
     self::$counts['install'] = 1 + (int) self::$counts['install'];
   }
 
-  function uninstall() {
+  public function uninstall() {
     self::$counts['uninstall'] = 1 + (int) self::$counts['uninstall'];
   }
 
-  function disable() {
+  public function disable() {
     self::$counts['disable'] = 1 + (int) self::$counts['disable'];
   }
 
-  function enable() {
+  public function enable() {
     self::$counts['enable'] = 1 + (int) self::$counts['enable'];
   }
 
@@ -53,13 +53,10 @@ class test_extension_manager_paymenttest extends CRM_Core_Payment {
    * @param array $params
    *   Assoc array of input parameters for this transaction.
    *
-   * @return array
-   *   the result in an nice formatted array (or an error object)
-   * @abstract
    */
-  function doDirectPayment(&$params) {
+  public function doDirectPayment(&$params) {
   }
 
-  function checkConfig() {
+  public function checkConfig() {
   }
 }
index ded9788d6640013bb0803eef3d5f62c6f672867b..e37529a21b661a1b3f0d1a97f10978b3b1b43114 100644 (file)
@@ -10,7 +10,6 @@ describe('Mailing Controllers', function() {
       ctrl = $controller('mailingCtrl', {$scope: scope});
     }));
 
-
     it('should check if 5 groups are there', function() {
       expect(scope.cool_api.length).toBe(3);
 
index 9b708b70dd657608c72e2a85800fd143116cd9a2..d11c92465442f059587f17a0ffee680447fd6ce6 100644 (file)
@@ -708,7 +708,7 @@ class CiviSeleniumTestCase extends PHPUnit_Extensions_SeleniumTestCase {
    * @param int $row
    * @param string $prefix
    */
-  function webtestNewDialogContact(
+  public function webtestNewDialogContact(
     $fname = 'Anthony', $lname = 'Anderson', $email = 'anthony@anderson.biz',
     $type = 4, $selectId = 's2id_contact_id', $row = 1, $prefix = '') {
     // 4 - Individual profile
@@ -809,7 +809,6 @@ class CiviSeleniumTestCase extends PHPUnit_Extensions_SeleniumTestCase {
    * @throws PHPUnit_Framework_AssertionFailedError
    * @return int
    */
-
   public function webtestAddPaymentProcessor($processorName = 'Test Processor', $processorType = 'Dummy', $processorSettings = NULL, $financialAccount = 'Deposit Bank Account') {
     if (!$processorName) {
       $this->fail("webTestAddPaymentProcessor requires $processorName.");
@@ -936,7 +935,7 @@ class CiviSeleniumTestCase extends PHPUnit_Extensions_SeleniumTestCase {
     if (!$filePath) {
       $filePath = '/tmp/testfile_' . substr(sha1(rand()), 0, 7) . '.txt';
       $fp = @fopen($filePath, 'w');
-      fputs($fp, 'Test file created by selenium test.');
+      fwrite($fp, 'Test file created by selenium test.');
       @fclose($fp);
     }
 
@@ -1067,7 +1066,7 @@ class CiviSeleniumTestCase extends PHPUnit_Extensions_SeleniumTestCase {
    * @return null
    *   of newly created online contribution page.
    */
-  function webtestAddContributionPage(
+  public function webtestAddContributionPage(
     $hash = NULL,
     $rand = NULL,
     $pageTitle = NULL,
@@ -1627,8 +1626,7 @@ class CiviSeleniumTestCase extends PHPUnit_Extensions_SeleniumTestCase {
   /**
    * @return bool
    */
-  static
-  public function checkDoLocalDBTest() {
+  public static function checkDoLocalDBTest() {
     if (defined('CIVICRM_WEBTEST_LOCAL_DB') &&
       CIVICRM_WEBTEST_LOCAL_DB
     ) {
@@ -1649,7 +1647,7 @@ class CiviSeleniumTestCase extends PHPUnit_Extensions_SeleniumTestCase {
    * @param array $match
    *   Associative array of field name => expected value. Empty if asserting
    *                      that a DELETE occurred
-   * @param boolean $delete
+   * @param bool $delete
    *   are we checking that a DELETE action occurred?
    */
   public function assertDBState($daoName, $id, $match, $delete = FALSE) {
@@ -1732,7 +1730,7 @@ class CiviSeleniumTestCase extends PHPUnit_Extensions_SeleniumTestCase {
    * @param bool $taxRate
    * @param bool $isDefault
    */
-  function _testAddFinancialAccount(
+  public function _testAddFinancialAccount(
     $financialAccountTitle,
     $financialAccountDescription = FALSE,
     $accountingCode = FALSE,
@@ -1823,7 +1821,7 @@ class CiviSeleniumTestCase extends PHPUnit_Extensions_SeleniumTestCase {
    * @param bool $taxRate
    * @param bool $isDefault
    */
-  function _testEditFinancialAccount(
+  public function _testEditFinancialAccount(
     $editfinancialAccount,
     $financialAccountTitle = FALSE,
     $financialAccountDescription = FALSE,
index efcd1ec6874530d2655f7d1b2c1c134025c97e79..c3a4ce49b90a45ffb5f2bc7a71ca158a7059efed 100644 (file)
@@ -91,7 +91,7 @@ class Contact extends CiviUnitTestCase {
    *
    * @param int $contactID
    *   Id of the contact to delete.
-   * @return boolean
+   * @return bool
    *   true if contact deleted, false otherwise
    */
   public static function delete($contactID) {
index e9ab99183cabb777059e9a17fd90f7e690c0f85f..64d8023d96afbbc1770b81c0b9fbfed5ec3051fe 100644 (file)
@@ -35,7 +35,7 @@ class Participant extends PHPUnit_Framework_Testcase {
    * Helper function to delete a participant
    *
    * @param int $participantId
-   * @return boolean
+   * @return bool
    *   true if participant deleted, false otherwise
    */
   public static function delete($participantId) {
index c8166af949a11c4635371b9a862ad4c7f898fb32..f8a5403564d11b0b159bfa2b400afb43287d36ca 100644 (file)
@@ -47,7 +47,7 @@ class WebTest_Case_CaseDashboardTest extends CiviSeleniumTestCase {
       'edit-2-access-all-cases-and-activities',
       'edit-2-access-my-cases-and-activities',
       'edit-2-administer-civicase',
-      'edit-2-delete-in-civicase'
+      'edit-2-delete-in-civicase',
     );
     $this->changePermissions($permission);