since release 2.8.0 and #129 the argument $client has the wrong type and breaks the library

Author: BenjaminLeibinger

src/HetznerAPIClient.php

@@ -2,7 +2,6 @@
 
 namespace LKDev\HetznerCloud;
 
-use GuzzleHttp\Client;
 use LKDev\HetznerCloud\Clients\GuzzleClient;
 use LKDev\HetznerCloud\Models\Actions\Actions;
 use LKDev\HetznerCloud\Models\Certificates\Certificates;
@@ -122,18 +121,12 @@ public function setBaseUrl(string $baseUrl): self
         return $this;
     }
 
-    /**
-     * @return Client
-     */
     public function getHttpClient(): GuzzleClient
     {
         return $this->httpClient;
     }
 
-    /**
-     * @return Client
-     */
-    public function setHttpClient(Client $client): self
+    public function setHttpClient(GuzzleClient $client): self
     {
         $this->httpClient = $client;
 

src/Models/Model.php

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

src/Models/Networks/Network.php

@@ -2,7 +2,7 @@
 
 namespace LKDev\HetznerCloud\Models\Networks;
 
-use GuzzleHttp\Client;
+use LKDev\HetznerCloud\Clients\GuzzleClient;
 use LKDev\HetznerCloud\APIResponse;
 use LKDev\HetznerCloud\HetznerAPIClient;
 use LKDev\HetznerCloud\Models\Actions\Action;
@@ -71,9 +71,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

@@ -2,7 +2,7 @@
 
 namespace LKDev\HetznerCloud\Models\Networks;
 
-use GuzzleHttp\Client;
+use LKDev\HetznerCloud\Clients\GuzzleClient;
 use LKDev\HetznerCloud\Models\Model;
 
 /**
@@ -24,9 +24,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 +35,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

@@ -2,7 +2,7 @@
 
 namespace LKDev\HetznerCloud\Models\Networks;
 
-use GuzzleHttp\Client;
+use LKDev\HetznerCloud\Clients\GuzzleClient;
 use LKDev\HetznerCloud\Models\Model;
 
 /**
@@ -36,9 +36,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 +49,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

@@ -2,7 +2,7 @@
 
 namespace LKDev\HetznerCloud\Models\PlacementGroups;
 
-use GuzzleHttp\Client;
+use LKDev\HetznerCloud\Clients\GuzzleClient;
 use LKDev\HetznerCloud\APIResponse;
 use LKDev\HetznerCloud\HetznerAPIClient;
 use LKDev\HetznerCloud\Models\Actions\Action;
@@ -49,9 +49,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

@@ -9,7 +9,7 @@
 
 namespace LKDev\HetznerCloud\Models\Servers;
 
-use GuzzleHttp\Client;
+use LKDev\HetznerCloud\Clients\GuzzleClient;
 use LKDev\HetznerCloud\APIResponse;
 use LKDev\HetznerCloud\HetznerAPIClient;
 use LKDev\HetznerCloud\Models\Actions\Action;
@@ -180,9 +180,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

@@ -9,7 +9,7 @@
 
 namespace LKDev\HetznerCloud\Models\Volumes;
 
-use GuzzleHttp\Client;
+use LKDev\HetznerCloud\Clients\GuzzleClient;
 use LKDev\HetznerCloud\APIResponse;
 use LKDev\HetznerCloud\HetznerAPIClient;
 use LKDev\HetznerCloud\Models\Actions\Action;
@@ -65,9 +65,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);