pamil / Sylius

src/Sylius/Bundle/CoreBundle/DataFixtures/ORM/LoadCurrencyData.php

Unused Use Statements -1 removed lines

@@ -13,7 +13,6 @@
 
 use Doctrine\Common\Persistence\ObjectManager;
 use Sylius\Bundle\CoreBundle\DataFixtures\DataFixture;
-use Sylius\Component\Currency\Model\CurrencyInterface;
 
 /**
  * Default currency fixtures.

Spacing +1 added lines, -1 removed lines

@@ -50,7 +50,7 @@
                 $currency->setBase(true);
             }
 
-            $this->setReference('Sylius.Currency.'.$code, $currency);
+            $this->setReference('Sylius.Currency.' . $code, $currency);
 
             $manager->persist($currency);
         }

src/Sylius/Bundle/CoreBundle/DataFixtures/ORM/LoadOrdersData.php

Doc Comments +3 added lines

@@ -126,6 +126,9 @@
         $order->addShipment($shipment);
     }
 
+    /**
+     * @param OrderInterface $order
+     */
     protected function dispatchEvents($order)
     {
         $dispatcher = $this->get('event_dispatcher');

Spacing +3 added lines, -3 removed lines

@@ -47,7 +47,7 @@ 
             $order->setChannel($channel);
 
             for ($j = 0, $items = rand(3, 6); $j <= $items; ++$j) {
-                $variant = $this->getReference('Sylius.Variant-'.rand(1, SYLIUS_FIXTURES_TOTAL_VARIANTS - 1));
+                $variant = $this->getReference('Sylius.Variant-' . rand(1, SYLIUS_FIXTURES_TOTAL_VARIANTS - 1));
 
                 /* @var $item OrderItemInterface */
                 $item = $orderItemFactory->createNew();
@@ -72,11 +72,11 @@ 
             $order->complete();
             $paymentState = $this->faker->boolean(50) ? PaymentInterface::STATE_COMPLETED : PaymentInterface::STATE_NEW;
 
-            $order->setCustomer($this->getReference('Sylius.Customer-'.rand(1, 15)));
+            $order->setCustomer($this->getReference('Sylius.Customer-' . rand(1, 15)));
             $this->createPayment($order, $paymentState);
 
             $order->setCompletedAt($this->faker->dateTimeThisDecade);
-            $this->setReference('Sylius.Order-'.$i, $order);
+            $this->setReference('Sylius.Order-' . $i, $order);
 
             $manager->persist($order);
         }

src/Sylius/Bundle/CoreBundle/DataFixtures/ORM/LoadProductArchetypeData.php

Doc Comments +1 added lines, -1 removed lines

@@ -57,7 +57,7 @@
      * @param string $code
      * @param array  $nameTranslations
      * @param array  $options
-     * @param array  $properties
+     * @param string[]  $properties
      *
      * @return ArchetypeInterface
      */

Spacing +3 added lines, -3 removed lines

@@ -72,13 +72,13 @@
         }
 
         foreach ($options as $option) {
-            $archetype->addOption($this->getReference('Sylius.Option.'.$option));
+            $archetype->addOption($this->getReference('Sylius.Option.' . $option));
         }
         foreach ($properties as $attribute) {
-            $archetype->addAttribute($this->getReference('Sylius.Attribute.'.$attribute));
+            $archetype->addAttribute($this->getReference('Sylius.Attribute.' . $attribute));
         }
 
-        $this->setReference('Sylius.Archetype.'.$code, $archetype);
+        $this->setReference('Sylius.Archetype.' . $code, $archetype);
 
         return $archetype;
     }

src/Sylius/Bundle/CoreBundle/DataFixtures/ORM/LoadProductsData.php

Doc Comments +2 added lines, -2 removed lines

@@ -313,7 +313,7 @@ 
      * Adds taxons to given product.
      *
      * @param ProductInterface $product
