From: Kurund Jalmi <kurund@civicrm.org>
Date: Sat, 16 Mar 2013 19:13:28 +0000 (+0530)
Subject: worked on CRM-12012, for multi-client cases, make sure case roles are added for all... 
X-Git-Url: https://vcs.fsf.org/?a=commitdiff_plain;h=14a679f113a389167271d1ae46573bd06af44a0c;p=civicrm-core.git

worked on CRM-12012, for multi-client cases, make sure case roles are added for all clients when added/updated case role for any client
---

diff --git a/CRM/Case/BAO/Case.php b/CRM/Case/BAO/Case.php
index f5b8397509..30764eac00 100644
--- a/CRM/Case/BAO/Case.php
+++ b/CRM/Case/BAO/Case.php
@@ -3102,5 +3102,26 @@ WHERE id IN (' . implode(',', $copiedActivityIds) . ')';
       $relationshipTypes[] = $caseRelationships->relationship_type_id;
     }
   }
+
+  /**
+   * Function to get the list of clients for a case
+   *
+   * @param int $caseId
+   *
+   * @return array $clients associated array with client ids
+   * @static
+   */
+  static function getCaseClients($caseId) {
+    $clients = array();
+    $caseContact = new CRM_Case_DAO_CaseContact();
+    $caseContact->case_id = $caseId;
+    $caseContact->find();
+
+    while($caseContact->fetch()) {
+      $clients[] = $caseContact->contact_id;
+    }
+
+    return $clients;
+  }
 }
 
diff --git a/CRM/Contact/Page/AJAX.php b/CRM/Contact/Page/AJAX.php
index 1ae994dbfe..7da94ba777 100644
--- a/CRM/Contact/Page/AJAX.php
+++ b/CRM/Contact/Page/AJAX.php
@@ -284,44 +284,56 @@ class CRM_Contact_Page_AJAX {
   static function relationship() {
     $relType         = CRM_Utils_Array::value('rel_type', $_REQUEST);
     $relContactID    = CRM_Utils_Array::value('rel_contact', $_REQUEST);
-    $sourceContactID = CRM_Utils_Array::value('contact_id', $_REQUEST);
-    $relationshipID  = CRM_Utils_Array::value('rel_id', $_REQUEST);
+    $sourceContactID = CRM_Utils_Array::value('contact_id', $_REQUEST); // we no longer need this.
+    $relationshipID  = CRM_Utils_Array::value('rel_id', $_REQUEST); // this used only to determine add or update mode
     $caseID          = CRM_Utils_Array::value('case_id', $_REQUEST);
 
-    $relationParams = array(
-      'relationship_type_id' => $relType . '_a_b',
-      'contact_check' => array($relContactID => 1),
-      'is_active' => 1,
-      'case_id' => $caseID,
-      'start_date' => date("Ymd"),
-    );
-
-    $relationIds = array('contact' => $sourceContactID);
-    if ($relationshipID && $relationshipID != 'null') {
-      $relationIds['relationship'] = $relationshipID;
-      $relationIds['contactTarget'] = $relContactID;
-    }
+    // check if there are multiple clients for this case, if so then we need create
+    // relationship and also activities for each contacts
 
-    $return = CRM_Contact_BAO_Relationship::create($relationParams, $relationIds);
-    $status = 'process-relationship-fail';
-    if (CRM_Utils_Array::value(0, $return[4])) {
-      $relationshipID = $return[4][0];
-      $status = 'process-relationship-success';
-    }
+    // get case client list
+    $clientList = CRM_Case_BAO_Case::getCaseClients($caseID);
 
-    $caseRelationship = array();
-    if ($relationshipID && $relationshipID != 'null') {
-      // we should return phone and email
-      $caseRelationship = CRM_Case_BAO_Case::getCaseRoles($sourceContactID,
-        $caseID, $relationshipID
+    foreach($clientList as $sourceContactID) {
+      $relationParams = array(
+        'relationship_type_id' => $relType . '_a_b',
+        'contact_check' => array($relContactID => 1),
+        'is_active' => 1,
+        'case_id' => $caseID,
+        'start_date' => date("Ymd"),
       );
 
-      //create an activity for case role assignment.CRM-4480
-      CRM_Case_BAO_Case::createCaseRoleActivity($caseID, $relationshipID, $relContactID);
+      $relationIds = array('contact' => $sourceContactID);
+
+      // check if we are editing/updating existing relationship
+      if ($relationshipID && $relationshipID != 'null') {
+        // here we need to retrieve appropriate relationshipID based on client id and relationship type id
+        $caseRelationships = new CRM_Contact_DAO_Relationship();
+        $caseRelationships->case_id = $caseID;
+        $caseRelationships->relationship_type_id = $relType;
+        $caseRelationships->contact_id_a = $sourceContactID;
+        $caseRelationships->find();
+
+        while($caseRelationships->fetch()) {
+          $relationIds['relationship'] = $caseRelationships->id;
+          $relationIds['contactTarget'] = $relContactID;
+        }
+        $caseRelationships->free();
+      }
+
+      // create new or update existing relationship
+      $return = CRM_Contact_BAO_Relationship::create($relationParams, $relationIds);
+
+      $status = 'process-relationship-fail';
+      if (CRM_Utils_Array::value(0, $return[4])) {
+        $relationshipID = $return[4][0];
+        $status = 'process-relationship-success';
+
+        //create an activity for case role assignment.CRM-4480
+        CRM_Case_BAO_Case::createCaseRoleActivity($caseID, $relationshipID, $relContactID);
+      }
     }
-    $relation = CRM_Utils_Array::value($relationshipID, $caseRelationship, array());
 
-    $relation['rel_id'] = $relationshipID;
     $relation['status'] = $status;
     echo json_encode($relation);
     CRM_Utils_System::civiExit();