refactoring

Author: ousid

src/Concerns/InteractsWithHttpRequests.php

@@ -5,6 +5,7 @@
 use Coderflex\LaravelSendy\Exceptions\InvalidApiKeyException;
 use Coderflex\LaravelSendy\Exceptions\InvalidApiUrlException;
 use Exception;
+use Illuminate\Http\Client\Response;
 use Illuminate\Support\Facades\Http;
 
 /**
@@ -16,7 +17,7 @@
  */
 trait InteractsWithHttpRequests
 {
-    public function __call(string $function, array $args): mixed
+    public function __call(string $function, array $args): Response
     {
         $options = ['get', 'post', 'put', 'delete', 'patch'];
         $path = $args[0] ?? null;
@@ -65,7 +66,7 @@ protected function sendRequest(string $type, string $request, array $data = [],
             $client = Http::withHeaders(array_merge([
                 'Content-Type' => 'application/json',
                 'Accept' => 'application/json',
-            ], $headers ?? []));
+            ], $headers));
 
             return $async
                 ? $client->async()->{$type}($url, $payload)

src/Facades/Sendy.php

@@ -11,6 +11,7 @@
  * @method static \Coderflex\LaravelSendy\Resources\Lists lists()
  * @method static \Coderflex\LaravelSendy\Resources\Brands brands()
  * @method static \Coderflex\LaravelSendy\Resources\Campaigns campaigns()
+ * @method static mixed post(string $endpoint, array $data, bool $async = false)
  */
 class Sendy extends Facade
 {

src/Resources/Brands.php

@@ -2,12 +2,17 @@
 
 namespace Coderflex\LaravelSendy\Resources;
 
-use Coderflex\LaravelSendy\Facades\Sendy;
+use Coderflex\LaravelSendy\Sendy;
+use Illuminate\Http\Client\Response;
 
 class Brands
 {
-    public function get()
+    public function __construct(
+        protected Sendy $sendy
+    ) {}
+
+    public function get(): Response
     {
-        return Sendy::post('/api/brands/get-brands.php');
+        return $this->sendy->post('/api/brands/get-brands.php');
     }
 }

src/Resources/Campaigns.php

@@ -3,23 +3,28 @@
 namespace Coderflex\LaravelSendy\Resources;
 
 use Coderflex\LaravelSendy\DTOs\Campaigns\CampaignDTO;
-use Coderflex\LaravelSendy\Facades\Sendy;
+use Coderflex\LaravelSendy\Sendy;
+use Illuminate\Http\Client\Response;
 
 class Campaigns
 {
-    public function create(array $data, bool $async = false)
+    public function __construct(
+        protected Sendy $sendy
+    ) {}
+
+    public function create(array $data, bool $async = false): Response
     {
         $data = CampaignDTO::validate($data);
 
-        return Sendy::post('/api/campaigns/create.php', $data, $async);
+        return $this->sendy->post('/api/campaigns/create.php', $data, $async);
     }
 
-    public function createAndSend(array $data, bool $async = false)
+    public function createAndSend(array $data, bool $async = false): Response
     {
         $data = array_merge($data, [
             'send_compaign' => 1,
         ]);
 
-        return Sendy::post('/api/campaigns/create.php', $data, $async);
+        return $this->sendy->post('/api/campaigns/create.php', $data, $async);
     }
 }

src/Resources/Lists.php

@@ -3,19 +3,19 @@
 namespace Coderflex\LaravelSendy\Resources;
 
 use Coderflex\LaravelSendy\DTOs\Lists\ListsDTO;
-use Coderflex\LaravelSendy\Facades\Sendy;
+use Coderflex\LaravelSendy\Sendy;
+use Illuminate\Http\Client\Response;
 
 class Lists
 {
-    /**
-     * Get all lists for a specific brand.
-     *
-     * @return array
-     */
-    public function get(array $data, bool $async = false)
+    public function __construct(
+        protected Sendy $sendy
+    ) {}
+
+    public function get(array $data, bool $async = false): Response
     {
         $data = ListsDTO::validate($data);
 
-        return Sendy::post('/api/lists/get-lists.php', $data, $async);
+        return $this->sendy->post('/api/lists/get-lists.php', $data, $async);
     }
 }

src/Resources/Subscribers.php

@@ -6,44 +6,49 @@
 use Coderflex\LaravelSendy\DTOs\Subscribers\SubscribeDTO;
 use Coderflex\LaravelSendy\DTOs\Subscribers\SubscriberStatusDTO;
 use Coderflex\LaravelSendy\DTOs\Subscribers\UnsubscribeDTO;
-use Coderflex\LaravelSendy\Facades\Sendy;
+use Coderflex\LaravelSendy\Sendy;
+use Illuminate\Http\Client\Response;
 
 class Subscribers
 {
-    public function subscribe(array $data, bool $async = false)
+    public function __construct(
+        protected Sendy $sendy
+    ) {}
+
+    public function subscribe(array $data, bool $async = false): Response
     {
         $data = SubscribeDTO::validate($data);
 
-        return Sendy::post('subscribe', $data, $async);
+        return $this->sendy->post('subscribe', $data, $async);
     }
 
-    public function unsubscribe(array $data, bool $async = false)
+    public function unsubscribe(array $data, bool $async = false): Response
     {
         $data = UnsubscribeDTO::validate($data);
 
-        return Sendy::post('api/subscribers/unsubscribe.php', $data, $async);
+        return $this->sendy->post('api/subscribers/unsubscribe.php', $data, $async);
     }
 
-    public function delete(array $data, bool $async = false)
+    public function delete(array $data, bool $async = false): Response
     {
         $data = DeleteSubscriberDTO::validate($data);
 
-        return Sendy::post('api/subscribers/delete.php', $data, $async);
+        return $this->sendy->post('api/subscribers/delete.php', $data, $async);
     }
 
-    public function status(array $data, bool $async = false)
+    public function status(array $data, bool $async = false): Response
     {
         $data = SubscriberStatusDTO::validate($data);
 
-        return Sendy::post('api/subscribers/subscription-status.php', $data, $async);
+        return $this->sendy->post('api/subscribers/subscription-status.php', $data, $async);
     }
 
-    public function count(int $listId, bool $async = false)
+    public function count(int $listId, bool $async = false): Response
     {
         $data = [
             'list_id' => $listId,
         ];
 
-        return Sendy::post('api/subscribers/subscriber-count.php', $data, $async);
+        return $this->sendy->post('api/subscribers/subscriber-count.php', $data, $async);
     }
 }

src/Sendy.php

@@ -10,21 +10,21 @@ class Sendy
 
     public function subscribers(): Resources\Subscribers
     {
-        return new Resources\Subscribers;
+        return new Resources\Subscribers($this);
     }
 
     public function lists(): Resources\Lists
     {
-        return new Resources\Lists;
+        return new Resources\Lists($this);
     }
 
     public function brands(): Resources\Brands
     {
-        return new Resources\Brands;
+        return new Resources\Brands($this);
     }
 
     public function campaigns(): Resources\Campaigns
     {
-        return new Resources\Campaigns;
+        return new Resources\Campaigns($this);
     }
 }