CRM-21052 - Activity.create API - Abide by setting `civicaseActivityRevisions`
[civicrm-core.git] / api / v3 / Case.php
index df6332a3eb258491d183aa39cba3c053bf60e271..bc263b0944a68c1c9cfea308800993818fdd9541 100644 (file)
  * @param array $params
  *
  * @code
- * //REQUIRED:
+ * //REQUIRED for create:
  * 'case_type_id' => int OR
  * 'case_type' => str (provide one or the other)
  * 'contact_id' => int // case client
  * 'subject' => str
+ * //REQUIRED for update:
+ * 'id' => case Id
  *
  * //OPTIONAL
  * 'medium_id' => int // see civicrm option values for possibilities
  *   api result array
  */
 function civicrm_api3_case_create($params) {
+  _civicrm_api3_case_format_params($params);
 
-  if (!empty($params['id'])) {
-    return civicrm_api3_case_update($params);
+  if (empty($params['id'])) {
+    // Creating a new case, so make sure we have the necessary parameters
+    civicrm_api3_verify_mandatory($params, NULL, array(
+        'contact_id',
+        'subject',
+        array('case_type', 'case_type_id'),
+      )
+    );
   }
+  else {
+    // Update an existing case
+    // FIXME: Some of this logic should move to the BAO object?
+    // FIXME: Should we check if case with ID actually exists?
+    if (!isset($params['case_id']) && isset($params['id'])) {
+      $params['case_id'] = $params['id'];
+    }
 
-  civicrm_api3_verify_mandatory($params, NULL, array(
-    'contact_id',
-    'subject',
-    array('case_type', 'case_type_id'))
-  );
-  _civicrm_api3_case_format_params($params);
+    if (array_key_exists('creator_id', $params)) {
+      throw new API_Exception('You cannot update creator id');
+    }
 
-  // If format_params didn't find what it was looking for, return error
-  if (empty($params['case_type_id'])) {
-    throw new API_Exception('Invalid case_type. No such case type exists.');
-  }
-  if (empty($params['case_type'])) {
-    throw new API_Exception('Invalid case_type_id. No such case type exists.');
-  }
+    $mergedCaseId = $origContactIds = array();
 
-  // Fixme: can we safely pass raw params to the BAO?
-  $newParams = array(
-    'case_type_id' => $params['case_type_id'],
-    'creator_id' => $params['creator_id'],
-    'status_id' => $params['status_id'],
-    'start_date' => $params['start_date'],
-    'end_date' => CRM_Utils_Array::value('end_date', $params),
-    'subject' => $params['subject'],
-  );
+    // get original contact id and creator id of case
+    if (!empty($params['contact_id'])) {
+      $origContactIds = CRM_Case_BAO_Case::retrieveContactIdsByCaseId($params['id']);
+      $origContactId = CRM_Utils_Array::first($origContactIds);
+    }
+
+    // FIXME: Refactor as separate method to get contactId
+    if (count($origContactIds) > 1) {
+      // check valid orig contact id
+      if (empty($params['orig_contact_id'])) {
+        throw new API_Exception('Case is linked with more than one contact id. Provide the required params orig_contact_id to be replaced');
+      }
+      if (!empty($params['orig_contact_id']) && !in_array($params['orig_contact_id'], $origContactIds)) {
+        throw new API_Exception('Invalid case contact id (orig_contact_id)');
+      }
+      $origContactId = $params['orig_contact_id'];
+    }
+
+    // check for same contact id for edit Client
+    if (!empty($params['contact_id']) && !in_array($params['contact_id'], $origContactIds)) {
+      $mergedCaseId = CRM_Case_BAO_Case::mergeCases($params['contact_id'], $params['case_id'], $origContactId, NULL, TRUE);
+    }
+
+    // If we merged cases then update the merged case
+    if (!empty($mergedCaseId[0])) {
+      $params['id'] = $mergedCaseId[0];
+    }
+  }
 
-  $caseBAO = CRM_Case_BAO_Case::create($newParams);
+  // Create/update the case
+  $caseBAO = CRM_Case_BAO_Case::create($params);
 
   if (!$caseBAO) {
     throw new API_Exception('Case not created. Please check input params.');
   }
 
-  foreach ((array) $params['contact_id'] as $cid) {
-    $contactParams = array('case_id' => $caseBAO->id, 'contact_id' => $cid);
-    CRM_Case_BAO_CaseContact::create($contactParams);
+  if (isset($params['contact_id'])) {
+    foreach ((array) $params['contact_id'] as $cid) {
+      $contactParams = array('case_id' => $caseBAO->id, 'contact_id' => $cid);
+      CRM_Case_BAO_CaseContact::create($contactParams);
+    }
+  }
+
+  if (!isset($params['id'])) {
+    // As the API was not passed an id we have created a new case.
+    // Only run the xmlProcessor for new cases to get all configuration for the new case.
+    _civicrm_api3_case_create_xmlProcessor($params, $caseBAO);
+  }
+
+  // return case
+  $values = array();
+  _civicrm_api3_object_to_array($caseBAO, $values[$caseBAO->id]);
+
+  return civicrm_api3_create_success($values, $params, 'Case', 'create', $caseBAO);
+}
+
+/**
+ * When creating a new case, run the xmlProcessor to get all necessary params/configuration
+ *  for the new case, as cases use an xml file to store their configuration.
+ * @param $params
+ * @param $caseBAO
+ */
+function _civicrm_api3_case_create_xmlProcessor($params, $caseBAO) {
+  // Format params for xmlProcessor
+  if (isset($caseBAO->id)) {
+    $params['id'] = $caseBAO->id;
   }
 
   // Initialize XML processor with $params
   $xmlProcessor = new CRM_Case_XMLProcessor_Process();
   $xmlProcessorParams = array(
-    'clientID' => $params['contact_id'],
-    'creatorID' => $params['creator_id'],
+    'clientID' => CRM_Utils_Array::value('contact_id', $params),
+    'creatorID' => CRM_Utils_Array::value('creator_id', $params),
     'standardTimeline' => 1,
     'activityTypeName' => 'Open Case',
-    'caseID' => $caseBAO->id,
-    'subject' => $params['subject'],
+    'caseID' => CRM_Utils_Array::value('id', $params),
+    'subject' => CRM_Utils_Array::value('subject', $params),
     'location' => CRM_Utils_Array::value('location', $params),
-    'activity_date_time' => $params['start_date'],
+    'activity_date_time' => CRM_Utils_Array::value('start_date', $params),
     'duration' => CRM_Utils_Array::value('duration', $params),
     'medium_id' => CRM_Utils_Array::value('medium_id', $params),
     'details' => CRM_Utils_Array::value('details', $params),
@@ -120,12 +174,6 @@ function civicrm_api3_case_create($params) {
 
   // Do it! :-D
   $xmlProcessor->run($params['case_type'], $xmlProcessorParams);
-
-  // return case
-  $values = array();
-  _civicrm_api3_object_to_array($caseBAO, $values[$caseBAO->id]);
-
-  return civicrm_api3_create_success($values, $params, 'Case', 'create', $caseBAO);
 }
 
 /**
@@ -446,7 +494,7 @@ function _civicrm_api3_case_deprecation() {
 }
 
 /**
- * Update a specified case.
+ * @deprecated Update a specified case.  Use civicrm_api3_case_create() instead.
  *
  * @param array $params
  *   //REQUIRED:
@@ -479,7 +527,7 @@ function civicrm_api3_case_update($params) {
   // get original contact id and creator id of case
   if (!empty($params['contact_id'])) {
     $origContactIds = CRM_Case_BAO_Case::retrieveContactIdsByCaseId($params['id']);
-    $origContactId = $origContactIds[1];
+    $origContactId = CRM_Utils_Array::first($origContactIds);
   }
 
   if (count($origContactIds) > 1) {
@@ -645,7 +693,16 @@ function _civicrm_api3_case_read(&$cases, $options) {
  * @param array $params
  */
 function _civicrm_api3_case_format_params(&$params) {
-  // figure out case type id from case type and vice-versa
+  // Format/include custom params
+  $values = array();
+  _civicrm_api3_custom_format_params($params, $values, 'Case');
+  $params = array_merge($params, $values);
+
+  if (empty($params['case_type_id']) && empty($params['case_type'])) {
+    return;
+  }
+
+  // figure out case_type_id from case_type and vice-versa
   $caseTypes = CRM_Case_PseudoConstant::caseType('name', FALSE);
   if (empty($params['case_type_id'])) {
     $params['case_type_id'] = array_search($params['case_type'], $caseTypes);
@@ -662,6 +719,7 @@ function _civicrm_api3_case_format_params(&$params) {
   }
 }
 
+
 /**
  * It actually works a lot better to use the CaseContact api instead of the Case api
  * for entityRef fields so we can perform the necessary joins,