pamil / Sylius

src/Sylius/Bundle/ResourceBundle/DependencyInjection/Driver/AbstractDriver.php

Unused Use Statements -1 removed lines

@@ -17,7 +17,6 @@
 use Sylius\Component\Resource\Metadata\MetadataInterface;
 use Symfony\Component\DependencyInjection\ContainerBuilder;
 use Symfony\Component\DependencyInjection\Definition;
-use Symfony\Component\DependencyInjection\Parameter;
 use Symfony\Component\DependencyInjection\Reference;
 
 /**

src/Sylius/Behat/Page/Admin/Order/UpdatePage.php

Spacing +6 added lines, -6 removed lines

@@ -45,11 +45,11 @@ 
      */
     private function specifyAddress(AddressInterface $address, $addressType)
     {
-        $this->specifyElementValue($addressType.'_first_name', $address->getFirstName());
-        $this->specifyElementValue($addressType.'_last_name', $address->getLastName());
-        $this->specifyElementValue($addressType.'_street', $address->getStreet());
-        $this->specifyElementValue($addressType.'_city', $address->getCity());
-        $this->specifyElementValue($addressType.'_postcode', $address->getPostcode());
+        $this->specifyElementValue($addressType . '_first_name', $address->getFirstName());
+        $this->specifyElementValue($addressType . '_last_name', $address->getLastName());
+        $this->specifyElementValue($addressType . '_street', $address->getStreet());
+        $this->specifyElementValue($addressType . '_city', $address->getCity());
+        $this->specifyElementValue($addressType . '_postcode', $address->getPostcode());
 
         $this->chooseCountry($address->getCountryCode(), $addressType);
     }
@@ -114,7 +114,7 @@ 
      */
     private function chooseCountry($country, $addressType)
     {
-        $this->getElement($addressType.'_country')->selectOption((null !== $country) ? $country : 'Select');
+        $this->getElement($addressType . '_country')->selectOption((null !== $country) ? $country : 'Select');
     }
 
     /**

src/Sylius/Bundle/PayumBundle/Action/Paypal/ExpressCheckout/ConvertPaymentAction.php

Doc Comments -1 removed lines

@@ -103,7 +103,6 @@
 
     /**
      * @param int $price
-     * @param string $currencyCode
      *
      * @return float
      */

Spacing +12 added lines, -12 removed lines

@@ -57,33 +57,33 @@
 
         $m = 0;
         foreach ($order->getItems() as $item) {
-            $details['L_PAYMENTREQUEST_0_NAME'.$m] = $item->getVariant()->getProduct()->getName();
-            $details['L_PAYMENTREQUEST_0_AMT'.$m] = $this->formatPrice($item->getDiscountedUnitPrice());
-            $details['L_PAYMENTREQUEST_0_QTY'.$m] = $item->getQuantity();
+            $details['L_PAYMENTREQUEST_0_NAME' . $m] = $item->getVariant()->getProduct()->getName();
+            $details['L_PAYMENTREQUEST_0_AMT' . $m] = $this->formatPrice($item->getDiscountedUnitPrice());
+            $details['L_PAYMENTREQUEST_0_QTY' . $m] = $item->getQuantity();
 
             ++$m;
         }
 
         if (0 !== $taxTotal = $order->getAdjustmentsTotalRecursively(AdjustmentInterface::TAX_ADJUSTMENT)) {
-            $details['L_PAYMENTREQUEST_0_NAME'.$m] = 'Tax Total';
-            $details['L_PAYMENTREQUEST_0_AMT'.$m] = $this->formatPrice($taxTotal);
-            $details['L_PAYMENTREQUEST_0_QTY'.$m] = 1;
+            $details['L_PAYMENTREQUEST_0_NAME' . $m] = 'Tax Total';
+            $details['L_PAYMENTREQUEST_0_AMT' . $m] = $this->formatPrice($taxTotal);
+            $details['L_PAYMENTREQUEST_0_QTY' . $m] = 1;
 
             ++$m;
         }
 
         if (0 !== $promotionTotal = $order->getOrderPromotionTotal()) {
-            $details['L_PAYMENTREQUEST_0_NAME'.$m] = 'Discount';
-            $details['L_PAYMENTREQUEST_0_AMT'.$m] = $this->formatPrice($promotionTotal);
-            $details['L_PAYMENTREQUEST_0_QTY'.$m] = 1;
+            $details['L_PAYMENTREQUEST_0_NAME' . $m] = 'Discount';
+            $details['L_PAYMENTREQUEST_0_AMT' . $m] = $this->formatPrice($promotionTotal);
+            $details['L_PAYMENTREQUEST_0_QTY' . $m] = 1;
 
             ++$m;
         }
 
         if (0 !== $shippingTotal = $order->getShippingTotal()) {
-            $details['L_PAYMENTREQUEST_0_NAME'.$m] = 'Shipping Total';
-            $details['L_PAYMENTREQUEST_0_AMT'.$m] = $this->formatPrice($shippingTotal);
-            $details['L_PAYMENTREQUEST_0_QTY'.$m] = 1;
+            $details['L_PAYMENTREQUEST_0_NAME' . $m] = 'Shipping Total';
+            $details['L_PAYMENTREQUEST_0_AMT' . $m] = $this->formatPrice($shippingTotal);
+            $details['L_PAYMENTREQUEST_0_QTY' . $m] = 1;
         }
 
         $request->setResult($details);

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

Unused Use Statements -1 removed lines

