Merge pull request #6 from keithbrink/analysis-qBeyOV

Apply fixes from StyleCI

Author: keithbrink

src/AmazonPrepInfo.php

@@ -35,12 +35,14 @@ class AmazonPrepInfo extends AmazonInboundCore implements Iterator
     protected $i = 0;
 
     /**
-     * Sets the seller SKU(s). (Required*)
+     * Sets the seller SKU(s). (Required*).
      *
      * This method sets the list of seller SKUs to be sent in the next request.
      * If this parameter is set, ASINs cannot be set.
+     *
      * @param array|string $s <p>A list of Seller SKUs, or a single SKU string. (max: 20)</p>
-     * @return boolean <b>FALSE</b> if improper input
+     *
+     * @return bool <b>FALSE</b> if improper input
      */
     public function setSkus($s)
     {
@@ -69,19 +71,21 @@ public function setSkus($s)
     private function resetSkus()
     {
         foreach ($this->options as $op=>$junk) {
-            if (preg_match("#SellerSKUList#", $op)) {
+            if (preg_match('#SellerSKUList#', $op)) {
                 unset($this->options[$op]);
             }
         }
     }
 
     /**
-     * Sets the ASIN(s). (Required*)
+     * Sets the ASIN(s). (Required*).
      *
      * This method sets the list of ASINs to be sent in the next request.
      * If this parameter is set, Seller SKUs cannot be set.
+     *
      * @param array|string $s <p>A list of ASINs, or a single ASIN string. (max: 20)</p>
-     * @return boolean <b>FALSE</b> if improper input
+     *
+     * @return bool <b>FALSE</b> if improper input
      */
     public function setAsins($s)
     {
@@ -110,7 +114,7 @@ public function setAsins($s)
     private function resetAsins()
     {
         foreach ($this->options as $op=>$junk) {
-            if (preg_match("#ASINList#", $op)) {
+            if (preg_match('#ASINList#', $op)) {
                 unset($this->options[$op]);
             }
         }
@@ -126,13 +130,15 @@ private function resetAsins()
      * Other methods are available for fetching specific values from the list.
      * A list of items that were deemed invalid can also be retrieved using
      * <i>getInvalidItemList</i>.
-     * @return boolean <b>FALSE</b> if something goes wrong
+     *
+     * @return bool <b>FALSE</b> if something goes wrong
      */
     public function fetchPrepInstructions()
     {
         if (!array_key_exists('SellerSKUList.Id.1', $this->options) &&
                 !array_key_exists('ASINList.Id.1', $this->options)) {
-            $this->log("Product IDs must be set in order to get prep instructions!", 'Warning');
+            $this->log('Product IDs must be set in order to get prep instructions!', 'Warning');
+
             return false;
         }
 
@@ -181,8 +187,10 @@ protected function preparePrep()
      * Parses XML response into array.
      *
      * This is what reads the response XML and converts it into an array.
+     *
      * @param SimpleXMLElement $xml <p>The XML response from Amazon.</p>
-     * @return boolean <b>FALSE</b> if no XML data is found
+     *
+     * @return bool <b>FALSE</b> if no XML data is found
      */
     protected function parseXml($xml)
     {
@@ -202,20 +210,20 @@ protected function parseXml($xml)
             foreach ($list->children() as $x) {
                 $temp = [];
                 if (isset($x->SellerSKU)) {
-                    $temp['SellerSKU'] = (string)$x->SellerSKU;
+                    $temp['SellerSKU'] = (string) $x->SellerSKU;
                 }
-                $temp['ASIN'] = (string)$x->ASIN;
-                $temp['BarcodeInstruction'] = (string)$x->BarcodeInstruction;
-                $temp['PrepGuidance'] = (string)$x->PrepGuidance;
+                $temp['ASIN'] = (string) $x->ASIN;
+                $temp['BarcodeInstruction'] = (string) $x->BarcodeInstruction;
+                $temp['PrepGuidance'] = (string) $x->PrepGuidance;
                 foreach ($x->PrepInstructionList->children() as $z) {
-                    $temp['PrepInstructionList'][] = (string)$z;
+                    $temp['PrepInstructionList'][] = (string) $z;
                 }
                 if (isset($x->AmazonPrepFeesDetailsList)) {
                     foreach ($x->AmazonPrepFeesDetailsList->children() as $z) {
                         $fee = [];
-                        $fee['PrepInstruction'] = (string)$z->PrepInstruction;
-                        $fee['Amount']['Value'] = (string)$z->Amount->Value;
-                        $fee['Amount']['CurrencyCode'] = (string)$z->Amount->CurrencyCode;
+                        $fee['PrepInstruction'] = (string) $z->PrepInstruction;
+                        $fee['Amount']['Value'] = (string) $z->Amount->Value;
+                        $fee['Amount']['CurrencyCode'] = (string) $z->Amount->CurrencyCode;
                         $temp['AmazonPrepFees'][] = $fee;
                     }
                 }
@@ -234,12 +242,12 @@ protected function parseXml($xml)
         if ($invList) {
             foreach ($invList->children() as $x) {
                 $temp = [];
-                $temp['ErrorReason'] = (string)$x->ErrorReason;
+                $temp['ErrorReason'] = (string) $x->ErrorReason;
                 if (isset($x->SellerSKU)) {
-                    $temp['SellerSKU'] = (string)$x->SellerSKU;
+                    $temp['SellerSKU'] = (string) $x->SellerSKU;
                 }
                 if (isset($x->ASIN)) {
-                    $temp['ASIN'] = (string)$x->ASIN;
+                    $temp['ASIN'] = (string) $x->ASIN;
                 }
                 $this->invalidList[] = $temp;
             }
@@ -252,8 +260,10 @@ protected function parseXml($xml)
      * Prep instructions will only include this data if SKUs were sent when
      * retrieving the list of prep instructions.
      * This method will return <b>FALSE</b> if the list has not yet been filled.
+     *
      * @param int $i [optional] <p>List index to retrieve the value from. Defaults to 0.</p>
-     * @return string|boolean single value, or <b>FALSE</b> if Non-numeric index
+     *
+     * @return string|bool single value, or <b>FALSE</b> if Non-numeric index
      */
     public function getSku($i = 0)
     {
@@ -268,8 +278,10 @@ public function getSku($i = 0)
      * Returns the ASIN for the specified item preperation instruction.
      *
      * This method will return <b>FALSE</b> if the list has not yet been filled.
+     *
      * @param int $i [optional] <p>List index to retrieve the value from. Defaults to 0.</p>
-     * @return string|boolean single value, or <b>FALSE</b> if Non-numeric index
+     *
+     * @return string|bool single value, or <b>FALSE</b> if Non-numeric index
      */
     public function getAsin($i = 0)
     {
@@ -285,8 +297,10 @@ public function getAsin($i = 0)
      *
      * Possible values are "RequiresFNSKULabel" and "CanUseOriginalBarcode".
      * This method will return <b>FALSE</b> if the list has not yet been filled.
+     *
      * @param int $i [optional] <p>List index to retrieve the value from. Defaults to 0.</p>
-     * @return string|boolean single value, or <b>FALSE</b> if Non-numeric index
+     *
+     * @return string|bool single value, or <b>FALSE</b> if Non-numeric index
      */
     public function getBarcodeInstruction($i = 0)
     {
@@ -303,8 +317,10 @@ public function getBarcodeInstruction($i = 0)
      * Possible values are "ConsultHelpDocuments", "NoAdditionalPrepRequired",
      * and "SeePrepInstructionsList".
      * This method will return <b>FALSE</b> if the list has not yet been filled.
+     *
      * @param int $i [optional] <p>List index to retrieve the value from. Defaults to 0.</p>
-     * @return string|boolean single value, or <b>FALSE</b> if Non-numeric index
+     *
+     * @return string|bool single value, or <b>FALSE</b> if Non-numeric index
      */
     public function getPrepGuidance($i = 0)
     {
@@ -321,8 +337,10 @@ public function getPrepGuidance($i = 0)
      * Possible values are "Polybagging", "BubbleWrapping", "Taping", "BlackShrinkWrapping",
      * "Labeling", and "HangGarment".
      * This method will return <b>FALSE</b> if the list has not yet been filled.
+     *
      * @param int $i [optional] <p>List index to retrieve the value from. Defaults to 0.</p>
-     * @return array|boolean simple array, or <b>FALSE</b> if Non-numeric index
+     *
+     * @return array|bool simple array, or <b>FALSE</b> if Non-numeric index
      */
     public function getPrepInstructions($i = 0)
     {
@@ -348,8 +366,11 @@ public function getPrepInstructions($i = 0)
      * Prep instructions will only include this data if SKUs were sent when
      * retrieving the list of prep instructions.
      * This method will return <b>FALSE</b> if the list has not yet been filled.
+     *
      * @param int $i [optional] <p>List index to retrieve the value from. Defaults to 0.</p>
-     * @return array|boolean multi-dimensional array, or <b>FALSE</b> if Non-numeric index
+     *
+     * @return array|bool multi-dimensional array, or <b>FALSE</b> if Non-numeric index
+     *
      * @see getPrepInstructions
      */
     public function getAmazonPrepFees($i = 0)
@@ -374,8 +395,11 @@ public function getAmazonPrepFees($i = 0)
      * <li><b>PrepInstructionList</b> (see <i>getPrepInstructions</i> for details)</li>
      * <li><b>AmazonPrepFees</b> (see <i>getAmazonPrepFees</i> for details)</li>
      * </ul>
+     *
      * @param int $i [optional] <p>List index of the instruction to return. Defaults to NULL.</p>
-     * @return array|boolean multi-dimensional array, or <b>FALSE</b> if list not filled yet
+     *
+     * @return array|bool multi-dimensional array, or <b>FALSE</b> if list not filled yet
+     *
      * @see getPrepInstructionList
      * @see getAmazonPrepFees
      */
@@ -400,8 +424,10 @@ public function getPrepList($i = null)
      * <li><b>SellerSKU</b> or <b>ASIN</b></li>
      * <li><b>ErrorReason</b></li>
      * </ul>
+     *
      * @param int $i [optional] <p>List index of the item to return. Defaults to NULL.</p>
-     * @return array|boolean multi-dimensional array, or <b>FALSE</b> if list not filled yet
+     *
+     * @return array|bool multi-dimensional array, or <b>FALSE</b> if list not filled yet
      */
     public function getInvalidItemList($i = null)
     {
@@ -416,7 +442,8 @@ public function getInvalidItemList($i = null)
     }
 
     /**
-     * Iterator function
+     * Iterator function.
+     *
      * @return array
      */
     public function current()
@@ -425,15 +452,16 @@ public function current()
     }
 
     /**
-     * Iterator function
+     * Iterator function.
      */
     public function rewind()
     {
         $this->i = 0;
     }
 
     /**
-     * Iterator function
+     * Iterator function.
+     *
      * @return int
      */
     public function key()
@@ -442,16 +470,17 @@ public function key()
     }
 
     /**
-     * Iterator function
+     * Iterator function.
      */
     public function next()
     {
         $this->i++;
     }
 
     /**
-     * Iterator function
-     * @return boolean
+     * Iterator function.
+     *
+     * @return bool
      */
     public function valid()
     {