Fix #52

Author: keithbrink

src/AmazonProductInfo.php

@@ -70,7 +70,7 @@ public function setSKUs($s)
                 $this->resetSKUs();
                 $i = 1;
                 foreach ($s as $x) {
-                    $this->options['SellerSKUList.SellerSKU.'.$i] = $x;
+                    $this->options['SellerSKUList.SellerSKU.' . $i] = $x;
                     $i++;
                 }
             } else {
@@ -116,7 +116,7 @@ public function setASINs($s)
                 $this->resetASINs();
                 $i = 1;
                 foreach ($s as $x) {
-                    $this->options['ASINList.ASIN.'.$i] = $x;
+                    $this->options['ASINList.ASIN.' . $i] = $x;
                     $i++;
                 }
             } else {
@@ -203,7 +203,7 @@ public function fetchCompetitivePricing()
 
         $this->prepareCompetitive();
 
-        $url = $this->urlbase.$this->urlbranch;
+        $url = $this->urlbase . $this->urlbranch;
 
         $query = $this->genQuery();
 
@@ -273,7 +273,7 @@ public function fetchLowestOffer()
 
         $this->prepareLowest();
 
-        $url = $this->urlbase.$this->urlbranch;
+        $url = $this->urlbase . $this->urlbranch;
 
         $query = $this->genQuery();
 
@@ -338,7 +338,7 @@ public function fetchMyPrice()
 
         $this->prepareMyPrice();
 
-        $url = $this->urlbase.$this->urlbranch;
+        $url = $this->urlbase . $this->urlbranch;
 
         $query = $this->genQuery();
 
@@ -394,19 +394,16 @@ protected function prepareMyPrice()
      */
     public function fetchCategories()
     {
-        if (! array_key_exists('SellerSKUList.SellerSKU.1', $this->options) && ! array_key_exists(
-            'ASINList.ASIN.1',
-            $this->options
-        )
-        ) {
+        if (! array_key_exists('SellerSKUList.SellerSKU.1', $this->options) &&
+            ! array_key_exists('ASINList.ASIN.1', $this->options)) {
             $this->log('Product IDs must be set in order to look them up!', 'Warning');
 
             return false;
         }
 
         $this->prepareCategories();
 
-        $url = $this->urlbase.$this->urlbranch;
+        $url = $this->urlbase . $this->urlbranch;
 
         $query = $this->genQuery();
 
@@ -448,6 +445,7 @@ protected function prepareCategories()
         } else {
             if (array_key_exists('ASINList.ASIN.1', $this->options)) {
                 $this->options['Action'] = 'GetProductCategoriesForASIN';
+                $this->options['ASIN'] = $this->options['ASINList.ASIN.1'];
                 $this->resetSKUs();
             }
         }
@@ -471,7 +469,7 @@ public function fetchMatchingProduct()
 
         $this->prepareMatchingProduct();
 
-        $url = $this->urlbase.$this->urlbranch;
+        $url = $this->urlbase . $this->urlbranch;
 
         $query = $this->genQuery();
 

tests/classes/AmazonProductInfoTest.php

@@ -227,6 +227,15 @@ public function testFetchCategories()
         resetLog();
         $this->object->setMock(true, 'fetchCategories.xml');
         $this->assertFalse($this->object->fetchCategories()); //no IDs yet
+
+        // Test ASINs
+        $this->object->setASINs('ASINTEST');
+        $this->assertNull($this->object->fetchCategories());
+        $o = $this->object->getOptions();
+        $this->assertEquals('GetProductCategoriesForASIN', $o['Action']);
+        $this->assertEquals('ASINTEST', $o['ASIN']);
+
+        // Test SKUs
         $this->object->setSKUs('789');
         $this->assertNull($this->object->fetchCategories());
         $o = $this->object->getOptions();
@@ -250,7 +259,8 @@ public function testGetProductCategories($o)
         $list = $o->getProduct(null);
         $this->assertIsArray($list);
         $this->assertArrayHasKey(0, $list);
-        $this->assertArrayNotHasKey(1, $list);
+        $this->assertArrayHasKey(1, $list);
+        $this->assertArrayNotHasKey(2, $list);
         $this->assertEquals($product, $list[0]);
 
         $default = $o->getProduct();
@@ -308,4 +318,4 @@ public function testGetMatchingProduct($o)
     }
 }
 
-require_once __DIR__.'/../helperFunctions.php';
+require_once __DIR__ . '/../helperFunctions.php';