pamil / Sylius

tests/Controller/OrderApiTest.php

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

@@ -91,6 +91,6 @@
 
   <info>php %command.full_name% web --symlink --relative</info>
 
-EOT;
+eot;
     }
 }

Spacing +3 added lines, -3 removed lines

@@ -59,7 +59,7 @@ 
         $this->loadFixturesFromFile('authentication/api_administrator.yml');
         $orderData = $this->loadFixturesFromFile('resources/orders.yml');
 
-        $this->client->request('GET', '/api/v1/orders/'.$orderData['order-001']->getId(), [], [], static::$authorizedHeaderWithContentType);
+        $this->client->request('GET', '/api/v1/orders/' . $orderData['order-001']->getId(), [], [], static::$authorizedHeaderWithContentType);
 
         $response = $this->client->getResponse();
         $this->assertResponse($response, 'order/show_response', Response::HTTP_OK);
@@ -159,12 +159,12 @@ 
         $this->loadFixturesFromFile('authentication/api_administrator.yml');
         $orders = $this->loadFixturesFromFile('resources/orders.yml');
 
-        $this->client->request('DELETE', '/api/v1/orders/'.$orders['order-001']->getId(), [], [], static::$authorizedHeaderWithContentType, []);
+        $this->client->request('DELETE', '/api/v1/orders/' . $orders['order-001']->getId(), [], [], static::$authorizedHeaderWithContentType, []);
 
         $response = $this->client->getResponse();
         $this->assertResponseCode($response, Response::HTTP_NO_CONTENT);
 
-        $this->client->request('GET', '/api/v1/orders/'.$orders['order-001']->getId(), [], [], static::$authorizedHeaderWithContentType);
+        $this->client->request('GET', '/api/v1/orders/' . $orders['order-001']->getId(), [], [], static::$authorizedHeaderWithContentType);
 
         $response = $this->client->getResponse();
         $this->assertResponse($response, 'error/not_found_response', Response::HTTP_NOT_FOUND);

src/Sylius/Bundle/ApiBundle/Controller/TokenController.php

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

@@ -12,10 +12,10 @@
 namespace Sylius\Bundle\ApiBundle\Controller;
 
 use FOS\RestBundle\Decoder\DecoderProviderInterface;
-use Symfony\Component\HttpFoundation\Request;
-use Symfony\Component\HttpFoundation\ParameterBag;
 use OAuth2\OAuth2;
 use OAuth2\OAuth2ServerException;
+use Symfony\Component\HttpFoundation\ParameterBag;
+use Symfony\Component\HttpFoundation\Request;
 use Symfony\Component\HttpFoundation\Response;
 use Symfony\Component\HttpKernel\Exception\BadRequestHttpException;
 

tests/Controller/ProvinceApiTest.php

Spacing +3 added lines, -3 removed lines

@@ -46,7 +46,7 @@ 
         $this->loadFixturesFromFile('authentication/api_administrator.yml');
         $countryData = $this->loadFixturesFromFile('resources/countries.yml');
 
-        $this->client->request('GET', '/api/v1/countries/'.$countryData['country_BE']->getCode().'/provinces/'.$countryData['province_BE_limburg']->getCode(), [], [], static::$authorizedHeaderWithContentType);
+        $this->client->request('GET', '/api/v1/countries/' . $countryData['country_BE']->getCode() . '/provinces/' . $countryData['province_BE_limburg']->getCode(), [], [], static::$authorizedHeaderWithContentType);
 
         $response = $this->client->getResponse();
         $this->assertResponse($response, 'province/show_response', Response::HTTP_OK);
@@ -60,12 +60,12 @@ 
         $this->loadFixturesFromFile('authentication/api_administrator.yml');
         $countryData = $this->loadFixturesFromFile('resources/countries.yml');
 
-        $this->client->request('DELETE', '/api/v1/countries/'.$countryData['country_BE']->getCode().'/provinces/'.$countryData['province_BE_limburg']->getCode(), [], [], static::$authorizedHeaderWithContentType);
+        $this->client->request('DELETE', '/api/v1/countries/' . $countryData['country_BE']->getCode() . '/provinces/' . $countryData['province_BE_limburg']->getCode(), [], [], static::$authorizedHeaderWithContentType);
 
         $response = $this->client->getResponse();
         $this->assertResponseCode($response, Response::HTTP_NO_CONTENT);
 
