Do not try to explode() return parameter when it already is an array

systopia · Mar 19, 2024 · 2375e46 · 2375e46
1 parent 3aa7846
commit 2375e46
Showing 1 changed file with 6 additions and 3 deletions.
diff --git a/Civi/RemoteToolsRequest.php b/Civi/RemoteToolsRequest.php
@@ -346,9 +346,12 @@ public function wasExecuted()
      */
     public function getOriginalReturnFields()
     {
-        $return_string = \CRM_Utils_Array::value('return', $this->original_request, '');
-        if ($return_string) {
-            return explode(',', $return_string);
+        $return = $this->original_request['return'] ?? '';
+        if (is_array($return)) {
+            return $return;
+        }
+        if (is_string($return)) {
+            return explode(',', $return);
         } else {
             return [];
         }