pamil / Sylius

src/Sylius/Bundle/ShippingBundle/DependencyInjection/SyliusShippingExtension.php

Unused Use Statements -1 removed lines

@@ -16,7 +16,6 @@
 use Symfony\Component\DependencyInjection\ContainerBuilder;
 use Symfony\Component\DependencyInjection\Loader\XmlFileLoader;
 use Symfony\Component\DependencyInjection\Parameter;
-use Symfony\Component\DependencyInjection\Reference;
 
 /**
  * @author Paweł Jędrzejewski <pawel@sylius.org>

src/Sylius/Component/Core/Model/ChannelPricingInterface.php

Doc Comments +2 added lines

@@ -27,6 +27,7 @@ 
 
     /**
      * @param OptionInterface $option
+     * @return void
      */
     public function setOption(OptionInterface $option = null);
 
@@ -41,6 +42,7 @@ 
      * Set internal value.
      *
      * @param string $value
+     * @return void
      */
     public function setValue($value);
 

src/Sylius/Component/Core/Model/ProductVariantInterface.php

Doc Comments +8 added lines

@@ -41,6 +41,7 @@ 
 
     /**
      * @param null|TaxonInterface $taxon
+     * @return void
      */
     public function setParent(TaxonInterface $taxon = null);
 
@@ -63,11 +64,13 @@ 
 
     /**
      * @param TaxonInterface $taxon
+     * @return void
      */
     public function addChild(TaxonInterface $taxon);
 
     /**
      * @param TaxonInterface $taxon
+     * @return void
      */
     public function removeChild(TaxonInterface $taxon);
 
@@ -78,6 +81,7 @@ 
 
     /**
      * @param string $name
+     * @return void
      */
     public function setName($name);
 
@@ -88,6 +92,7 @@ 
 
     /**
      * @param string $permalink
+     * @return void
      */
     public function setPermalink($permalink);
 
@@ -98,6 +103,7 @@ 
 
     /**
      * @param int $left
+     * @return void
      */
     public function setLeft($left);
 
@@ -108,6 +114,7 @@ 
 
     /**
      * @param int $right
+     * @return void
      */
     public function setRight($right);
 
@@ -118,6 +125,7 @@ 
 
     /**
      * @param int $level
+     * @return void
      */
     public function setLevel($level);
 }

src/Sylius/Component/Core/Promotion/Action/FixedDiscountPromotionActionCommand.php

Unused Use Statements -1 removed lines

@@ -14,7 +14,6 @@
 use Sylius\Bundle\PromotionBundle\Form\Type\Action\FixedDiscountConfigurationType;
 use Sylius\Component\Core\Distributor\ProportionalIntegerDistributorInterface;
 use Sylius\Component\Core\Promotion\Applicator\UnitsPromotionAdjustmentsApplicatorInterface;
-use Sylius\Component\Currency\Converter\CurrencyConverterInterface;
 use Sylius\Component\Promotion\Model\PromotionInterface;
 use Sylius\Component\Promotion\Model\PromotionSubjectInterface;
 use Webmozart\Assert\Assert;

src/Sylius/Component/Core/Shipping/Calculator/FlatRateCalculator.php

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

@@ -13,7 +13,7 @@
 
 use Sylius\Component\Core\Model\ShipmentInterface;
 use Sylius\Component\Shipping\Calculator\CalculatorInterface;
