pamil / Sylius

src/Sylius/Component/Payment/Model/PaymentMethodInterface.php

Doc Comments +6 added lines

@@ -32,16 +32,19 @@ 
 
     /**
      * @param Collection $variants
+     * @return void
      */
     public function setVariants(Collection $variants);
 
     /**
      * @param VariantInterface $variant
+     * @return void
      */
     public function addVariant(VariantInterface $variant);
 
     /**
      * @param VariantInterface $variant
+     * @return void
      */
     public function removeVariant(VariantInterface $variant);
 
@@ -64,16 +67,19 @@ 
 
     /**
      * @param Collection $options
+     * @return void
      */
     public function setOptions(Collection $options);
 
     /**
      * @param OptionInterface $option
+     * @return void
      */
     public function addOption(OptionInterface $option);
 
     /**
      * @param OptionInterface $option
+     * @return void
      */
     public function removeOption(OptionInterface $option);
 

src/Sylius/Bundle/PaymentBundle/spec/Form/Type/PaymentMethodTypeSpec.php

Braces +1 added lines, -1 removed lines

@@ -40,7 +40,7 @@
         $this->shouldImplement(FormTypeInterface::class);
     }
 
-    function it_adds_add_code_event_subscriber(FormBuilder $builder) 
+    function it_adds_add_code_event_subscriber(FormBuilder $builder)
     {
         $builder->add(Argument::any(), Argument::cetera())->willReturn($builder);
       

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

Spacing +9 added lines, -9 removed lines

@@ -180,7 +180,7 @@ 
     {
         $product = $this->createProduct($productName, $price);
 
-        $product->setDescription('Awesome '.$productName);
+        $product->setDescription('Awesome ' . $productName);
 
         if ($this->sharedStorage->has('channel')) {
             $product->addChannel($this->sharedStorage->get('channel'));
@@ -213,7 +213,7 @@ 
     {
         $product = $this->createProduct($productName, $price);
 
-        $product->setDescription('Awesome '.$productName);
+        $product->setDescription('Awesome ' . $productName);
 
         foreach ($channels as $channel) {
             $product->addChannel($channel);
@@ -251,7 +251,7 @@ 
         $product->setCode(StringInflector::nameToUppercaseCode($productName));
         $product->setSlug($this->slugGenerator->generate($productName));
 
-        $product->setDescription('Awesome '.$productName);
+        $product->setDescription('Awesome ' . $productName);
 
         if ($this->sharedStorage->has('channel')) {
             $channel = $this->sharedStorage->get('channel');
@@ -373,7 +373,7 @@ 
      */
     public function thisProductHasAttributeWithValue(ProductInterface $product, $productAttributeType, $productAttributeName, $value)
     {
-        $attribute = $this->createProductAttribute($productAttributeType,$productAttributeName);
+        $attribute = $this->createProductAttribute($productAttributeType, $productAttributeName);
         $attributeValue = $this->createProductAttributeValue($value, $attribute);
         $product->addAttribute($attributeValue);
 
@@ -385,8 +385,8 @@ 
      */
     public function thisProductHasPercentAttributeWithValue(ProductInterface $product, $productAttributeName, $value)
     {
-        $attribute = $this->createProductAttribute('percent',$productAttributeName);
-        $attributeValue = $this->createProductAttributeValue($value/100, $attribute);
+        $attribute = $this->createProductAttribute('percent', $productAttributeName);
+        $attributeValue = $this->createProductAttributeValue($value / 100, $attribute);
         $product->addAttribute($attributeValue);
 
         $this->objectManager->flush();
@@ -410,9 +410,9 @@ 
      */
     public function thisProductHasPercentAttributeWithValueAtPosition(ProductInterface $product, $productAttributeName, $position)
     {
-        $attribute = $this->createProductAttribute('percent',$productAttributeName);
+        $attribute = $this->createProductAttribute('percent', $productAttributeName);
         $attribute->setPosition($position);
-        $attributeValue = $this->createProductAttributeValue(rand(1,100)/100, $attribute);
+        $attributeValue = $this->createProductAttributeValue(rand(1, 100) / 100, $attribute);
 
         $product->addAttribute($attributeValue);
 
@@ -583,7 +583,7 @@ 
 
         /** @var ImageInterface $productImage */
         $productImage = $this->productImageFactory->createNew();
-        $productImage->setFile(new UploadedFile($filesPath.$imagePath, basename($imagePath)));
+        $productImage->setFile(new UploadedFile($filesPath . $imagePath, basename($imagePath)));
         $productImage->setCode($imageCode);
         $this->imageUploader->upload($productImage);
 

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

@@ -15,6 +15,7 @@ 
 use Behat\Gherkin\Node\TableNode;
 use Behat\Mink\Element\NodeElement;
 use Doctrine\Common\Persistence\ObjectManager;
+use Sylius\Behat\Service\SharedStorageInterface;
 use Sylius\Component\Attribute\Factory\AttributeFactoryInterface;
 use Sylius\Component\Core\Formatter\StringInflector;
 use Sylius\Component\Core\Model\ChannelInterface;
@@ -24,7 +25,6 @@ 
 use Sylius\Component\Core\Model\ProductVariantInterface;
 use Sylius\Component\Core\Pricing\Calculators;
 use Sylius\Component\Core\Repository\ProductRepositoryInterface;
-use Sylius\Behat\Service\SharedStorageInterface;
 use Sylius\Component\Core\Uploader\ImageUploaderInterface;
 use Sylius\Component\Product\Factory\ProductFactoryInterface;
 use Sylius\Component\Product\Generator\SlugGeneratorInterface;
@@ -32,11 +32,11 @@ 
 use Sylius\Component\Product\Model\ProductAttributeValueInterface;
 use Sylius\Component\Product\Model\ProductOptionInterface;
 use Sylius\Component\Product\Model\ProductOptionValueInterface;
+use Sylius\Component\Product\Resolver\ProductVariantResolverInterface;
 use Sylius\Component\Resource\Factory\FactoryInterface;
-use Sylius\Component\Shipping\Model\ShippingCategoryInterface;
 use Sylius\Component\Resource\Model\TranslationInterface;
+use Sylius\Component\Shipping\Model\ShippingCategoryInterface;
 use Sylius\Component\Taxation\Model\TaxCategoryInterface;
-use Sylius\Component\Product\Resolver\ProductVariantResolverInterface;
 use Symfony\Component\HttpFoundation\File\UploadedFile;
 
 /**

src/Sylius/Behat/Page/Shop/Checkout/SelectPaymentPage.php

Indentation +13 added lines, -13 removed lines

@@ -100,25 +100,25 @@
     /**
      * {@inheritdoc}
      */
-     public function isNextStepButtonUnavailable()
-     {
-         return $this->getElement('next_step')->hasClass('disabled');
-     }
+        public function isNextStepButtonUnavailable()
+        {
+            return $this->getElement('next_step')->hasClass('disabled');
+        }
 
     /**
      * {@inheritdoc}
      */
-     public function getPaymentMethods()
-     {
-         $inputs = $this->getSession()->getPage()->findAll('css', '#payment_methods .item .content label');
+        public function getPaymentMethods()
+        {
+            $inputs = $this->getSession()->getPage()->findAll('css', '#payment_methods .item .content label');
 
-         $paymentMethods = [];
-         foreach ($inputs as $input) {
-             $paymentMethods[] = trim($input->getText());
-         }
+            $paymentMethods = [];
+            foreach ($inputs as $input) {
+                $paymentMethods[] = trim($input->getText());
+            }
 
-         return $paymentMethods;
-     }
+            return $paymentMethods;
+        }
 
     /**
      * {@inheritdoc}

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

Doc Comments +3 added lines

@@ -39,6 +39,7 @@ 
      * Sets all option values.
      *
      * @param Collection $optionValues
+     * @return void
      */
     public function setValues(Collection $optionValues);
 
@@ -46,6 +47,7 @@ 
      * Adds option value.
      *
      * @param OptionValueInterface $optionValue
+     * @return void
      */
     public function addValue(OptionValueInterface $optionValue);
 
@@ -53,6 +55,7 @@ 
      * Removes option value.
      *
      * @param OptionValueInterface $optionValue
+     * @return void
      */
     public function removeValue(OptionValueInterface $optionValue);
 

src/Sylius/Behat/Page/Admin/ExchangeRate/UpdatePageInterface.php

Doc Comments +1 added lines

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

src/Sylius/Component/Currency/Model/ExchangeRateInterface.php

Doc Comments +3 added lines

@@ -39,6 +39,7 @@ 
      * Sets all option values.
      *
      * @param Collection $optionValues
+     * @return void
      */
     public function setValues(Collection $optionValues);
 
@@ -46,6 +47,7 @@ 
      * Adds option value.
      *
      * @param OptionValueInterface $optionValue
+     * @return void
      */
     public function addValue(OptionValueInterface $optionValue);
 
@@ -53,6 +55,7 @@ 
      * Removes option value.
      *
      * @param OptionValueInterface $optionValue
+     * @return void
      */
     public function removeValue(OptionValueInterface $optionValue);
 

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

Spacing +1 added lines, -1 removed lines

@@ -116,7 +116,7 @@
      */
     public function iChangeRatioTo($ratio)
     {
-        $this->updatePage->changeRatio((float)$ratio);
+        $this->updatePage->changeRatio((float) $ratio);
     }
 
     /**

tests/Controller/ProvinceApiTest.php

Spacing +3 added lines, -3 removed lines

@@ -40,7 +40,7 @@ 
         $this->loadFixturesFromFile('authentication/api_administrator.yml');
         $countryData = $this->loadFixturesFromFile('resources/countries.yml');
 
-        $this->client->request('GET', '/api/countries/'.$countryData['country_BE']->getCode().'/provinces/'.$countryData['province_BE_limburg']->getCode(), [], [], static::$authorizedHeaderWithContentType);
+        $this->client->request('GET', '/api/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);
@@ -51,12 +51,12 @@ 
         $this->loadFixturesFromFile('authentication/api_administrator.yml');
         $countryData = $this->loadFixturesFromFile('resources/countries.yml');
 
-        $this->client->request('DELETE', '/api/countries/'.$countryData['country_BE']->getCode().'/provinces/'.$countryData['province_BE_limburg']->getCode(), [], [], static::$authorizedHeaderWithContentType);
+        $this->client->request('DELETE', '/api/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/countries/'.$countryData['country_BE']->getCode().'/provinces/'.$countryData['province_BE_limburg']->getCode(), [], [], static::$authorizedHeaderWithContentType);
+        $this->client->request('GET', '/api/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);