pamil / Sylius

src/Sylius/Bundle/AddressingBundle/DependencyInjection/Configuration.php

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

@@ -22,8 +22,8 @@
 use Sylius\Bundle\AddressingBundle\Form\Type\ZoneType;
 use Sylius\Bundle\ResourceBundle\Controller\ResourceController;
 use Sylius\Bundle\ResourceBundle\Form\Type\ResourceChoiceType;
-use Sylius\Bundle\ResourceBundle\SyliusResourceBundle;
 use Sylius\Bundle\ResourceBundle\Form\Type\ResourceFromIdentifierType;
+use Sylius\Bundle\ResourceBundle\SyliusResourceBundle;
 use Sylius\Component\Addressing\Model\Address;
 use Sylius\Component\Addressing\Model\AddressInterface;
 use Sylius\Component\Addressing\Model\Country;

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/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);
     }
 
     /**

src/Sylius/Bundle/ResourceBundle/EventListener/ORMMappedSuperClassSubscriber.php

Doc Comments +1 added lines, -1 removed lines

@@ -70,7 +70,7 @@
 
     /**
      * @param ClassMetadataInfo $metadata
-     * @param $configuration
+     * @param \Doctrine\ORM\Configuration $configuration
      */
     private function setAssociationMappings(ClassMetadataInfo $metadata, $configuration)
     {

src/Sylius/Bundle/ResourceBundle/EventListener/ORMTranslatableListener.php

Doc Comments +1 added lines, -1 removed lines

@@ -153,7 +153,7 @@
      * Check if a unique constraint has been defined.
      *
      * @param ClassMetadata $metadata
-     * @param array         $columns
+     * @param string[]         $columns
      *
      * @return bool
      */

Spacing +2 added lines, -2 removed lines

@@ -134,7 +134,7 @@ 
         }
 
         /** @var MetadataInterface $translationResourceMetadata */
-        $translationResourceMetadata = $this->resourceMetadataRegistry->get($resourceMetadata->getAlias().'_translation');
+        $translationResourceMetadata = $this->resourceMetadataRegistry->get($resourceMetadata->getAlias() . '_translation');
 
         $metadata->mapOneToMany([
             'fieldName' => 'translations',
@@ -194,7 +194,7 @@ 
         if (!$this->hasUniqueConstraint($metadata, $columns)) {
             $constraints = isset($metadata->table['uniqueConstraints']) ? $metadata->table['uniqueConstraints'] : [];
 
-            $constraints[$metadata->getTableName().'_uniq_trans'] = [
+            $constraints[$metadata->getTableName() . '_uniq_trans'] = [
                 'columns' => $columns,
             ];
 

src/Sylius/Bundle/ResourceBundle/Validator/EnabledValidator.php

Doc Comments +1 added lines, -1 removed lines

@@ -40,7 +40,7 @@
     }
 
     /**
-     * @param mixed $value
+     * @param ToggleableInterface $value
      */
     private function ensureValueImplementsToggleableInterface($value)
     {