-use Sylius\Component\Shipping\Model\ShipmentInterface  as BaseShipmentInterface;
+use Sylius\Component\Shipping\Model\ShipmentInterface as BaseShipmentInterface;
 use Webmozart\Assert\Assert;
 
 /**

src/Sylius/Bundle/CoreBundle/Tests/Fixture/CurrencyFixtureTest.php

Indentation +1 added lines, -1 removed lines

@@ -36,7 +36,7 @@
      */
     public function currencies_can_be_set()
     {
-       $this->assertConfigurationIsValid([['currencies' => ['EUR', 'USD', 'PLN']]], 'currencies');
+        $this->assertConfigurationIsValid([['currencies' => ['EUR', 'USD', 'PLN']]], 'currencies');
     }
 
     /**

src/Sylius/Behat/Context/Ui/Admin/ManagingProductVariantsContext.php

Spacing +4 added lines, -4 removed lines

@@ -166,7 +166,7 @@ 
      */
     public function iSetItsPriceTo($price = null, $channel = null)
     {
-        $this->createPage->specifyPrice($price, (null === $channel) ? $this->sharedStorage->get('channel') :$channel);
+        $this->createPage->specifyPrice($price, (null === $channel) ? $this->sharedStorage->get('channel') : $channel);
     }
 
     /**
@@ -425,7 +425,7 @@ 
     public function iShouldBeNotifiedThatPricesInAllChannelsMustBeDefinedForTheVariant($position)
     {
         Assert::same(
-            $this->generatePage->getPricesValidationMessage($position-1),
+            $this->generatePage->getPricesValidationMessage($position - 1),
             'You must define price for every channel.'
         );
     }
@@ -473,7 +473,7 @@ 
     {
         Assert::true(
             $this->indexPage->isSingleResourceWithSpecificElementOnPage(['name' => $productVariantName], sprintf('td > div.ui.label:contains("%s")', $quantity)),
-            sprintf('The product variant %s should have %s items on hand, but it does not.',$productVariantName, $quantity)
+            sprintf('The product variant %s should have %s items on hand, but it does not.', $productVariantName, $quantity)
         );
     }
 
@@ -486,7 +486,7 @@ 
 
         Assert::true(
             $this->indexPage->isSingleResourceWithSpecificElementOnPage(['name' => $productVariantName], sprintf('td > div.ui.label:contains("%s")', $quantity)),
-            sprintf('The product variant %s should have %s items on hand, but it does not.',$productVariantName, $quantity)
+            sprintf('The product variant %s should have %s items on hand, but it does not.', $productVariantName, $quantity)
         );
     }
 

tests/Controller/CurrencyApiTest.php

Spacing +3 added lines, -3 removed lines

@@ -120,7 +120,7 @@ 
         $this->loadFixturesFromFile('authentication/api_administrator.yml');
         $currencies = $this->loadFixturesFromFile('resources/currencies.yml');
 
-        $this->client->request('GET', '/api/currencies/'.$currencies['currency_1']->getCode(), [], [], static::$authorizedHeaderWithAccept);
+        $this->client->request('GET', '/api/currencies/' . $currencies['currency_1']->getCode(), [], [], static::$authorizedHeaderWithAccept);
 
         $response = $this->client->getResponse();
         $this->assertResponse($response, 'currency/show_response', Response::HTTP_OK);
@@ -141,12 +141,12 @@ 
         $this->loadFixturesFromFile('authentication/api_administrator.yml');
         $currencies = $this->loadFixturesFromFile('resources/currencies.yml');
 
-        $this->client->request('DELETE', '/api/currencies/'.$currencies['currency_1']->getCode(), [], [], static::$authorizedHeaderWithContentType, []);
+        $this->client->request('DELETE', '/api/currencies/' . $currencies['currency_1']->getCode(), [], [], static::$authorizedHeaderWithContentType, []);
 
         $response = $this->client->getResponse();
         $this->assertResponseCode($response, Response::HTTP_NO_CONTENT);
 
-        $this->client->request('GET', '/api/currencies/'.$currencies['currency_1']->getCode(), [], [], static::$authorizedHeaderWithAccept);
+        $this->client->request('GET', '/api/currencies/' . $currencies['currency_1']->getCode(), [], [], static::$authorizedHeaderWithAccept);
 
         $response = $this->client->getResponse();
         $this->assertResponse($response, 'error/not_found_response', Response::HTTP_NOT_FOUND);

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

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