Merge pull request #35 from VincentBean/master

Add delete and put methods

Author: ahinkle

src/Api/AbstractApi.php

@@ -49,8 +49,8 @@ protected function constructRequest()
     /**
      * Send a GET request with query parameters.
      *
-     * @param  string  $path
-     * @param  string  $parameters
+     * @param string $path
+     * @param string $parameters
      *
      * @return \Illuminate\Http\Client\Response
      */
@@ -63,8 +63,8 @@ protected function get($path, $parameters = [])
     /**
      * Send a POST request with query parameters.
      *
-     * @param  string  $path
-     * @param  string  $parameters
+     * @param string $path
+     * @param string $parameters
      *
      * @return \Illuminate\Http\Client\Response
      */
@@ -74,12 +74,40 @@ protected function post($path, $parameters = [])
             ->post($this->apiRequest.$path, $parameters));
     }
 
+    /**
+     * Send a PUT request.
+     *
+     * @param $path
+     * @param array $parameters
+     *
+     * @return \Illuminate\Http\Client\Response|void
+     */
+    protected function put($path, $parameters = [])
+    {
+        return $this->checkExceptions(Http::withToken($this->magento->token)
+            ->put($this->apiRequest.$path, $parameters));
+    }
+
+    /**
+     * Send a DELETE request.
+     *
+     * @param $path
+     * @param array $parameters
+     *
+     * @return \Illuminate\Http\Client\Response|void
+     */
+    protected function delete($path, $parameters = [])
+    {
+        return $this->checkExceptions(Http::withToken($this->magento->token)
+            ->delete($this->apiRequest.$path, $parameters));
+    }
+
     /**
      * Check for any type of invalid API Responses.
      *
      * @param \Illuminate\Http\Client\Response $response
-     * @throws \Exception
      * @return void
+     * @throws \Exception
      */
     protected function checkExceptions($response)
     {
@@ -93,8 +121,8 @@ protected function checkExceptions($response)
     /**
      * Validates the usage of the store code as needed.
      *
-     * @throws \Exception
      * @return void
+     * @throws \Exception
      */
     protected function validateSingleStoreCode()
     {

src/Api/Custom.php

@@ -6,6 +6,8 @@
 
 class Custom extends AbstractApi
 {
+    const HTTP_METHODS = ['get', 'post', 'put', 'delete'];
+
     /**
      * @var string Magento API endpoint
      */
@@ -18,7 +20,7 @@ class Custom extends AbstractApi
      */
     public function __construct(string $endpoint, Magento $magento)
     {
-        $this->endpoint = $endpoint;
+        $this->endpoint = '/'.rtrim(ltrim($endpoint, '/'), '/').'/';
 
         parent::__construct($magento);
     }
@@ -32,8 +34,8 @@ public function __construct(string $endpoint, Magento $magento)
      */
     public function __call($method, $args)
     {
-        if ($method == 'get' || $method == 'post') {
-            $args[0] = rtrim($this->endpoint, '/').'/'.$args[0];
+        if (in_array($method, self::HTTP_METHODS)) {
+            $args[0] = $this->endpoint.ltrim($args[0], '/');
         }
 
         return call_user_func_array([$this, $method], $args);