pamil / Sylius

src/Sylius/Behat/Page/Shop/Product/ShowPageInterface.php

Indentation +1 added lines, -1 removed lines

@@ -64,5 +64,5 @@
      *
      * @return bool
      */
-     public function hasAttributeWithValue($attributeName, $AttributeValue);
+        public function hasAttributeWithValue($attributeName, $AttributeValue);
 }

Doc Comments +5 added lines

@@ -31,6 +31,7 @@ 
 
     /**
      * @param string $name
+     * @return void
      */
     public function setName($name);
 
@@ -41,6 +42,7 @@ 
 
     /**
      * @param VariableInterface|null $object
+     * @return void
      */
     public function setObject(VariableInterface $object = null);
 
@@ -51,16 +53,19 @@ 
 
     /**
      * @param Collection $options
+     * @return void
      */
     public function setOptions(Collection $options);
 
     /**
      * @param OptionValueInterface $option
+     * @return void
      */
     public function addOption(OptionValueInterface $option);
 
     /**
      * @param OptionValueInterface $option
+     * @return void
      */
     public function removeOption(OptionValueInterface $option);
 

src/Sylius/Behat/Page/Shop/Cart/SummaryPage.php

Indentation +2 added lines, -2 removed lines

@@ -132,7 +132,7 @@ 
      */
     public function hasItemNamed($name)
     {
-       return $this->hasItemWith($name, '.sylius-product-name');
+        return $this->hasItemWith($name, '.sylius-product-name');
     }
 
     /**
@@ -140,7 +140,7 @@ 
      */
     public function hasItemWithVariantNamed($variantName)
     {
-       return $this->hasItemWith($variantName, '.sylius-product-variant-name');
+        return $this->hasItemWith($variantName, '.sylius-product-variant-name');
     }
 
     /**

src/Sylius/Bundle/CoreBundle/spec/Form/EventSubscriber/AddAuthorGuestTypeFormSubscriberSpec.php

Spacing +1 added lines, -1 removed lines

@@ -41,7 +41,7 @@
 
     function it_listens_on_pre_set_data_event()
     {
-        $this->getSubscribedEvents()->shouldReturn([FormEvents::PRE_SET_DATA => 'preSetData',]);
+        $this->getSubscribedEvents()->shouldReturn([FormEvents::PRE_SET_DATA => 'preSetData', ]);
     }
 
     function it_adds_author_guest_form_type_if_user_is_not_logged_in_and_review_subject_does_not_have_author(

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

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

@@ -14,19 +14,19 @@
 use Behat\Behat\Context\Context;
 use Doctrine\Common\Persistence\ObjectManager;
 use SM\Factory\FactoryInterface as StateMachineFactoryInterface;
+use Sylius\Component\Core\Model\AddressInterface;
 use Sylius\Component\Core\Model\ChannelInterface;
 use Sylius\Component\Core\Model\CouponInterface;
-use Sylius\Component\Core\OrderProcessing\OrderRecalculatorInterface;
-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\ShippingMethodInterface;
+use Sylius\Component\Core\OrderProcessing\OrderRecalculatorInterface;
 use Sylius\Component\Core\OrderProcessing\OrderShipmentProcessorInterface;
 use Sylius\Component\Core\Repository\OrderRepositoryInterface;
 use Sylius\Component\Core\Test\Services\SharedStorageInterface;
+use Sylius\Component\Order\Modifier\OrderItemQuantityModifierInterface;
 use Sylius\Component\Order\OrderTransitions;
 use Sylius\Component\Payment\Factory\PaymentFactoryInterface;
 use Sylius\Component\Payment\Model\PaymentInterface;

Spacing +3 added lines, -3 removed lines

@@ -310,7 +310,7 @@ 
     {
         $customer = $user->getCustomer();
         for ($i = 0; $i < $numberOfOrders; $i++) {
-            $order = $this->createOrder($customer, '#00000'.$i);
+            $order = $this->createOrder($customer, '#00000' . $i);
             $order->setPaymentState(PaymentInterface::STATE_COMPLETED);
             $order->setCompletedAt(new \DateTime());
 
@@ -358,7 +358,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->setPaymentState(PaymentInterface::STATE_COMPLETED);
             $order->setCompletedAt(new \DateTime());
 
@@ -502,7 +502,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/Setup/PromotionContext.php

Doc Comments +4 added lines, -1 removed lines

@@ -175,6 +175,7 @@ 
 
     /**
      * @Given /^([^"]+) gives ("[^"]+%") discount on shipping to every order$/
+     * @param integer $discount
      */
     public function itGivesPercentageDiscountOnShippingToEveryOrder(PromotionInterface $promotion, $discount)
     {
@@ -429,7 +430,7 @@ 
     }
 
     /**
-     * @param array $taxonCodes
+     * @param string[] $taxonCodes
      *
      * @return array
      */
@@ -458,6 +459,7 @@ 
      * @param PromotionInterface $promotion
      * @param int $discount
      * @param array $configuration
