pamil / Sylius

tests/Controller/ZoneApiTest.php

Upper-Lower-Casing +3 added lines, -3 removed lines

@@ -34,7 +34,7 @@ 
             },
             "author": "Christie Golden"
         }
-EOT;
+eot;
 
         $this->client->request('POST', '/books/', [], [], ['CONTENT_TYPE' => 'application/json'], $data);
         $response = $this->client->getResponse();
@@ -61,7 +61,7 @@ 
             },
             "author": "Christie Golden"
         }
-EOT;
+eot;
 
         $this->client->request('PUT', '/books/'. $objects["book1"]->getId(), [], [], ['CONTENT_TYPE' => 'application/json'], $data);
         $response = $this->client->getResponse();
@@ -80,7 +80,7 @@ 
         {
             "author": "Christie Golden"
         }
-EOT;
+eot;
 
         $this->client->request('PATCH', '/books/'. $objects["book1"]->getId(), [], [], ['CONTENT_TYPE' => 'application/json'], $data);
         $response = $this->client->getResponse();

Spacing +8 added lines, -8 removed lines

@@ -146,7 +146,7 @@ 
         $this->loadFixturesFromFile('authentication/api_administrator.yml');
         $zones = $this->loadFixturesFromFile('resources/zones.yml');
 
-        $this->client->request('GET', '/api/zones/'.$zones['zone_eu']->getCode(), [], [], static::$authorizedHeaderWithAccept);
+        $this->client->request('GET', '/api/zones/' . $zones['zone_eu']->getCode(), [], [], static::$authorizedHeaderWithAccept);
 
         $response = $this->client->getResponse();
         $this->assertResponse($response, 'zone/show_response', Response::HTTP_OK);
@@ -184,7 +184,7 @@ 
         $this->loadFixturesFromFile('authentication/api_administrator.yml');
         $zones = $this->loadFixturesFromFile('resources/zones.yml');
 
-        $this->client->request('PUT', '/api/zones/'.$zones['zone_eu']->getCode(), [], [], static::$authorizedHeaderWithContentType);
+        $this->client->request('PUT', '/api/zones/' . $zones['zone_eu']->getCode(), [], [], static::$authorizedHeaderWithContentType);
 
         $response = $this->client->getResponse();
         $this->assertResponse($response, 'zone/update_validation_fail_response', Response::HTTP_BAD_REQUEST);
@@ -211,12 +211,12 @@ 
         }
 EOT;
 
-        $this->client->request('PUT', '/api/zones/'.$zones['zone_eu']->getCode(), [], [], static::$authorizedHeaderWithContentType, $data);
+        $this->client->request('PUT', '/api/zones/' . $zones['zone_eu']->getCode(), [], [], static::$authorizedHeaderWithContentType, $data);
 
         $response = $this->client->getResponse();
         $this->assertResponseCode($response, Response::HTTP_NO_CONTENT);
 
-        $this->client->request('GET', '/api/zones/'.$zones['zone_eu']->getCode(), [], [], static::$authorizedHeaderWithAccept);
+        $this->client->request('GET', '/api/zones/' . $zones['zone_eu']->getCode(), [], [], static::$authorizedHeaderWithAccept);
 
         $response = $this->client->getResponse();
         $this->assertResponse($response, 'zone/update_response', Response::HTTP_OK);
@@ -262,12 +262,12 @@ 
         }
 EOT;
 
-        $this->client->request('PATCH', '/api/zones/'.$zones['zone_eu']->getCode(), [], [], static::$authorizedHeaderWithContentType, $data);
+        $this->client->request('PATCH', '/api/zones/' . $zones['zone_eu']->getCode(), [], [], static::$authorizedHeaderWithContentType, $data);
 
         $response = $this->client->getResponse();
         $this->assertResponseCode($response, Response::HTTP_NO_CONTENT);
 
-        $this->client->request('GET', '/api/zones/'.$zones['zone_eu']->getCode(), [], [], static::$authorizedHeaderWithAccept);
+        $this->client->request('GET', '/api/zones/' . $zones['zone_eu']->getCode(), [], [], static::$authorizedHeaderWithAccept);
 
         $response = $this->client->getResponse();
         $this->assertResponse($response, 'zone/update_response', Response::HTTP_OK);
