pamil / Sylius

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,
             ];
 

Unused Use Statements -1 removed lines

@@ -21,7 +21,6 @@
 use Sylius\Component\Resource\Metadata\RegistryInterface;
 use Sylius\Component\Resource\Model\TranslatableInterface;
 use Sylius\Component\Resource\Model\TranslationInterface;
-use Sylius\Component\Resource\Translation\Provider\TranslationLocaleProviderInterface;
 use Sylius\Component\Resource\Translation\TranslatableEntityLocaleAssignerInterface;
 use Symfony\Component\DependencyInjection\ContainerInterface;
 

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/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 = []);
 

src/Sylius/Bundle/ThemeBundle/Loader/CircularDependencyFoundException.php

Doc Comments +1 added lines, -1 removed lines

@@ -37,7 +37,7 @@
     }
 
     /**
-     * @param array $themes
+     * @param ThemeInterface[] $themes
      *
      * @return array
      */

src/Sylius/Bundle/ThemeBundle/Translation/Finder/TranslationFilesFinder.php

Doc Comments +1 added lines, -1 removed lines

@@ -53,7 +53,7 @@
     /**
      * @param string $path
      *
-     * @return array
+     * @return \Symfony\Component\Finder\Finder
      */
     private function getFiles($path)
     {

src/Sylius/Component/Core/Resolver/DefaultShippingMethodResolver.php

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

@@ -12,9 +12,9 @@
 namespace Sylius\Component\Core\Resolver;
 
 use Sylius\Component\Core\Model\ChannelInterface;
+use Sylius\Component\Core\Model\ShipmentInterface as CoreShipmentInterface;
 use Sylius\Component\Shipping\Exception\UnresolvedDefaultShippingMethodException;
 use Sylius\Component\Shipping\Model\ShipmentInterface;
-use Sylius\Component\Core\Model\ShipmentInterface as CoreShipmentInterface;
 use Sylius\Component\Shipping\Repository\ShippingMethodRepositoryInterface;
 use Sylius\Component\Shipping\Resolver\DefaultShippingMethodResolverInterface;
 use Webmozart\Assert\Assert;