INFRA-132 - CRM/Case - phpcbf
authorTim Otten <totten@civicrm.org>
Tue, 6 Jan 2015 21:09:19 +0000 (13:09 -0800)
committerTim Otten <totten@civicrm.org>
Tue, 6 Jan 2015 23:04:26 +0000 (15:04 -0800)
21 files changed:
CRM/Case/Audit/Audit.php
CRM/Case/BAO/Case.php
CRM/Case/BAO/CaseType.php
CRM/Case/BAO/Query.php
CRM/Case/Form/Activity.php
CRM/Case/Form/Activity/ChangeCaseStatus.php
CRM/Case/Form/Activity/ChangeCaseType.php
CRM/Case/Form/Activity/LinkCases.php
CRM/Case/Form/Case.php
CRM/Case/Form/CaseView.php
CRM/Case/Form/Report.php
CRM/Case/Form/Search.php
CRM/Case/Form/Task/Result.php
CRM/Case/Info.php
CRM/Case/Page/DashBoard.php
CRM/Case/Page/Tab.php
CRM/Case/PseudoConstant.php
CRM/Case/Selector/Search.php
CRM/Case/XMLProcessor/Process.php
CRM/Case/XMLProcessor/Report.php
CRM/Case/XMLRepository.php

index 8517a7098c3abb58797d2b43e703727418631288..afbbecdc158160f11a5d02ee634290d8221a4f43 100644 (file)
@@ -220,10 +220,10 @@ class CRM_Case_Audit_Audit {
   static
   public function run($xmlString, $clientID, $caseID, $printReport = FALSE) {
     /*
-$fh = fopen('C:/temp/audit2.xml', 'w');
-fwrite($fh, $xmlString);
-fclose($fh);
-*/
+    $fh = fopen('C:/temp/audit2.xml', 'w');
+    fwrite($fh, $xmlString);
+    fclose($fh);
+    */
 
     $audit = new CRM_Case_Audit_Audit($xmlString, 'audit.conf.xml');
     $activities = $audit->getActivities($printReport);
index 14c84b89f138ba9eabe804c59a02445ed68341a7..987e089196255541e92f3d07f01ff5166fad18e3 100644 (file)
@@ -749,7 +749,6 @@ LEFT JOIN civicrm_option_group aog ON aog.name='activity_type'
       $allCases = FALSE;
     }
 
-
     $condition = " AND civicrm_case.is_deleted = 0 ";
 
     if (!$allCases) {
@@ -809,7 +808,6 @@ AND civicrm_case.status_id != $closedId";
     // we're going to use the usual actions, so doesn't make sense to duplicate definitions
     $actions = CRM_Case_Selector_Search::links();
 
-
     // check is the user has view/edit signer permission
     $permissions = array(CRM_Core_Permission::VIEW);
     if (CRM_Core_Permission::check('access all cases and activities') ||
@@ -826,8 +824,7 @@ AND civicrm_case.status_id != $closedId";
       foreach ($resultFields as $donCare => $field) {
         $casesList[$result->case_id][$field] = $result->$field;
         if ($field == 'contact_type') {
-          $casesList[$result->case_id]['contact_type_icon'] = CRM_Contact_BAO_Contact_Utils::getImage($result->contact_sub_type ?
-              $result->contact_sub_type : $result->contact_type
+          $casesList[$result->case_id]['contact_type_icon'] = CRM_Contact_BAO_Contact_Utils::getImage($result->contact_sub_type ? $result->contact_sub_type : $result->contact_type
           );
           $casesList[$result->case_id]['action'] = CRM_Core_Action::formLink($actions['primaryActions'], $mask,
             array(
@@ -999,7 +996,6 @@ SELECT case_status.label AS case_status, status_id, civicrm_case_type.title AS c
  LEFT JOIN  civicrm_email ON (civicrm_email.contact_id = civicrm_contact.id )
      WHERE  civicrm_relationship.contact_id_a = %1 AND civicrm_relationship.case_id = %2';
 
-
     $params = array(
       1 => array($contactID, 'Positive'),
       2 => array($caseID, 'Positive'),
@@ -1179,7 +1175,6 @@ SELECT case_status.label AS case_status, status_id, civicrm_case_type.title AS c
     //EXIT;
     $dao = CRM_Core_DAO::executeQuery($query, $params);
 
-
     $activityTypes = CRM_Case_PseudoConstant::caseActivityType(FALSE, TRUE);
     $activityStatus = CRM_Core_PseudoConstant::activityStatus();
     $activityPriority = CRM_Core_PseudoConstant::get('CRM_Activity_DAO_Activity', 'priority_id');
@@ -1342,7 +1337,6 @@ SELECT case_status.label AS case_status, status_id, civicrm_case_type.title AS c
         }
       }
 
-
       $values[$dao->id]['links'] = $url;
       $values[$dao->id]['class'] = "";
 
index e57e0cb4511759e2dfde40aa6d1101b846889546..699ec1ca21b3a7722061df82ea9dd4ca7d96b170 100644 (file)
@@ -89,7 +89,7 @@ class CRM_Case_BAO_CaseType extends CRM_Case_DAO_CaseType {
   }
 
   protected function assignTestValue($fieldName, &$fieldDef, $counter) {
-    if ($fieldName  == 'definition') {
+    if ($fieldName == 'definition') {
       $this->{$fieldName} = "<CaseType><name>TestCaseType{$counter}</name></CaseType>";
     } else {
       parent::assignTestValue($fieldName, $fieldDef, $counter);
@@ -145,12 +145,14 @@ class CRM_Case_BAO_CaseType extends CRM_Case_DAO_CaseType {
                 $xmlFile .= "</ActivityTypes>\n";
               }
               break;
+
             case 'sequence': // passthrough
             case 'timeline':
               if ($setVal) {
                 $xmlFile .= "<{$index}>true</{$index}>\n";
               }
               break;
+
             default:
               $xmlFile .= "<{$index}>{$setVal}</{$index}>\n";
           }
@@ -347,7 +349,7 @@ class CRM_Case_BAO_CaseType extends CRM_Case_DAO_CaseType {
    * @return bool
    */
   public static function isValidName($caseType) {
-    return preg_match('/^[a-zA-Z0-9_]+$/',  $caseType);
+    return preg_match('/^[a-zA-Z0-9_]+$/', $caseType);
   }
 
   /**
@@ -378,7 +380,7 @@ class CRM_Case_BAO_CaseType extends CRM_Case_DAO_CaseType {
       // if file-based definition explicitly disables "forkable" option, then don't allow changes to definition
       $fileDefinition = CRM_Case_XMLRepository::singleton()->retrieveFile($caseTypeName);
       if ($fileDefinition && isset($fileDefinition->forkable)) {
-        return CRM_Utils_String::strtobool((string)$fileDefinition->forkable);
+        return CRM_Utils_String::strtobool((string) $fileDefinition->forkable);
       }
     }
     return TRUE;
index 2f460c9a7b40c6d350156ba626835678cf8a3f7e..0890a48650268b8a405c287418f4cbdb4ac461fe 100644 (file)
@@ -718,7 +718,7 @@ case_relation_type.id = case_relationship.relationship_type_id )";
     }
 
     CRM_Core_Form_Date::buildDateRange($form, 'case_from', 1, '_start_date_low', '_start_date_high', ts('From'), FALSE);
-    CRM_Core_Form_Date::buildDateRange($form, 'case_to',   1, '_end_date_low',   '_end_date_high',   ts('From'), FALSE);
+    CRM_Core_Form_Date::buildDateRange($form, 'case_to', 1, '_end_date_low', '_end_date_high', ts('From'), FALSE);
 
     $form->assign('validCiviCase', TRUE);
 
@@ -770,5 +770,6 @@ case_relation_type.id = case_relationship.relationship_type_id )";
    * @param $row
    * @param int $id
    */
-  public static function searchAction(&$row, $id) {}
+  public static function searchAction(&$row, $id) {
+  }
 }
index ada4d36d115612fee572f09c1e4c0dfec5974e92..43c05b0d4f6f6b22736d75fbda464d00be9fe98f 100644 (file)
@@ -319,14 +319,14 @@ class CRM_Case_Form_Activity extends CRM_Activity_Form_Activity {
       $this->_relatedContacts[] = $rgc = CRM_Case_BAO_Case::getRelatedAndGlobalContacts($val);
       $contName = CRM_Case_BAO_Case::getContactNames($val);
       foreach ($contName as $nkey => $nval) {
-        array_push($this->_relatedContacts[$i][0] , $this->_relatedContacts[$i][0]['managerOf']= $nval['display_name']);
+        array_push($this->_relatedContacts[$i][0], $this->_relatedContacts[$i][0]['managerOf'] = $nval['display_name']);
       }
       $i++;
     }
 
     //add case client in send a copy selector.CRM-4438.
     foreach ($this->_caseId as $key => $val) {
-      $relatedContacts[] = $relCon= CRM_Case_BAO_Case::getContactNames($val);
+      $relatedContacts[] = $relCon = CRM_Case_BAO_Case::getContactNames($val);
     }
 
     if (!empty($relatedContacts)) {
@@ -558,7 +558,7 @@ class CRM_Case_Form_Activity extends CRM_Activity_Form_Activity {
       foreach ($this->_caseId as $key => $val) {
         $newActParams['case_id'] = $val;
         $activity = CRM_Activity_BAO_Activity::create($newActParams);
-        $vvalue[] = array('case_id'=> $val, 'actId'=> $activity->id);
+        $vvalue[] = array('case_id' => $val, 'actId' => $activity->id);
         // call end post process, after the activity has been created/updated.
         $this->endPostProcess($newActParams, $activity);
       }
index 21fda360de36af1cba489d61b32190589e477292..5b9d506865ef6758dae0cc279149d23f876c039a 100644 (file)
@@ -152,7 +152,8 @@ class CRM_Case_Form_Activity_ChangeCaseStatus {
         // FIXME: Is there an existing function to close a relationship?
         $query = 'UPDATE civicrm_relationship SET end_date=%2 WHERE id=%1';
         foreach ($rels as $relId => $relData) {
-          $relParams = array(1 => array($relId, 'Integer'),
+          $relParams = array(
+          1 => array($relId, 'Integer'),
             2 => array($params['end_date'], 'Timestamp'),
           );
           CRM_Core_DAO::executeQuery($query, $relParams);
index 0d2150370140746c0cc0b84ab4e6435eb5d56cde..0fbc58421d5dff35f504a518fb8b29a38ad8dbee 100644 (file)
@@ -174,7 +174,8 @@ class CRM_Case_Form_Activity_ChangeCaseType {
 
     if ($activity->subject == 'null') {
       $activity->subject = ts('Case type changed from %1 to %2',
-        array(1 => CRM_Utils_Array::value($form->_defaults['case_type_id'], $allCaseTypes),
+        array(
+      1 => CRM_Utils_Array::value($form->_defaults['case_type_id'], $allCaseTypes),
           2 => CRM_Utils_Array::value($params['case_type_id'], $allCaseTypes),
         )
       );
index ef335cb5d8e599b17e2c4fdd6be0a8e8dfe3bdd5..0539ec4775510efa064fa6a6c56b334d675929b3 100644 (file)
@@ -128,7 +128,8 @@ class CRM_Case_Form_Activity_LinkCases {
    *
    * @return void
    */
-  public static function beginPostProcess(&$form, &$params) {}
+  public static function beginPostProcess(&$form, &$params) {
+  }
 
   /**
    * Process the form submission
index 925e1122b1c5598dc89fbf447cbca2292878e151..959f97a4a798565b4a9004121c656394f8b35945 100644 (file)
@@ -121,7 +121,8 @@ class CRM_Case_Form_Case extends CRM_Core_Form {
     }
 
     if (!$this->_caseId) {
-      $caseAttributes = array('case_type' => CRM_Case_PseudoConstant::caseType(),
+      $caseAttributes = array(
+      'case_type' => CRM_Case_PseudoConstant::caseType(),
         'case_status' => CRM_Case_PseudoConstant::caseStatus(),
         'encounter_medium' => CRM_Case_PseudoConstant::encounterMedium(),
       );
@@ -181,7 +182,6 @@ class CRM_Case_Form_Case extends CRM_Core_Form {
       $this->assign('clientName', $contact->display_name);
     }
 
-
     $session = CRM_Core_Session::singleton();
     $this->_currentUserId = $session->get('userID');
 
@@ -364,7 +364,6 @@ class CRM_Case_Form_Case extends CRM_Core_Form {
     $params = $this->controller->exportValues($this->_name);
     $params['now'] = date("Ymd");
 
-
     // 1. call begin post process
     if ($this->_activityTypeFile) {
       $className = "CRM_Case_Form_Activity_{$this->_activityTypeFile}";
index a21f4585f2d6abd515f30e849a9c4f5d5a82eb3b..ccdf0df41dabc94732c51db65a91ed5e88c93983 100644 (file)
@@ -159,7 +159,6 @@ class CRM_Case_Form_CaseView extends CRM_Core_Form {
       $recentOther
     );
 
-
     //get the related cases for given case.
     $relatedCases = $this->get('relatedCases');
     if (!isset($relatedCases)) {
@@ -259,7 +258,7 @@ class CRM_Case_Form_CaseView extends CRM_Core_Form {
         );
       }
       else if ($type == $pdfActivityType ) {
-         $url = CRM_Utils_System::url('civicrm/activity/pdf/add',
+        $url = CRM_Utils_System::url('civicrm/activity/pdf/add',
           "action=add&context=standalone&reset=1&cid={$this->_contactID}&caseid={$this->_caseID}&atype=$type",
           FALSE, NULL, FALSE );
       }
index 10798d66abd1fd9ce3b7e3376b362baf23c06ecf..e563b211d95c9d6bae6a08844a94e82ddede57ab 100644 (file)
@@ -84,7 +84,8 @@ class CRM_Case_Form_Report extends CRM_Core_Form {
       return;
     }
 
-    $includeActivites = array(1 => ts('All Activities'),
+    $includeActivites = array(
+    1 => ts('All Activities'),
       2 => ts('Exclude Completed Activities'),
     );
     $includeActivitesGroup = $this->addRadio('include_activities',
index 02eba915bc7b3738b7f21b8bb87b56a2a51c972d..b6382ed17127cf7f10d04917a145445af2a27001 100644 (file)
@@ -135,7 +135,6 @@ class CRM_Case_Form_Search extends CRM_Core_Form_Search {
       );
     }
 
-
     $this->_queryParams = CRM_Contact_BAO_Query::convertFormValues($this->_formValues);
     $selector = new CRM_Case_Selector_Search($this->_queryParams,
       $this->_action,
@@ -405,7 +404,7 @@ class CRM_Case_Form_Search extends CRM_Core_Form_Search {
       }
     }
     else {
-        // First, if "all" is stored in the session, default to all cases, otherwise default to no selection.
+      // First, if "all" is stored in the session, default to all cases, otherwise default to no selection.
       $session = CRM_Core_Session::singleton();
       if (CRM_Utils_Request::retrieve('all', 'Positive', $session)) {
         $this->_formValues['case_owner'] = 1;
index 5289241c0bb2ee9a66717b3812bfcc48c82c566c..c18819d722c681f3707e2d45d78d6ed68dcc916e 100644 (file)
@@ -45,7 +45,8 @@ class CRM_Case_Form_Task_Result extends CRM_Case_Form_Task {
    *
    * @return void
    */
-  public function preProcess() {}
+  public function preProcess() {
+  }
 
   /**
    * Build the form object
index 6dd43cecbe329c12474f1478eab93fc2df9ec5ef..1b0db3c88feef319e9dcc65f5eef3cc4c017bf16 100644 (file)
@@ -152,7 +152,8 @@ class CRM_Case_Info extends CRM_Core_Component_Info {
    * @return array
    */
   public function registerTab() {
-    return array('title' => ts('Cases'),
+    return array(
+    'title' => ts('Cases'),
       'url' => 'case',
       'weight' => 50,
     );
@@ -163,7 +164,8 @@ class CRM_Case_Info extends CRM_Core_Component_Info {
    * @return array
    */
   public function registerAdvancedSearchPane() {
-    return array('title' => ts('Cases'),
+    return array(
+    'title' => ts('Cases'),
       'weight' => 50,
     );
   }
@@ -190,7 +192,8 @@ class CRM_Case_Info extends CRM_Core_Component_Info {
       );
       if ($atype) {
         $shortCuts = array_merge($shortCuts, array(
-          array('path' => 'civicrm/case/add',
+          array(
+        'path' => 'civicrm/case/add',
               'query' => "reset=1&action=add&atype=$atype&context=standalone",
               'ref' => 'new-case',
               'title' => ts('Case'),
@@ -218,7 +221,7 @@ class CRM_Case_Info extends CRM_Core_Component_Info {
       &&
       (!$oldValue || !in_array('CiviCase', $oldValue))
     ) {
-      $pathToCaseSampleTpl =  __DIR__ . '/xml/configuration.sample/';
+      $pathToCaseSampleTpl = __DIR__ . '/xml/configuration.sample/';
       $config = CRM_Core_Config::singleton();
       CRM_Admin_Form_Setting_Component::loadCaseSampleData($config->dsn, $pathToCaseSampleTpl . 'case_sample.mysql.tpl');
       if (!CRM_Case_BAO_Case::createCaseViews()) {
index fa66d50816e909a5fa0383d6c674afd634ae9c6f..9d9ac409aa7c39c77f079482908ebf3dfbaa7fa1 100644 (file)
@@ -90,7 +90,7 @@ class CRM_Case_Page_DashBoard extends CRM_Core_Page {
     $upcoming = CRM_Case_BAO_Case::getCases($allCases, $userID, 'upcoming');
     $recent   = CRM_Case_BAO_Case::getCases($allCases, $userID, 'recent');
 
-    foreach($upcoming as $key=>$value) {
+    foreach($upcoming as $key => $value) {
       if(strtotime($value['case_scheduled_activity_date']) < time()) {
         $upcoming[$key]['activity_status'] = 'status-overdue';
       }
index b9314f5ac8812aea8253cfff4b62dbc266c3c2fa..279c025beff45c88be970ecc0c8c1320e0109962 100644 (file)
@@ -130,7 +130,6 @@ class CRM_Case_Page_Tab extends CRM_Core_Page {
         $this->_id
       );
 
-
     $controller->setEmbedded(TRUE);
 
     $controller->run();
@@ -206,8 +205,7 @@ class CRM_Case_Page_Tab extends CRM_Core_Page {
     if ($this->_action & CRM_Core_Action::VIEW) {
       $this->view();
     }
-    elseif (($this->_action &
-        (CRM_Core_Action::UPDATE | CRM_Core_Action::ADD |
+    elseif (($this->_action & (CRM_Core_Action::UPDATE | CRM_Core_Action::ADD |
           CRM_Core_Action::DELETE | CRM_Core_Action::RENEW
         )
       ) ||
index 04c6eef8dbdf7439adf7cfc3a11196aa7f353581..d22e4d85240aee08b3cba019643ea35d534bccbd 100644 (file)
@@ -87,7 +87,7 @@ class CRM_Case_PseudoConstant extends CRM_Core_PseudoConstant {
    * @static
    */
   public static function caseStatus($column = 'label', $onlyActive = TRUE, $condition = NULL, $fresh = FALSE) {
-    $cacheKey = "{$column}_" . (int)$onlyActive;
+    $cacheKey = "{$column}_" . (int) $onlyActive;
     if (!$condition) {
       $condition = 'AND filter = 0';
     }
@@ -174,7 +174,7 @@ class CRM_Case_PseudoConstant extends CRM_Core_PseudoConstant {
    * @static
    */
   public static function encounterMedium($column = 'label', $onlyActive = TRUE) {
-    $cacheKey = "{$column}_" . (int)$onlyActive;
+    $cacheKey = "{$column}_" . (int) $onlyActive;
     if (!isset(self::$encounterMedium[$cacheKey])) {
       self::$encounterMedium[$cacheKey] = CRM_Core_OptionGroup::values('encounter_medium',
         FALSE, FALSE, FALSE, NULL,
@@ -255,7 +255,7 @@ class CRM_Case_PseudoConstant extends CRM_Core_PseudoConstant {
    * @param bool|string $name pseudoconstant to be flushed
    */
   public static function flush($name = 'cache') {
-   if (isset(self::$$name)) {
+    if (isset(self::$$name)) {
       self::$$name = NULL;
     }
   }
index d6e9159cb59813c01a63d809f129239459b5eb04..10af1d82a07d058dcc03c40c27e363829c633e1d 100644 (file)
@@ -146,11 +146,11 @@ class CRM_Case_Selector_Search extends CRM_Core_Selector_Base {
   @access public
    */
   function __construct(&$queryParams,
-    $action           = CRM_Core_Action::NONE,
+    $action = CRM_Core_Action::NONE,
     $additionalClause = NULL,
-    $single           = FALSE,
-    $limit            = NULL,
-    $context          = 'search'
+    $single = FALSE,
+    $limit = NULL,
+    $context = 'search'
   ) {
     // submitted form values
     $this->_queryParams = &$queryParams;
@@ -369,8 +369,7 @@ class CRM_Case_Selector_Search extends CRM_Core_Selector_Base {
         $result->case_id
       );
 
-      $row['contact_type'] = CRM_Contact_BAO_Contact_Utils::getImage($result->contact_sub_type ?
-        $result->contact_sub_type : $result->contact_type
+      $row['contact_type'] = CRM_Contact_BAO_Contact_Utils::getImage($result->contact_sub_type ? $result->contact_sub_type : $result->contact_type
       );
 
       //adding case manager to case selector.CRM-4510.
index e3bf4fd41bea37a221dbe2505e660b5fb00b2786..8eaadbba6a33132a8ca7e6cbef98915262fbdb69 100644 (file)
@@ -389,7 +389,8 @@ AND        ca.case_id = %2
 AND        a.is_deleted = 0
 ";
 
-    $sqlParams = array(1 => array($params['activityTypeID'], 'Integer'),
+    $sqlParams = array(
+    1 => array($params['activityTypeID'], 'Integer'),
       2 => array($params['caseID'], 'Integer'),
     );
     $count = CRM_Core_DAO::singleValueQuery($query, $sqlParams);
index 5ee01eef3cb376092223d31e672dc7891f0a0aeb..f2ebe640f9402aa1c09c36de6b287256ec762d60 100644 (file)
@@ -44,7 +44,8 @@ class CRM_Case_XMLProcessor_Report extends CRM_Case_XMLProcessor {
   /**
    *
    */
-  public function __construct() {}
+  public function __construct() {
+  }
 
   /**
    * @param int $clientID
@@ -250,14 +251,12 @@ AND    ac.case_id = %1
       $this->getRedactionRules();
     }
 
-
     $index = $activityID . '_' . (int) $anyActivity;
 
     if ($clientID) {
       $index = $index . '_' . $clientID;
     }
 
-
     if (!array_key_exists($index, $activityInfos)) {
       $activityInfos[$index] = array();
       $selectCaseActivity = "";
@@ -788,7 +787,8 @@ LIMIT  1
     }
 
     // next get activity set Informtion
-    $activitySet = array('label' => $form->getActivitySetLabel($xml, $activitySetName),
+    $activitySet = array(
+    'label' => $form->getActivitySetLabel($xml, $activitySetName),
       'includeActivities' => 'All',
       'redact' => 'false',
     );
index 5710603aa89ba0d442ddeae0090c11d1c2f1ddb7..0be96d708bf9c6ff888ccbd6aad8dd9d02e39253 100644 (file)
@@ -139,7 +139,8 @@ class CRM_Case_XMLRepository {
    * @param string $caseType
    * @return null|string file path
    */
-  public function findXmlFile($caseType) { // first check custom templates directory
+  public function findXmlFile($caseType) {
+    // first check custom templates directory
     $fileName = NULL;
 
     if (!$fileName || !file_exists($fileName)) {