@@ -305,12 +305,12 @@ 
         $this->loadFixturesFromFile('authentication/api_administrator.yml');
         $zones = $this->loadFixturesFromFile('resources/zones.yml');
 
-        $this->client->request('DELETE', '/api/zones/'.$zones['zone_eu']->getCode(), [], [], static::$authorizedHeaderWithContentType);
+        $this->client->request('DELETE', '/api/zones/' . $zones['zone_eu']->getCode(), [], [], static::$authorizedHeaderWithContentType);
 
         $response = $this->client->getResponse();
         $this->assertResponseCode($response, Response::HTTP_NO_CONTENT);
 
-        $this->client->request('GET', '/api/zones/'.$zones['zone_eu']->getCode(), [], [], static::$authorizedHeaderWithAccept);
+        $this->client->request('GET', '/api/zones/' . $zones['zone_eu']->getCode(), [], [], static::$authorizedHeaderWithAccept);
 
         $response = $this->client->getResponse();
         $this->assertResponseCode($response, Response::HTTP_NOT_FOUND);

tests/Controller/CheckoutApiTestCase.php

Upper-Lower-Casing +3 added lines, -3 removed lines

@@ -34,7 +34,7 @@ 
             },
             "author": "Christie Golden"
         }
-EOT;
+eot;
 
         $this->client->request('POST', '/books/', [], [], ['CONTENT_TYPE' => 'application/json'], $data);
         $response = $this->client->getResponse();
@@ -61,7 +61,7 @@ 
             },
             "author": "Christie Golden"
         }
-EOT;
+eot;
 
         $this->client->request('PUT', '/books/'. $objects["book1"]->getId(), [], [], ['CONTENT_TYPE' => 'application/json'], $data);
         $response = $this->client->getResponse();
@@ -80,7 +80,7 @@ 
         {
             "author": "Christie Golden"
         }
-EOT;
+eot;
 
         $this->client->request('PATCH', '/books/'. $objects["book1"]->getId(), [], [], ['CONTENT_TYPE' => 'application/json'], $data);
         $response = $this->client->getResponse();

tests/Controller/ChannelApiTest.php

Spacing +1 added lines, -1 removed lines

@@ -40,7 +40,7 @@
         $this->loadFixturesFromFile('authentication/api_administrator.yml');
         $channelData = $this->loadFixturesFromFile('resources/channels.yml');
 
-        $this->client->request('GET', '/api/channels/'.$channelData['channel-web']->getId(), [], [], static::$authorizedHeaderWithContentType);
+        $this->client->request('GET', '/api/channels/' . $channelData['channel-web']->getId(), [], [], static::$authorizedHeaderWithContentType);
 
         $response = $this->client->getResponse();
         $this->assertResponse($response, 'channel/show_response', Response::HTTP_OK);

src/Sylius/Behat/Context/Setup/UserContext.php

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

@@ -15,9 +15,9 @@
 use Sylius\Behat\Page\Admin\Channel\IndexPageInterface;
 use Sylius\Behat\Page\Admin\Channel\UpdatePageInterface;
 use Sylius\Behat\Page\Shop\HomePageInterface;
+use Sylius\Behat\Service\SharedStorageInterface;
 use Sylius\Bundle\ThemeBundle\Model\ThemeInterface;
 use Sylius\Component\Core\Model\ChannelInterface;
-use Sylius\Behat\Service\SharedStorageInterface;
 use Webmozart\Assert\Assert;
 
 /**

src/Sylius/Behat/Page/Shop/Checkout/CompletePageInterface.php

Doc Comments +13 added lines

@@ -20,6 +20,7 @@ 
 {
     /**
      * @param string $paymentMethod
+     * @return void
      */
     public function selectPaymentMethod($paymentMethod);
     
@@ -37,12 +38,24 @@ 
      */
     public function getItemSubtotal($itemName);
 
+    /**
+     * @return void
+     */
     public function nextStep();
 
+    /**
+     * @return void
+     */
     public function changeShippingMethod();
 
+    /**
+     * @return void
+     */
     public function changeShippingMethodByStepLabel();
 
