pamil / Sylius

src/Sylius/Behat/Page/Shop/Account/ResetPasswordPageInterface.php

Doc Comments +4 added lines

@@ -26,10 +26,14 @@
      */
     public function checkValidationMessageFor($element, $message);
 
+    /**
+     * @return void
+     */
     public function reset();
 
     /**
      * @param string $email
+     * @return void
      */
     public function specifyEmail($email);
 }

src/Sylius/Behat/Service/Accessor/TableAccessor.php

Doc Comments +2 added lines, -2 removed lines

@@ -91,8 +91,8 @@
     }
 
     /**
-     * @param array $columns
-     * @param array $fields
+     * @param NodeElement[] $columns
+     * @param string[] $fields
      *
      * @return bool
      */

src/Sylius/Behat/Service/Mocker/PaypalApiMocker.php

Doc Comments +1 added lines, -1 removed lines

@@ -102,7 +102,7 @@
 
     /**
      * @param int $statusCode
-     * @param mixed $streamMock
+     * @param Mock $streamMock
      *
      * @return Mock
      */

src/Sylius/Behat/Service/ResponseLoader.php

Doc Comments +2 added lines, -2 removed lines

@@ -85,7 +85,7 @@ 
 
     /**
      * @param string $source
-     * @param mixed $content
+     * @param string $content
      *
      * @throws \RuntimeException
      */
@@ -98,7 +98,7 @@ 
 
     /**
      * @param string $source
-     * @param mixed $content
+     * @param string $content
      *
      * @throws \RuntimeException
      */

src/Sylius/Behat/Service/SecurityServiceInterface.php

Doc Comments +5 added lines

@@ -22,14 +22,19 @@
      * @param string $email
      *
      * @throws \InvalidArgumentException
+     * @return void
      */
     public function logIn($email);
     
+    /**
+     * @return void
+     */
     public function logOut();
 
     /**
      * @param UserInterface $user
      * @param callable $action
+     * @return void
      */
     public function performActionAs(UserInterface $user, callable $action);
 }

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()
     {

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;

src/Sylius/Bundle/CoreBundle/Test/Services/PaymentMethodNameToGatewayConverter.php

Doc Comments +1 added lines, -1 removed lines

@@ -67,7 +67,7 @@
      *
      * @throws \RuntimeException
      *
-     * @return string
+     * @return integer
      */
     private function tryToMapPaymentMethodName($paymentMethodName)
     {

src/Sylius/Bundle/GridBundle/Doctrine/ORM/ExpressionBuilder.php

Doc Comments +1 added lines

@@ -187,6 +187,7 @@
 
     /**
      * {@inheritdoc}
+     * @param string $field
      */
     private function getFieldName($field)
     {

Spacing +7 added lines, -7 removed lines

@@ -65,7 +65,7 @@ 
         $parameterName = $this->getParameterName($field);
         $this->queryBuilder->setParameter($parameterName, $value);
 
-        return $this->queryBuilder->expr()->eq($this->getFieldName($field), ':'.$parameterName);
+        return $this->queryBuilder->expr()->eq($this->getFieldName($field), ':' . $parameterName);
     }
 
     /**
@@ -76,7 +76,7 @@ 
         $parameterName = $this->getParameterName($field);
         $this->queryBuilder->setParameter($parameterName, $value);
 
-        return $this->queryBuilder->expr()->neq($this->getFieldName($field), ':'.$parameterName);
+        return $this->queryBuilder->expr()->neq($this->getFieldName($field), ':' . $parameterName);
     }
 
     /**
@@ -87,7 +87,7 @@ 
         $parameterName = $this->getParameterName($field);
         $this->queryBuilder->setParameter($parameterName, $value);
 
-        $this->queryBuilder->andWhere($this->getFieldName($field).' < :'.$parameterName);
+        $this->queryBuilder->andWhere($this->getFieldName($field) . ' < :' . $parameterName);
     }
 
     /**
@@ -98,7 +98,7 @@ 
         $parameterName = $this->getParameterName($field);
         $this->queryBuilder->setParameter($parameterName, $value);
 
-        $this->queryBuilder->andWhere($this->getFieldName($field).' <= :'.$parameterName);
+        $this->queryBuilder->andWhere($this->getFieldName($field) . ' <= :' . $parameterName);
     }
 
     /**
@@ -109,7 +109,7 @@ 
         $parameterName = $this->getParameterName($field);
         $this->queryBuilder->setParameter($parameterName, $value);
 
-        $this->queryBuilder->andWhere($this->getFieldName($field).' > :'.$parameterName);
+        $this->queryBuilder->andWhere($this->getFieldName($field) . ' > :' . $parameterName);
     }
 
     /**
@@ -120,7 +120,7 @@ 
         $parameterName = $this->getParameterName($field);
         $this->queryBuilder->setParameter($parameterName, $value);
 
-        $this->queryBuilder->andWhere($this->getFieldName($field).' >= :'.$parameterName);
+        $this->queryBuilder->andWhere($this->getFieldName($field) . ' >= :' . $parameterName);
     }
 
     /**
@@ -193,7 +193,7 @@ 
     private function getFieldName($field)
     {
         if (false === strpos($field, '.')) {
-            return $this->queryBuilder->getRootAlias().'.'.$field;
+            return $this->queryBuilder->getRootAlias() . '.' . $field;
         }
 
         return $field;

src/Sylius/Bundle/ResourceBundle/Controller/RedirectHandler.php

Doc Comments +1 added lines

@@ -80,6 +80,7 @@
 
     /**
      * {@inheritdoc}
+     * @param string|null $url
      */
     public function redirect(RequestConfiguration $configuration, $url, $status = 302)
     {

Spacing +2 added lines, -2 removed lines

@@ -85,11 +85,11 @@
     {
         if ($configuration->isHeaderRedirection()) {
             return new Response('', 200, [
-                'X-SYLIUS-LOCATION' => $url.$configuration->getRedirectHash(),
+                'X-SYLIUS-LOCATION' => $url . $configuration->getRedirectHash(),
             ]);
         }
 
-        return new RedirectResponse($url.$configuration->getRedirectHash(), $status);
+        return new RedirectResponse($url . $configuration->getRedirectHash(), $status);
     }
 
     /**