MDL-67115 google: php74 params order for implode()
authorEloy Lafuente (stronk7) <stronk7@moodle.org>
Sun, 19 Jan 2020 17:35:25 +0000 (18:35 +0100)
committerEloy Lafuente (stronk7) <stronk7@moodle.org>
Sun, 19 Jan 2020 17:47:28 +0000 (18:47 +0100)
Passing parameters to implode() in reverse order is deprecated, use
implode($glue, $parts) instead of implode($parts, $glue).

This commit corresponds to general search of wrong cases.

lib/google/readme_moodle.txt
lib/google/src/Google/Http/REST.php

index 15358b7..5aae6a3 100644 (file)
@@ -38,6 +38,8 @@ Here are the files that we have added.
 
 Local changes (to reapply until upstream upgrades contain them):
     * MDL-67034 php74 compliance fixes
+    * MDL-67115 php74 implode() compliance fixes. This is fixed in upstream library v2.2.4
+      (verify that https://github.com/googleapis/google-api-php-client/pull/1683 is applied)
 
 
 Information
index 491c068..b690046 100644 (file)
@@ -170,7 +170,7 @@ class Google_Http_REST
     }
 
     if (count($queryVars)) {
-      $requestUrl .= '?' . implode($queryVars, '&');
+      $requestUrl .= '?' . implode('&', $queryVars);
     }
 
     return $requestUrl;