pamil / Sylius

src/Sylius/Bundle/FixturesBundle/spec/Suite/SuiteFactorySpec.php

Braces +2 added lines, -1 removed lines

@@ -222,7 +222,8 @@
      *
      * @return \Generator
      */
-    private function createGenerator(Collaborator ...$collaborators) {
+    private function createGenerator(Collaborator ...$collaborators)
+    {
         foreach ($collaborators as $collaborator) {
             yield $collaborator->getWrappedObject() => [];
         }

Unused Use Statements -1 removed lines

@@ -13,7 +13,6 @@
 
 use PhpSpec\ObjectBehavior;
 use PhpSpec\Wrapper\Collaborator;
-use Prophecy\Argument;
 use Sylius\Bundle\FixturesBundle\Fixture\FixtureInterface;
 use Sylius\Bundle\FixturesBundle\Fixture\FixtureRegistryInterface;
 use Sylius\Bundle\FixturesBundle\Listener\ListenerInterface;

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

Doc Comments +9 added lines

@@ -31,10 +31,19 @@
      */
     public function isAvailableInChannel($channelName);
 
+    /**
+     * @return void
+     */
     public function enable();
 
+    /**
+     * @return void
+     */
     public function disable();
 
+    /**
+     * @return void
+     */
     public function removeZone();
 
     /**

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

Doc Comments +11 added lines, -1 removed lines

@@ -20,6 +20,7 @@ 
 {
     /**
      * @param string $shippingMethod
+     * @return void
      */
     public function selectShippingMethod($shippingMethod);
 
@@ -48,14 +49,23 @@ 
      */
     public function getItemSubtotal($itemName);
 
+    /**
+     * @return void
+     */
     public function nextStep();
 
+    /**
+     * @return void
+     */
     public function changeAddress();
 
+    /**
+     * @return void
+     */
     public function changeAddressByStepLabel();
 
     /**
-     * @return mixed string
+     * @return string string
      */
     public function getPurchaserEmail();
 

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

Spacing +1 added lines, -1 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);
     }
 
     /**

Doc Comments +2 added lines, -1 removed lines

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

src/Sylius/Bundle/PromotionBundle/Form/EventListener/AbstractConfigurationSubscriber.php

Doc Comments -1 removed lines

@@ -129,7 +129,6 @@
     }
 
     /**
-     * @param PromotionDynamicTypeInterface|null $type
      * @param FormInterface $form
      *
      * @return null|string

src/Sylius/Component/Core/Payment/Provider/OrderPaymentProvider.php

Doc Comments +1 added lines, -1 removed lines

@@ -106,7 +106,7 @@
      * @param PaymentInterface $payment
      * @param OrderInterface $order
      *
-     * @return PaymentMethodInterface|null
+     * @return \Sylius\Component\Payment\Model\PaymentMethodInterface|null
      */
     private function getDefaultPaymentMethod(PaymentInterface $payment, OrderInterface $order)
     {

src/Sylius/Component/Grid/Data/ExpressionBuilderInterface.php

Doc Comments +1 added lines, -1 removed lines

@@ -81,7 +81,7 @@
 
     /**
      * @param string $field
-     * @param mixed $value
+     * @param string $value
      *
      * @return self
      */

app/migrations/Version20161214153137.php

Spacing +2 added lines, -2 removed lines

@@ -33,7 +33,7 @@ 
 
         $this->addSql('CREATE TABLE sylius_product_variant_translation (id INT AUTO_INCREMENT NOT NULL, translatable_id INT NOT NULL, name VARCHAR(255) DEFAULT NULL, locale VARCHAR(255) NOT NULL, INDEX IDX_8DC18EDC2C2AC5D3 (translatable_id), UNIQUE INDEX sylius_product_variant_translation_uniq_trans (translatable_id, locale), PRIMARY KEY(id)) DEFAULT CHARACTER SET utf8 COLLATE utf8_unicode_ci ENGINE = InnoDB');
         $this->addSql('ALTER TABLE sylius_product_variant_translation ADD CONSTRAINT FK_8DC18EDC2C2AC5D3 FOREIGN KEY (translatable_id) REFERENCES sylius_product_variant (id) ON DELETE CASCADE');
-        $this->addSql('INSERT INTO sylius_product_variant_translation (translatable_id, name, locale) SELECT id, name, "'.$defaultLocale.'" from sylius_product_variant WHERE sylius_product_variant.name IS NOT null');
+        $this->addSql('INSERT INTO sylius_product_variant_translation (translatable_id, name, locale) SELECT id, name, "' . $defaultLocale . '" from sylius_product_variant WHERE sylius_product_variant.name IS NOT null');
         $this->addSql('ALTER TABLE sylius_product_variant DROP name');
     }
 
@@ -47,7 +47,7 @@ 
         $this->abortIf($this->connection->getDatabasePlatform()->getName() != 'mysql', 'Migration can only be executed safely on \'mysql\'.');
 
         $this->addSql('ALTER TABLE sylius_product_variant ADD name VARCHAR(255) DEFAULT NULL COLLATE utf8_unicode_ci');
-        $this->addSql('UPDATE sylius_product_variant SET name = (SELECT name FROM sylius_product_variant_translation WHERE sylius_product_variant_translation.translatable_id = sylius_product_variant.id AND sylius_product_variant_translation.locale = "'.$defaultLocale.'")');
+        $this->addSql('UPDATE sylius_product_variant SET name = (SELECT name FROM sylius_product_variant_translation WHERE sylius_product_variant_translation.translatable_id = sylius_product_variant.id AND sylius_product_variant_translation.locale = "' . $defaultLocale . '")');
         $this->addSql('DROP TABLE sylius_product_variant_translation');
     }
 }

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

Doc Comments +18 added lines

@@ -18,67 +18,85 @@
  */
 interface CreatePageInterface extends BaseCreatePageInterface
 {
+    /**
+     * @return void
+     */
     public function enable();
 
+    /**
+     * @return void
+     */
     public function disable();
 
     /**
      * @param string $name
+     * @return void
      */
     public function nameIt($name);
 
     /**
      * @param string $code
+     * @return void
      */
     public function specifyCode($code);
 
     /**
      * @param string $description
+     * @return void
      */
     public function describeItAs($description);
 
     /**
      * @param string $hostname
+     * @return void
      */
     public function setHostname($hostname);
 
     /**
      * @param string $contactEmail
+     * @return void
      */
     public function setContactEmail($contactEmail);
 
     /**
      * @param string $color
+     * @return void
      */
     public function defineColor($color);
 
     /**
      * @param string $language
+     * @return void
      */
     public function chooseLocale($language);
 
     /**
      * @param string $currencyCode
+     * @return void
      */
     public function chooseCurrency($currencyCode);
 
     /**
      * @param string $taxZone
+     * @return void
      */
     public function chooseDefaultTaxZone($taxZone);
 
     /**
      * @param string $locale
+     * @return void
      */
     public function chooseDefaultLocale($locale);
 
     /**
      * @param string $currency
+     * @return void
      */
     public function chooseBaseCurrency($currency);
 
     /**
      * @param string $taxCalculationStrategy
+     * @return void
      */
     public function chooseTaxCalculationStrategy($taxCalculationStrategy);
 }