pamil / Sylius

src/Sylius/Behat/Service/SharedSecurityServiceInterface.php

Doc Comments +1 added lines

@@ -27,6 +27,7 @@
 
     /**
      * @param string $value
+     * @return void
      */
     public function setValue($value);
 }

Unused Use Statements -1 removed lines

@@ -12,7 +12,6 @@
 namespace Sylius\Behat\Service;
 
 use Sylius\Component\Core\Model\AdminUserInterface;
-use Sylius\Component\Core\Model\ShopUserInterface;
 
 /**
  * @author Arkadiusz Krakowiak <arkadiusz.krakowiak@lakion.com>

src/Sylius/Bundle/AttributeBundle/Doctrine/ORM/Subscriber/LoadMetadataSubscriber.php

Doc Comments +1 added lines, -1 removed lines

@@ -58,7 +58,7 @@
     }
 
     /**
-     * @return array
+     * @return string[]
      */
     public function getRecipients()
     {

Spacing +1 added lines, -1 removed lines

@@ -87,7 +87,7 @@
             'targetEntity' => $subjectClass,
             'inversedBy' => 'attributes',
             'joinColumns' => [[
-                'name' => $subject.'_id',
+                'name' => $subject . '_id',
                 'referencedColumnName' => $targetEntityMetadata->fieldMappings['id']['columnName'],
                 'nullable' => false,
                 'onDelete' => 'CASCADE',

src/Sylius/Bundle/CoreBundle/Command/AbstractInstallCommand.php

Doc Comments +7 added lines, -7 removed lines

@@ -46,7 +46,7 @@ 
     }
 
     /**
-     * @param $id
+     * @param string $id
      *
      * @return object
      */
@@ -72,7 +72,7 @@ 
     }
 
     /**
-     * @param array $headers
+     * @param string[] $headers
      * @param array $rows
      * @param OutputInterface $output
      */
@@ -144,9 +144,9 @@ 
     /**
      * @param OutputInterface $output
      * @param string          $question
-     * @param array           $constraints
+     * @param \Symfony\Component\Validator\Constraints\NotBlank[]           $constraints
      *
-     * @return mixed
+     * @return string
      */
     protected function askHidden(OutputInterface $output, $question, array $constraints = [])
     {
@@ -159,7 +159,7 @@ 
      * @param array $constraints
      * @param mixed $default
      *
-     * @return mixed
+     * @return string
      */
     protected function ask(OutputInterface $output, $question, array $constraints = [], $default = null)
     {
@@ -167,7 +167,7 @@ 
     }
 
     /**
-     * @param mixed $value
+     * @param string $value
      * @param array $constraints
      *
      * @return bool
@@ -195,7 +195,7 @@ 
      * @param string          $default
      * @param bool         $hidden
      *
-     * @return mixed
+     * @return string
      */
     private function proceedAskRequest(OutputInterface $output, $question, array $constraints = [], $default = null, $hidden = false)
     {

src/Sylius/Bundle/CoreBundle/Command/SetupCommand.php

Doc Comments +1 added lines, -1 removed lines

@@ -215,7 +215,7 @@
     /**
      * @param OutputInterface $output
      *
-     * @return mixed
+     * @return string
      */
     private function getAdministratorPassword(OutputInterface $output)
     {

Upper-Lower-Casing +1 added lines, -1 removed lines

@@ -30,7 +30,7 @@
             ->setDescription('Initialize default permissions & roles in the application.')
             ->setHelp(<<<EOT
 The <info>%command.name%</info> command initializes default RBAC setup.
-EOT
+eot
             )
         ;
     }

src/Sylius/Bundle/CoreBundle/Context/CustomerContext.php

Doc Comments +1 added lines, -1 removed lines

@@ -45,7 +45,7 @@
     /**
      * Gets customer based on currently logged user.
      *
-     * @return CustomerInterface|null
+     * @return null|\Sylius\Component\Customer\Model\CustomerInterface
      */
     public function getCustomer()
     {

src/Sylius/Bundle/OrderBundle/NumberAssigner/OrderNumberAssignerInterface.php

Doc Comments +1 added lines

@@ -27,6 +27,7 @@
 
     /**
      * @param string $value
+     * @return void
      */
     public function setValue($value);
 }

src/Sylius/Bundle/ReviewBundle/EventListener/ReviewCreateListener.php

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

@@ -11,9 +11,9 @@
 
 namespace Sylius\Bundle\ReviewBundle\EventListener;
 
+use Sylius\Component\Customer\Context\CustomerContextInterface;
 use Sylius\Component\Resource\Exception\UnexpectedTypeException;
 use Sylius\Component\Review\Model\ReviewInterface;
-use Sylius\Component\Customer\Context\CustomerContextInterface;
 use Symfony\Component\EventDispatcher\GenericEvent;
 
 /**

src/Sylius/Bundle/ThemeBundle/Configuration/ConfigurationSourceFactoryInterface.php

Doc Comments +1 added lines

@@ -27,6 +27,7 @@
 
     /**
      * @param string $value
+     * @return void
      */
     public function setValue($value);
 }

src/Sylius/Bundle/ThemeBundle/Tests/Functional/AssetTest.php

Doc Comments +3 added lines

@@ -101,6 +101,9 @@
         return $webDirectory;
     }
 
+    /**
+     * @param string $webDirectory
+     */
     private function assertFileContent($lines, $webDirectory)
     {
         foreach ($lines as $line) {

Spacing +4 added lines, -4 removed lines

@@ -22,7 +22,7 @@ 
     {
         parent::tearDown();
 
-        file_put_contents(__DIR__.'/../Fixtures/themes/FirstTestTheme/TestBundle/public/theme_asset.txt', 'Theme asset'.PHP_EOL);
+        file_put_contents(__DIR__ . '/../Fixtures/themes/FirstTestTheme/TestBundle/public/theme_asset.txt', 'Theme asset' . PHP_EOL);
     }
 
     /**
@@ -58,7 +58,7 @@ 
         $client->getContainer()->get('sylius.theme.asset.assets_installer')->installAssets($webDirectory, $symlinkMask);
 
         sleep(1);
-        file_put_contents(__DIR__.'/../Fixtures/themes/FirstTestTheme/TestBundle/public/theme_asset.txt', 'Theme asset modified');
+        file_put_contents(__DIR__ . '/../Fixtures/themes/FirstTestTheme/TestBundle/public/theme_asset.txt', 'Theme asset modified');
         clearstatcache();
 
         $client->getContainer()->get('sylius.theme.asset.assets_installer')->installAssets($webDirectory, $symlinkMask);
@@ -91,7 +91,7 @@ 
 
     private function createWebDirectory()
     {
-        $webDirectory = $this->getTmpDirPath(self::TEST_CASE).'/web';
+        $webDirectory = $this->getTmpDirPath(self::TEST_CASE) . '/web';
         if (!is_dir($webDirectory)) {
             mkdir($webDirectory, 0777, true);
         }
@@ -110,7 +110,7 @@ 
 
             list($expectedText, $assetFile) = explode(': ', $line);
 
-            $contents = file_get_contents($webDirectory.$assetFile);
+            $contents = file_get_contents($webDirectory . $assetFile);
 
             $this->assertEquals($expectedText, trim($contents));
         }