Fixed Code clean up for batch 15
authorpratikshad <pratiksha.dubey@webaccess.co.in>
Thu, 15 Jan 2015 12:33:19 +0000 (18:03 +0530)
committerpratikshad <pratiksha.dubey@webaccess.co.in>
Thu, 15 Jan 2015 12:33:19 +0000 (18:03 +0530)
52 files changed:
CRM/Activity/Form/ActivityView.php
CRM/Activity/Import/Parser/Activity.php
CRM/Badge/BAO/Badge.php
CRM/Batch/Page/AJAX.php
CRM/Campaign/BAO/Query.php
CRM/Case/BAO/Case.php
CRM/Case/PseudoConstant.php
CRM/Contact/Form/Search/Builder.php
CRM/Contact/Form/Task/AddToGroup.php
CRM/Contact/Form/Task/Map.php
CRM/Contact/Page/AJAX.php
CRM/Contact/Page/View/UserDashBoard/GroupContact.php
CRM/Core/BAO/CustomQuery.php
CRM/Core/BAO/Dashboard.php
CRM/Core/BAO/PreferencesDate.php
CRM/Core/I18n.php
CRM/Core/Invoke.php
CRM/Core/State.php
CRM/Custom/Import/Parser/Api.php
CRM/Custom/Page/Option.php
CRM/Event/Badge/Logo.php
CRM/Event/Form/Task/AddToGroup.php
CRM/Extension/Browser.php
CRM/Financial/BAO/ExportFormat/CSV.php
CRM/Financial/Form/BatchTransaction.php
CRM/Financial/Form/FinancialTypeAccount.php
CRM/Price/BAO/PriceFieldValue.php
CRM/SMS/Form/Upload.php
CRM/Upgrade/Incremental/Legacy.php
CRM/Upgrade/Incremental/php/FourOne.php
CRM/Utils/Mail/Incoming.php
CRM/Utils/Rule.php
CRM/Utils/System/WordPress.php
Civi/Core/Transaction/Frame.php
api/Wrapper.php
api/v3/Relationship.php
api/v3/ReportTemplate.php
api/v3/SystemLog.php
bin/setup.sh
install/civicrm.php
tests/extensions/test.extension.manager.moduletest/moduletest.php
tests/phpunit/CRM/Core/Smarty/plugins/CrmScopeTest.php
tests/phpunit/CRM/Member/Import/Parser/MembershipTest.php
tests/phpunit/CRM/Queue/RunnerTest.php
tests/phpunit/WebTest/Import/ImportCiviSeleniumTestCase.php
tests/phpunit/WebTest/Member/InheritedMembershipTest.php
tests/phpunit/api/v3/ContactTest.php
tests/phpunit/api/v3/EventTest.php
tests/phpunit/api/v3/ParticipantPaymentTest.php
tests/phpunit/api/v3/ParticipantTest.php
tests/phpunit/api/v3/RelationshipTest.php
tests/phpunit/api/v3/ReportTemplateTest.php

