radowoj / yaah

src/Field.php

Doc Comments -1 removed lines

@@ -142,7 +142,6 @@
 
     /**
      * Detect type of range passed as argument (int, float, date)
-     * @param array $value value to detect type of
      */
     protected function setValueRangeAutodetect(array $range)
     {

tests/FieldTest.php

Unused Use Statements +1 added lines, -1 removed lines

@@ -2,7 +2,7 @@
 

 namespace Radowoj\Yaah;

 

-use PHPUnit\Framework\TestCase;

+use PHPUnit\Framework\TestCase;
 use Radowoj\Yaah\Field;

 

 class FieldTest extends TestCase

src/Auction.php

Doc Comments +1 added lines, -1 removed lines

@@ -21,7 +21,7 @@
 
     /**
      * Sets photos for auction
-     * @param array $photos array of photo file paths
+     * @param string[] $photos array of photo file paths
      */
     public function setPhotos(array $photos)
     {

src/Decorators/AuctionArrayMapDecorator.php

Spacing +2 added lines, -2 removed lines

@@ -22,7 +22,7 @@ 
     public function fromArray(array $humanReadableArray)

     {

         $fields = [];

-        foreach($humanReadableArray as $key => $value) {

+        foreach ($humanReadableArray as $key => $value) {

             if (array_key_exists($key, $this->map)) {

                 $fields[$this->map[$key]] = $value;

             }

@@ -37,7 +37,7 @@ 
         $flippedMap = array_flip($this->map);

         $humanReadableArray = [];

 

-        foreach($fields as $key => $value) {

+        foreach ($fields as $key => $value) {

             if (array_key_exists($key, $flippedMap)) {

                 $humanReadableArray[$flippedMap[$key]] = $value;

             }

tests/AuctionTest.php

Unused Use Statements +1 added lines, -1 removed lines

@@ -2,7 +2,7 @@
 

 namespace Radowoj\Yaah;

 

-use PHPUnit\Framework\TestCase;

+use PHPUnit\Framework\TestCase;
 use Radowoj\Yaah\Auction;

 

 class AuctionTest extends TestCase

src/HelperFactory/HelperFactory.php

Doc Comments +3 added lines

@@ -11,6 +11,9 @@
 {

     protected $config = null;

 

+    /**
+     * @return \SoapClient
+     */
     abstract protected function getSoapClient();

 

     protected function setConfig($configArray)

Unused Use Statements +3 added lines, -3 removed lines

@@ -2,9 +2,9 @@
 

 namespace Radowoj\Yaah\HelperFactory;

 

-use Radowoj\Yaah\Config;

-use Radowoj\Yaah\Client;

-use Radowoj\Yaah\Helper;

+use Radowoj\Yaah\Config;
+use Radowoj\Yaah\Client;
+use Radowoj\Yaah\Helper;
 use Radowoj\Yaah\Exception;

 

 abstract class HelperFactory

tests/HelperTest.php

Unused Use Statements +1 added lines, -2 removed lines

@@ -2,8 +2,7 @@
 

 namespace Radowoj\Yaah;

 

-use PHPUnit\Framework\TestCase;

-use Radowoj\Yaah\Field;

+use PHPUnit\Framework\TestCase;
 use SoapClient;

 

 class HelperTest extends TestCase

Spacing +1 added lines, -1 removed lines

@@ -143,7 +143,7 @@
         $this->soapClient->expects($this->once())

             ->method('doSomethingNotImplementedInHelper')

             ->with(

-                $this->callback(function($params){

+                $this->callback(function($params) {

                     return array_key_exists('param1', $params)

                         && array_key_exists('webapiKey', $params)

                         && $params['param1'] == 1337

tests/ClientTest.php

Unused Use Statements +4 added lines, -5 removed lines

@@ -2,11 +2,10 @@
 

 namespace Radowoj\Yaah;

 

-use PHPUnit\Framework\TestCase;

-use Radowoj\Yaah\Config;

-use Radowoj\Yaah\Client;

-

-use SoapClient;

+use PHPUnit\Framework\TestCase;
+use Radowoj\Yaah\Config;
+use Radowoj\Yaah\Client;
+use SoapClient;
 use SoapFault;

 

 class ClientTest extends TestCase

src/Helper.php

Spacing +3 added lines, -3 removed lines

@@ -28,7 +28,7 @@ 
 
         $items = $data->sellFormFieldsForCategory->sellFormFieldsList->item;
 
-        return array_map(function ($item) {
+        return array_map(function($item) {
             return [
                 'fid' => $item->sellFormId,
                 'title' => $item->sellFormTitle,
@@ -86,7 +86,7 @@ 
      */
     public function finishAuctions(array $auctionIds, $cancelAllBids = 0, $finishCancelReason = '')
     {
-        $finishItemsList = array_map(function ($auctionId) use ($cancelAllBids, $finishCancelReason) {
+        $finishItemsList = array_map(function($auctionId) use ($cancelAllBids, $finishCancelReason) {
             return [
                 'finishItemId' => $auctionId,
                 'finishCancelAllBids' => $cancelAllBids,
@@ -134,7 +134,7 @@ 
      */
     public function getAuctionByItemId($itemId)
     {
-        $response =  $this->client->doGetItemFields(['itemId' => $itemId]);
+        $response = $this->client->doGetItemFields(['itemId' => $itemId]);
         if (isset($response->itemFields->item)) {
             $auction = new Auction();
             $auction->fromApiRepresentation($response->itemFields->item);