-     * @param array $taxonCodes
+     * @param string[] $taxonCodes
      */
     protected function setTaxons(ProductInterface $product, array $taxonCodes)
     {
@@ -330,7 +330,7 @@ 
      * Set channels.
      *
      * @param ProductInterface $product
-     * @param array $channelCodes
+     * @param string[] $channelCodes
      */
     protected function setChannels(ProductInterface $product, array $channelCodes)
     {

Spacing +11 added lines, -11 removed lines

@@ -118,7 +118,7 @@ 
 
         $this->generateVariants($product);
 
-        $this->setReference('Sylius.Product.'.$i, $product);
+        $this->setReference('Sylius.Product.' . $i, $product);
 
         return $product;
     }
@@ -162,7 +162,7 @@ 
 
         $this->generateVariants($product);
 
-        $this->setReference('Sylius.Product.'.$i, $product);
+        $this->setReference('Sylius.Product.' . $i, $product);
 
         return $product;
     }
@@ -199,7 +199,7 @@ 
 
         $this->generateVariants($product);
 
-        $this->setReference('Sylius.Product.'.$i, $product);
+        $this->setReference('Sylius.Product.' . $i, $product);
 
         return $product;
     }
@@ -236,7 +236,7 @@ 
         $this->addAttribute($product, 'book_isbn', $isbn);
         $this->addAttribute($product, 'book_pages', $this->faker->randomNumber(3));
 
-        $this->setReference('Sylius.Product.'.$i, $product);
+        $this->setReference('Sylius.Product.' . $i, $product);
 
         return $product;
     }
@@ -259,7 +259,7 @@ 
             $variant->setCode($this->getUniqueCode());
             $variant->setOnHand($this->faker->randomNumber(1));
 
-            $this->setReference('Sylius.Variant-'.$this->totalVariants, $variant);
+            $this->setReference('Sylius.Variant-' . $this->totalVariants, $variant);
 
             ++$this->totalVariants;
         }
@@ -281,12 +281,12 @@ 
         $variant->setTaxCategory($this->getTaxCategory('taxable'));
 
         $mainTaxon = $product->getMainTaxon();
-        $image = clone $this->getReference('Sylius.Image.'.$mainTaxon->getCode());
+        $image = clone $this->getReference('Sylius.Image.' . $mainTaxon->getCode());
         $variant->addImage($image);
 
         $product->addVariant($variant);
 
-        $this->setReference('Sylius.Variant-'.$this->totalVariants, $variant);
+        $this->setReference('Sylius.Variant-' . $this->totalVariants, $variant);
 
         ++$this->totalVariants;
     }
@@ -302,7 +302,7 @@ 
     {
         /* @var $attribute AttributeValueInterface */
         $attribute = $this->getProductAttributeValueFactory()->createNew();
-        $attribute->setAttribute($this->getReference('Sylius.Attribute.'.$code));
+        $attribute->setAttribute($this->getReference('Sylius.Attribute.' . $code));
         $attribute->setProduct($product);
         $attribute->setValue($value);
 
@@ -320,7 +320,7 @@ 
         $taxons = new ArrayCollection();
 
         foreach ($taxonCodes as $taxonCode) {
-            $taxons->add($this->getReference('Sylius.Taxon.'.$taxonCode));
+            $taxons->add($this->getReference('Sylius.Taxon.' . $taxonCode));
         }
 
         $product->setTaxons($taxons);
@@ -335,7 +335,7 @@ 
     protected function setChannels(ProductInterface $product, array $channelCodes)
     {
         foreach ($channelCodes as $code) {
-            $product->addChannel($this->getReference('Sylius.Channel.'.$code));
+            $product->addChannel($this->getReference('Sylius.Channel.' . $code));
         }
     }
 
@@ -346,7 +346,7 @@ 
      */
     protected function getTaxCategory($code)
     {
-        return $this->getReference('Sylius.TaxCategory.'.$code);
+        return $this->getReference('Sylius.TaxCategory.' . $code);
     }
 
     /**

src/Sylius/Bundle/CoreBundle/DataFixtures/ORM/LoadPromotionsData.php

Doc Comments +2 added lines, -2 removed lines

@@ -137,8 +137,8 @@
      * @param string $description
      * @param int $priority
      * @param string $channel
-     * @param array $rules
-     * @param array $actions
+     * @param RuleInterface[] $rules
+     * @param ActionInterface[] $actions
      *
      * @return PromotionInterface
      */

Spacing +2 added lines, -2 removed lines

@@ -151,7 +151,7 @@ 
         $promotion->setCode($code);
         $promotion->setPriority($priority);
 
-        $promotion->addChannel($this->getReference('Sylius.Channel.'.$channel));
+        $promotion->addChannel($this->getReference('Sylius.Channel.' . $channel));
 
         foreach ($rules as $rule) {
             $promotion->addRule($rule);
@@ -160,7 +160,7 @@ 
             $promotion->addAction($action);
         }
 
-        $this->setReference('Sylius.Promotion.'.$code, $promotion);
+        $this->setReference('Sylius.Promotion.' . $code, $promotion);
 
         return $promotion;
     }

