change \GuzzleHttp\Promise\all to \GuzzleHttp\Promise\Utils::all (#361)

* change \GuzzleHttp\Promise\all to \GuzzleHttp\Promise\Utils::all * support old promises as well * run unit tests only on pull_request
php-opencloud · Jun 24, 2023 · aef36ab · aef36ab
1 parent 3f84692
commit aef36ab
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/.github/workflows/unit_tests.yml b/.github/workflows/unit_tests.yml
@@ -1,6 +1,5 @@
 name: Run Unit Tests
 on:
-  push:
   pull_request:
 jobs:
   tests:

diff --git a/src/ObjectStore/v1/Models/Container.php b/src/ObjectStore/v1/Models/Container.php
@@ -233,7 +233,9 @@ public function createLargeObject(array $data): StorageObject
         }
 
         /** @var Promise $p */
-        $p = \GuzzleHttp\Promise\all($promises);
+        $p = function_exists('\GuzzleHttp\Promise\all')
+            ? \GuzzleHttp\Promise\all($promises)
+            : \GuzzleHttp\Promise\Utils::all($promises);
         $p->wait();
 
         return $this->createObject([