@@ -11,7 +11,6 @@
 
 namespace Sylius\Bundle\CoreBundle\Fixture\Factory;
 
-use Doctrine\Common\Persistence\ObjectManager;
 use Sylius\Component\Core\Formatter\StringInflector;
 use Sylius\Component\Core\Model\TaxonInterface;
 use Sylius\Component\Locale\Model\LocaleInterface;

Doc Comments +1 added lines, -1 removed lines

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

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

Spacing +1 added lines, -1 removed lines

@@ -96,7 +96,7 @@
                 return $this->faker->email;
             })
             ->setDefault('username', function (Options $options) {
-                return $this->faker->firstName.' '.$this->faker->lastName;
+                return $this->faker->firstName . ' ' . $this->faker->lastName;
             })
             ->setDefault('enabled', true)
             ->setAllowedTypes('enabled', 'bool')

src/Sylius/Bundle/LocaleBundle/DependencyInjection/SyliusLocaleExtension.php

Unused Use Statements -2 removed lines

@@ -14,9 +14,7 @@
 use Sylius\Bundle\ResourceBundle\DependencyInjection\Extension\AbstractResourceExtension;
 use Symfony\Component\Config\FileLocator;
 use Symfony\Component\DependencyInjection\ContainerBuilder;
-use Symfony\Component\DependencyInjection\Extension\PrependExtensionInterface;
 use Symfony\Component\DependencyInjection\Loader\XmlFileLoader;
-use Symfony\Component\DependencyInjection\Reference;
 
 /**
  * @author Paweł Jędrzejewski <pawel@sylius.org>

Spacing +1 added lines, -1 removed lines

@@ -27,7 +27,7 @@
     public function load(array $config, ContainerBuilder $container)
     {
         $config = $this->processConfiguration($this->getConfiguration([], $container), $config);
-        $loader = new XmlFileLoader($container, new FileLocator(__DIR__.'/../Resources/config'));
+        $loader = new XmlFileLoader($container, new FileLocator(__DIR__ . '/../Resources/config'));
 
         $this->registerResources('sylius', $config['driver'], $config['resources'], $container);
 

src/Sylius/Bundle/CoreBundle/Context/CustomerAndChannelBasedCartContext.php

Unused Use Statements -1 removed lines

@@ -17,7 +17,6 @@
 use Sylius\Component\Customer\Context\CustomerContextInterface;
 use Sylius\Component\Order\Context\CartContextInterface;
 use Sylius\Component\Order\Context\CartNotFoundException;
-use Sylius\Component\Order\Model\OrderInterface;
 
 /**
  * @author Mateusz Zalewski <mateusz.zalewski@lakion.com>

src/Sylius/Bundle/CoreBundle/Remover/ReviewerReviewsRemover.php

Doc Comments +1 added lines, -1 removed lines

@@ -75,7 +75,7 @@
      * @param ReviewInterface $review
      * @param ReviewableInterface[] $reviewSubjectsToRecalculate
      *
-     * @return array
+     * @return ReviewableInterface[]
      */
     private function removeReviewsAndExtractSubject(ReviewInterface $review, array $reviewSubjectsToRecalculate)
     {

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

@@ -14,9 +14,9 @@
 use Doctrine\Common\Persistence\ObjectManager;
 use Sylius\Bundle\ResourceBundle\Doctrine\ORM\EntityRepository;
 use Sylius\Bundle\ReviewBundle\Updater\ReviewableRatingUpdaterInterface;
+use Sylius\Component\Review\Model\ReviewInterface;
 use Sylius\Component\Review\Model\ReviewableInterface;
 use Sylius\Component\Review\Model\ReviewerInterface;
-use Sylius\Component\Review\Model\ReviewInterface;
 
 /**
  * @author Grzegorz Sadowski <grzegorz.sadowski@lakion.com>

src/Sylius/Component/Order/Model/OrderInterface.php

Doc Comments +16 added lines

@@ -32,6 +32,7 @@ 
 
     /**
      * @param null|\DateTime $checkoutCompletedAt
+     * @return void
      */
     public function setCheckoutCompletedAt(\DateTime $checkoutCompletedAt = null);
 
@@ -40,6 +41,9 @@ 
      */
     public function isCheckoutCompleted();
 
+    /**
+     * @return void
+     */
     public function completeCheckout();
 
     /**
@@ -49,6 +53,7 @@ 
 
     /**
      * @param string
+     * @return void
      */
     public function setNumber($number);
 
@@ -59,6 +64,7 @@ 
 
     /**
      * @param string $notes
+     * @return void
      */
     public function setNotes($notes);
 
@@ -67,6 +73,9 @@ 
      */
     public function getItems();
 
+    /**
+     * @return void
+     */
     public function clearItems();
 
     /**
@@ -76,11 +85,13 @@ 
 
     /**
      * @param OrderItemInterface $item
+     * @return void
      */
     public function addItem(OrderItemInterface $item);
 
     /**
      * @param OrderItemInterface $item
+     * @return void
      */
     public function removeItem(OrderItemInterface $item);
 
@@ -96,6 +107,9 @@ 
      */
     public function getItemsTotal();
 
+    /**
+     * @return void
+     */
     public function recalculateItemsTotal();
 
     /**
@@ -115,6 +129,7 @@ 
 
     /**
      * @param string $state
+     * @return void
      */
     public function setState($state);
 
@@ -139,6 +154,7 @@ 
 
     /**
      * @param string|null $type
+     * @return void
      */
     public function removeAdjustmentsRecursively($type = null);
 }