src/Sylius/Bundle/CoreBundle/DataFixtures/ORM/LoadUsersData.php

Doc Comments +1 added lines, -1 removed lines

@@ -85,7 +85,7 @@
      * @param string $email
      * @param string $password
      * @param bool   $enabled
-     * @param array  $roles
+     * @param string[]  $roles
      * @param string $currencyCode
      *
      * @return UserInterface

Spacing +3 added lines, -3 removed lines

@@ -50,7 +50,7 @@ 
             }
 
             $user = $this->createUser(
-                $username.'@example.com',
+                $username . '@example.com',
                 $username,
                 $this->faker->boolean()
             );
@@ -60,8 +60,8 @@ 
             $manager->persist($user);
             $this->usernames[$username] = true;
 
-            $this->setReference('Sylius.User-'.$i, $user);
-            $this->setReference('Sylius.Customer-'.$i, $user->getCustomer());
+            $this->setReference('Sylius.User-' . $i, $user);
+            $this->setReference('Sylius.Customer-' . $i, $user->getCustomer());
         }
 
         $customer = $this->getCustomerFactory()->createNew();

src/Sylius/Bundle/CoreBundle/Fixture/Factory/TaxonExampleFactory.php

Doc Comments +1 added lines, -1 removed lines

@@ -110,7 +110,7 @@
     }
 
     /**
-     * @return array
+     * @return \Generator
      */
     private function getLocales()
     {

src/Sylius/Bundle/CoreBundle/Form/Type/CountryType.php

Doc Comments +2 added lines, -2 removed lines

@@ -93,7 +93,7 @@ 
     }
 
     /**
-     * @param $code
+     * @param string $code
      *
      * @return null|string
      */
@@ -103,7 +103,7 @@ 
     }
 
     /**
-     * @return array
+     * @return string[]
      */
     private function getAvailableCountries()
     {

src/Sylius/Bundle/CoreBundle/Routing/RouteProvider.php

Doc Comments +1 added lines

@@ -194,6 +194,7 @@
 
     /**
      * @param object $entity
+     * @param string $value
      *
      * @return Route
      */

Spacing +2 added lines, -2 removed lines

@@ -189,7 +189,7 @@ 
      */
     private function getFieldValue($entity, $fieldName)
     {
-        return $entity->{'get'.ucfirst($fieldName)}();
+        return $entity->{'get' . ucfirst($fieldName)}();
     }
 
     /**
@@ -207,7 +207,7 @@ 
         }
         $defaults = ['_sylius_entity' => $entity, $fieldName => $value];
 
-        return new Route($this->routeConfigs[$className]['prefix'].'/'.$value, $defaults);
+        return new Route($this->routeConfigs[$className]['prefix'] . '/' . $value, $defaults);
     }
 
     /**