pamil / Sylius

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

Unused Use Statements +8 added lines, -9 removed lines

@@ -14,31 +14,30 @@
 use Behat\Behat\Context\Context;
 use Doctrine\Common\Persistence\ObjectManager;
 use SM\Factory\FactoryInterface as StateMachineFactoryInterface;
+use Sylius\Behat\Service\SharedStorageInterface;
 use Sylius\Component\Core\Currency\CurrencyStorageInterface;
+use Sylius\Component\Core\Model\AddressInterface;
 use Sylius\Component\Core\Model\ChannelInterface;
 use Sylius\Component\Core\Model\ChannelPricingInterface;
-use Sylius\Component\Core\Model\PromotionCouponInterface;
-use Sylius\Component\Core\OrderCheckoutTransitions;
-use Sylius\Component\Order\Processor\OrderProcessorInterface;
-use Sylius\Component\Order\Modifier\OrderItemQuantityModifierInterface;
-use Sylius\Component\Core\Model\AddressInterface;
 use Sylius\Component\Core\Model\OrderInterface;
 use Sylius\Component\Core\Model\OrderItemInterface;
 use Sylius\Component\Core\Model\ProductInterface;
 use Sylius\Component\Core\Model\ProductVariantInterface;
+use Sylius\Component\Core\Model\PromotionCouponInterface;
 use Sylius\Component\Core\Model\ShippingMethodInterface;
+use Sylius\Component\Core\OrderCheckoutTransitions;
 use Sylius\Component\Core\Repository\OrderRepositoryInterface;
-use Sylius\Behat\Service\SharedStorageInterface;
+use Sylius\Component\Customer\Model\CustomerInterface;
+use Sylius\Component\Order\Modifier\OrderItemQuantityModifierInterface;
 use Sylius\Component\Order\OrderTransitions;
-use Sylius\Component\Payment\Model\PaymentInterface;
+use Sylius\Component\Order\Processor\OrderProcessorInterface;
 use Sylius\Component\Payment\Model\PaymentMethodInterface;
 use Sylius\Component\Payment\PaymentTransitions;
+use Sylius\Component\Product\Resolver\ProductVariantResolverInterface;
 use Sylius\Component\Resource\Factory\FactoryInterface;
 use Sylius\Component\Resource\Repository\RepositoryInterface;
 use Sylius\Component\Shipping\ShipmentTransitions;
-use Sylius\Component\Customer\Model\CustomerInterface;
 use Sylius\Component\User\Model\UserInterface;
-use Sylius\Component\Product\Resolver\ProductVariantResolverInterface;
 
 /**
  * @author Łukasz Chruściel <lukasz.chrusciel@lakion.com>

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

Unused Use Statements -2 removed lines

@@ -13,8 +13,6 @@
 
 use Behat\Mink\Driver\Selenium2Driver;
 use Behat\Mink\Exception\ElementNotFoundException;
-use Sylius\Behat\Behaviour\ChoosesCalculator;
-use Sylius\Behat\Behaviour\SpecifiesItsAmount;
 use Sylius\Behat\Behaviour\SpecifiesItsCode;
 use Sylius\Behat\Page\Admin\Crud\CreatePage as BaseCreatePage;
 use Sylius\Component\Core\Formatter\StringInflector;

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/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/PayumBundle/Action/Paypal/ExpressCheckout/ConvertPaymentAction.php

Spacing +12 added lines, -12 removed lines

@@ -57,33 +57,33 @@
 
         $m = 0;
         foreach ($order->getItems() as $item) {
-            $details['L_PAYMENTREQUEST_0_NAME'.$m] = $item->getVariant()->getProduct()->getName();
-            $details['L_PAYMENTREQUEST_0_AMT'.$m] = $this->formatPrice($item->getDiscountedUnitPrice());
-            $details['L_PAYMENTREQUEST_0_QTY'.$m] = $item->getQuantity();
+            $details['L_PAYMENTREQUEST_0_NAME' . $m] = $item->getVariant()->getProduct()->getName();
+            $details['L_PAYMENTREQUEST_0_AMT' . $m] = $this->formatPrice($item->getDiscountedUnitPrice());
+            $details['L_PAYMENTREQUEST_0_QTY' . $m] = $item->getQuantity();
 
             ++$m;
         }
 
         if (0 !== $taxTotal = $order->getAdjustmentsTotalRecursively(AdjustmentInterface::TAX_ADJUSTMENT)) {
-            $details['L_PAYMENTREQUEST_0_NAME'.$m] = 'Tax Total';
-            $details['L_PAYMENTREQUEST_0_AMT'.$m] = $this->formatPrice($taxTotal);
-            $details['L_PAYMENTREQUEST_0_QTY'.$m] = 1;
+            $details['L_PAYMENTREQUEST_0_NAME' . $m] = 'Tax Total';
+            $details['L_PAYMENTREQUEST_0_AMT' . $m] = $this->formatPrice($taxTotal);
+            $details['L_PAYMENTREQUEST_0_QTY' . $m] = 1;
 
             ++$m;
         }
 
         if (0 !== $promotionTotal = $order->getOrderPromotionTotal()) {
-            $details['L_PAYMENTREQUEST_0_NAME'.$m] = 'Discount';
-            $details['L_PAYMENTREQUEST_0_AMT'.$m] = $this->formatPrice($promotionTotal);
-            $details['L_PAYMENTREQUEST_0_QTY'.$m] = 1;
+            $details['L_PAYMENTREQUEST_0_NAME' . $m] = 'Discount';
+            $details['L_PAYMENTREQUEST_0_AMT' . $m] = $this->formatPrice($promotionTotal);
+            $details['L_PAYMENTREQUEST_0_QTY' . $m] = 1;
 
             ++$m;
         }
 
         if (0 !== $shippingTotal = $order->getShippingTotal()) {
-            $details['L_PAYMENTREQUEST_0_NAME'.$m] = 'Shipping Total';
-            $details['L_PAYMENTREQUEST_0_AMT'.$m] = $this->formatPrice($shippingTotal);
-            $details['L_PAYMENTREQUEST_0_QTY'.$m] = 1;
+            $details['L_PAYMENTREQUEST_0_NAME' . $m] = 'Shipping Total';
+            $details['L_PAYMENTREQUEST_0_AMT' . $m] = $this->formatPrice($shippingTotal);
+            $details['L_PAYMENTREQUEST_0_QTY' . $m] = 1;
         }
 
         $request->setResult($details);

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);