Fix Invalid argument errors introduced with 2.8.0

Superseeds #136

Author: LKaemmerling

src/HetznerAPIClient.php

@@ -60,7 +60,7 @@ class HetznerAPIClient
     /**
      * @var \LKDev\HetznerCloud\Clients\GuzzleClient
      */
-    protected $httpClient;
+    protected GuzzleClient $httpClient;
 
     /**
      * @param  string  $apiToken
@@ -123,17 +123,17 @@ public function setBaseUrl(string $baseUrl): self
     }
 
     /**
-     * @return Client
+     * @return GuzzleClient
      */
-    public function getHttpClient(): Client
+    public function getHttpClient(): GuzzleClient
     {
         return $this->httpClient;
     }
 
     /**
-     * @return Client
+     * @return GuzzleClient
      */
-    public function setHttpClient(Client $client): self
+    public function setHttpClient(GuzzleClient $client): self
     {
         $this->httpClient = $client;
 

src/Models/Model.php

@@ -3,21 +3,22 @@
 namespace LKDev\HetznerCloud\Models;
 
 use GuzzleHttp\Client;
+use LKDev\HetznerCloud\Clients\GuzzleClient;
 use LKDev\HetznerCloud\HetznerAPIClient;
 
 abstract class Model
 {
     /**
-     * @var \GuzzleHttp\Client
+     * @var GuzzleClient
      */
     protected $httpClient;
 
     /**
      * Model constructor.
      *
-     * @param  Client  $httpClient
+     * @param  GuzzleClient  $httpClient
      */
-    public function __construct(?Client $httpClient = null)
+    public function __construct(?GuzzleClient $httpClient = null)
     {
         $this->httpClient = $httpClient == null ? HetznerAPIClient::$instance->getHttpClient() : $httpClient;
     }
@@ -34,9 +35,9 @@ public static function parse($input)
     /**
      * Replaces or sets the http client.
      *
-     * @param  Client  $httpClient
+     * @param  GuzzleClient  $httpClient
      */
-    public function setHttpClient(?Client $httpClient = null)
+    public function setHttpClient(?GuzzleClient $httpClient = null)
     {
         $this->httpClient = $httpClient;
     }

src/Models/Networks/Network.php

@@ -4,6 +4,7 @@
 
 use GuzzleHttp\Client;
 use LKDev\HetznerCloud\APIResponse;
+use LKDev\HetznerCloud\Clients\GuzzleClient;
 use LKDev\HetznerCloud\HetznerAPIClient;
 use LKDev\HetznerCloud\Models\Actions\Action;
 use LKDev\HetznerCloud\Models\Contracts\Resource;
@@ -71,9 +72,9 @@ class Network extends Model implements Resource
      * Network constructor.
      *
      * @param  int  $id
-     * @param  Client|null  $httpClient
+     * @param  GuzzleClient|null  $httpClient
      */
-    public function __construct(int $id, ?Client $httpClient = null)
+    public function __construct(int $id, ?GuzzleClient $httpClient = null)
     {
         $this->id = $id;
         parent::__construct($httpClient);

src/Models/Networks/Route.php

@@ -3,6 +3,7 @@
 namespace LKDev\HetznerCloud\Models\Networks;
 
 use GuzzleHttp\Client;
+use LKDev\HetznerCloud\Clients\GuzzleClient;
 use LKDev\HetznerCloud\Models\Model;
 
 /**
@@ -24,9 +25,9 @@ class Route extends Model
      *
      * @param  string  $destination
      * @param  string  $gateway
-     * @param  Client|null  $client
+     * @param  GuzzleClient|null  $client
      */
-    public function __construct(string $destination, string $gateway, ?Client $client = null)
+    public function __construct(string $destination, string $gateway, ?GuzzleClient $client = null)
     {
         $this->destination = $destination;
         $this->gateway = $gateway;
@@ -35,10 +36,10 @@ public function __construct(string $destination, string $gateway, ?Client $clien
 
     /**
      * @param  $input
-     * @param  Client|null  $client
+     * @param  GuzzleClient|null  $client
      * @return array|Model
      */
-    public static function parse($input, ?Client $client = null)
+    public static function parse($input, ?GuzzleClient $client = null)
     {
         return collect($input)->map(function ($route) use ($client) {
             return new self($route->destination, $route->gateway, $client);

src/Models/Networks/Subnet.php

@@ -3,6 +3,7 @@
 namespace LKDev\HetznerCloud\Models\Networks;
 
 use GuzzleHttp\Client;
+use LKDev\HetznerCloud\Clients\GuzzleClient;
 use LKDev\HetznerCloud\Models\Model;
 
 /**
@@ -36,9 +37,9 @@ class Subnet extends Model
      * @param  string  $ipRange
      * @param  string  $networkZone
      * @param  string  $gateway
-     * @param  Client|null  $client
+     * @param  GuzzleClient|null  $client
      */
-    public function __construct(string $type, string $ipRange, string $networkZone, ?string $gateway = null, ?Client $client = null)
+    public function __construct(string $type, string $ipRange, string $networkZone, ?string $gateway = null, ?GuzzleClient $client = null)
     {
         $this->type = $type;
         $this->ipRange = $ipRange;
@@ -49,10 +50,10 @@ public function __construct(string $type, string $ipRange, string $networkZone,
 
     /**
      * @param  $input
-     * @param  Client|null  $client
+     * @param  GuzzleClient|null  $client
      * @return array|Model
      */
-    public static function parse($input, ?Client $client = null)
+    public static function parse($input, ?GuzzleClient $client = null)
     {
         return collect($input)->map(function ($subnet) use ($client) {
             return new self($subnet->type, $subnet->ip_range, $subnet->network_zone, $subnet->gateway, $client);

src/Models/PlacementGroups/PlacementGroup.php

@@ -4,6 +4,7 @@
 
 use GuzzleHttp\Client;
 use LKDev\HetznerCloud\APIResponse;
+use LKDev\HetznerCloud\Clients\GuzzleClient;
 use LKDev\HetznerCloud\HetznerAPIClient;
 use LKDev\HetznerCloud\Models\Actions\Action;
 use LKDev\HetznerCloud\Models\Contracts\Resource;
@@ -49,9 +50,9 @@ class PlacementGroup extends Model implements Resource
      * PlacementGroup constructor.
      *
      * @param  int  $id
-     * @param  Client|null  $httpClient
+     * @param  GuzzleClient|null  $httpClient
      */
-    public function __construct(int $id, ?Client $httpClient = null)
+    public function __construct(int $id, ?GuzzleClient $httpClient = null)
     {
         $this->id = $id;
         parent::__construct($httpClient);

src/Models/Servers/Server.php

@@ -11,6 +11,7 @@
 
 use GuzzleHttp\Client;
 use LKDev\HetznerCloud\APIResponse;
+use LKDev\HetznerCloud\Clients\GuzzleClient;
 use LKDev\HetznerCloud\HetznerAPIClient;
 use LKDev\HetznerCloud\Models\Actions\Action;
 use LKDev\HetznerCloud\Models\Contracts\Resource;
@@ -180,9 +181,9 @@ class Server extends Model implements Resource
 
     /**
      * @param  int  $serverId
-     * @param  Client|null  $httpClient
+     * @param  GuzzleClient|null  $httpClient
      */
-    public function __construct(int $serverId, ?Client $httpClient = null)
+    public function __construct(int $serverId, ?GuzzleClient $httpClient = null)
     {
         $this->id = $serverId;
         parent::__construct($httpClient);

src/Models/Volumes/Volume.php

@@ -11,6 +11,7 @@
 
 use GuzzleHttp\Client;
 use LKDev\HetznerCloud\APIResponse;
+use LKDev\HetznerCloud\Clients\GuzzleClient;
 use LKDev\HetznerCloud\HetznerAPIClient;
 use LKDev\HetznerCloud\Models\Actions\Action;
 use LKDev\HetznerCloud\Models\Contracts\Resource;
@@ -65,9 +66,9 @@ class Volume extends Model implements Resource
 
     /**
      * @param  int  $volumeId
-     * @param  Client|null  $httpClient
+     * @param  GuzzleClient|null  $httpClient
      */
-    public function __construct(?int $volumeId = null, ?Client $httpClient = null)
+    public function __construct(?int $volumeId = null, ?GuzzleClient $httpClient = null)
     {
         $this->id = $volumeId;
         parent::__construct($httpClient);