Merge pull request #19478 from eileenmcnaughton/ref

[REF] do not needlessly pass as reference, enforce valid param

CRM/Utils/Array.php

@@ -61,20 +61,17 @@ public static function value($key, $list, $default = NULL) {
    * @return mixed
    *   The value of the key, or null if the key is not found.
    */
-  public static function retrieveValueRecursive(&$params, $key) {
-    if (!is_array($params)) {
-      return NULL;
-    }
-    elseif ($value = CRM_Utils_Array::value($key, $params)) {
-      return $value;
+  public static function retrieveValueRecursive(array $params, string $key) {
+    if (isset($params[$key])) {
+      return $params[$key];
     }
-    else {
-      foreach ($params as $subParam) {
-        if (is_array($subParam) &&
-          $value = self::retrieveValueRecursive($subParam, $key)
-        ) {
-          return $value;
-        }
+    foreach ($params as $subParam) {
+      if (is_array($subParam) &&
+        // @todo - this will mishandle values like 0 and false
+        // but it's a little scary to fix.
+        $value = self::retrieveValueRecursive($subParam, $key)
+      ) {
+        return $value;
       }
     }
     return NULL;