index cee07678301018e018c67f5e675c38c897ce885a..59e1272310ff91fb2764df78e8e334a0f0540439 100644 (file)
@@ -61,7 +61,7 @@ class CRM_Activity_Form_ActivityView extends CRM_Core_Form {
     if (!in_array($context, array(
       'home',
       'dashlet',
-      'dashletFullscreen'
+      'dashletFullscreen',
     ))
     ) {
       $url = CRM_Utils_System::url('civicrm/contact/view', "reset=1&cid={$cid}&selectedChild=activity");
index bf077530dceb78199c970fc0d93915e842702d5c..01e7b6869f1c9caa35829811faad06a47aa0d1c3 100644 (file)
@@ -135,7 +135,7 @@ class CRM_Activity_Import_Parser_Activity extends CRM_Activity_Import_Parser {
    * @param array $values
    *   The array of values belonging to this line.
    *
-   * @return boolean
+   * @return bool
    */
   public function mapField(&$values) {
     return CRM_Import_Parser::VALID;
@@ -147,7 +147,7 @@ class CRM_Activity_Import_Parser_Activity extends CRM_Activity_Import_Parser {
    * @param array $values
    *   The array of values belonging to this line.
    *
-   * @return boolean
+   * @return bool
    *   the result of this processing
    */
   public function preview(&$values) {
@@ -160,7 +160,7 @@ class CRM_Activity_Import_Parser_Activity extends CRM_Activity_Import_Parser {
    * @param array $values
    *   The array of values belonging to this line.
    *
-   * @return boolean
+   * @return bool
    *   the result of this processing
    */
   public function summary(&$values) {
@@ -245,7 +245,7 @@ class CRM_Activity_Import_Parser_Activity extends CRM_Activity_Import_Parser {
    * @param array $values
    *   The array of values belonging to this line.
    *
-   * @return boolean
+   * @return bool
    *   the result of this processing
    */
   public function import($onDuplicate, &$values) {
index 183a7e74b348283461a13e580afadaf2a528a608..14611420375e8cbe8fa4d8a045b99999a135a916 100644 (file)
@@ -268,8 +268,7 @@ class CRM_Badge_BAO_Badge {
       $data = $formattedRow['values'];
 
       if ($formattedRow['barcode']['type'] == 'barcode') {
-        $data['current_value'] =
-          $formattedRow['values']['contact_id'] . '-' . $formattedRow['values']['participant_id'];
+        $data['current_value'] = $formattedRow['values']['contact_id'] . '-' . $formattedRow['values']['participant_id'];
       }
       else {
         // view participant url
index 98263c5f9aba4da27f5229e8738d2f6609101ae7..1de2a868bd985dac906e60f3b9f56b263023be50 100644 (file)
@@ -65,16 +65,11 @@ class CRM_Batch_Page_AJAX {
     );
 
     $sEcho = CRM_Utils_Type::escape($_REQUEST['sEcho'], 'Integer');
-    $offset =
-      isset($_REQUEST['iDisplayStart']) ? CRM_Utils_Type::escape($_REQUEST['iDisplayStart'], 'Integer') : 0;
-    $rowCount =
-      isset($_REQUEST['iDisplayLength']) ? CRM_Utils_Type::escape($_REQUEST['iDisplayLength'], 'Integer') : 25;
-    $sort =
-      isset($_REQUEST['iSortCol_0']) ? CRM_Utils_Array::value(CRM_Utils_Type::escape($_REQUEST['iSortCol_0'], 'Integer'), $sortMapper) : NULL;
-    $sortOrder =
-      isset($_REQUEST['sSortDir_0']) ? CRM_Utils_Type::escape($_REQUEST['sSortDir_0'], 'String') : 'asc';
-    $context =
-      isset($_REQUEST['context']) ? CRM_Utils_Type::escape($_REQUEST['context'], 'String') : NULL;
+    $offset = isset($_REQUEST['iDisplayStart']) ? CRM_Utils_Type::escape($_REQUEST['iDisplayStart'], 'Integer') : 0;
+    $rowCount = isset($_REQUEST['iDisplayLength']) ? CRM_Utils_Type::escape($_REQUEST['iDisplayLength'], 'Integer') : 25;
+    $sort = isset($_REQUEST['iSortCol_0']) ? CRM_Utils_Array::value(CRM_Utils_Type::escape($_REQUEST['iSortCol_0'], 'Integer'), $sortMapper) : NULL;
+    $sortOrder = isset($_REQUEST['sSortDir_0']) ? CRM_Utils_Type::escape($_REQUEST['sSortDir_0'], 'String') : 'asc';
+    $context = isset($_REQUEST['context']) ? CRM_Utils_Type::escape($_REQUEST['context'], 'String') : NULL;
 
     $params = $_REQUEST;
     if ($sort && $sortOrder) {
index 0e6b778a6629137ab356eebf0e39c531d9d8c7fc..137aa33b8762d1e9666dbd5f4831e3317570e338 100755 (executable)
@@ -267,7 +267,7 @@ civicrm_activity_assignment.record_type_id = $assigneeID ) ";
    *
    * @return array|null
    */
-  static function defaultReturnProperties(
+  public static function defaultReturnProperties(
     $mode,
     $includeCustomFields = TRUE
   ) {
@@ -406,7 +406,7 @@ INNER JOIN  civicrm_custom_group grp on fld.custom_group_id = grp.id
     while ($dao->fetch()) {
       foreach (array(
                  'ward',
-                 'precinct'
+                 'precinct',
                ) as $name) {
         if (stripos($name, $dao->label) !== FALSE) {
           $fieldId = $dao->id;
@@ -571,7 +571,7 @@ INNER JOIN  civicrm_custom_group grp on fld.custom_group_id = grp.id
   /**
    * Build the campaign clause for component serach.
    *
-   **/
+   */
   public static function componentSearchClause(&$params, &$query) {
     $op = CRM_Utils_Array::value('op', $params, '=');
     $campaign = CRM_Utils_Array::value('campaign', $params);
@@ -586,7 +586,7 @@ INNER JOIN  civicrm_custom_group grp on fld.custom_group_id = grp.id
     if (is_array($campaign)) {
       foreach (array(
                  'current_campaign',
-                 'past_campaign'
+                 'past_campaign',
                ) as $ignore) {
         $index = array_search($ignore, $campaign);
         if ($index !== FALSE) {
index bd27990f46eabf326277f8b18734fd271a0e6ebf..7d0d5e9ad6e3bfb710100a94d5469eeb00b2aab7 100644 (file)
@@ -198,7 +198,7 @@ class CRM_Case_BAO_Case extends CRM_Case_DAO_Case {
    *
    * @param int $caseID
    *
-   * @return Void
+   * @return;
    */
   public static function deleteCaseContact($caseID) {
     $caseContact = new CRM_Case_DAO_CaseContact();
@@ -1857,7 +1857,7 @@ SELECT case_status.label AS case_status, status_id, civicrm_case_type.title AS c
     $where = $groupBy = ' ';
 
     if (!$caseID) {
-      return;
+      return NULL;
     }
 
     if ($latestDate) {
@@ -2001,7 +2001,7 @@ SELECT case_status.label AS case_status, status_id, civicrm_case_type.title AS c
    */
   public static function getCaseManagerContact($caseType, $caseId) {
     if (!$caseType || !$caseId) {
-      return;
+      return NULL;
     }
 
     $caseManagerContact = array();
@@ -3379,7 +3379,7 @@ WHERE id IN (' . implode(',', $copiedActivityIds) . ')';
    *
    * @param string $fieldName
    * @param string $context
-   *   @see CRM_Core_DAO::buildOptionsContext.
+   * @see CRM_Core_DAO::buildOptionsContext
    * @param array $props
    *   Whatever is known about this dao object.
    *
index 26fdd21fa4b419f98950ed81b9d452f6c6543aa9..8b45a10986addcfe9d90ffcc00ad39fb640a9154 100644 (file)
@@ -106,7 +106,6 @@ class CRM_Case_PseudoConstant extends CRM_Core_PseudoConstant {
    * @return array
    *   array reference of all redaction rules
    */
-
   public static function redactionRule($filter = NULL) {
     // if ( ! self::$redactionRule ) {
     self::$redactionRule = array();
index 1101237727c27c2b659d9f45c4766af7b08cfb21..2e74aaea819dd9eb2aa2fef02fe2bd24d7e364c6 100644 (file)
@@ -213,7 +213,7 @@ class CRM_Contact_Form_Search_Builder extends CRM_Contact_Form_Search {
             // hack to handle custom data of type state and country
             if (in_array($type, array(
               'Country',
-              'StateProvince'
+              'StateProvince',
             ))) {
               $type = "Integer";
             }
@@ -423,7 +423,7 @@ class CRM_Contact_Form_Search_Builder extends CRM_Contact_Form_Search {
    * FIXME: When our core fields contain reliable metadata this will be much simpler.
    * @return array
    *   (string => string) key: field_name value: api entity name
-   * Note: options are fetched via ajax using the api "getoptions" method
+   *   Note: options are fetched via ajax using the api "getoptions" method
    */
   public static function fieldOptions() {
     // Hack to add options not retrieved by getfields
@@ -450,7 +450,7 @@ class CRM_Contact_Form_Search_Builder extends CRM_Contact_Form_Search {
       'member',
       'contribution',
       'case',
-      'grant'
+      'grant',
     );
     CRM_Contact_BAO_Query_Hook::singleton()->alterSearchBuilderOptions($entities, $options);
     foreach ($entities as $entity) {
@@ -468,7 +468,7 @@ class CRM_Contact_Form_Search_Builder extends CRM_Contact_Form_Search {
         }
         elseif (in_array(substr($field, 0, 3), array(
               'is_',
-              'do_'
+              'do_',
             )) || CRM_Utils_Array::value('data_type', $info) == 'Boolean'
         ) {
           $options[$field] = 'yesno';
index 1f643851c0e5504c719e7f591d8b378decd4c0b0..1146e0df890e200f934f0da42921541f9d8f9a42 100644 (file)
@@ -234,20 +234,20 @@ class CRM_Contact_Form_Task_AddToGroup extends CRM_Contact_Form_Task {
     $status = array(
       ts('%count contact added to group', array(
           'count' => $added,
-          'plural' => '%count contacts added to group'
-        ))
+          'plural' => '%count contacts added to group',
+      )),
     );
     if ($notAdded) {
       $status[] = ts('%count contact was already in group', array(
           'count' => $notAdded,
-          'plural' => '%count contacts were already in group'
+          'plural' => '%count contacts were already in group',
         ));
     }
     $status = '<ul><li>' . implode('</li><li>', $status) . '</li></ul>';
     CRM_Core_Session::setStatus($status, ts('Added Contact to %1', array(
           1 => $groupName,
           'count' => $added,
-          'plural' => 'Added Contacts to %1'
+          'plural' => 'Added Contacts to %1',
         )), 'success', array('expires' => 0));
 
     if ($this->_context === 'amtg') {
index 0a225a2ab597a665331cf2975427176dfbaadabb..65c94569d9812b6c74b34ed715f4cd523b810e3c 100644 (file)
@@ -149,7 +149,7 @@ class CRM_Contact_Form_Task_Map extends CRM_Contact_Form_Task {
    * @param $addBreadCrumb
    * @param string $type
    *
-   * @return string
+   * @return;
    *   the location of the file we have created
    */
   public static function createMapXML($ids, $locationId, &$page, $addBreadCrumb, $type = 'Contact') {
index 5018815665a5896141f86af6a2e1ca378078dff5..4cd911fb46111a9e17833b4811473234424c3f7c 100644 (file)
@@ -161,7 +161,7 @@ class CRM_Contact_Page_AJAX {
         'id',
         's',
         'q',
-        'action'
+        'action',
       );
       foreach ($_GET as $param => $val) {
         if (empty($val) ||
@@ -1062,11 +1062,10 @@ LIMIT {$offset}, {$rowCount}
       $addressVal["error_message"] = "no contact id found";
     }
     else {
-      $entityBlock =
-        array(
-          'contact_id' => $contactId,
-          'entity_id' => $contactId,
-        );
+      $entityBlock = array(
+        'contact_id' => $contactId,
+        'entity_id' => $contactId,
+      );
       $addressVal = CRM_Core_BAO_Address::getValues($entityBlock);
     }
 
index e8570e9201f483c2bcb8f9c14a507f9fe7140aa7..3fb41fcf8c169e1749d5d546d0fc1fc75f19cf06 100644 (file)
@@ -37,7 +37,7 @@ class CRM_Contact_Page_View_UserDashBoard_GroupContact extends CRM_Contact_Page_
   /**
    * called when action is browse
    *
-   * @return null
+   * @return;
    */
   public function browse() {
     $count = CRM_Contact_BAO_GroupContact::getContactGroup(
@@ -84,7 +84,7 @@ class CRM_Contact_Page_View_UserDashBoard_GroupContact extends CRM_Contact_Page_
   public function edit($groupId = NULL) {
     $this->assign('edit', $this->_edit);
     if (!$this->_edit) {
-      return;
+      return NULL;
     }
 
     $action = CRM_Utils_Request::retrieve('action', 'String',
@@ -93,10 +93,8 @@ class CRM_Contact_Page_View_UserDashBoard_GroupContact extends CRM_Contact_Page_
     );
 
     if ($action == CRM_Core_Action::DELETE) {
-      $groupContactId =
-        CRM_Utils_Request::retrieve('gcid', 'Positive', CRM_Core_DAO::$_nullObject, TRUE);
-      $status =
-        CRM_Utils_Request::retrieve('st', 'String', CRM_Core_DAO::$_nullObject, TRUE);
+      $groupContactId = CRM_Utils_Request::retrieve('gcid', 'Positive', CRM_Core_DAO::$_nullObject, TRUE);
+      $status = CRM_Utils_Request::retrieve('st', 'String', CRM_Core_DAO::$_nullObject, TRUE);
       if (is_numeric($groupContactId) && $status) {
         CRM_Contact_Page_View_GroupContact::del($groupContactId, $status, $this->_contactId);
       }
@@ -132,7 +130,7 @@ class CRM_Contact_Page_View_UserDashBoard_GroupContact extends CRM_Contact_Page_
    * the main function that is called when the page loads,
    * it decides the which action has to be taken for the page.
    *
-   * @return null
+   * @return;
    */
   public function run() {
     $this->edit();
index 4e0c1c9fe8ee38df6b84d924f97bf58aa74451cc..77633ff315f927f925d20923b309721b27642d8b 100644 (file)
@@ -256,8 +256,6 @@ SELECT label, value
    * Generate the select clause and the associated tables
    * for the from clause
    *
-   * @param NULL
-   *
    * @return void
    */
   public function select() {
@@ -323,8 +321,6 @@ SELECT label, value
    * Generate the where clause and also the english language
    * equivalent
    *
-   * @param NULL
-   *
    * @return void
    */
   public function where() {
@@ -357,7 +353,7 @@ SELECT label, value
           $wildcard = $isSerialized ? $wildcard : TRUE;
           $options = CRM_Utils_Array::value('values', civicrm_api3('contact', 'getoptions', array(
                 'field' => $name,
-                'context' => 'search'
+                'context' => 'search',
               ), array()));
           $qillValue = '';
           $sqlOP = $wildcard ? ' OR ' : ' AND ';
@@ -555,8 +551,6 @@ SELECT label, value
    * Function that does the actual query generation
    * basically ties all the above functions together
    *
-   * @param NULL
-   *
    * @return array
    *   array of strings
    */
index a8e4068bc977e615aef34cbf081227de9f892215..d25c8fe457143f5631c9480b3948bae72c01339c 100644 (file)
@@ -159,7 +159,7 @@ class CRM_Core_BAO_Dashboard extends CRM_Core_DAO_Dashboard {
     $dashlets = array();
     $getDashlets = civicrm_api3("Dashboard", "get", array(
         'domain_id' => CRM_Core_Config::domainID(),
-        'option.limit' => 0
+        'option.limit' => 0,
       ));
     $contactID = CRM_Core_Session::singleton()->get('userID');
     $allDashlets = CRM_Utils_Array::index(array('name'), $getDashlets['values']);
@@ -198,7 +198,7 @@ class CRM_Core_BAO_Dashboard extends CRM_Core_DAO_Dashboard {
    *   Comma separated list.
    * @param string $operator
    *
-   * @return boolean
+   * @return bool
    *   true if use has permission else false
    */
   public static function checkPermission($permission, $operator) {
@@ -226,7 +226,7 @@ class CRM_Core_BAO_Dashboard extends CRM_Core_DAO_Dashboard {
         // hack to handle case permissions
         if (!$componentName && in_array($key, array(
             'access my cases and activities',
-            'access all cases and activities'
+            'access all cases and activities',
           ))
         ) {
           $componentName = 'CiviCase';
@@ -365,7 +365,7 @@ class CRM_Core_BAO_Dashboard extends CRM_Core_DAO_Dashboard {
     $dashletIDs = array();
     if (is_array($columns)) {
       foreach ($columns as $colNo => $dashlets) {
-        if (!is_integer($colNo)) {
+        if (!is_int($colNo)) {
           continue;
         }
         $weight = 1;
index 7d173f81dbaea65e010e374a14926dac19168de2..e85135433bb6e109fab6b9660b904b705fc17164 100644 (file)
@@ -75,7 +75,7 @@ class CRM_Core_BAO_PreferencesDate extends CRM_Core_DAO_PreferencesDate {
    * @param bool $is_active
    *   Value we want to set the is_active field.
    *
-   * @return Object
+   * @return;
    *   DAO object on sucess, null otherwise
    *
    */
index 2b532c53c3ac64d618415466940999e240ecbb8f..9a22988ae02d5d2cbef0de4695e65d3d04246e42 100644 (file)
@@ -152,7 +152,9 @@ class CRM_Core_I18n {
         if ($code == 'en_US') {
           continue;
         }
-        if (!in_array($code, $codes)) unset($all[$code]);
+        if (!in_array($code, $codes)) {
+          unset($all[$code]);
+        }
       }
     }
 
@@ -176,7 +178,6 @@ class CRM_Core_I18n {
    *
    * @param string $str
    *   source string.
-   * @param mixed arguments, can be passed in an array or through single variables
    *
    * @return string
    *   modified string
@@ -368,7 +369,7 @@ class CRM_Core_I18n {
    *
    * @return void
    */
-  function localizeArray(
+  public function localizeArray(
     &$array,
     $params = array()
   ) {
@@ -411,7 +412,7 @@ class CRM_Core_I18n {
    * @param $key
    *   Key of the extension (can be 'civicrm', or 'org.example.foo').
    *
-   * @return Boolean
+   * @return Bool
    *   True if the domain was changed for an extension.
    */
   public function setGettextDomain($key) {
index c5056fa2a00658836ec810a3c3b5277835785197..b1da8497d374c0120c4639feaa98c9fbb178b765 100644 (file)
@@ -70,7 +70,7 @@ class CRM_Core_Invoke {
    */
   public static function _invoke($args) {
     if ($args[0] !== 'civicrm') {
-      return;
+      return NULL;
     }
 
     if (!defined('CIVICRM_SYMFONY_PATH')) {
@@ -226,7 +226,7 @@ class CRM_Core_Invoke {
       // check that we are permissioned to access this page
       if (!CRM_Core_Permission::checkMenuItem($item)) {
         CRM_Utils_System::permissionDenied();
-        return;
+        return NULL;
       }
 
       // check if ssl is set
@@ -289,7 +289,7 @@ class CRM_Core_Invoke {
         }
         $title = CRM_Utils_Array::value('title', $item);
         if (strstr($item['page_callback'], '_Page')) {
-          $object = new $item['page_callback'] ($title, $mode);
+          $object = new $item['page_callback']($title, $mode);
           $object->urlPath = explode('/', $_GET[$config->userFrameworkURLVar]);
         }
         elseif (strstr($item['page_callback'], '_Controller')) {
@@ -299,7 +299,7 @@ class CRM_Core_Invoke {
             $addSequence = $addSequence ? 'true' : 'false';
             unset($pageArgs['addSequence']);
           }
-          $object = new $item['page_callback'] ($title, TRUE, $mode, NULL, $addSequence);
+          $object = new $item['page_callback']($title, TRUE, $mode, NULL, $addSequence);
         }
         else {
           CRM_Core_Error::fatal();
index 8b003a089590f44679f2fedee8a64c27659a068b..b5baf68f676fed0de9e8ee857785e17d6684ecb9 100644 (file)
@@ -108,8 +108,6 @@ class CRM_Core_State {
   /**
    * Given an CRM Form, jump to the previous page
    *
-   * @param object the CRM_Core_Form element under consideration
-   *
    * @return mixed
    *   does a jump to the back state
    */
@@ -126,8 +124,6 @@ class CRM_Core_State {
   /**
    * Given an CRM Form, jump to the next page
    *
-   * @param object the CRM_Core_Form element under consideration
-   *
    * @return mixed
    *   does a jump to the nextstate
    */
@@ -161,8 +157,6 @@ class CRM_Core_State {
    * Mark this page as valid for the QFC framework. This is needed as
    * we build more advanced functionality into the StateMachine
    *
-   * @param object the QFC data container
-   *
    * @return void
    */
   public function validate(&$data) {
@@ -173,8 +167,6 @@ class CRM_Core_State {
    * Mark this page as invalid for the QFC framework. This is needed as
    * we build more advanced functionality into the StateMachine
    *
-   * @param object the QFC data container
-   *
    * @return void
    */
   public function invalidate(&$data) {
@@ -193,8 +185,6 @@ class CRM_Core_State {
   /**
    * Setter for name
    *
-   * @param string
-   *
    * @return void
    */
   public function setName($name) {
index 514847194fb59e26ddae0b327065ffa4ed846247..915312f2aead6d59723307931d2ef1e658fc33ae 100644 (file)
@@ -30,7 +30,7 @@ class CRM_Custom_Import_Parser_Api extends CRM_Custom_Import_Parser {
     $importableFields = $this->getGroupFieldsForImport($customGroupID, $this);
     $this->_fields = array_merge(array(
         'do_not_import' => array('title' => ts('- do not import -')),
-        'contact_id' => array('title' => ts('Contact ID'))
+        'contact_id' => array('title' => ts('Contact ID')),
       ), $importableFields);
   }
 
@@ -59,7 +59,7 @@ class CRM_Custom_Import_Parser_Api extends CRM_Custom_Import_Parser {
    * @param array $values
    *   The array of values belonging to this line.
    *
-   * @return boolean
+   * @return bool
    */
   public function mapField(&$values) {
     return CRM_Import_Parser::VALID;
@@ -71,7 +71,7 @@ class CRM_Custom_Import_Parser_Api extends CRM_Custom_Import_Parser {
    * @param array $values
    *   The array of values belonging to this line.
    *
-   * @return boolean
+   * @return bool
    *   the result of this processing
    */
   public function preview(&$values) {
@@ -82,7 +82,7 @@ class CRM_Custom_Import_Parser_Api extends CRM_Custom_Import_Parser {
    * @param array $values
    *   The array of values belonging to this line.
    *
-   * @return boolean
+   * @return bool
    *   the result of this processing
    *   It is called from both the preview & the import actions
    *
@@ -136,7 +136,7 @@ class CRM_Custom_Import_Parser_Api extends CRM_Custom_Import_Parser {
    * @param array $values
    *   The array of values belonging to this line.
    *
-   * @return boolean
+   * @return bool
    *   the result of this processing
    */
   public function import($onDuplicate, &$values) {
index bbf3a05389fe5e072b175a1932bc4f0c41b45005..dc44b216fe57c8a45157664a55ced7771e7ef250 100644 (file)
@@ -69,8 +69,6 @@ class CRM_Custom_Page_Option extends CRM_Core_Page {
   /**
    * Get the action links for this page.
    *
-   * @param null
-   *
    * @return array
    *   array of action links that we need to display for the browse screen
    */
@@ -113,8 +111,6 @@ class CRM_Custom_Page_Option extends CRM_Core_Page {
   /**
    * Browse all custom group fields.
    *
-   * @param null
-   *
    * @return void
    */
   public function browse() {
@@ -265,8 +261,6 @@ ORDER BY weight, label
    * This method is called after the page is created. It checks for the
    * type of action and executes that action.
    *
-   * @param null
-   *
    * @return void
    */
   public function run() {
@@ -288,7 +282,7 @@ ORDER BY weight, label
       array(
         'title' => ts('Custom Data Fields'),
         'url' => CRM_Utils_System::url('civicrm/admin/custom/group/field', 'reset=1&gid=' . $this->_gid),
-      )
+      ),
     );
     CRM_Utils_System::appendBreadCrumb($breadcrumb);
 
index 016979c6da550aae78f2c3dad2bdbb15be3ba144..fa34bd66b8f55fe02aa640e86c3815d8c7050288 100644 (file)
@@ -44,7 +44,7 @@ class CRM_Event_Badge_Logo extends CRM_Event_Badge {
         'cap' => 'round',
         'join' => 'round',
         'dash' => '2,2',
-        'color' => array(0, 0, 200)
+        'color' => array(0, 0, 200),
       ));
 
     $this->pdf->SetFontSize(8);
index eabde5fb764a10f8036c64fbeac0828d81428473..e5707ce5a5dad6c128c4bbc6d891048fc7c50707 100644 (file)
@@ -178,7 +178,7 @@ class CRM_Event_Form_Task_AddToGroup extends CRM_Event_Form_Task {
   /**
    * Global validation rules for the form
    *
-   * @param array $fields
+   * @param array $params
    *   Posted values of the form.
    *
    * @return array
@@ -236,20 +236,20 @@ class CRM_Event_Form_Task_AddToGroup extends CRM_Event_Form_Task {
     $status = array(
       ts('%count contact added to group', array(
           'count' => $added,
-          'plural' => '%count contacts added to group'
-        ))
+          'plural' => '%count contacts added to group',
+      )),
     );
     if ($notAdded) {
       $status[] = ts('%count contact was already in group', array(
           'count' => $notAdded,
-          'plural' => '%count contacts were already in group'
+          'plural' => '%count contacts were already in group',
         ));
     }
     $status = '<ul><li>' . implode('</li><li>', $status) . '</li></ul>';
     CRM_Core_Session::setStatus($status, ts('Added Contact to %1', array(
           1 => $groupName,
           'count' => $added,
-          'plural' => 'Added Contacts to %1'
+          'plural' => 'Added Contacts to %1',
         )), 'success', array('expires' => 0));
   }
 }
index 43bfcb8acb2b9efd8a66daa6d03a24ec201002c2..bed1913d9a8922e7bc7688321dc0fef3b93f00ea 100644 (file)
@@ -220,7 +220,7 @@ class CRM_Extension_Browser {
    * extensions.
    *
    *
-   * @return Array
+   * @return array
    *   list of extension names
    */
   private function grabRemoteKeyList() {
index ce76cb19018e527ea0e393b954510544f5ebaa9d..7860dd25790c825110f2fdecfe7825907d43302e 100644 (file)
@@ -173,8 +173,7 @@ class CRM_Financial_BAO_ExportFormat_CSV extends CRM_Financial_BAO_ExportFormat
       $financialItems = array();
       $this->_batchIds = $batchId;
       while ($dao->fetch()) {
-        $creditAccountName = $creditAccountType =
-        $creditAccount = NULL;
+        $creditAccountName = $creditAccountType = $creditAccount = NULL;
         if ($dao->credit_account) {
           $creditAccountName = $dao->credit_account_name;
           $creditAccountType = $dao->credit_account_type_code;
index 3f9d3f030cbbdd80f55f35b9cd807c2d192436cf..129df0d25b4d1871678fd66f7d87130ea58c68b8 100644 (file)
@@ -57,18 +57,17 @@ class CRM_Financial_Form_BatchTransaction extends CRM_Contribute_Form {
       $batchTitle = CRM_Core_DAO::getFieldValue('CRM_Batch_BAO_Batch', self::$_entityID, 'title');
       CRM_Utils_System::setTitle(ts('Accounting Batch - %1', array(1 => $batchTitle)));
 
-      $columnHeaders =
-        array(
-          'created_by' => ts('Created By'),
-          'status' => ts('Status'),
-          'description' => ts('Description'),
-          'payment_instrument' => ts('Payment Instrument'),
-          'item_count' => ts('Entered Transactions'),
-          'assigned_item_count' => ts('Assigned Transactions'),
-          'total' => ts('Entered Total'),
-          'assigned_total' => ts('Assigned Total'),
-          'opened_date' => ts('Opened'),
-        );
+      $columnHeaders = array(
+        'created_by' => ts('Created By'),
+        'status' => ts('Status'),
+        'description' => ts('Description'),
+        'payment_instrument' => ts('Payment Instrument'),
+        'item_count' => ts('Entered Transactions'),
+        'assigned_item_count' => ts('Assigned Transactions'),
+        'total' => ts('Entered Total'),
+        'assigned_total' => ts('Assigned Total'),
+        'opened_date' => ts('Opened'),
+      );
       $this->assign('columnHeaders', $columnHeaders);
     }
   }
index 4574abe3f0c2c2cf7d9c85fb26dceb2467fca407..9b7210e3520ea76dacf969ac4d530951aa8f00c3 100644 (file)
@@ -171,7 +171,7 @@ class CRM_Financial_Form_FinancialTypeAccount extends CRM_Contribute_Form {
           '6' => 1, //Asset
           '7' => 4, //cost of sales
           '8' => 1, //premium inventory
-          '9' => 3 //discount account is,
+          '9' => 3, //discount account is,
         );
 
         $financialAccountType = CRM_Utils_Array::value($this->_submitValues['account_relationship'], $financialAccountType);
@@ -181,8 +181,8 @@ class CRM_Financial_Form_FinancialTypeAccount extends CRM_Contribute_Form {
       }
       else {
         $financialAccountSelect = array(
-            'select' => ts('- select -'),
-          ) + CRM_Contribute_PseudoConstant::financialAccount();
+          'select' => ts('- select -'),
+        ) + CRM_Contribute_PseudoConstant::financialAccount();
       }
     }
     if ($this->_action == CRM_Core_Action::UPDATE) {
@@ -193,7 +193,7 @@ class CRM_Financial_Form_FinancialTypeAccount extends CRM_Contribute_Form {
         '6' => 1, //Asset
         '7' => 4, //cost of sales
         '8' => 1, //premium inventory
-        '9' => 3 //discount account is,
+        '9' => 3, //discount account is,
       );
 
       $financialAccountType = $financialAccountType[$this->_defaultValues['account_relationship']];
@@ -223,7 +223,7 @@ class CRM_Financial_Form_FinancialTypeAccount extends CRM_Contribute_Form {
         array(
           'type' => 'cancel',
           'name' => ts('Cancel'),
-        )
+        ),
       )
     );
     $this->addFormRule(array('CRM_Financial_Form_FinancialTypeAccount', 'formRule'), $this);
index a0a9f5e171cf2f04f85ea4b0c2c64fb0403225e5..da14f6d76f2dd39017d9604ba9ed74cdb4eeb503 100644 (file)
@@ -80,7 +80,7 @@ class CRM_Price_BAO_PriceFieldValue extends CRM_Price_DAO_PriceFieldValue {
   public static function create(&$params, $ids = array()) {
     $id = CRM_Utils_Array::value('id', $params, CRM_Utils_Array::value('id', $ids));
     if (!is_array($params) || empty($params)) {
-      return;
+      return NULL;
     }
     if (!$id && empty($params['name'])) {
       $params['name'] = strtolower(CRM_Utils_String::munge($params['label'], '_', 242));
@@ -221,7 +221,7 @@ class CRM_Price_BAO_PriceFieldValue extends CRM_Price_DAO_PriceFieldValue {
    * @param int $id
    *   Id.
    *
-   * @return boolean
+   * @return bool
    *
    */
   public static function del($id) {
@@ -240,9 +240,9 @@ class CRM_Price_BAO_PriceFieldValue extends CRM_Price_DAO_PriceFieldValue {
    *
    * @param int $entityId
    *   Id.
-   * @param string $entityTableEntity table.
+   * @param string $entityTable table.
    *   Entity table.
-   * @param string $financialTypeIDFinancial type id.
+   * @param string $financialTypeID type id.
    *   Financial type id.
    *
    */
index 174a8047d125c28596f9723feb27be7dd5eb71b8..033b1803059d411ec781eb66924102efec5859ff 100644 (file)
@@ -309,7 +309,6 @@ class CRM_SMS_Form_Upload extends CRM_Core_Form {
     $errors = array();
     $template = CRM_Core_Smarty::singleton();
 
-
     $domain = CRM_Core_BAO_Domain::getDomain();
 
     $mailing = new CRM_Mailing_BAO_Mailing();
@@ -337,7 +336,6 @@ class CRM_SMS_Form_Upload extends CRM_Core_Form {
       $urls[$key]++;
     }
 
-
     $skipTextFile = $self->get('skipTextFile');
 
     if (!$params['upload_type']) {
@@ -382,7 +380,6 @@ class CRM_SMS_Form_Upload extends CRM_Core_Form {
       /* Do a full token replacement on a dummy verp, the current
              * contact and domain, and the first organization. */
 
-
       // here we make a dummy mailing object so that we
       // can retrieve the tokens that we need to replace
       // so that we do get an invalid token error
@@ -415,7 +412,7 @@ class CRM_SMS_Form_Upload extends CRM_Core_Form {
       }
       if (!empty($dataErrors)) {
         $errors['textFile'] = ts('The following errors were detected in %1:', array(
-            1 => $name
+          1 => $name,
           )) . ' <ul>' . implode('', $dataErrors) . '</ul>';
       }
     }
index b893d3b2b796199ff01db9f0b31aadd46eaf208c..19f538f52dc3b930b7416113dc7137181fdb87c0 100644 (file)
@@ -83,7 +83,7 @@ SELECT  id
       $config = CRM_Core_Config::singleton();
       $preUpgradeMessage .= '<br />' . ts("As per <a href='%1'>the related blog post</a>, we are making contact names, addresses and mailings monolingual; the values entered for the default locale (%2) will be preserved and values for other locales removed.", array(
             1 => 'http://civicrm.org/blogs/shot/multilingual-civicrm-3440-making-some-fields-monolingual',
-            2 => $config->lcMessages
+            2 => $config->lcMessages,
           ));
     }
 
@@ -203,7 +203,7 @@ SELECT  id
 
       $message .= '<br />' . ts("The default copies of the message templates listed below will be updated to handle new features or correct a problem. Your installation has customized versions of these message templates, and you will need to apply the updates manually after running this upgrade. <a href='%1' style='color:white; text-decoration:underline; font-weight:bold;' target='_blank'>Click here</a> for detailed instructions. %2", array(
             1 => 'http://wiki.civicrm.org/confluence/display/CRMDOC/Message+Templates#MessageTemplates-UpgradesandCustomizedSystemWorkflowTemplates',
-            2 => $html
+            2 => $html,
           ));
     }
   }
@@ -250,7 +250,7 @@ SELECT  id
           $subTypeTemplates = implode(',', $subTypeTemplates);
           $postUpgradeMessage .= '<br />' . ts('You are using custom template for contact subtypes: %1.', array(1 => $subTypeTemplates)) . '<br />' . ts('You need to move these subtype templates to the SubType directory in %1 and %2 respectively.', array(
                 1 => 'CRM/Contact/Form/Edit',
-                2 => 'CRM/Contact/Page/View'
+                2 => 'CRM/Contact/Page/View',
               ));
         }
       }
@@ -550,7 +550,7 @@ SELECT  count( id ) as statusCount
     $config = CRM_Core_Config::singleton();
     if (is_callable(array(
       $config->userSystem,
-      'replacePermission'
+      'replacePermission',
     ))) {
       $config->userSystem->replacePermission('access CiviEvent', array('access CiviEvent', 'edit all events'));
     }
index 3c3bc842164a72af0ad9fd1c19658de698aeaa56..c66c3cf0e7593d1d836d48edc58701a82da73772 100644 (file)
@@ -334,7 +334,7 @@ AND    v.is_active = 1
     $dao->free();
 
     // Do the regular upgrade
-    $upgrade = new CRM_Upgrade_Form;
+    $upgrade = new CRM_Upgrade_Form();
     $upgrade->processSQL($rev);
   }
 
index 320899b9762a82ebe2ede8a3e97ba82be9369f79..612214c53f57dceee4c18751f9831e862a377897 100644 (file)
@@ -324,7 +324,7 @@ class CRM_Utils_Mail_Incoming {
     // we definitely need a contact id for the from address
     // if we dont have one, skip this email
     if (empty($params['from']['id'])) {
-      return;
+      return NULL;
     }
 
     $emailFields = array('to', 'cc', 'bcc');
index 65944518963515ecf2bedba812b9e32229f9b938..c9b791157adb69af4c1f1f04664a93b506ced839 100644 (file)
@@ -555,8 +555,6 @@ class CRM_Utils_Rule {
    * See how file rules are written in HTML/QuickForm/file.php
    * Checks to make sure the uploaded file is ascii
    *
-   * @param array Uploaded file info (from $_FILES)
-   *
    * @return bool
    *   true if file has been uploaded, false otherwise
    */
@@ -572,8 +570,6 @@ class CRM_Utils_Rule {
   /**
    * Checks to make sure the uploaded file is in UTF-8, recodes if it's not
    *
-   * @param array Uploaded file info (from $_FILES)
-   *
    * @return bool
    *   whether file has been uploaded properly and is now in UTF-8
    */
@@ -602,8 +598,6 @@ class CRM_Utils_Rule {
    * See how file rules are written in HTML/QuickForm/file.php
    * Checks to make sure the uploaded file is html
    *
-   * @param array Uploaded file info (from $_FILES)
-   *
    * @return bool
    *   true if file has been uploaded, false otherwise
    */
@@ -624,7 +618,7 @@ class CRM_Utils_Rule {
    * @param array $options
    *   The daoName and fieldName (optional ).
    *
-   * @return boolean
+   * @return bool
    *   true if object exists
    */
   public static function objectExists($value, $options) {
index 6f19973d5a2d9ea940a82176d145e5171fc9af29..d748d112597aa2cb9fdcf31b067906bf81ca8742 100644 (file)
@@ -237,8 +237,6 @@ class CRM_Utils_System_WordPress extends CRM_Utils_System_Base {
   /**
    * Rewrite various system urls to https
    *
-   * @param null
-   *
    * @return void
    */
   public function mapConfigToSSL() {
@@ -286,7 +284,7 @@ class CRM_Utils_System_WordPress 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,
@@ -667,7 +665,7 @@ class CRM_Utils_System_WordPress extends CRM_Utils_System_Base {
   /**
    * Check is user logged in.
    *
-   * @return boolean
+   * @return bool
    */
   public function isUserLoggedIn() {
     $isloggedIn = FALSE;
@@ -755,7 +753,7 @@ class CRM_Utils_System_WordPress extends CRM_Utils_System_Base {
    * @param CRM_Core_Form $form
    */
   public function getLoginDestination(&$form) {
-    return;
+    return NULL;
   }
 
   /**
index d07de48e672ec9a1fe57026ac258eb9b3f9547ad..8e565f2b93e04fe835eb78ca3512c599e3b3eed5 100644 (file)
@@ -168,13 +168,13 @@ class Frame {
     if ($id) {
       $this->callbacks[$phase][$id] = array(
         'callback' => $callback,
-        'parameters' => (is_array($params) ? $params : array($params))
+        'parameters' => (is_array($params) ? $params : array($params)),
       );
     }
     else {
       $this->callbacks[$phase][] = array(
         'callback' => $callback,
-        'parameters' => (is_array($params) ? $params : array($params))
+        'parameters' => (is_array($params) ? $params : array($params)),
       );
     }
   }
index 07e38f01810eacb9d46983b60fe4a43f25f14f82..f53c8fdb835d6fcc6d7f80c35938bfd9461261bf 100644 (file)
@@ -11,7 +11,7 @@ interface API_Wrapper {
    * @return array
    *   modified $apiRequest
    */
-  function fromApiInput($apiRequest);
+  public function fromApiInput($apiRequest);
 
   /**
    * @param array $apiRequest
@@ -20,5 +20,5 @@ interface API_Wrapper {
    * @return array
    *   modified $result
    */
-  function toApiOutput($apiRequest, $result);
+  public function toApiOutput($apiRequest, $result);
 }
index 17a92bdf45f87168723306a1c5102e44449f1a96..ff0dfd1f59ee5d2c5325f5269326fd2ef1a2d7e2 100644 (file)
@@ -47,7 +47,7 @@
  *
  * @return array
  *   API Result Array
- * {@getfields relationship_create}
+ *   {@getfields relationship_create}
  */
 function civicrm_api3_relationship_create($params) {
   _civicrm_api3_handle_relationship_type($params);
@@ -74,7 +74,7 @@ function _civicrm_api3_relationship_create_spec(&$params) {
  *
  * @return array
  *   API Result Array
- * {@getfields relationship_delete}
+ *   {@getfields relationship_delete}
  * @example RelationshipDelete.php Delete Example
  *
  */
@@ -104,9 +104,9 @@ function civicrm_api3_relationship_delete($params) {
  * -  if you pass in contact_id - it just returns all relationships for 'contact_id'
  * -  if you don't pass in contact_id then it does a filter on the relationship table (DAO based search)
  *
- * @return Array
+ * @return array
  *   API Result Array
- * {@getfields relationship_get}
+ *   {@getfields relationship_get}
  * @example RelationshipGet.php
  * @access  public
  */
index 17ecda8705bd3baad962987fb606a455b922c2d5..05755081b4aff096af5957878a196a617101e521 100644 (file)
@@ -12,7 +12,7 @@
  *
  * @return array
  *   details of found Option Values
- * {@getfields OptionValue_get}
+ *   {@getfields OptionValue_get}
  */
 function civicrm_api3_report_template_get($params) {
   require_once 'api/v3/OptionValue.php';
@@ -33,7 +33,7 @@ function civicrm_api3_report_template_get($params) {
  *
  * @return array
  *   Array of newly created option_value property values.
- * {@getfields OptionValue_create}
+ *   {@getfields OptionValue_create}
  */
 function civicrm_api3_report_template_create($params) {
   require_once 'api/v3/OptionValue.php';
@@ -74,7 +74,7 @@ function _civicrm_api3_report_template_create_spec(&$params) {
  *
  * @return array
  *   Api result
- * {@getfields ReportTemplate_create}
+ *   {@getfields ReportTemplate_create}
  */
 function civicrm_api3_report_template_delete($params) {
   require_once 'api/v3/OptionValue.php';
@@ -164,7 +164,7 @@ function civicrm_api3_report_template_getstatistics($params) {
  * @param array $params
  *   Input parameters.
  *
- * @return array
+ * @return;
  *   details of found instances
  */
 function _civicrm_api3_report_template_getrows_spec(&$params) {
index 8410be956589f4964c184837a5c0556f49665188..89f6dac6d8aba7fc4fb74c29085504f306c00e34 100644 (file)
@@ -60,7 +60,7 @@ function civicrm_api3_system_log_create($params) {
 /**
  * @param array $params
  *
- * @return array
+ * @return;
  */
 function _civicrm_api3_system_log_create_spec(&$params) {
   require_once 'api/v3/System.php';
index eb31a8328feff404465bf7f24a6ecfb3d2fb230d..5273907a6943f23b9d30f7b2d267fa33a7d405c7 100755 (executable)
@@ -151,7 +151,7 @@ if [ -n "$DO_DOWNLOAD" ]; then
     fi
 
     BOWER=$(pickcmd node_modules/bower/bin/bower bower)
-    $BOWER install
+    #$BOWER install
   popd
 fi
 
index bc2186f4f3bf936d7597dc1a4ab1cf73ed1ec4b6..cab7d8c3d03b74ad8f45aaf7e76d6bee5faebad5 100644 (file)
@@ -71,7 +71,7 @@ function civicrm_write_file($name, &$buffer) {
   if (!$fd) {
     die("Cannot open $name");
   }
-  fputs($fd, $buffer);
+  fwrite($fd, $buffer);
   fclose($fd);
 }
 
index 908a4d37b31042f8490c0bd5a3f6bb6bf8c9c58b..984a0e36b87c9466f8a27bea5c7cd92c2a225a1f 100644 (file)
@@ -1,35 +1,35 @@
 <?php
 
 /**
- * Implements hook_civicrm_install
+ * Implements civicrm_install
  */
 function moduletest_civicrm_install() {
   CRM_Extension_Manager_ModuleTest::incHookCount('moduletest', 'install');
 }
 
 /**
- * Implements hook_civicrm_postInstall
+ * Implements civicrm_postInstall
  */
 function moduletest_civicrm_postInstall() {
   CRM_Extension_Manager_ModuleTest::incHookCount('moduletest', 'postInstall');
 }
 
 /**
- * Implements hook_civicrm_uninstall
+ * Implements civicrm_uninstall
  */
 function moduletest_civicrm_uninstall() {
   CRM_Extension_Manager_ModuleTest::incHookCount('moduletest', 'uninstall');
 }
 
 /**
- * Implements hook_civicrm_enable
+ * Implements civicrm_enable
  */
 function moduletest_civicrm_enable() {
   CRM_Extension_Manager_ModuleTest::incHookCount('moduletest', 'enable');
 }
 
 /**
- * Implements hook_civicrm_disable
+ * Implements civicrm_disable
  */
 function moduletest_civicrm_disable() {
   CRM_Extension_Manager_ModuleTest::incHookCount('moduletest', 'disable');
index 7c6efc32b69d6c1d04305c72377bbdd224fb88f1..83155d3b87ae2874c003e1e8fc211b9883431e77 100644 (file)
@@ -28,12 +28,12 @@ class CRM_Core_Smarty_plugins_CrmScopeTest extends CiviUnitTestCase {
     $cases[] = array('x=1 x=2 x=1', '{crmScope x=1}x={$x} {crmScope x=2}x={$x}{/crmScope} x={$x}{/crmScope}');
     $cases[] = array(
       'x=1 x=2 x=3 x=2 x=1',
-      '{crmScope x=1}x={$x} {crmScope x=2}x={$x} {crmScope x=3}x={$x}{/crmScope} x={$x}{/crmScope} x={$x}{/crmScope}'
+      '{crmScope x=1}x={$x} {crmScope x=2}x={$x} {crmScope x=3}x={$x}{/crmScope} x={$x}{/crmScope} x={$x}{/crmScope}',
     );
     $cases[] = array('x=1,y=9', '{crmScope x=1 y=9}x={$x},y={$y}{/crmScope}');
     $cases[] = array(
       'x=1,y=9 x=1,y=8 x=1,y=9',
-      '{crmScope x=1 y=9}x={$x},y={$y} {crmScope y=8}x={$x},y={$y}{/crmScope} x={$x},y={$y}{/crmScope}'
+      '{crmScope x=1 y=9}x={$x},y={$y} {crmScope y=8}x={$x},y={$y}{/crmScope} x={$x},y={$y}{/crmScope}',
     );
     $cases[] = array('x=', 'x={$x}');
     return $cases;
index 8e67a619ee482748bfc45dc0ea0114a33570e49b..6b545c373505513d529b77c3fc7ce86dc4f0a450 100644 (file)
@@ -92,7 +92,7 @@ class CRM_Member_Import_Parser_MembershipTest extends CiviUnitTestCase {
       'civicrm_membership',
       'civicrm_membership_log',
       'civicrm_contribution',
-      'civicrm_membership_payment'
+      'civicrm_membership_payment',
     );
     $this->quickCleanup($tablesToTruncate);
     $this->relationshipTypeDelete($this->_relationshipTypeId);
index e9cede7ab163e9ade65eee5f14b7db6cd527d31b..75b305ab22525cc89c9a3a36486092f355f4db5e 100644 (file)
@@ -241,8 +241,7 @@ class CRM_Queue_RunnerTest extends CiviUnitTestCase {
    *
    * @return bool
    */
-  static
-  public function _recordValue($taskCtx, $value) {
+  static public function _recordValue($taskCtx, $value) {
     self::$_recordedValues[] = $value;
     return TRUE;
   }
@@ -252,8 +251,7 @@ class CRM_Queue_RunnerTest extends CiviUnitTestCase {
    *
    * @return bool
    */
-  static
-  public function _returnFalse($taskCtx) {
+  static public function _returnFalse($taskCtx) {
     return FALSE;
   }
 
@@ -263,8 +261,7 @@ class CRM_Queue_RunnerTest extends CiviUnitTestCase {
    *
    * @throws Exception
    */
-  static
-  public function _throwException($taskCtx, $value) {
+  static public function _throwException($taskCtx, $value) {
     throw new Exception("Manufactured error: $value");
   }
 
@@ -275,8 +272,7 @@ class CRM_Queue_RunnerTest extends CiviUnitTestCase {
    *
    * @return bool
    */
-  static
-  public function _enqueueNumbers($taskCtx, $low, $high) {
+  static public function _enqueueNumbers($taskCtx, $low, $high) {
     for ($i = $low; $i <= $high; $i++) {
       $taskCtx->queue->createItem(new CRM_Queue_Task(
         array('CRM_Queue_RunnerTest', '_recordValue'),
index f91dd87ff624f54be2534039ceb08c1ffb638206..c67a67a4b0490f776a53761de010cfb53042153e 100644 (file)
@@ -55,7 +55,7 @@ class ImportCiviSeleniumTestCase extends CiviSeleniumTestCase {
    *                             saveMapping        : save current mapping?
    *                             saveMappingName    : to override mapping name
    */
-  function importCSVComponent(
+  public function importCSVComponent(
     $component,
     $headers,
     $rows,
@@ -417,7 +417,7 @@ class ImportCiviSeleniumTestCase extends CiviSeleniumTestCase {
 
     if (!empty($other['callbackImportSummary']) && is_callable(array(
         $this,
-        $other['callbackImportSummary']
+        $other['callbackImportSummary'],
       ))
     ) {
       $callbackImportSummary = $other['callbackImportSummary'];
index 9508c9ca66c6ad674cf34e5fb3c3adc8ba614e4b..c4dbbada1ac887baf13e5512f3b02473bdfb5dd5 100644 (file)
@@ -118,7 +118,7 @@ class WebTest_Member_InheritedMembershipTest extends CiviSeleniumTestCase {
     foreach (array(
                'joinDate',
                'startDate',
-               'endDate'
+               'endDate',
              ) as $date) {
       $$date = CRM_Utils_Date::customFormat($$date, $configVars->dateformatFull);
     }
index 49904ca9410a2bcab535ef9c8059420e1935d57c..64082d036ae5de7fa4b3ba1956a0af6f914668d1 100644 (file)
@@ -1025,7 +1025,6 @@ class api_v3_ContactTest extends CiviUnitTestCase {
    *  Test civicrm_update() Deliberately exclude contact_type as it should still
    *  cope using civicrm_api CRM-7645
    */
-
   public function testUpdateCreateWithID() {
     //  Insert a row in civicrm_contact creating individual contact
     $op = new PHPUnit_Extensions_Database_Operation_Insert();
@@ -1686,8 +1685,8 @@ class api_v3_ContactTest extends CiviUnitTestCase {
   }
 
   /**
-    * Test id only format
-    */
+   * Test id only format
+   */
   public function testContactGetFormatID_only() {
     $this->createContactFromXML();
     $description = "This demonstrates use of the 'format.id_only' param.
@@ -1701,8 +1700,8 @@ class api_v3_ContactTest extends CiviUnitTestCase {
   }
 
   /**
-    * Test id only format
-    */
+   * Test id only format
+   */
   public function testContactGetFormatSingleValue() {
     $this->createContactFromXML();
     $description = "This demonstrates use of the 'format.single_value' param.
index 356aeac4ee2bc549284f021533488bd9582681d7..0f5662e986d9fe33bdf83733ca8d70e418f958c7 100644 (file)
@@ -110,8 +110,9 @@ class api_v3_EventTest extends CiviUnitTestCase {
     $this->quickCleanup($tablesToTruncate, TRUE);
   }
 
-  ///////////////// civicrm_event_get methods
-
+  /**
+   * civicrm_event_get methods
+   */
   public function testGetEventById() {
     $params = array(
       'id' => $this->_events[1]['id'],
@@ -252,7 +253,7 @@ class api_v3_EventTest extends CiviUnitTestCase {
         'participant_status' => 1,
         'role_id' => 1,
         'contact_id' => $contactID,
-        'event_id' => $this->_eventIds[0]
+        'event_id' => $this->_eventIds[0],
       ));
     $currentEvent = $this->callAPIAndDocument('Event', 'getsingle', $getEventParams, __FUNCTION__, __FILE__, $description, $subfile, 'getsingle');
     $this->assertEquals(1, $currentEvent['is_full'], ' is full is set in line ' . __LINE__);
@@ -261,7 +262,7 @@ class api_v3_EventTest extends CiviUnitTestCase {
     $this->contactDelete($contactID);
   }
 
-  /*
+  /**
    * Legacy support for Contribution Type ID. We need to ensure this is supported
    * as an alias for financial_type_id
    */
@@ -295,7 +296,7 @@ class api_v3_EventTest extends CiviUnitTestCase {
 
     $check = $this->callAPISuccess($this->_entity, 'get', array(
         'return.custom_' . $ids['custom_field_id'] => 1,
-        'id' => $result['id']
+        'id' => $result['id'],
       ));
     $this->assertEquals("custom string", $check['values'][$check['id']]['custom_' . $ids['custom_field_id']], ' in line ' . __LINE__);
 
index 1a6a23767c4a2e15032be00f815b5090cd204692..3d17b76cbadb47af4d9edb0a208a0af1b49a46c6 100644 (file)
@@ -55,22 +55,22 @@ class api_v3_ParticipantPaymentTest extends CiviUnitTestCase {
 
     $this->_participantID = $this->participantCreate(array(
         'contactID' => $this->_contactID,
-        'eventID' => $this->_eventID
+        'eventID' => $this->_eventID,
       ));
     $this->_contactID2 = $this->individualCreate();
     $this->_participantID2 = $this->participantCreate(array(
         'contactID' => $this->_contactID2,
-        'eventID' => $this->_eventID
+        'eventID' => $this->_eventID,
       ));
     $this->_participantID3 = $this->participantCreate(array(
         'contactID' => $this->_contactID2,
-        'eventID' => $this->_eventID
+        'eventID' => $this->_eventID,
       ));
 
     $this->_contactID3 = $this->individualCreate();
     $this->_participantID4 = $this->participantCreate(array(
         'contactID' => $this->_contactID3,
-        'eventID' => $this->_eventID
+        'eventID' => $this->_eventID,
       ));
   }
 
index de6feff96ea8176905b19b79562cba73698d90ed..a748e5029b8f113026714508839a5678f9f1b5c5 100644 (file)
@@ -64,16 +64,16 @@ class api_v3_ParticipantTest extends CiviUnitTestCase {
 
     $this->_participantID = $this->participantCreate(array(
         'contact_id' => $this->_contactID,
-        'event_id' => $this->_eventID
+        'event_id' => $this->_eventID,
       ));
     $this->_contactID2 = $this->individualCreate();
     $this->_participantID2 = $this->participantCreate(array(
         'contact_id' => $this->_contactID2,
-        'event_id' => $this->_eventID
+        'event_id' => $this->_eventID,
       ));
     $this->_participantID3 = $this->participantCreate(array(
         'contact_id' => $this->_contactID2,
-        'event_id' => $this->_eventID
+        'event_id' => $this->_eventID,
       ));
     $this->_params = array(
       'contact_id' => $this->_contactID,
@@ -190,7 +190,7 @@ class api_v3_ParticipantTest extends CiviUnitTestCase {
     $event = $this->eventCreate(NULL);
     $this->callAPISuccess('participant', 'create', array(
         'event_id' => $event['id'],
-        'contact_id' => $this->_contactID
+        'contact_id' => $this->_contactID,
       ));
 
     $description = "use nested get to get an event";
index dbd8f607d5cced29942b63f0012ebf5ef2b49603..aa4e8573688ff26432645ae063a6a2f4c580c86f 100644 (file)
@@ -717,7 +717,7 @@ class api_v3_RelationshipTest extends CiviUnitTestCase {
     $rel2 = $this->callAPISuccess('relationship', 'create', $rel2Params);
   }
 
-  /*
+  /**
    * Test using various operators
    */
   public function testGetTypeOperators() {
index 3e89285651db21494cf2e8660793f09b8fac9dc8..f7d02211d42084fc29263bfe54fdf7cfaaf91d7e 100644 (file)
@@ -173,14 +173,10 @@ class api_v3_ReportTemplateTest extends CiviUnitTestCase {
   public static function getReportTemplates() {
     $reportsToSkip = array(
       'activity' => 'does not respect function signature on from clause',
-      'walklist' => 'Notice: Undefined index: type in CRM_Report_Form_Walklist_Walklist line 155.
-                       (suspect the select function should be removed in favour of the parent (state province field)
-                      also, type should be added to state province & others? & potentially getAddressColumns fn should be
-                      used per other reports',
+      'walklist' => 'Notice: Undefined index: type in CRM_Report_Form_Walklist_Walklist line 155.(suspect the select function should be removed in favour of the parent (state province field) also, type should be added to state province & others? & potentially getAddressColumns fn should be used per other reports',
       'contribute/repeat' => 'Reports with important functionality in postProcess are not callable via the api. For variable setting recommend beginPostProcessCommon, for temp table creation recommend From fn',
       'contribute/topDonor' => 'construction of query in postProcess makes inaccessible ',
-      'contribute/sybunt' => 'e notice - (ui gives fatal error at civicrm/report/contribute/sybunt&reset=1&force=1
-                                e-notice is on yid_valueContribute/Sybunt.php(214) because at the force url "yid_relative" not "yid_value" is defined',
+      'contribute/sybunt' => 'e notice - (ui gives fatal error at civicrm/report/contribute/sybunt&reset=1&force=1 e-notice is on yid_valueContribute/Sybunt.php(214) because at the force url "yid_relative" not "yid_value" is defined',
       'contribute/lybunt' => 'same as sybunt - fatals on force url & test identifies why',
       'event/income' => 'I do no understand why but error is Call to undefined method CRM_Report_Form_Event_Income::from() in CRM/Report/Form.php on line 2120',
       'contact/relationship' => '(see contribute/repeat), property declaration issue, Undefined property: CRM_Report_Form_Contact_Relationship::$relationType in /Contact/Relationship.php(486):',