pamil / Sylius

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/Shop/Cart/SummaryPage.php

Indentation +2 added lines, -2 removed lines

@@ -132,7 +132,7 @@ 
      */
     public function hasItemNamed($name)
     {
-       return $this->hasItemWith($name, '.sylius-product-name');
+        return $this->hasItemWith($name, '.sylius-product-name');
     }
 
     /**
@@ -140,7 +140,7 @@ 
      */
     public function hasItemWithVariantNamed($variantName)
     {
-       return $this->hasItemWith($variantName, '.sylius-product-variant-name');
+        return $this->hasItemWith($variantName, '.sylius-product-variant-name');
     }
 
     /**

Spacing +1 added lines, -1 removed lines

@@ -227,7 +227,7 @@
     {
         $cartTotalText = $this->getElement('cart_total')->getText();
 
-        if (strpos($cartTotalText, ',') !== false ) {
+        if (strpos($cartTotalText, ',') !== false) {
             return strstr($cartTotalText, ',', true);
         }
 

src/Sylius/Behat/Extension/MultiContainerExtension/Context/Environment/Handler/ContextServiceEnvironmentHandler.php

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

@@ -18,8 +18,8 @@
 use Behat\Testwork\Environment\Handler\EnvironmentHandler;
 use Behat\Testwork\Suite\Exception\SuiteConfigurationException;
 use Behat\Testwork\Suite\Suite;
-use Sylius\Behat\Extension\MultiContainerExtension\Context\Environment\UninitializedContextServiceEnvironment;
 use Sylius\Behat\Extension\MultiContainerExtension\ContextRegistry;
+use Sylius\Behat\Extension\MultiContainerExtension\Context\Environment\UninitializedContextServiceEnvironment;
 use Symfony\Component\DependencyInjection\ContainerInterface;
 
 /**

src/Sylius/Behat/Extension/MultiContainerExtension/Context/Environment/UninitializedContextServiceEnvironment.php

Doc Comments +1 added lines, -1 removed lines

@@ -77,7 +77,7 @@
     }
 
     /**
-     * @return array[]
+     * @return integer[]
      */
     public function getContextsServicesIds()
     {

src/Sylius/Behat/Extension/MultiContainerExtension/Loader/XmlFileLoader.php

Doc Comments +2 added lines, -2 removed lines

@@ -415,9 +415,9 @@
      * Get child elements by name.
      *
      * @param \DOMNode $node
-     * @param mixed    $name
+     * @param string    $name
      *
-     * @return array
+     * @return string
      */
     private function getChildren(\DOMNode $node, $name)
     {

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

@@ -15,15 +15,15 @@
 use Symfony\Component\Config\FileLocatorInterface;
 use Symfony\Component\Config\Resource\FileResource;
 use Symfony\Component\Config\Util\XmlUtils;
+use Symfony\Component\DependencyInjection\Alias;
 use Symfony\Component\DependencyInjection\ContainerBuilder;
-use Symfony\Component\DependencyInjection\DefinitionDecorator;
 use Symfony\Component\DependencyInjection\ContainerInterface;
-use Symfony\Component\DependencyInjection\Alias;
 use Symfony\Component\DependencyInjection\Definition;
-use Symfony\Component\DependencyInjection\Loader\FileLoader;
-use Symfony\Component\DependencyInjection\Reference;
+use Symfony\Component\DependencyInjection\DefinitionDecorator;
 use Symfony\Component\DependencyInjection\Exception\InvalidArgumentException;
 use Symfony\Component\DependencyInjection\Exception\RuntimeException;
+use Symfony\Component\DependencyInjection\Loader\FileLoader;
+use Symfony\Component\DependencyInjection\Reference;
 use Symfony\Component\ExpressionLanguage\Expression;
 
 /**

Spacing +7 added lines, -7 removed lines

@@ -177,7 +177,7 @@ 
                 if (in_array($key, ['factory-class', 'factory-method', 'factory-service'])) {
                     @trigger_error(sprintf('The "%s" attribute of service "%s" in file "%s" is deprecated since version 2.6 and will be removed in 3.0. Use the "factory" element instead.', $key, (string) $service->getAttribute('id'), $file), E_USER_DEPRECATED);
                 }
-                $method = 'set'.str_replace('-', '', $key);
+                $method = 'set' . str_replace('-', '', $key);
                 $definition->$method(XmlUtils::phpize($value));
             }
         }
@@ -384,7 +384,7 @@ 
             // this is used by DefinitionDecorator to overwrite a specific
             // argument of the parent definition
             if ($arg->hasAttribute('index')) {
-                $key = 'index_'.$arg->getAttribute('index');
+                $key = 'index_' . $arg->getAttribute('index');
             }
 
             switch ($arg->getAttribute('type')) {
@@ -442,7 +442,7 @@ 
      */
     public function validateSchema(\DOMDocument $dom)
     {
-        $schemaLocations = ['http://symfony.com/schema/dic/services' => str_replace('\\', '/', __DIR__.'/schema/dic/services/services-1.0.xsd')];
+        $schemaLocations = ['http://symfony.com/schema/dic/services' => str_replace('\\', '/', __DIR__ . '/schema/dic/services/services-1.0.xsd')];
 
         if ($element = $dom->documentElement->getAttributeNS('http://www.w3.org/2001/XMLSchema-instance', 'schemaLocation')) {
             $items = preg_split('/\s+/', $element);
@@ -452,7 +452,7 @@ 
                 }
 
                 if (($extension = $this->container->getExtension($items[$i])) && false !== $extension->getXsdValidationBasePath()) {
-                    $path = str_replace($extension->getNamespace(), str_replace('\\', '/', $extension->getXsdValidationBasePath()).'/', $items[$i + 1]);
+                    $path = str_replace($extension->getNamespace(), str_replace('\\', '/', $extension->getXsdValidationBasePath()) . '/', $items[$i + 1]);
 
                     if (!is_file($path)) {
                         throw new RuntimeException(sprintf('Extension "%s" references a non-existent XSD file "%s"', get_class($extension), $path));
@@ -475,10 +475,10 @@ 
                     $parts = explode('/', str_replace('\\', '/', $tmpfile));
                 }
             }
-            $drive = '\\' === DIRECTORY_SEPARATOR ? array_shift($parts).'/' : '';
-            $location = 'file:///'.$drive.implode('/', array_map('rawurlencode', $parts));
+            $drive = '\\' === DIRECTORY_SEPARATOR ? array_shift($parts) . '/' : '';
+            $location = 'file:///' . $drive . implode('/', array_map('rawurlencode', $parts));
 
-            $imports .= sprintf('  <xsd:import namespace="%s" schemaLocation="%s" />'."\n", $namespace, $location);
+            $imports .= sprintf('  <xsd:import namespace="%s" schemaLocation="%s" />' . "\n", $namespace, $location);
         }
 
         $source = <<<EOF

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

@@ -491,7 +491,7 @@
     <xsd:import namespace="http://www.w3.org/XML/1998/namespace"/>
 $imports
 </xsd:schema>
-EOF
+eof
         ;
 
         $valid = @$dom->schemaValidateSource($source);

src/Sylius/Behat/Extension/MultiContainerExtension/ServiceContainer/MultiContainerExtension.php

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

@@ -16,8 +16,8 @@
 use Behat\Testwork\ServiceContainer\Extension;
 use Behat\Testwork\ServiceContainer\ExtensionManager;
 use Sylius\Behat\Extension\MultiContainerExtension\ContainerConfiguration;
-use Sylius\Behat\Extension\MultiContainerExtension\Context\Environment\Handler\ContextServiceEnvironmentHandler;
 use Sylius\Behat\Extension\MultiContainerExtension\ContextRegistry;
+use Sylius\Behat\Extension\MultiContainerExtension\Context\Environment\Handler\ContextServiceEnvironmentHandler;
 use Sylius\Behat\Extension\MultiContainerExtension\Loader\XmlFileLoader;
 use Sylius\Behat\Extension\MultiContainerExtension\ScopeManipulator;
 use Symfony\Bridge\ProxyManager\LazyProxy\Instantiator\RuntimeInstantiator;

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

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

Doc Comments +8 added lines

@@ -18,11 +18,19 @@
  */
 interface UpdatePageInterface extends BaseUpdatePageInterface
 {
+    /**
+     * @return void
+     */
     public function enable();
+
+    /**
+     * @return void
+     */
     public function disable();
 
     /**
      * @param string $exchangeRate
+     * @return void
      */
     public function changeExchangeRate($exchangeRate);