pamil / Sylius

src/Sylius/Component/Shipping/Model/ShippingMethodInterface.php

Unused Use Statements -1 removed lines

@@ -11,7 +11,6 @@
 
 namespace spec\Sylius\Component\Core\Promotion\Filter;
 
-use Doctrine\Common\Collections\Collection;
 use PhpSpec\ObjectBehavior;
 use Sylius\Component\Core\Model\OrderItemInterface;
 use Sylius\Component\Core\Model\ProductInterface;

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/Component/Shipping/spec/Checker/ShippingMethodEligibilityCheckerSpec.php

Unused Use Statements -1 removed lines

@@ -12,7 +12,6 @@
 namespace spec\Sylius\Component\Shipping\Checker;
 
 use PhpSpec\ObjectBehavior;
-use Sylius\Component\Registry\ServiceRegistryInterface;
 use Sylius\Component\Shipping\Checker\ShippingMethodEligibilityChecker;
 use Sylius\Component\Shipping\Checker\ShippingMethodEligibilityCheckerInterface;
 use Sylius\Component\Shipping\Model\ShippableInterface;

src/Sylius/Component/User/Model/UserOAuthInterface.php

Doc Comments +4 added lines

@@ -26,6 +26,7 @@ 
 
     /**
      * @param string $provider
+     * @return void
      */
     public function setProvider($provider);
 
@@ -36,6 +37,7 @@ 
 
     /**
      * @param string $identifier
+     * @return void
      */
     public function setIdentifier($identifier);
 
@@ -46,6 +48,7 @@ 
 
     /**
      * @param string $accessToken
+     * @return void
      */
     public function setAccessToken($accessToken);
 
@@ -56,6 +59,7 @@ 
 
    /**
     * @param string $refreshToken
+    * @return void
     */
    public function setRefreshToken($refreshToken);
 }

Indentation +5 added lines, -5 removed lines

@@ -52,10 +52,10 @@
     /**
      * @return string
      */
-   public function getRefreshToken();
+    public function getRefreshToken();
 
-   /**
-    * @param string $refreshToken
-    */
-   public function setRefreshToken($refreshToken);
+    /**
+     * @param string $refreshToken
+     */
+    public function setRefreshToken($refreshToken);
 }

src/Sylius/Component/Addressing/Provider/ProvinceNamingProvider.php

Spacing +2 added lines, -2 removed lines

@@ -48,7 +48,7 @@ 
         }
 
         $province = $this->provinceRepository->findOneBy(['code' => $address->getProvinceCode()]);
-        Assert::notNull($province,sprintf('Province with code "%s" not found.', $address->getProvinceCode()));
+        Assert::notNull($province, sprintf('Province with code "%s" not found.', $address->getProvinceCode()));
 
         return $province->getName();
     }
@@ -67,7 +67,7 @@ 
         }
 
         $province = $this->provinceRepository->findOneBy(['code' => $address->getProvinceCode()]);
-        Assert::notNull($province,sprintf('Province with code "%s" not found.', $address->getProvinceCode()));
+        Assert::notNull($province, sprintf('Province with code "%s" not found.', $address->getProvinceCode()));
 
         return $province->getAbbreviation() ?: $province->getName();
     }

src/Sylius/Component/Core/Test/Services/EmailChecker.php

Spacing +1 added lines, -1 removed lines

@@ -137,7 +137,7 @@
         $messages = [];
 
         /** @var SplFileInfo $file */
-        foreach($finder as $file) {
+        foreach ($finder as $file) {
             $messages[] = unserialize($file->getContents());
         }
 

src/Sylius/Component/Core/Model/ProductVariant.php

Spacing +2 added lines, -2 removed lines

@@ -92,10 +92,10 @@
             $string .= '(';
 
             foreach ($this->getOptionValues() as $option) {
-                $string .= $option->getOption()->getName().': '.$option->getValue().', ';
+                $string .= $option->getOption()->getName() . ': ' . $option->getValue() . ', ';
             }
 
-            $string = substr($string, 0, -2).')';
+            $string = substr($string, 0, -2) . ')';
         }
 
         return $string;

src/Sylius/Component/Order/spec/Updater/UnpaidOrdersStateUpdaterSpec.php

Indentation +2 added lines, -2 removed lines

@@ -52,8 +52,8 @@
         StateMachineInterface $secondOrderStateMachine
     ) {
         $orderRepository->findOrdersUnpaidSince(Argument::type(\DateTime::class))->willReturn([
-           $firstOrder,
-           $secondOrder
+            $firstOrder,
+            $secondOrder
         ]);
 
         $stateMachineFactory->get($firstOrder, 'sylius_order')->willReturn($firstOrderStateMachine);

src/Sylius/Component/Order/Updater/UnpaidOrdersStateUpdater.php

Spacing +1 added lines, -1 removed lines

@@ -53,7 +53,7 @@
 
     public function cancel()
     {
-        $expiredUnpaidOrders = $this->orderRepository->findOrdersUnpaidSince(new \DateTime('-'.$this->expirationPeriod));
+        $expiredUnpaidOrders = $this->orderRepository->findOrdersUnpaidSince(new \DateTime('-' . $this->expirationPeriod));
         foreach ($expiredUnpaidOrders as $expiredUnpaidOrder) {
             $this->cancelOrder($expiredUnpaidOrder);
         }

src/Sylius/Component/Order/Remover/ExpiredCartsRemover.php

Spacing +1 added lines, -1 removed lines

@@ -40,7 +40,7 @@
 
     public function remove()
     {
-        $expiredCarts = $this->orderRepository->findCartsNotModifiedSince(new \DateTime('-'.$this->expirationPeriod));
+        $expiredCarts = $this->orderRepository->findCartsNotModifiedSince(new \DateTime('-' . $this->expirationPeriod));
         foreach ($expiredCarts as $expiredCart) {
             $this->orderRepository->remove($expiredCart);
         }