pamil / Sylius

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

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

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

@@ -16,8 +16,8 @@
 use Sylius\Bundle\ReviewBundle\Doctrine\ORM\ReviewRepository;
 use Sylius\Bundle\ReviewBundle\Form\Type\ReviewType;
 use Sylius\Component\Resource\Factory\Factory;
-use Sylius\Component\Review\Model\ReviewerInterface;
 use Sylius\Component\Review\Model\ReviewInterface;
+use Sylius\Component\Review\Model\ReviewerInterface;
 use Symfony\Component\Config\Definition\Builder\ArrayNodeDefinition;
 use Symfony\Component\Config\Definition\Builder\TreeBuilder;
 use Symfony\Component\Config\Definition\ConfigurationInterface;

src/Sylius/Bundle/ReviewBundle/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

@@ -12,9 +12,9 @@
 namespace spec\Sylius\Component\Review\Model;
 
 use PhpSpec\ObjectBehavior;
+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 Mateusz Zalewski <mateusz.zalewski@lakion.com>

src/Sylius/Bundle/ThemeBundle/Filesystem/FilesystemInterface.php

Doc Comments +11 added lines, -1 removed lines

@@ -33,6 +33,7 @@ 
      *
      * @throws FileNotFoundException When originFile doesn't exist
      * @throws IOException           When copy fails
+     * @return void
      */
     public function copy($originFile, $targetFile, $override = false);
 
@@ -43,13 +44,14 @@ 
      * @param int                       $mode The directory mode
      *
      * @throws IOException On any directory creation failure
+     * @return void
      */
     public function mkdir($dirs, $mode = 0777);
 
     /**
      * Checks the existence of files or directories.
      *
-     * @param string|array|\Traversable $files A filename, an array of files, or a \Traversable instance to check
+     * @param string $files A filename, an array of files, or a \Traversable instance to check
      *
      * @return bool true if the file exists, false otherwise
      */
@@ -63,6 +65,7 @@ 
      * @param int                       $atime The access time as a Unix timestamp
      *
      * @throws IOException When touch fails
+     * @return void
      */
     public function touch($files, $time = null, $atime = null);
 
@@ -72,6 +75,7 @@ 
      * @param string|array|\Traversable $files A filename, an array of files, or a \Traversable instance to remove
      *
      * @throws IOException When removal fails
+     * @return void
      */
     public function remove($files);
 
@@ -84,6 +88,7 @@ 
      * @param bool                      $recursive Whether change the mod recursively or not
      *
      * @throws IOException When the change fail
+     * @return void
      */
     public function chmod($files, $mode, $umask = 0000, $recursive = false);
 
@@ -95,6 +100,7 @@ 
      * @param bool                      $recursive Whether change the owner recursively or not
      *
      * @throws IOException When the change fail
+     * @return void
      */
     public function chown($files, $user, $recursive = false);
 
@@ -106,6 +112,7 @@ 
      * @param bool                      $recursive Whether change the group recursively or not
      *
      * @throws IOException When the change fail
+     * @return void
      */
     public function chgrp($files, $group, $recursive = false);
 
@@ -118,6 +125,7 @@ 
      *
      * @throws IOException When target file or directory already exists
      * @throws IOException When origin cannot be renamed
+     * @return void
      */
     public function rename($origin, $target, $overwrite = false);
 
@@ -129,6 +137,7 @@ 
      * @param bool   $copyOnWindows Whether to copy files if on Windows
      *
      * @throws IOException When symlink fails
+     * @return void
      */
     public function symlink($originDir, $targetDir, $copyOnWindows = false);
 
@@ -145,6 +154,7 @@ 
      *                                - $options['delete'] Whether to delete files that are not in the source directory (defaults to false)
      *
      * @throws IOException When file type is unknown
+     * @return void
      */
     public function mirror($originDir, $targetDir, \Traversable $iterator = null, $options = []);