+     * @param RuleInterface $rule
      */
     private function createUnitFixedPromotion(PromotionInterface $promotion, $discount, array $configuration = [], $rule = null)
     {
@@ -468,6 +470,7 @@ 
      * @param PromotionInterface $promotion
      * @param int $discount
      * @param array $configuration
+     * @param RuleInterface $rule
      */
     private function createUnitPercentagePromotion(PromotionInterface $promotion, $discount, array $configuration = [], $rule = null)
     {

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

Unused Use Statements -1 removed lines

@@ -18,7 +18,6 @@
 use Sylius\Component\Core\Model\ShippingMethodInterface;
 use Sylius\Component\Core\Test\Services\SharedStorageInterface;
 use Sylius\Component\Resource\Factory\FactoryInterface;
-use Sylius\Component\Resource\Repository\RepositoryInterface;
 use Sylius\Component\Shipping\Calculator\DefaultCalculators;
 use Sylius\Component\Shipping\Repository\ShippingMethodRepositoryInterface;
 use Sylius\Component\Taxation\Model\TaxCategoryInterface;

Spacing +1 added lines, -1 removed lines

@@ -218,6 +218,6 @@
      */
     private function generateCodeFromNameAndZone($shippingMethodName, $zoneCode = null)
     {
-        return str_replace([' ', '-'], '_', strtolower($shippingMethodName)).'_'.strtolower($zoneCode);
+        return str_replace([' ', '-'], '_', strtolower($shippingMethodName)) . '_' . strtolower($zoneCode);
     }
 }

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

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

@@ -12,8 +12,8 @@
 namespace Sylius\Behat\Context\Ui\Admin;
 
 use Behat\Behat\Context\Context;
-use Sylius\Behat\Page\Admin\Crud\UpdatePageInterface;
 use Sylius\Behat\Page\Admin\Crud\IndexPageInterface;
+use Sylius\Behat\Page\Admin\Crud\UpdatePageInterface;
 use Sylius\Behat\Page\Admin\Customer\CreatePageInterface;
 use Sylius\Component\User\Model\CustomerInterface;
 use Webmozart\Assert\Assert;

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

Doc Comments +2 added lines

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

src/Sylius/Behat/Context/Ui/CheckoutContext.php

Doc Comments +1 added lines

@@ -337,6 +337,7 @@
 
     /**
      * @When /^I proceed selecting "([^"]*)" as shipping country with "([^"]*)" payment method$/
+     * @param string $paymentMethodName
      */
     public function iProceedSelectingShippingCountryAndPaymentMethod($shippingCountry, $paymentMethodName)
     {

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

@@ -13,18 +13,18 @@
 
 use Behat\Behat\Context\Context;
 use Sylius\Behat\Page\Shop\Checkout\AddressingPageInterface;
-use Sylius\Behat\Page\Shop\Checkout\PaymentPageInterface;
-use Sylius\Behat\Page\Shop\Checkout\ShippingPageInterface;
-use Sylius\Behat\Page\Shop\Checkout\SummaryPageInterface;
-use Sylius\Behat\Page\Shop\Order\OrderPaymentsPageInterface;
 use Sylius\Behat\Page\Shop\Checkout\AddressingStepInterface;
 use Sylius\Behat\Page\Shop\Checkout\FinalizeStepInterface;
+use Sylius\Behat\Page\Shop\Checkout\PaymentPageInterface;
 use Sylius\Behat\Page\Shop\Checkout\PaymentStepInterface;
 use Sylius\Behat\Page\Shop\Checkout\SecurityStepInterface;
+use Sylius\Behat\Page\Shop\Checkout\ShippingPageInterface;
 use Sylius\Behat\Page\Shop\Checkout\ShippingStepInterface;
+use Sylius\Behat\Page\Shop\Checkout\SummaryPageInterface;
 use Sylius\Behat\Page\Shop\Checkout\ThankYouPageInterface;
-use Sylius\Component\Core\Formatter\StringInflector;
+use Sylius\Behat\Page\Shop\Order\OrderPaymentsPageInterface;
 use Sylius\Behat\Service\SecurityServiceInterface;
+use Sylius\Component\Core\Formatter\StringInflector;
 use Sylius\Component\Core\Model\AddressInterface;
 use Sylius\Component\Core\Model\OrderInterface;
 use Sylius\Component\Core\Model\ProductInterface;

Spacing +3 added lines, -3 removed lines

@@ -530,7 +530,7 @@ 
      */
     public function iShouldSeeThisShippingAddressAsShippingAddress($fullName)
     {
-        $address = $this->sharedStorage->get('shipping_address_'.StringInflector::nameToLowercaseCode($fullName));
+        $address = $this->sharedStorage->get('shipping_address_' . StringInflector::nameToLowercaseCode($fullName));
         Assert::true(
             $this->summaryPage->hasShippingAddress($address),
             'Shipping address is improper.'
@@ -542,7 +542,7 @@ 
      */
     public function iShouldSeeThisBillingAddressAsBillingAddress($fullName)
     {
-        $address = $this->sharedStorage->get('billing_address_'.StringInflector::nameToLowercaseCode($fullName));
+        $address = $this->sharedStorage->get('billing_address_' . StringInflector::nameToLowercaseCode($fullName));
         Assert::true(
             $this->summaryPage->hasBillingAddress($address),
             'Billing address is improper.'
@@ -622,7 +622,7 @@ 
     {
         Assert::true(
             $this->summaryPage->hasShippingTotal($price),
-            sprintf('The shipping total should be %s, but it is not.',$price)
+            sprintf('The shipping total should be %s, but it is not.', $price)
         );
     }