Apply changes to avoid PHP deprecations

These changes are introduced to mitigate deprecations from PHP 8.1
and to avoid the lower boundaries test of PHP 8.1 to fail due to
PHPUnit resorting to the use of `$GLOBALS`.

These changes are introduced before the requirement updates to
allow them to be forward and backward compatible.

Author: agustingomes

src/MessageBird/Resources/AvailablePhoneNumbers.php

@@ -69,6 +69,10 @@ public function getList(string $countryCode, array $parameters = [])
      */
     private function processRequest(?string $body): Objects\Number
     {
+        if ($body === null) {
+            throw new Exceptions\ServerException('Got an invalid JSON response from the server.');
+        }
+
         try {
             $body = json_decode($body, true, 512, JSON_THROW_ON_ERROR);
         } catch (\JsonException $e) {

src/MessageBird/Resources/Base.php

@@ -117,13 +117,13 @@ public function create($object, ?array $query = null)
      */
     public function processRequest(?string $body)
     {
-        try {
-            $body = json_decode($body, null, 512, \JSON_THROW_ON_ERROR);
-        } catch (\JsonException $e) {
+        if ($body === null) {
             throw new Exceptions\ServerException('Got an invalid JSON response from the server.');
         }
 
-        if ($body === null) {
+        try {
+            $body = json_decode($body, null, 512, \JSON_THROW_ON_ERROR);
+        } catch (\JsonException $e) {
             throw new Exceptions\ServerException('Got an invalid JSON response from the server.');
         }
 

src/MessageBird/Resources/Voice/Base.php

@@ -70,17 +70,17 @@ public function getList($parameters = [])
      */
     public function processRequest(?string $body)
     {
+        if ($body === null) {
+            throw new Exceptions\ServerException('Got an invalid JSON response from the server.');
+        }
+
         try {
             $body = json_decode($body, null, 512, \JSON_THROW_ON_ERROR);
         } catch (\JsonException $e) {
             throw new Exceptions\ServerException('Got an invalid JSON response from the server.');
 
         }
 
-        if ($body === null) {
-            throw new Exceptions\ServerException('Got an invalid JSON response from the server.');
-        }
-
         if (empty($body->errors)) {
             return $this->object->loadFromArray($body->data[0]);
         }

src/MessageBird/Resources/Voice/Legs.php

@@ -92,13 +92,13 @@ public function getList(string $callId, array $parameters = [])
      */
     public function processRequest(?string $body): Objects\Voice\Leg
     {
-        try {
-            $body = json_decode($body, null, 512, \JSON_THROW_ON_ERROR);
-        } catch (\JsonException $e) {
+        if ($body === null) {
             throw new Exceptions\ServerException('Got an invalid JSON response from the server.');
         }
 
-        if ($body === null) {
+        try {
+            $body = json_decode($body, null, 512, \JSON_THROW_ON_ERROR);
+        } catch (\JsonException $e) {
             throw new Exceptions\ServerException('Got an invalid JSON response from the server.');
         }
 

tests/Unit/RequestValidatorTest.php

@@ -173,27 +173,28 @@ public function __construct($array)
         $this->array = $array;
     }
 
-    function rewind()
+    function rewind(): void
     {
-        return reset($this->array);
+        reset($this->array);
     }
 
-    function current()
+    function current(): array
     {
         return [current($this->array)];
     }
 
+    #[\ReturnTypeWillChange]
     function key()
     {
         return current($this->array)['name'];
     }
 
-    function next()
+    function next(): void
     {
-        return [next($this->array)];
+        next($this->array);
     }
 
-    function valid()
+    function valid(): bool
     {
         return key($this->array) !== null;
     }