pamil / Sylius

src/Sylius/Bundle/UserBundle/Command/DemoteUserCommand.php

Spacing +2 added lines, -2 removed lines

@@ -51,13 +51,13 @@
 
         foreach ($securityRoles as $securityRole) {
             if (!$user->hasRole($securityRole)) {
-                $output->writeln(sprintf('<error>User "%s" didn\'t have "%s" Security role.</error>', (string)$user, $securityRole));
+                $output->writeln(sprintf('<error>User "%s" didn\'t have "%s" Security role.</error>', (string) $user, $securityRole));
                 $error = true;
                 continue;
             }
 
             $user->removeRole($securityRole);
-            $output->writeln(sprintf('Security role <comment>%s</comment> has been removed from user <comment>%s</comment>', $securityRole, (string)$user));
+            $output->writeln(sprintf('Security role <comment>%s</comment> has been removed from user <comment>%s</comment>', $securityRole, (string) $user));
         }
 
         if (!$error) {

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

Spacing +1 added lines, -1 removed lines

@@ -163,7 +163,7 @@
      */
     public function orderBy($field, $direction)
     {
-        $this->orderBys = [ $field => $direction ];
+        $this->orderBys = [$field => $direction];
     }
 
     /**

src/Sylius/Bundle/GridBundle/spec/Doctrine/PHPCRODM/DataSourceSpec.php

Spacing +1 added lines, -1 removed lines

@@ -76,7 +76,7 @@
     {
         $this->shouldThrow(
             new \RuntimeException('Unknown restrict condition "foo"')
-        )->during('restrict', [ $comparison, 'foo' ]);
+        )->during('restrict', [$comparison, 'foo']);
     }
 
     function it_should_return_the_expression_builder(

src/Sylius/Bundle/UserBundle/Command/PromoteUserCommand.php

Spacing +2 added lines, -2 removed lines

@@ -51,13 +51,13 @@
 
         foreach ($securityRoles as $securityRole) {
             if ($user->hasRole($securityRole)) {
-                $output->writeln(sprintf('<error>User "%s" did already have "%s" security role.</error>', (string)$user, $securityRole));
+                $output->writeln(sprintf('<error>User "%s" did already have "%s" security role.</error>', (string) $user, $securityRole));
                 $error = true;
                 continue;
             }
 
             $user->addRole($securityRole);
-            $output->writeln(sprintf('Security role <comment>%s</comment> has been added to user <comment>%s</comment>', $securityRole, (string)$user));
+            $output->writeln(sprintf('Security role <comment>%s</comment> has been added to user <comment>%s</comment>', $securityRole, (string) $user));
         }
 
         if (!$error) {

src/Sylius/Bundle/ResourceBundle/spec/Doctrine/ODM/PHPCR/EventListener/NameFilterListenerSpec.php

Spacing +1 added lines, -1 removed lines

@@ -49,7 +49,7 @@
         $documentManager->getClassMetadata('stdClass')->willReturn($metadata);
         $metadata->nodename = null;
 
-        $this->shouldThrow(new \RuntimeException('In order to use the node name filter on "stdClass" it is necessary to map a field as the "nodename"'))->during('onEvent', [ $event ]);
+        $this->shouldThrow(new \RuntimeException('In order to use the node name filter on "stdClass" it is necessary to map a field as the "nodename"'))->during('onEvent', [$event]);
     }
 
     function it_should_clean_the_name(

src/Sylius/Bundle/ResourceBundle/spec/Doctrine/ODM/PHPCR/EventListener/NameResolverListenerSpec.php

Spacing +1 added lines, -1 removed lines

@@ -50,7 +50,7 @@
         $metadata->idGenerator = 'foo';
 
         $this->shouldThrow(new \RuntimeException('Document of class "stdClass" must be using the GENERATOR_TYPE_PARENT identificatio strategy (value 3), it is current using "foo" (this may be an automatic configuration: be sure to map both the `nodename` and the `parentDocument`).'))->during(
-            'onEvent', [ $event ]
+            'onEvent', [$event]
         );
     }
 

src/Sylius/Bundle/ResourceBundle/spec/Doctrine/ODM/PHPCR/EventListener/DefaultParentListenerSpec.php

Spacing +2 added lines, -2 removed lines

@@ -58,7 +58,7 @@ 
             'A default parent path has been specified, but no parent mapping has been applied to document "stdClass"'
         ))->during(
             'onPreCreate',
-            [ $event ]
+            [$event]
         );
     }
 
@@ -84,7 +84,7 @@ 
             'Document at default parent path "/path/to" does not exist. `autocreate` was set to "false"'
         ))->during(
             'onPreCreate',
-            [ $event ]
+            [$event]
         );
     }
 

src/Sylius/Behat/Page/Admin/Channel/UpdatePageInterface.php

Doc Comments +12 added lines

@@ -19,17 +19,25 @@ 
  */
 interface UpdatePageInterface extends BaseUpdatePageInterface
 {
+    /**
+     * @return void
+     */
     public function enable();
 
+    /**
+     * @return void
+     */
     public function disable();
 
     /**
      * @param string $themeName
+     * @return void
      */
     public function setTheme($themeName);
 
     /**
      * @throws ElementNotFoundException
+     * @return void
      */
     public function unsetTheme();
 
@@ -40,6 +48,7 @@ 
 
     /**
      * @param string $language
+     * @return void
      */
     public function chooseLocale($language);
 
@@ -52,6 +61,7 @@ 
 
     /**
      * @param string $currencyCode
+     * @return void
      */
     public function chooseCurrency($currencyCode);
 
@@ -64,6 +74,7 @@ 
 
     /**
      * @param string $shippingMethod
+     * @return void
      */
     public function chooseShippingMethod($shippingMethod);
 
@@ -76,6 +87,7 @@ 
 
     /**
      * @param string $paymentMethod
+     * @return void
      */
     public function choosePaymentMethod($paymentMethod);
 

src/Sylius/Behat/Page/Admin/Country/UpdatePageInterface.php

Doc Comments +15 added lines

@@ -18,7 +18,14 @@ 
  */
 interface UpdatePageInterface extends BaseUpdatePageInterface
 {
+    /**
+     * @return void
+     */
     public function enable();
+
+    /**
+     * @return void
+     */
     public function disable();
 
     /**
@@ -44,28 +51,36 @@ 
      * @param string $name
      * @param string $code
      * @param string|null $abbreviation
+     * @return void
      */
     public function addProvince($name, $code, $abbreviation = null);
 
     /**
      * @param string $provinceName
+     * @return void
      */
     public function removeProvince($provinceName);
 
+    /**
+     * @return void
+     */
     public function clickAddProvinceButton();
 
     /**
      * @param string $provinceName
+     * @return void
      */
     public function nameProvince($provinceName);
 
     /**
      * @param string $provinceName
+     * @return void
      */
     public function removeProvinceName($provinceName);
 
     /**
      * @param string $provinceCode
+     * @return void
      */
     public function specifyProvinceCode($provinceCode);
 }