Merge pull request #1 from oceanapplications/analysis-zYoya0

Apply fixes from StyleCI

src/client.php

@@ -3,21 +3,21 @@
 namespace OceanApplications\currencylayer;
 
 /**
- * PHP client for the currencylayer API
- * 
+ * PHP client for the currencylayer API.
+ *
  * @version 1.3.1
+ *
  * @link https://currencylayer.com/
  */
 class client
 {
-
     /**
-     * API base URL
+     * API base URL.
      */
     const ENDPOINT = 'http://apilayer.net/api';
-    
+
     /**
-     * API endpoint parameters
+     * API endpoint parameters.
      */
     private $source = null;
     private $currencies = null;
@@ -28,10 +28,10 @@ class client
     private $start_date = null;
     private $end_date = null;
     private $access_key = null;
-    
+
     /**
-     * Constructor
-     * 
+     * Constructor.
+     *
      * @param string $access_key
      */
     public function __construct($access_key = null)
@@ -41,183 +41,197 @@ public function __construct($access_key = null)
 
     /**
      * @param $source
+     *
      * @return $this
      */
     public function source($source)
     {
         $this->source = $source;
+
         return $this;
     }
 
     /**
      * @param $currencies
+     *
      * @return $this
      */
     public function currencies($currencies)
     {
         $this->currencies = $currencies;
+
         return $this;
     }
 
     /**
      * @param $from
+     *
      * @return mixed
      */
     public function from($from)
     {
         $this->from = $from;
+
         return $from;
     }
 
     /**
      * @param $to
+     *
      * @return $this
      */
     public function to($to)
     {
         $this->to = $to;
+
         return $this;
     }
 
     /**
      * @param $amount
+     *
      * @return $this
      */
     public function amount($amount)
     {
         $this->amount = $amount;
+
         return $this;
     }
 
     /**
      * @param $date
+     *
      * @return $this
      */
     public function date($date)
     {
         $this->date = $date;
+
         return $this;
     }
 
     /**
      * @param $start_date
+     *
      * @return $this
      */
     public function start_date($start_date)
     {
         $this->start_date = $start_date;
+
         return $this;
     }
 
     /**
      * @param $end_date
+     *
      * @return $this
      */
     public function end_date($end_date)
     {
         $this->end_date = $end_date;
+
         return $this;
     }
 
     /**
-     * Request the API's "live" endpoint
-     * 
+     * Request the API's "live" endpoint.
+     *
      * @return array
      */
     public function live()
     {
-        return $this->request('/live', array(
+        return $this->request('/live', [
             'currencies' => $this->currencies,
-            'source' => $this->source
-        ));
+            'source'     => $this->source,
+        ]);
     }
-    
+
     /**
-     * Request the API's "convert" endpoint
-     * 
+     * Request the API's "convert" endpoint.
+     *
      * @return array
      */
     public function convert()
     {
-        return $this->request('/convert', array(
-            'from' => $this->from,
-            'to' => $this->to,
+        return $this->request('/convert', [
+            'from'   => $this->from,
+            'to'     => $this->to,
             'amount' => $this->amount,
-            'date' => $this->date
-        ));
+            'date'   => $this->date,
+        ]);
     }
-    
+
     /**
-     * Request the API's "historical" endpoint
-     * 
+     * Request the API's "historical" endpoint.
+     *
      * @return array
      */
     public function historical()
     {
-        $this->request('/historical', array(
-            'date' => $this->date,
+        $this->request('/historical', [
+            'date'       => $this->date,
             'currencies' => $this->currencies,
-            'source' => $this->source
-        ));
+            'source'     => $this->source,
+        ]);
     }
-    
+
     /**
-     * Request the API's "timeframe" endpoint
-     * 
+     * Request the API's "timeframe" endpoint.
+     *
      * @return array
      */
     public function timeframe()
     {
-        return $this->request('/timeframe', array(
+        return $this->request('/timeframe', [
             'currencies' => $this->currencies,
-            'source' => $this->source,
+            'source'     => $this->source,
             'start_date' => $this->start_date,
-            'end_date' => $this->end_date
-        ));
+            'end_date'   => $this->end_date,
+        ]);
     }
 
     /**
-     * Request the API's "change" endpoint
-     * 
+     * Request the API's "change" endpoint.
+     *
      * @return array
      */
     public function change()
     {
-        return $this->request('/change', array(
+        return $this->request('/change', [
             'currencies' => $this->currencies,
-            'source' => $this->source,
+            'source'     => $this->source,
             'start_date' => $this->start_date,
-            'end_date' => $this->end_date
-        ));
+            'end_date'   => $this->end_date,
+        ]);
     }
 
     /**
-     * Execute the API request
-     * 
+     * Execute the API request.
+     *
      * @param string $endpoint
-     * @param array $params
+     * @param array  $params
+     *
      * @throws \InvalidArgumentException
+     *
      * @return string
      */
     protected function request($endpoint, $params)
     {
-
         $params['access_key'] = $this->access_key;
-        $url = self::ENDPOINT . $endpoint . '?' . http_build_query($params);
+        $url = self::ENDPOINT.$endpoint.'?'.http_build_query($params);
         $ch = curl_init();
         curl_setopt($ch, CURLOPT_URL, $url);
         curl_setopt($ch, CURLOPT_RETURNTRANSFER, true);
         $json = curl_exec($ch);
         curl_close($ch);
         $rsp = json_decode($json, true);
-    
+
         if (array_key_exists('error', $rsp)) {
             throw new \InvalidArgumentException($rsp['error']);
         }
-        
+
         return $rsp;
     }
-
 }
-
-?>