Merge pull request #243 from Rickyxstar/master

Use product custom fields endpoints form official documentation to stop random 401 status codes.

Author: bookernath

src/Bigcommerce/Api/Client.php

@@ -495,7 +495,7 @@ public static function getProductImages($id)
      */
     public static function getProductCustomFields($id)
     {
-        return self::getCollection('/products/' . $id . '/customfields', 'ProductCustomField');
+        return self::getCollection('/products/' . $id . '/custom_fields', 'ProductCustomField');
     }
 
     /**
@@ -506,7 +506,7 @@ public static function getProductCustomFields($id)
      */
     public static function getProductCustomField($product_id, $id)
     {
-        return self::getResource('/products/' . $product_id . '/customfields/' . $id, 'ProductCustomField');
+        return self::getResource('/products/' . $product_id . '/custom_fields/' . $id, 'ProductCustomField');
     }
 
     /**
@@ -518,7 +518,7 @@ public static function getProductCustomField($product_id, $id)
      */
     public static function createProductCustomField($product_id, $object)
     {
-        return self::createResource('/products/' . $product_id . '/customfields', $object);
+        return self::createResource('/products/' . $product_id . '/custom_fields', $object);
     }
 
     /**
@@ -542,7 +542,7 @@ public static function getProductReviews($id)
      */
     public static function updateProductCustomField($product_id, $id, $object)
     {
-        return self::updateResource('/products/' . $product_id . '/customfields/' . $id, $object);
+        return self::updateResource('/products/' . $product_id . '/custom_fields/' . $id, $object);
     }
 
     /**
@@ -554,7 +554,7 @@ public static function updateProductCustomField($product_id, $id, $object)
      */
     public static function deleteProductCustomField($product_id, $id)
     {
-        return self::deleteResource('/products/' . $product_id . '/customfields/' . $id);
+        return self::deleteResource('/products/' . $product_id . '/custom_fields/' . $id);
     }
 
     /**
@@ -1916,7 +1916,7 @@ public static function updateOptionValue($optionId, $optionValueId, $object)
             $object
         );
     }
-    
+
     /**
      * Returns all webhooks.
      *
@@ -1926,7 +1926,7 @@ public static function listWebhooks()
     {
         return self::getCollection('/hooks');
     }
-    
+
     /**
      * Returns data for a specific web-hook.
      *
@@ -1937,7 +1937,7 @@ public static function getWebhook($id)
     {
         return self::getResource('/hooks/' . $id);
     }
-    
+
     /**
      * Creates a web-hook.
      *
@@ -1948,7 +1948,7 @@ public static function createWebhook($object)
     {
         return self::createResource('/hooks', $object);
     }
-    
+
     /**
      * Updates the given webhook.
      *
@@ -1960,7 +1960,7 @@ public static function updateWebhook($id, $object)
     {
         return self::updateResource('/hooks/' . $id, $object);
     }
-    
+
     /**
      * Delete the given webhook.
      *
@@ -2040,7 +2040,7 @@ public static function deleteShippingMethod($zoneId, $methodId)
     {
         return self::deleteResource('/shipping/zones/'. $zoneId . '/methods/'. $methodId);
     }
-    
+
     /**
      * Get collection of product skus by Product
      *

test/Unit/Api/ClientTest.php

@@ -449,7 +449,7 @@ public function testGettingProductCustomFieldsReturnsCollectionOfProductCustomFi
     {
         $this->connection->expects($this->once())
             ->method('get')
-            ->with($this->basePath . '/products/1/customfields', false)
+            ->with($this->basePath . '/products/1/custom_fields', false)
             ->will($this->returnValue(array(array(), array())));
 
         $collection = Client::getProductCustomFields(1);
@@ -474,7 +474,7 @@ public function testGettingASpecifiedProductCustomFieldReturnsThatProductCustomF
     {
         $this->connection->expects($this->once())
             ->method('get')
-            ->with($this->basePath . '/products/1/customfields/1', false)
+            ->with($this->basePath . '/products/1/custom_fields/1', false)
             ->will($this->returnValue(array(array(), array())));
 
         $resource = Client::getProductCustomField(1, 1);
@@ -560,7 +560,7 @@ public function testCreatingAProductCustomFieldPostsToTheProductCustomFieldResou
     {
         $this->connection->expects($this->once())
             ->method('post')
-            ->with($this->basePath . '/products/1/customfields', (object)array());
+            ->with($this->basePath . '/products/1/custom_fields', (object)array());
 
         Client::createProductCustomField(1, array());
     }
@@ -578,7 +578,7 @@ public function testUpdatingAProductCustomFieldPutsToTheProductCustomFieldResour
     {
         $this->connection->expects($this->once())
             ->method('put')
-            ->with($this->basePath . '/products/1/customfields/1', (object)array());
+            ->with($this->basePath . '/products/1/custom_fields/1', (object)array());
 
         Client::updateProductCustomField(1, 1, array());
     }
@@ -596,7 +596,7 @@ public function testDeletingAProductCustomFieldDeletesToTheProductCustomFieldRes
     {
         $this->connection->expects($this->once())
             ->method('delete')
-            ->with($this->basePath . '/products/1/customfields/1');
+            ->with($this->basePath . '/products/1/custom_fields/1');
 
         Client::deleteProductCustomField(1, 1);
     }
@@ -852,8 +852,8 @@ public function testDeletingAllGiftCertificatesDeletesToTheAllGiftCertificatesRe
 
         Client::deleteAllGiftCertificates();
     }
-    
-    
+
+
     public function testGettingWebhooksReturnsAllWebhooks()
     {
         $this->connection->expects($this->once())
@@ -866,7 +866,7 @@ public function testGettingWebhooksReturnsAllWebhooks()
             $this->assertInstanceOf('Bigcommerce\\Api\\Resource', $resource);
         }
     }
-    
+
     public function testGettingSpecifiedWebhookReturnsTheSpecifiedWebhook()
     {
         $this->connection->expects($this->once())
@@ -876,7 +876,7 @@ public function testGettingSpecifiedWebhookReturnsTheSpecifiedWebhook()
         $resource = Client::getWebhook(1);
         $this->assertInstanceOf('Bigcommerce\\Api\\Resource', $resource);
     }
-    
+
     public function testCreatingWebhookPostsToTheSpecifiedResource()
     {
         $this->connection->expects($this->once())
@@ -891,7 +891,7 @@ public function testUpdatingWebhookPutsToTheSpecifiedResource()
             ->with($this->basePath . '/hooks/1', (object)array());
         Client::updateWebhook(1, array());
     }
-    
+
     public function testDeleteWebhookDeletesToTheSpecifiedResource()
     {
         $this->connection->expects($this->once())