+    /**
+     * @return void
+     */
     public function changeAddressByStepLabel();
 
     /**

src/Sylius/Component/Core/Resolver/ZoneAndChannelBasedShippingMethodsResolver.php

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

@@ -19,10 +19,10 @@
 use Sylius\Component\Core\Model\OrderInterface;
 use Sylius\Component\Core\Model\ShipmentInterface;
 use Sylius\Component\Core\Model\ShippingMethodInterface;
+use Sylius\Component\Core\Repository\ShippingMethodRepositoryInterface;
 use Sylius\Component\Core\Resolver\ZoneAndChannelBasedShippingMethodsResolver;
 use Sylius\Component\Shipping\Checker\ShippingMethodEligibilityCheckerInterface;
 use Sylius\Component\Shipping\Model\ShippingSubjectInterface;
-use Sylius\Component\Core\Repository\ShippingMethodRepositoryInterface;
 use Sylius\Component\Shipping\Resolver\ShippingMethodsResolverInterface;
 
 /**

src/Sylius/Component/Core/StateResolver/OrderPaymentStateResolver.php

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

@@ -13,11 +13,11 @@
 
 use SM\Factory\FactoryInterface;
 use SM\StateMachine\StateMachineInterface;
-use Sylius\Component\Order\Model\OrderInterface;
-use Sylius\Component\Order\StateResolver\StateResolverInterface;
 use Sylius\Component\Core\Model\ShipmentInterface;
 use Sylius\Component\Core\OrderShippingStates;
 use Sylius\Component\Core\OrderShippingTransitions;
+use Sylius\Component\Order\Model\OrderInterface;
+use Sylius\Component\Order\StateResolver\StateResolverInterface;
 
 /**
  * @author Paweł Jędrzejewski <pawel@sylius.org>

src/Sylius/Behat/Page/Admin/Taxon/UpdatePage.php

Spacing +3 added lines, -3 removed lines

@@ -152,7 +152,7 @@ 
 
         $imageForm = $this->getLastImageElement();
         $imageForm->fillField('Code', $code);
-        $imageForm->find('css', 'input[type="file"]')->attachFile($filesPath.$path);
+        $imageForm->find('css', 'input[type="file"]')->attachFile($filesPath . $path);
     }
 
     /**
@@ -163,7 +163,7 @@ 
         $filesPath = $this->getParameter('files_path');
 
         $imageForm = $this->getImageElementByCode($code);
-        $imageForm->find('css', 'input[type="file"]')->attachFile($filesPath.$path);
+        $imageForm->find('css', 'input[type="file"]')->attachFile($filesPath . $path);
     }
 
     /**
@@ -261,7 +261,7 @@ 
     private function getImageElementByCode($code)
     {
         $images = $this->getElement('images');
-        $inputCode = $images->find('css', 'input[value="'.$code.'"]');
+        $inputCode = $images->find('css', 'input[value="' . $code . '"]');
 
         if (null === $inputCode) {
             return null;

src/Sylius/Behat/Page/Admin/Taxon/CreatePage.php

Spacing +1 added lines, -1 removed lines

@@ -54,7 +54,7 @@
 
         $imageForm = $this->getLastImageElement();
         $imageForm->fillField('Code', $code);
-        $imageForm->find('css', 'input[type="file"]')->attachFile($filesPath.$path);
+        $imageForm->find('css', 'input[type="file"]')->attachFile($filesPath . $path);
     }
 
     /**

Unused Use Statements -1 removed lines

@@ -14,7 +14,6 @@
 use Behat\Mink\Driver\Selenium2Driver;
 use Behat\Mink\Element\NodeElement;
 use Behat\Mink\Exception\ElementNotFoundException;
-use Behat\Mink\Exception\UnsupportedDriverActionException;
 use Sylius\Behat\Behaviour\SpecifiesItsCode;
 use Sylius\Behat\Page\Admin\Crud\CreatePage as BaseCreatePage;
 use Sylius\Component\Core\Model\TaxonInterface;

Doc Comments +1 added lines

@@ -199,6 +199,7 @@
 
     /**
      * {@inheritdoc}
+     * @param string $name
      */
     protected function getElement($name, array $parameters = [])
     {