-        $this->client->request('GET', '/api/v1/countries/'.$countryData['country_BE']->getCode().'/provinces/'.$countryData['province_BE_limburg']->getCode(), [], [], static::$authorizedHeaderWithContentType);
+        $this->client->request('GET', '/api/v1/countries/' . $countryData['country_BE']->getCode() . '/provinces/' . $countryData['province_BE_limburg']->getCode(), [], [], static::$authorizedHeaderWithContentType);
 
         $response = $this->client->getResponse();
         $this->assertResponse($response, 'error/not_found_response', Response::HTTP_NOT_FOUND);

src/Sylius/Component/Attribute/Model/AttributeValueInterface.php

Doc Comments +4 added lines

@@ -36,11 +36,13 @@ 
 
     /**
      * @param string $locale
+     * @return void
      */
     public function setCurrentLocale($locale);
 
     /**
      * @param string $locale
+     * @return void
      */
     public function setFallbackLocale($locale);
 
@@ -51,11 +53,13 @@ 
 
     /**
      * @param TranslationInterface $translation
+     * @return void
      */
     public function addTranslation(TranslationInterface $translation);
 
     /**
      * @param TranslationInterface $translation
+     * @return void
      */
     public function removeTranslation(TranslationInterface $translation);
 }

src/Sylius/Component/Product/Model/Product.php

Doc Comments +1 added lines

@@ -505,6 +505,7 @@
 
     /**
      * {@inheritdoc}
+     * @param string $localeCode
      */
     private function getAttributeInDifferentLocale(ProductAttributeValueInterface $attributeValue, $localeCode)
     {

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

@@ -13,8 +13,8 @@
 
 use Sylius\Component\Grid\DataExtractor\DataExtractorInterface;
 use Sylius\Component\Grid\Definition\Field;
-use Webmozart\Assert\Assert;
 use Symfony\Component\OptionsResolver\OptionsResolver;
+use Webmozart\Assert\Assert;
 
 /**
  * @author Mateusz Zalewski <mateusz.zalewski@lakion.com>

app/migrations/Version20170109143010.php

Spacing +1 added lines, -1 removed lines

@@ -35,7 +35,7 @@
         $defaultLocale = $this->container->getParameter('locale');
 
         $this->addSql('ALTER TABLE sylius_product_attribute_value ADD locale_code VARCHAR(255) NOT NULL');
-        $this->addSql('UPDATE sylius_product_attribute_value SET locale_code = "'.$defaultLocale.'"');
+        $this->addSql('UPDATE sylius_product_attribute_value SET locale_code = "' . $defaultLocale . '"');
     }
 
     /**

src/Sylius/Behat/Page/Admin/ShippingMethod/IndexPageInterface.php

Doc Comments +1 added lines

@@ -27,6 +27,7 @@
 
     /**
      * @param string $value
+     * @return void
      */
     public function setValue($value);
 }

src/Sylius/Bundle/CoreBundle/Fixture/GeographicalFixture.php

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

@@ -4,9 +4,9 @@
 
 use Doctrine\Common\Persistence\ObjectManager;
 use PhpSpec\ObjectBehavior;
-use Sylius\Component\Addressing\Factory\ZoneFactoryInterface;
 use Sylius\Bundle\CoreBundle\Fixture\GeographicalFixture;
 use Sylius\Bundle\FixturesBundle\Fixture\FixtureInterface;
+use Sylius\Component\Addressing\Factory\ZoneFactoryInterface;
 use Sylius\Component\Addressing\Model\CountryInterface;
 use Sylius\Component\Addressing\Model\ProvinceInterface;
 use Sylius\Component\Addressing\Model\ZoneInterface;

src/Sylius/Bundle/ProductBundle/Form/DataTransformer/ProductVariantToProductOptionsTransformer.php

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

@@ -15,8 +15,8 @@
 use Sylius\Component\Product\Model\ProductOptionValueInterface;
 use Sylius\Component\Product\Model\ProductVariantInterface;
 use Symfony\Component\Form\DataTransformerInterface;
-use Symfony\Component\Form\Exception\UnexpectedTypeException;
 use Symfony\Component\Form\Exception\TransformationFailedException;
+use Symfony\Component\Form\Exception\UnexpectedTypeException;
 
 /**
  * @author Paweł Jędrzejewski <pawel@sylius.org>