Merge pull request #881 from compucorp/MAE-955-fix-export-case

MAE-955: Fix Cases export on CiviCRM versions >= 5.41.0
compucorp · Nov 15, 2022 · a3fbd7f · a3fbd7f
2 parents 650dd8d + 9cad1ec
commit a3fbd7f
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 25 deletions.
diff --git a/CRM/Case/Export/Form/Map.php b/CRM/Case/Export/Form/Map.php
diff --git a/CRM/Case/Export/Form/Select.php b/CRM/Case/Export/Form/Select.php
diff --git a/api/v3/Case/Getdetails.php b/api/v3/Case/Getdetails.php
@@ -70,16 +70,16 @@ function _civicrm_api3_case_getdetails_spec(array &$spec) {
  * @throws API_Exception
  */
 function civicrm_api3_case_getdetails(array $params) {
-  list(
+  [
     'resultMetadata' => $resultMetadata,
     'params' => $params,
     'toReturn' => $toReturn,
-    'sql' => $sql
-  ) = CaseDetailsQuery::get($params);
+    'sql' => $sql,
+  ] = CaseDetailsQuery::get($params);
 
   // Call the case api.
   $result = civicrm_api3_case_get(['sequential' => 0] + $params, $sql);
-  if (!empty($result['values'])) {
+  if (!empty($result['values']) && is_array($result['values'])) {
     $ids = array_keys($result['values']);
 
     // Remove legacy cruft.