Complete package restructure (#2)

* Complete package restructure

Restructured whole package to support twilio sms service and dynamical prediction based on gateway priority configuration.

* DOC: PHP Doc fix

* TASK: Reformat code

Author: vigneshgurusamy

src/Adapters/Adapter.php

@@ -0,0 +1,38 @@
+<?php
+
+namespace Linkstreet\LaravelSms\Adapters;
+
+use Linkstreet\LaravelSms\Exceptions\AdapterException;
+
+class Adapter
+{
+    /**
+     * List of adapter implementations
+     *
+     * @var array
+     */
+    private static $list = [
+        'twilio' => Twilio\TwilioAdapter::class,
+        'kap' => Kap\KapAdapter::class,
+    ];
+
+    /**
+     * @param string $adapter
+     * @return mixed|static
+     * @throws AdapterException
+     */
+    public static function find(string $adapter)
+    {
+        return isset(self::$list[$adapter])
+            ? self::$list[$adapter]
+            : AdapterException::notFound($adapter);
+    }
+
+    /**
+     * @return array
+     */
+    public static function all()
+    {
+        return self::$list;
+    }
+}

src/Adapters/BaseAdapter.php

@@ -0,0 +1,21 @@
+<?php
+
+namespace Linkstreet\LaravelSms\Adapters;
+
+use GuzzleHttp\Client;
+
+trait HttpClient
+{
+    protected $client;
+
+    /**
+     * @param Client|null $client
+     * @return self
+     */
+    public function setClient($client = null)
+    {
+        $this->client = $client ?? new Client();
+
+        return $this;
+    }
+}

src/Adapters/HttpClient.php

@@ -2,66 +2,69 @@
 
 namespace Linkstreet\LaravelSms\Adapters\Kap;
 
-use GuzzleHttp\Client;
 use GuzzleHttp\Psr7\Request;
-use Illuminate\Support\Facades\Config;
-use Linkstreet\LaravelSms\Adapters\BaseAdapter;
+use Linkstreet\LaravelSms\Adapters\HttpClient;
 use Linkstreet\LaravelSms\Contracts\AdapterInterface;
+use Linkstreet\LaravelSms\Contracts\ResponseInterface;
 use Linkstreet\LaravelSms\Exceptions\AdapterException;
+use Linkstreet\LaravelSms\Model\Device;
 
 /**
  * KapAdapter
  */
-class KapAdapter extends BaseAdapter implements AdapterInterface
+class KapAdapter implements AdapterInterface
 {
+    use HttpClient;
+
+    /**
+     * @var array
+     */
+    private $config;
+
     /**
      * Create a instance of Kap Adapter
-     *
-     * @param array|null configuration for KAP adapter
+     * @param array configuration for KAP adapter
      */
-    public function __construct($config = null)
+    public function __construct(array $config)
     {
-        $this->config = $this->requiredConfig($config);
-        $this->setClient(new Client());
+        $this->config = $config;
+
+        $this->setClient();
     }
 
     /**
      * {@inheritdoc}
      */
-    public function send($devices, $message)
+    public function send(Device $device, string $message): ResponseInterface
     {
-        $this->response = $this->client->send($this->buildRequest(), $this->buildOptions($devices, $message));
-        return new KapResponse($devices, $this->response);
+        $this->checkForMissingConfiguration();
+
+        $response = $this->client->send($this->buildRequest(), $this->buildOptions($device, $message));
+
+        return new KapResponse($device, $response);
     }
 
     /**
      * Build Guzzle request object
-     *
      * @return Request
      */
-    private function buildRequest()
+    private function buildRequest(): Request
     {
-        return new Request('POST', 'http://api.kapsystem.com/api/v3/sendsms/json');
+        return new Request('POST', 'https://api.kapsystem.com/api/v3/sendsms/json');
     }
 
     /**
      * Build Guzzle query options with json payload
-     *
-     * @param DeviceCollection $devices
-     * @param Message $message
+     * @param Device $device
+     * @param string $message
      * @return array
      */
-    private function buildOptions($devices, $message)
+    private function buildOptions(Device $device, string $message): array
     {
-        $recipients = [];
-
-        foreach ($devices as $device) {
-            $recipients[]['gsm'] = $device->getNumber();
-        }
-
         return [
             'debug' => false,
-            'timeout' => 10,
+            'verify' => false,
+            'timeout' => 20,
             'json' => [
                 'authentication' => [
                     'username' => $this->config['username'],
@@ -70,43 +73,27 @@ private function buildOptions($devices, $message)
                 'messages' => [
                     [
                         'sender' => $this->config['sender'],
-                        'text' => $message->getMessage(),
+                        'text' => $message,
                         'type' => 'longSMS',
-                        'recipients' => $recipients
+                        'recipients' => [
+                            ['gsm' => $device->getNumberWithoutPlusSign()]
+                        ],
                     ]
                 ]
             ]
         ];
     }
 
     /**
-     * Check & retrieve the config for KAP adapter
-     *
-     * @param array|null configuration for KAP adapter
-     * @return array
+     * Check for valid configuration
      * @throws AdapterException
      */
-    private function requiredConfig($config = null)
+    private function checkForMissingConfiguration()
     {
-        if (is_null($config)) {
-            $config = Config::get('sms.adapter.kap');
-        }
-
-        // Check for username in the config
-        if (!isset($config['username']) || empty($config['username'])) {
-            throw new AdapterException('Invalid username');
-        }
-
-        // Check for password in the config
-        if (!isset($config['password']) || empty($config['password'])) {
-            throw new AdapterException('Invalid password');
-        }
+        $config = $this->config;
 
-        // Check for sender id in the config
-        if (!isset($config['sender']) || empty($config['sender'])) {
-            throw new AdapterException('Invalid sender id');
+        if (!isset($config['username'], $config['password'], $config['sender'])) {
+            throw AdapterException::missingConfiguration();
         }
-
-        return $config;
     }
 }