pamil / Sylius

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

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

@@ -14,30 +14,29 @@
 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\ChannelInterface;
-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\ChannelInterface;
 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>

Spacing +4 added lines, -4 removed lines

@@ -368,7 +368,7 @@ 
     ) {
         $customer = $user->getCustomer();
         for ($i = 0; $i < $numberOfOrders; $i++) {
-            $order = $this->createOrder($customer, '#00000'.$i);
+            $order = $this->createOrder($customer, '#00000' . $i);
             $this->checkoutUsing($order, $shippingMethod, clone $address, $paymentMethod);
             $this->applyPaymentTransitionOnOrder($order, PaymentTransitions::TRANSITION_COMPLETE);
 
@@ -415,7 +415,7 @@ 
         $total = $this->getPriceFromString($total);
 
         for ($i = 0; $i < $numberOfOrders; $i++) {
-            $order = $this->createOrder($customers[rand(0, $numberOfCustomers - 1)], '#'.uniqid());
+            $order = $this->createOrder($customers[rand(0, $numberOfCustomers - 1)], '#' . uniqid());
             $order->setState(OrderInterface::STATE_NEW); // Temporary, we should use checkout to place these orders.
             $this->applyPaymentTransitionOnOrder($order, PaymentTransitions::TRANSITION_COMPLETE);
 
@@ -445,7 +445,7 @@ 
         $total = $this->getPriceFromString($total);
 
         for ($i = 0; $i < $numberOfOrders; $i++) {
-            $order = $this->createOrder($customers[rand(0, $numberOfCustomers - 1)], '#'.uniqid(), $product->getChannels()->first());
+            $order = $this->createOrder($customers[rand(0, $numberOfCustomers - 1)], '#' . uniqid(), $product->getChannels()->first());
             $order->setState(OrderInterface::STATE_NEW);
             $this->applyPaymentTransitionOnOrder($order, PaymentTransitions::TRANSITION_COMPLETE);
 
@@ -621,7 +621,7 @@ 
             $customer = $this->customerFactory->createNew();
             $customer->setEmail(sprintf('john%s@doe.com', uniqid()));
             $customer->setFirstname('John');
-            $customer->setLastname('Doe'.$i);
+            $customer->setLastname('Doe' . $i);
 
             $customers[] = $customer;
 

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

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

@@ -12,9 +12,8 @@
 namespace Sylius\Behat\Context\Transform;
 
 use Behat\Behat\Context\Context;
-use Sylius\Component\Core\Model\OrderInterface;
-use Sylius\Component\Core\Repository\OrderRepositoryInterface;
 use Sylius\Component\Core\Repository\CustomerRepositoryInterface;
+use Sylius\Component\Core\Repository\OrderRepositoryInterface;
 use Webmozart\Assert\Assert;
 
 /**

src/Sylius/Behat/Context/Transform/ProductVariantContext.php

Unused Use Statements -1 removed lines

@@ -13,7 +13,6 @@
 
 use Behat\Behat\Context\Context;
 use Sylius\Component\Core\Repository\ProductRepositoryInterface;
-use Sylius\Component\Core\Repository\ProductVariantRepositoryInterface;
 use Sylius\Component\Resource\Repository\RepositoryInterface;
 use Webmozart\Assert\Assert;
 

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

Doc Comments +1 added lines

@@ -205,6 +205,7 @@
      * @Then the product :productName should appear in the shop
      * @Then the product :productName should be in the shop
      * @Then this product should still be named :productName
+     * @param string $productName
      */
     public function theProductShouldAppearInTheShop($productName)
     {

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

@@ -15,17 +15,17 @@
 use Sylius\Behat\NotificationType;
 use Sylius\Behat\Page\Admin\Crud\CreatePageInterface;
 use Sylius\Behat\Page\Admin\Crud\UpdatePageInterface;
+use Sylius\Behat\Page\Admin\ProductReview\IndexPageInterface as ProductReviewIndexPageInterface;
 use Sylius\Behat\Page\Admin\Product\CreateConfigurableProductPageInterface;
 use Sylius\Behat\Page\Admin\Product\CreateSimpleProductPageInterface;
 use Sylius\Behat\Page\Admin\Product\IndexPageInterface;
 use Sylius\Behat\Page\Admin\Product\UpdateConfigurableProductPageInterface;
 use Sylius\Behat\Page\Admin\Product\UpdateSimpleProductPageInterface;
-use Sylius\Behat\Page\Admin\ProductReview\IndexPageInterface as ProductReviewIndexPageInterface;
 use Sylius\Behat\Service\NotificationCheckerInterface;
 use Sylius\Behat\Service\Resolver\CurrentProductPageResolverInterface;
+use Sylius\Behat\Service\SharedStorageInterface;
 use Sylius\Component\Core\Model\ChannelInterface;
 use Sylius\Component\Core\Model\ProductInterface;
-use Sylius\Behat\Service\SharedStorageInterface;
 use Sylius\Component\Currency\Model\CurrencyInterface;
 use Sylius\Component\Product\Model\ProductAssociationTypeInterface;
 use Sylius\Component\Taxonomy\Model\TaxonInterface;

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

Doc Comments +2 added lines, -1 removed lines

@@ -150,6 +150,7 @@ 
 
     /**
      * @Then the :productVariantCode variant of the :product product should appear in the shop
+     * @param string $productVariantCode
      */
     public function theProductVariantShouldAppearInTheShop($productVariantCode, ProductInterface $product)
     {
@@ -451,7 +452,7 @@ 
 
     /**
      * @param string $element
-     * @param $message
+     * @param string $message
      */
     private function assertValidationMessage($element, $message)
     {

Spacing +2 added lines, -2 removed lines

@@ -288,7 +288,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)
         );
     }
 
@@ -301,7 +301,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)
         );
     }
 

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/Context/Ui/ThemeContext.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/Admin/Account/LoginPageInterface.php

Doc Comments +5 added lines

@@ -25,15 +25,20 @@
      */
     public function hasValidationErrorWith($message);
 
+    /**
+     * @return void
+     */
     public function logIn();
 
     /**
      * @param string $password
+     * @return void
      */
     public function specifyPassword($password);
 
     /**
      * @param string $userName
+     * @return void
      */
     public function specifyUserName($userName);
 }

src/Sylius/Behat/Page/Admin/Administrator/CreatePageInterface.php

Doc Comments +7 added lines

@@ -39,25 +39,32 @@
      */
     public function checkGenerationValidation($message);
 
+    /**
+     * @return void
+     */
     public function generate();
 
     /**
      * @param int $amount
+     * @return void
      */
     public function specifyAmount($amount);
 
     /**
      * @param int $codeLength
+     * @return void
      */
     public function specifyCodeLength($codeLength);
 
     /**
      * @param \DateTime $date
+     * @return void
      */
     public function setExpiresAt(\DateTime $date);
 
     /**
      * @param int $limit
+     * @return void
      */
     public function setUsageLimit($limit);
 }

src/Sylius/Behat/Page/Admin/Administrator/UpdatePageInterface.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);
 }