fixed query for CRM-13147
authoryashodha <yashodha.chaku@webaccess.co.in>
Thu, 1 Aug 2013 13:38:28 +0000 (19:08 +0530)
committeryashodha <yashodha.chaku@webaccess.co.in>
Thu, 1 Aug 2013 13:38:28 +0000 (19:08 +0530)
CRM/Case/BAO/Case.php

index 8e7f197e1d3a4d4db2fdf491083c2edd79ee3e1b..904e0bee92ecb874972c1443dde3da9c624f0f59 100644 (file)
@@ -46,6 +46,7 @@ class CRM_Case_BAO_Case extends CRM_Case_DAO_Case {
    * @static
    */
   static $_exportableFields = NULL;
+
   function __construct() {
     parent::__construct();
   }
@@ -57,7 +58,7 @@ class CRM_Case_BAO_Case extends CRM_Case_DAO_Case {
    * case object. the params array could contain additional unused name/value
    * pairs
    *
-   * @param array  $params (reference ) an assoc array of name/value pairs
+   * @param array $params (reference ) an assoc array of name/value pairs
    * @param array $ids    the array that holds all the db ids
    *
    * @return object CRM_Case_BAO_Case object
@@ -321,7 +322,7 @@ class CRM_Case_BAO_Case extends CRM_Case_DAO_Case {
    * Delete the record that are associated with this case
    * record are deleted from case
    *
-   * @param  int  $caseId id of the case to delete
+   * @param  int $caseId id of the case to delete
    *
    * @return void
    * @access public
@@ -373,12 +374,12 @@ class CRM_Case_BAO_Case extends CRM_Case_DAO_Case {
   /**
    * Function to enable disable case related relationships
    *
-   *  @param int      $caseId case id
-   *  @param boolean  $enable action
+   * @param int $caseId case id
+   * @param boolean $enable action
    *
-   *  @return void
-   *  @access public
-   *  @static
+   * @return void
+   * @access public
+   * @static
    */
   static function enableDisableCaseRelationships($caseId, $enable) {
     $contactIds = self::retrieveContactIdsByCaseId($caseId);
@@ -387,8 +388,8 @@ class CRM_Case_BAO_Case extends CRM_Case_DAO_Case {
         $roles = self::getCaseRoles($cid, $caseId);
         if (!empty($roles)) {
           $relationshipIds = implode(',', array_keys($roles));
-          $enable          = (int)$enable;
-          $query           = "UPDATE civicrm_relationship SET is_active = {$enable}
+          $enable = (int) $enable;
+          $query = "UPDATE civicrm_relationship SET is_active = {$enable}
                         WHERE id IN ( {$relationshipIds} )";
           CRM_Core_DAO::executeQuery($query);
         }
@@ -399,7 +400,7 @@ class CRM_Case_BAO_Case extends CRM_Case_DAO_Case {
   /**
    * Delete the activities related to case
    *
-   * @param  int  $activityId id of the activity
+   * @param  int $activityId id of the activity
    *
    * @return void
    * @access public
@@ -414,7 +415,7 @@ class CRM_Case_BAO_Case extends CRM_Case_DAO_Case {
   /**
    * Retrieve contact_id by case_id
    *
-   * @param int    $caseId  ID of the case
+   * @param int $caseId  ID of the case
    *
    * @return array
    * @access public
@@ -461,7 +462,7 @@ class CRM_Case_BAO_Case extends CRM_Case_DAO_Case {
   /**
    * Retrieve contact names by caseId
    *
-   * @param int    $caseId  ID of the case
+   * @param int $caseId  ID of the case
    *
    * @return array
    *
@@ -506,7 +507,7 @@ class CRM_Case_BAO_Case extends CRM_Case_DAO_Case {
   /**
    * Retrieve case_id by contact_id
    *
-   * @param int     $contactId      ID of the contact
+   * @param int $contactId      ID of the contact
    * @param boolean $includeDeleted include the deleted cases in result
    *
    * @return array
@@ -669,11 +670,11 @@ LEFT JOIN civicrm_option_group aog ON aog.name='activity_type'
    * Retrieve cases related to particular contact or whole contact
    * used in Dashboad and Tab
    *
-   * @param boolean    $allCases
+   * @param boolean $allCases
    *
-   * @param int        $userID
+   * @param int $userID
    *
-   * @param String     $type /upcoming,recent,all/
+   * @param String $type /upcoming,recent,all/
    *
    * @return array     Array of Cases
    *
@@ -842,9 +843,9 @@ AND civicrm_case.status_id != $closedId";
       $allCases = FALSE;
     }
 
-    $caseTypes    = CRM_Case_PseudoConstant::caseType();
+    $caseTypes = CRM_Case_PseudoConstant::caseType();
     $caseStatuses = CRM_Case_PseudoConstant::caseStatus();
-    $caseTypes    = array_flip($caseTypes);
+    $caseTypes = array_flip($caseTypes);
 
     // get statuses as headers for the table
     $url = CRM_Utils_System::url('civicrm/case/search', "reset=1&force=1&all=1&status=");
@@ -863,10 +864,10 @@ AND civicrm_case.status_id != $closedId";
       $case_owner = 1;
     }
     else {
-      $all               = 0;
+      $all = 0;
       $case_owner = 2;
       $myCaseWhereClause = " AND case_relationship.contact_id_b = {$userID}";
-      $myGroupByClause   = " GROUP BY CONCAT(case_relationship.case_id,'-',case_relationship.contact_id_b)";
+      $myGroupByClause = " GROUP BY CONCAT(case_relationship.case_id,'-',case_relationship.contact_id_b)";
     }
 
     $seperator = CRM_Core_DAO::VALUE_SEPARATOR;
@@ -908,8 +909,8 @@ SELECT case_status.label AS case_status, status_id, case_type.label AS case_type
   /**
    * Function to get Case roles
    *
-   * @param int   $contactID contact id
-   * @param int   $caseID case id
+   * @param int $contactID contact id
+   * @param int $caseID case id
    * @return returns case role / relationships
    *
    * @static
@@ -963,9 +964,9 @@ SELECT case_status.label AS case_status, status_id, case_type.label AS case_type
   /**
    * Function to get Case Activities
    *
-   * @param int    $caseID case id
-   * @param array  $params posted params
-   * @param int    $contactID contact id
+   * @param int $caseID case id
+   * @param array $params posted params
+   * @param int $contactID contact id
    *
    * @return returns case activities
    *
@@ -976,6 +977,7 @@ SELECT case_status.label AS case_status, status_id, case_type.label AS case_type
 
     $activityContacts = CRM_Core_OptionGroup::values('activity_contacts', FALSE, FALSE, FALSE, NULL, 'name');
     $assigneeID = CRM_Utils_Array::key('Activity Assignees', $activityContacts);
+    $sourceID = CRM_Utils_Array::key('Activity Source', $activityContacts);
     $targetID = CRM_Utils_Array::key('Activity Targets', $activityContacts);
 
     // CRM-5081 - formatting the dates to omit seconds.
@@ -999,19 +1001,20 @@ SELECT case_status.label AS case_status, status_id, case_type.label AS case_type
                           ca.weight as weight,
                           GROUP_CONCAT(ef.file_id) as attachment_ids ";
 
-    $from = 'FROM civicrm_case_activity cca
+    $from = "
+      FROM civicrm_case_activity cca
                   INNER JOIN civicrm_activity ca ON ca.id = cca.activity_id
-                  INNER JOIN civicrm_activity_contact cac ON cac.activity_id = ca.id
+                  INNER JOIN civicrm_activity_contact cac ON cac.activity_id = ca.id AND cac.record_type_id = {$sourceID}
                   INNER JOIN civicrm_contact cc ON cc.id = cac.contact_id
-                  INNER JOIN civicrm_option_group cog ON cog.name = "activity_type"
+                  INNER JOIN civicrm_option_group cog ON cog.name = 'activity_type'
                   INNER JOIN civicrm_option_value cov ON cov.option_group_id = cog.id
                          AND cov.value = ca.activity_type_id AND cov.is_active = 1
-                  LEFT JOIN civicrm_entity_file ef on ef.entity_table = "civicrm_activity" AND ef.entity_id = ca.id
-                  LEFT OUTER JOIN civicrm_option_group og ON og.name="activity_status"
-                  LEFT OUTER JOIN civicrm_option_value ov ON ov.option_group_id=og.id AND ov.name="Scheduled"
+                  LEFT JOIN civicrm_entity_file ef on ef.entity_table = 'civicrm_activity'  AND ef.entity_id = ca.id
+                  LEFT OUTER JOIN civicrm_option_group og ON og.name = 'activity_status'
+                  LEFT OUTER JOIN civicrm_option_value ov ON ov.option_group_id=og.id AND ov.name = 'Scheduled'
                   LEFT JOIN civicrm_activity_contact caa
-                                ON caa.activity_id = ca.id AND caa.record_type_id = "$assigneeID"
-                  LEFT JOIN civicrm_contact acc ON acc.id = caa.contact_id  ';
+                                ON caa.activity_id = ca.id AND caa.record_type_id = {$assigneeID}
+                  LEFT JOIN civicrm_contact acc ON acc.id = caa.contact_id  ";
 
     $where = 'WHERE cca.case_id= %1
                     AND ca.is_current_revision = 1';
@@ -1080,30 +1083,29 @@ SELECT case_status.label AS case_status, status_id, case_type.label AS case_type
     $rp = CRM_Utils_Array::value('rp', $params);
 
     if (!$page) {
-
       $page = 1;
-
     }
     if (!$rp) {
       $rp = 10;
     }
 
     $start = (($page - 1) * $rp);
-
     $query = $select . $from . $where . $groupBy . $orderBy;
 
-    $params          = array(1 => array($caseID, 'Integer'));
-    $dao             = CRM_Core_DAO::executeQuery($query, $params);
+    $params = array(1 => array($caseID, 'Integer'));
+    $dao = CRM_Core_DAO::executeQuery($query, $params);
     $params['total'] = $dao->N;
 
     //FIXME: need to optimize/cache these queries
     $limit = " LIMIT $start, $rp";
     $query .= $limit;
+
+    //EXIT;
     $dao = CRM_Core_DAO::executeQuery($query, $params);
 
 
-    $activityTypes    = CRM_Case_PseudoConstant::caseActivityType(FALSE, TRUE);
-    $activityStatus   = CRM_Core_PseudoConstant::activityStatus();
+    $activityTypes = CRM_Case_PseudoConstant::caseActivityType(FALSE, TRUE);
+    $activityStatus = CRM_Core_PseudoConstant::activityStatus();
     $activityPriority = CRM_Core_PseudoConstant::get('CRM_Activity_DAO_Activity', 'priority_id');
 
     $url = CRM_Utils_System::url("civicrm/case/activity",
@@ -1114,10 +1116,10 @@ SELECT case_status.label AS case_status, status_id, case_type.label AS case_type
     if ($context == 'fulltext') {
       $contextUrl = "&context={$context}";
     }
-    $editUrl     = "{$url}&action=update{$contextUrl}";
-    $deleteUrl   = "{$url}&action=delete{$contextUrl}";
-    $restoreUrl  = "{$url}&action=renew{$contextUrl}";
-    $viewTitle   = ts('View this activity.');
+    $editUrl = "{$url}&action=update{$contextUrl}";
+    $deleteUrl = "{$url}&action=delete{$contextUrl}";
+    $restoreUrl = "{$url}&action=renew{$contextUrl}";
+    $viewTitle = ts('View this activity.');
     $statusTitle = ts('Edit status');
 
     $emailActivityTypeIDs = array(
@@ -1163,8 +1165,8 @@ SELECT case_status.label AS case_status, status_id, case_type.label AS case_type
 
     while ($dao->fetch()) {
 
-      $allowView   = self::checkPermission($dao->id, 'view', $dao->activity_type_id, $userID);
-      $allowEdit   = self::checkPermission($dao->id, 'edit', $dao->activity_type_id, $userID);
+      $allowView = self::checkPermission($dao->id, 'view', $dao->activity_type_id, $userID);
+      $allowEdit = self::checkPermission($dao->id, 'edit', $dao->activity_type_id, $userID);
       $allowDelete = self::checkPermission($dao->id, 'delete', $dao->activity_type_id, $userID);
 
       //do not have sufficient permission
@@ -1248,10 +1250,10 @@ SELECT case_status.label AS case_status, status_id, case_type.label AS case_type
         $url .= " | " . '<a href="#" onClick="Javascript:fileOnCase( \'copy\',' . $dao->id . ',' . $caseID . ' ); return false;">' . ts('Copy To Case') . '</a> ';
       }
       // if there are file attachments we will return how many and, if only one, add a link to it
-      if(!empty($dao->attachment_ids)){
-        $attachmentIDs = explode(',',$dao->attachment_ids);
+      if (!empty($dao->attachment_ids)) {
+        $attachmentIDs = explode(',', $dao->attachment_ids);
         $values[$dao->id]['no_attachments'] = count($attachmentIDs);
-        if($values[$dao->id]['no_attachments'] == 1){
+        if ($values[$dao->id]['no_attachments'] == 1) {
           // if there is only one it's easy to do a link - otherwise just flag it
           $attachmentViewUrl = CRM_Utils_System::url(
             "civicrm/file",
@@ -1290,7 +1292,7 @@ SELECT case_status.label AS case_status, status_id, case_type.label AS case_type
       }
 
       if ($allowEdit) {
-        $values[$dao->id]['status'] = '<a class="crm-activity-status crm-activity-status-' . $dao->id . ' ' . $values[$dao->id]['class'] . ' crm-activity-change-status crm-editable-enabled" activity_id='. $dao->id. ' current_status=' . $dao->status . ' case_id=' . $caseID . '" href="#" title=\'' . $statusTitle . '\'>' . $values[$dao->id]['status'] . '</a>';
+        $values[$dao->id]['status'] = '<a class="crm-activity-status crm-activity-status-' . $dao->id . ' ' . $values[$dao->id]['class'] . ' crm-activity-change-status crm-editable-enabled" activity_id=' . $dao->id . ' current_status=' . $dao->status . ' case_id=' . $caseID . '" href="#" title=\'' . $statusTitle . '\'>' . $values[$dao->id]['status'] . '</a>';
       }
     }
     $dao->free();
@@ -1301,7 +1303,7 @@ SELECT case_status.label AS case_status, status_id, case_type.label AS case_type
   /**
    * Function to get Case Related Contacts
    *
-   * @param int     $caseID case id
+   * @param int $caseID case id
    * @param boolean $skipDetails if true include details of contacts
    *
    * @return returns $searchRows array of returnproperties
@@ -1343,8 +1345,8 @@ SELECT case_status.label AS case_status, status_id, case_type.label AS case_type
   /**
    * Function that sends e-mail copy of activity
    *
-   * @param int     $activityId activity Id
-   * @param array   $contacts array of related contact
+   * @param int $activityId activity Id
+   * @param array $contacts array of related contact
    *
    * @return void
    * @access public
@@ -1482,8 +1484,8 @@ SELECT case_status.label AS case_status, status_id, case_type.label AS case_type
    * Retrieve count of activities having a particular type, and
    * associated with a particular case.
    *
-   * @param int    $caseId          ID of the case
-   * @param int    $activityTypeId  ID of the activity type
+   * @param int $caseId          ID of the case
+   * @param int $activityTypeId  ID of the activity type
    *
    * @return array
    *
@@ -1491,7 +1493,8 @@ SELECT case_status.label AS case_status, status_id, case_type.label AS case_type
    *
    */
   static function getCaseActivityCount($caseId, $activityTypeId) {
-    $queryParam = array(1 => array($caseId, 'Integer'),
+    $queryParam = array(
+      1 => array($caseId, 'Integer'),
       2 => array($activityTypeId, 'Integer'),
     );
     $query = "SELECT count(ca.id) as countact
@@ -1512,7 +1515,7 @@ SELECT case_status.label AS case_status, status_id, case_type.label AS case_type
   /**
    * Create an activity for a case via email
    *
-   * @param int    $file   email sent
+   * @param int $file   email sent
    *
    * @return $activity object of newly creted activity via email
    *
@@ -1603,7 +1606,7 @@ SELECT case_status.label AS case_status, status_id, case_type.label AS case_type
   /**
    * Function to retrive the scheduled activity type and date
    *
-   * @param  array  $cases  Array of contact and case id
+   * @param  array $cases  Array of contact and case id
    *
    * @return array  $activityInfo Array of scheduled activity type and date
    *
@@ -1655,12 +1658,14 @@ SELECT case_status.label AS case_status, status_id, case_type.label AS case_type
         self::$_exportableFields = array();
       }
 
-      $fields              = CRM_Case_DAO_Case::export();
+      $fields = CRM_Case_DAO_Case::export();
       $fields['case_role'] = array('title' => ts('Role in Case'));
-      $fields['case_type'] = array('title' => ts('Case Type'),
+      $fields['case_type'] = array(
+        'title' => ts('Case Type'),
         'name' => 'case_type',
       );
-      $fields['case_status'] = array('title' => ts('Case Status'),
+      $fields['case_status'] = array(
+        'title' => ts('Case Status'),
         'name' => 'case_status',
       );
 
@@ -1672,7 +1677,7 @@ SELECT case_status.label AS case_status, status_id, case_type.label AS case_type
   /**
    * Restore the record that are associated with this case
    *
-   * @param  int  $caseId id of the case to restore
+   * @param  int $caseId id of the case to restore
    *
    * @return true if success.
    * @access public
@@ -1687,8 +1692,8 @@ SELECT case_status.label AS case_status, status_id, case_type.label AS case_type
       }
     }
     //restore case
-    $case             = new CRM_Case_DAO_Case();
-    $case->id         = $caseId;
+    $case = new CRM_Case_DAO_Case();
+    $case->id = $caseId;
     $case->is_deleted = 0;
     $case->save();
 
@@ -1709,11 +1714,11 @@ SELECT case_status.label AS case_status, status_id, case_type.label AS case_type
         $results = array();
         CRM_Contact_BAO_Group::retrieve($searchParams, $results);
         if ($results) {
-          $groupInfo['id']    = $results['id'];
+          $groupInfo['id'] = $results['id'];
           $groupInfo['title'] = $results['title'];
-          $params             = array(array('group', 'IN', array($groupInfo['id'] => 1), 0, 0));
-          $return             = array('sort_name' => 1, 'display_name' => 1, 'email' => 1, 'phone' => 1);
-          $return             = array('contact_id' => 1, 'sort_name' => 1, 'display_name' => 1, 'email' => 1, 'phone' => 1);
+          $params = array(array('group', 'IN', array($groupInfo['id'] => 1), 0, 0));
+          $return = array('sort_name' => 1, 'display_name' => 1, 'email' => 1, 'phone' => 1);
+          $return = array('contact_id' => 1, 'sort_name' => 1, 'display_name' => 1, 'email' => 1, 'phone' => 1);
           list($globalContacts, $_) = CRM_Contact_BAO_Query::apiQuery($params, $return, NULL, $sort, $offset, $rowCount, TRUE, $returnOnlyCount);
 
           if ($returnOnlyCount) {
@@ -1721,7 +1726,7 @@ SELECT case_status.label AS case_status, status_id, case_type.label AS case_type
           }
 
           if ($showLinks) {
-            foreach($globalContacts as $idx => $contact) {
+            foreach ($globalContacts as $idx => $contact) {
               $globalContacts[$idx]['sort_name'] = '<a href="' . $contactViewUrl . $contact['contact_id'] . '">' . $contact['sort_name'] . '</a>';
             }
           }
@@ -1756,19 +1761,18 @@ SELECT case_status.label AS case_status, status_id, case_type.label AS case_type
   /**
    * Function to get Case ActivitiesDueDates with given criteria.
    *
-   * @param int      $caseID case id
-   * @param array    $criteriaParams given criteria
-   * @param boolean  $latestDate if set newest or oldest date is selceted.
+   * @param int $caseID case id
+   * @param array $criteriaParams given criteria
+   * @param boolean $latestDate if set newest or oldest date is selceted.
    *
    * @return returns case activities due dates
    *
    * @static
    */
-  static function getCaseActivityDates($caseID, $criteriaParams = array(
-  ), $latestDate = FALSE) {
-    $values     = array();
+  static function getCaseActivityDates($caseID, $criteriaParams = array(), $latestDate = FALSE) {
+    $values = array();
     $selectDate = " ca.activity_date_time";
-    $where      = $groupBy = ' ';
+    $where = $groupBy = ' ';
 
     if (!$caseID) {
       return;
@@ -1808,9 +1812,9 @@ SELECT case_status.label AS case_status, status_id, case_type.label AS case_type
   /**
    * Function to create activities when Case or Other roles assigned/modified/deleted.
    *
-   * @param int      $caseID case id
-   * @param int      $relationshipId relationship id
-   * @param int      $relContactId case role assigne contactId.
+   * @param int $caseID case id
+   * @param int $relationshipId relationship id
+   * @param int $relContactId case role assigne contactId.
    *
    * @return void on success creates activity and case activity
    *
@@ -1901,8 +1905,8 @@ SELECT case_status.label AS case_status, status_id, case_type.label AS case_type
    * Function to get case manger
    * contact which is assigned a case role of case manager.
    *
-   * @param int    $caseType case type
-   * @param int    $caseId   case id
+   * @param int $caseType case type
+   * @param int $caseId   case id
    *
    * @return array $caseManagerContact array of contact on success otherwise empty
    *
@@ -1947,16 +1951,15 @@ SELECT civicrm_contact.id as casemanager_id,
    *
    * @return array of case and related data keyed on case id
    */
-  static function getUnclosedCases($params = array(
-  ), $excludeCaseIds = array(), $excludeDeleted = TRUE) {
+  static function getUnclosedCases($params = array(), $excludeCaseIds = array(), $excludeDeleted = TRUE) {
     //params from ajax call.
     $where = array('( ca.end_date is null )');
     if ($caseType = CRM_Utils_Array::value('case_type', $params)) {
       $where[] = "( ov.label LIKE '%$caseType%' )";
     }
     if ($sortName = CRM_Utils_Array::value('sort_name', $params)) {
-      $config  = CRM_Core_Config::singleton();
-      $search  = ($config->includeWildCardInName) ? "%$sortName%" : "$sortName%";
+      $config = CRM_Core_Config::singleton();
+      $search = ($config->includeWildCardInName) ? "%$sortName%" : "$sortName%";
       $where[] = "( sort_name LIKE '$search' )";
     }
     if (is_array($excludeCaseIds) &&
@@ -1973,8 +1976,8 @@ SELECT civicrm_contact.id as casemanager_id,
     $doFilterCases = FALSE;
     if (!CRM_Core_Permission::check('access all cases and activities')) {
       $doFilterCases = TRUE;
-      $session       = CRM_Core_Session::singleton();
-      $filterCases   = CRM_Case_BAO_Case::getCases(FALSE, $session->get('userID'));
+      $session = CRM_Core_Session::singleton();
+      $filterCases = CRM_Case_BAO_Case::getCases(FALSE, $session->get('userID'));
     }
     $whereClause = implode(' AND ', $where);
 
@@ -2055,7 +2058,7 @@ LEFT JOIN  civicrm_case_contact ON ( civicrm_case.id = civicrm_case_contact.case
   /**
    * Retrieve cases related to particular contact.
    *
-   * @param int     $contactId contact id
+   * @param int $contactId contact id
    * @param boolean $excludeDeleted do not include deleted cases.
    *
    * @return an array of cases.
@@ -2098,8 +2101,8 @@ INNER JOIN  civicrm_case_contact ON ( civicrm_case.id = civicrm_case_contact.cas
   /**
    * Retrieve related cases for give case.
    *
-   * @param int     $mainCaseId     id of main case
-   * @param int     $contactId      id of contact
+   * @param int $mainCaseId     id of main case
+   * @param int $contactId      id of contact
    * @param boolean $excludeDeleted do not include deleted cases.
    *
    * @return an array of related cases.
@@ -2161,8 +2164,8 @@ INNER JOIN  civicrm_case_contact ON ( civicrm_case.id = civicrm_case_contact.cas
     $doFilterCases = FALSE;
     if (!CRM_Core_Permission::check('access all cases and activities')) {
       $doFilterCases = TRUE;
-      $session       = CRM_Core_Session::singleton();
-      $filterCases   = CRM_Case_BAO_Case::getCases(FALSE, $session->get('userID'));
+      $session = CRM_Core_Session::singleton();
+      $filterCases = CRM_Case_BAO_Case::getCases(FALSE, $session->get('userID'));
     }
 
     //2. fetch the details of related cases.
@@ -2179,7 +2182,7 @@ INNER JOIN  civicrm_case_contact ON ( civicrm_case.id = civicrm_case_contact.cas
                                                      AND case_type_og.id = case_type_ov.option_group_id )
      WHERE  {$whereClause}";
 
-    $dao            = CRM_Core_DAO::executeQuery($query);
+    $dao = CRM_Core_DAO::executeQuery($query);
     $contactViewUrl = CRM_Utils_System::url("civicrm/contact/view", "reset=1&cid=");
     $hasViewContact = CRM_Core_Permission::giveMeAllACLs();
 
@@ -2188,7 +2191,7 @@ INNER JOIN  civicrm_case_contact ON ( civicrm_case.id = civicrm_case_contact.cas
       if (!$doFilterCases || array_key_exists($dao->id, $filterCases)) {
         $caseViewStr = "reset=1&id={$dao->id}&cid={$dao->client_id}&action=view&context=case&selectedChild=case";
         $caseViewUrl = CRM_Utils_System::url("civicrm/contact/view/case", $caseViewStr);
-        $caseView    = "<a href='{$caseViewUrl}'>" . ts('View Case') . "</a>";
+        $caseView = "<a href='{$caseViewUrl}'>" . ts('View Case') . "</a>";
       }
       $clientView = $dao->client_name;
       if ($hasViewContact) {
@@ -2232,7 +2235,7 @@ INNER JOIN  civicrm_case_contact ON ( civicrm_case.id = civicrm_case_contact.cas
    * @static
    */
   static function mergeCases($mainContactId, $mainCaseId = NULL, $otherContactId = NULL,
-                             $otherCaseId = NULL, $changeClient = FALSE ) {
+                             $otherCaseId = NULL, $changeClient = FALSE) {
     $moveToTrash = TRUE;
 
     $duplicateContacts = FALSE;
@@ -2299,18 +2302,18 @@ INNER JOIN  civicrm_case_contact ON ( civicrm_case.id = civicrm_case_contact.cas
           }
 
           foreach ($table as $tableName => $tableColumns) {
-            $insert          = 'INSERT INTO ' . $tableName . ' (' . implode(', ', $tableColumns) . ') ';
+            $insert = 'INSERT INTO ' . $tableName . ' (' . implode(', ', $tableColumns) . ') ';
             $tableColumns[0] = $mainCaseId;
-            $select          = 'SELECT ' . implode(', ', $tableColumns);
-            $from            = ' FROM ' . $tableName;
-            $where           = " WHERE {$tableName}.entity_id = {$otherCaseId}";
-            $query           = $insert . $select . $from . $where;
-            $dao             = CRM_Core_DAO::executeQuery($query, CRM_Core_DAO::$_nullArray);
+            $select = 'SELECT ' . implode(', ', $tableColumns);
+            $from = ' FROM ' . $tableName;
+            $where = " WHERE {$tableName}.entity_id = {$otherCaseId}";
+            $query = $insert . $select . $from . $where;
+            $dao = CRM_Core_DAO::executeQuery($query, CRM_Core_DAO::$_nullArray);
           }
         }
-        
+
         $mainCase->free();
-        
+
         $mainCaseIds[] = $mainCaseId;
         //insert record for case contact.
         $otherCaseContact = new CRM_Case_DAO_CaseContact();
@@ -2564,8 +2567,10 @@ SELECT  id
         $mergeActType = array_search('Reassigned Case', $activityTypes);
         $mergeActSubject = ts("Case %1 reassigned client from %2 to %3. New Case ID is %4.",
           array(
-            1 => $otherCaseId, 2 => $otherContactDisplayName,
-            3 => $mainContactDisplayName, 4 => $mainCaseId
+            1 => $otherCaseId,
+            2 => $otherContactDisplayName,
+            3 => $mainContactDisplayName,
+            4 => $mainCaseId
           )
         );
       }
@@ -2573,8 +2578,10 @@ SELECT  id
         $mergeActType = array_search('Merge Case', $activityTypes);
         $mergeActSubject = ts("Case %1 copied from contact id %2 to contact id %3 via merge. New Case ID is %4.",
           array(
-            1 => $otherCaseId, 2 => $otherContactId,
-            3 => $mainContactId, 4 => $mainCaseId
+            1 => $otherCaseId,
+            2 => $otherContactId,
+            3 => $mainContactId,
+            4 => $mainCaseId
           )
         );
       }
@@ -2629,8 +2636,8 @@ WHERE id IN (' . implode(',', $copiedActivityIds) . ')';
    * Validate contact permission for
    * edit/view on activity record and build links.
    *
-   * @param array   $tplParams       params to be sent to template for sending email.
-   * @param array   $activityParams  info of the activity.
+   * @param array $tplParams       params to be sent to template for sending email.
+   * @param array $activityParams  info of the activity.
    *
    * @return void
    * @static
@@ -2668,10 +2675,10 @@ WHERE id IN (' . implode(',', $copiedActivityIds) . ')';
    * Validate contact permission for
    * given operation on activity record.
    *
-   * @param int     $activityId      activity record id.
-   * @param string  $operation       user operation.
-   * @param int     $actTypeId       activity type id.
-   * @param int     $contactId       contact id/if not pass consider logged in
+   * @param int $activityId      activity record id.
+   * @param string $operation       user operation.
+   * @param int $actTypeId       activity type id.
+   * @param int $contactId       contact id/if not pass consider logged in
    * @param boolean $checkComponent  do we need to check component enabled.
    *
    * @return boolean $allow  true/false
@@ -2766,7 +2773,10 @@ WHERE id IN (' . implode(',', $copiedActivityIds) . ')';
         if (!empty($hasPermissions)) {
           //need to check activity object specific.
           if (in_array($operation, array(
-            'view', 'edit'))) {
+            'view',
+            'edit'
+          ))
+          ) {
             //do we have supper permission.
             if (in_array('access all cases and activities', $hasPermissions[$operation])) {
               $allowOperations[$operation] = $allow = TRUE;
@@ -2843,7 +2853,15 @@ WHERE id IN (' . implode(',', $copiedActivityIds) . ')';
       $actTypeName = CRM_Utils_Array::value($actTypeId, $activityTypes);
 
       //do not allow multiple copy / edit action.
-      $singletonNames = array('Open Case', 'Reassigned Case', 'Merge Case', 'Link Cases', 'Assign Case Role', 'Email', 'Inbound Email');
+      $singletonNames = array(
+        'Open Case',
+        'Reassigned Case',
+        'Merge Case',
+        'Link Cases',
+        'Assign Case Role',
+        'Email',
+        'Inbound Email'
+      );
 
       //do not allow to delete these activities, CRM-4543
       $doNotDeleteNames = array('Open Case', 'Change Case Type', 'Change Case Status', 'Change Case Start Date');
@@ -2852,7 +2870,16 @@ WHERE id IN (' . implode(',', $copiedActivityIds) . ')';
       $allowEditNames = array('Open Case');
 
       // do not allow File on Case
-      $doNotFileNames = array('Open Case', 'Change Case Type', 'Change Case Status', 'Change Case Start Date','Reassigned Case', 'Merge Case', 'Link Cases', 'Assign Case Role');
+      $doNotFileNames = array(
+        'Open Case',
+        'Change Case Type',
+        'Change Case Status',
+        'Change Case Start Date',
+        'Reassigned Case',
+        'Merge Case',
+        'Link Cases',
+        'Assign Case Role'
+      );
 
       if (in_array($actTypeName, $singletonNames)) {
         $allow = FALSE;
@@ -2932,7 +2959,7 @@ WHERE id IN (' . implode(',', $copiedActivityIds) . ')';
   /**
    * Function to check whether activity is a case Activity
    *
-   * @param  int      $activityID   activity id
+   * @param  int $activityID   activity id
    *
    * @return boolean  $isCaseActivity true/false
    */
@@ -3127,16 +3154,16 @@ WHERE id IN (' . implode(',', $copiedActivityIds) . ')';
         break;
     }
     return $sql;
- }
-
- /**
-  * Function to add/copy relationships, when new client is added for a case
-  *
-  * @param int $caseId case id
-  * @param int $contactId contact id / new client id
-  *
-  * @return void
-  */
 }
+
 /**
+   * Function to add/copy relationships, when new client is added for a case
+   *
+   * @param int $caseId case id
+   * @param int $contactId contact id / new client id
+   *
+   * @return void
+   */
   static function addCaseRelationships($caseId, $contactId) {
     // get the case role / relationships for the case
     $caseRelationships = new CRM_Contact_DAO_Relationship();
@@ -3183,7 +3210,7 @@ WHERE id IN (' . implode(',', $copiedActivityIds) . ')';
     $caseContact->case_id = $caseId;
     $caseContact->find();
 
-    while($caseContact->fetch()) {
+    while ($caseContact->fetch()) {
       $clients[] = $caseContact->contact_id;
     }