TE-1211 Fixed cs.

spryker · Sep 26, 2019 · 5b7c8cd · 5b7c8cd
1 parent 7cba70f
commit 5b7c8cd
Show file tree

Hide file tree

Showing 23 changed files with 65 additions and 28 deletions.
diff --git a/src/Spryker/Shared/Router/Resolver/ControllerResolver.php b/src/Spryker/Shared/Router/Resolver/ControllerResolver.php
@@ -28,10 +28,9 @@ public function __construct(ContainerInterface $container)
     }
 
     /**
-     * {@inheritdoc}
+     * @param \Symfony\Component\HttpFoundation\Request $request
      *
-     * This method looks for a '_controller' request attribute that represents
-     * the controller.
+     * @return array|bool|callable|object|false
      */
     public function getController(Request $request)
     {
@@ -135,7 +134,7 @@ protected function injectContainerAndInitialize($controller)
     }
 
     /**
-     * {@inheritdoc}
+     * @inheritdoc
      *
      * @deprecated This method is deprecated as of 3.1 and will be removed in 4.0. Implement the ArgumentResolverInterface and inject it in the HttpKernel instead.
      */

diff --git a/src/Spryker/Yves/Router/Plugin/Console/Descriptor/Descriptor.php b/src/Spryker/Yves/Router/Plugin/Console/Descriptor/Descriptor.php
@@ -22,9 +22,13 @@ abstract class Descriptor implements DescriptorInterface
     protected $output;
 
     /**
-     * {@inheritdoc}
+     * @param \Symfony\Component\Console\Output\OutputInterface $output
+     * @param object $object
+     * @param array $options
      *
      * @throws \InvalidArgumentException
+     *
+     * @return void
      */
     public function describe(OutputInterface $output, $object, array $options = [])
     {
@@ -44,9 +48,7 @@ public function describe(OutputInterface $output, $object, array $options = [])
     }
 
     /**
-     * Returns the output.
-     *
-     * @return \Symfony\Component\Console\Output\OutputInterface The output
+     * @return \Symfony\Component\Console\Output\OutputInterface
      */
     protected function getOutput()
     {

diff --git a/src/Spryker/Yves/Router/Plugin/Console/Descriptor/TextDescriptor.php b/src/Spryker/Yves/Router/Plugin/Console/Descriptor/TextDescriptor.php
@@ -17,7 +17,10 @@
 class TextDescriptor extends Descriptor
 {
     /**
-     * {@inheritdoc}
+     * @param \Symfony\Component\Routing\RouteCollection $routes
+     * @param array $options
+     *
+     * @return void
      */
     protected function describeRouteCollection(RouteCollection $routes, array $options = [])
     {
@@ -56,7 +59,10 @@ protected function describeRouteCollection(RouteCollection $routes, array $optio
     }
 
     /**
-     * {@inheritdoc}
+     * @param \Spryker\Yves\Router\Route\Route $route
+     * @param array $options
+     *
+     * @return void
      */
     protected function describeRoute(Route $route, array $options = [])
     {

diff --git a/src/Spryker/Yves/Router/Plugin/Console/RouterDebugYvesConsole.php b/src/Spryker/Yves/Router/Plugin/Console/RouterDebugYvesConsole.php
@@ -54,9 +54,12 @@ protected function getFactory(): RouterFactory
     }
 
     /**
-     * {@inheritdoc}
+     * @param \Symfony\Component\Console\Input\InputInterface $input
+     * @param \Symfony\Component\Console\Output\OutputInterface $output
      *
-     * @throws \Symfony\Component\Console\Exception\InvalidArgumentException When route does not exist
+     * @throws \Symfony\Component\Console\Exception\InvalidArgumentException
+     *
+     * @return int|null|void
      */
     protected function execute(InputInterface $input, OutputInterface $output)
     {

diff --git a/src/Spryker/Yves/Router/RouterConfig.php b/src/Spryker/Yves/Router/RouterConfig.php
@@ -18,10 +18,10 @@ class RouterConfig extends AbstractBundleConfig
      * Specification:
      * - Returns a Router configuration which makes use of a Router cache.
      *
-     * @see \Symfony\Component\Routing\Router::setOptions()
-     *
      * @api
      *
+     * @see \Symfony\Component\Routing\Router::setOptions()
+     *
      * @return array
      */
     public function getRouterConfiguration(): array
@@ -39,10 +39,10 @@ public function getRouterConfiguration(): array
      * - Returns a Router configuration which does not make use of a Router cache.
      * - Fallback for development which is executed when the cached Router is not able to match.
      *
-     * @see \Symfony\Component\Routing\Router::setOptions()
-     *
      * @api
      *
+     * @see \Symfony\Component\Routing\Router::setOptions()
+     *
      * @return array
      */
     public function getDevelopmentRouterConfiguration(): array
@@ -102,10 +102,10 @@ public function getSslExcludedRouteNames(): array
      *
      * @example Incoming URL `/en/home` will be manipulated to `/home` because the router only knows URL's without any optional pre/suffix.
      *
-     * @see \Spryker\Yves\Router\Plugin\RouterEnhancer\LanguagePrefixRouterEnhancerPlugin
-     *
      * @api
      *
+     * @see \Spryker\Yves\Router\Plugin\RouterEnhancer\LanguagePrefixRouterEnhancerPlugin
+     *
      * @return array
      */
     public function getAllowedLanguages(): array

diff --git a/src/Spryker/Yves/Router/RouterFactory.php b/src/Spryker/Yves/Router/RouterFactory.php
@@ -34,7 +34,8 @@
 class RouterFactory extends AbstractFactory
 {
     /**
-     * @return \Spryker\Yves\Router\Router\ChainRouter */
+     * @return \Spryker\Yves\Router\Router\ChainRouter
+     */
     public function createRouter()
     {
         return new ChainRouter($this->getRouterPlugins());

diff --git a/src/Spryker/Zed/Router/Communication/Plugin/Console/Descriptor/Descriptor.php b/src/Spryker/Zed/Router/Communication/Plugin/Console/Descriptor/Descriptor.php
@@ -21,9 +21,13 @@ abstract class Descriptor implements DescriptorInterface
     protected $output;
 
     /**
-     * {@inheritdoc}
+     * @param \Symfony\Component\Console\Output\OutputInterface $output
+     * @param object $object
+     * @param array $options
      *
      * @throws \InvalidArgumentException
+     *
+     * @return void
      */
     public function describe(OutputInterface $output, $object, array $options = [])
     {

diff --git a/src/Spryker/Zed/Router/Communication/Plugin/Console/Descriptor/TextDescriptor.php b/src/Spryker/Zed/Router/Communication/Plugin/Console/Descriptor/TextDescriptor.php
@@ -17,7 +17,10 @@
 class TextDescriptor extends Descriptor
 {
     /**
-     * {@inheritdoc}
+     * @param \Symfony\Component\Routing\RouteCollection $routes
+     * @param array $options
+     *
+     * @return void
      */
     protected function describeRouteCollection(RouteCollection $routes, array $options = [])
     {
@@ -58,7 +61,10 @@ protected function describeRouteCollection(RouteCollection $routes, array $optio
     }
 
     /**
-     * {@inheritdoc}
+     * @param \Symfony\Component\Routing\Route $route
+     * @param array $options
+     *
+     * @return void
      */
     protected function describeRoute(Route $route, array $options = [])
     {

diff --git a/src/Spryker/Zed/Router/Communication/Plugin/Console/RouterDebugZedConsole.php b/src/Spryker/Zed/Router/Communication/Plugin/Console/RouterDebugZedConsole.php
@@ -49,9 +49,12 @@ protected function configure(): void
     }
 
     /**
-     * {@inheritdoc}
+     * @param \Symfony\Component\Console\Input\InputInterface $input
+     * @param \Symfony\Component\Console\Output\OutputInterface $output
      *
-     * @throws \Symfony\Component\Console\Exception\InvalidArgumentException When route does not exist
+     * @throws \Symfony\Component\Console\Exception\InvalidArgumentException
+     *
+     * @return int|null|void
      */
     protected function execute(InputInterface $input, OutputInterface $output)
     {

diff --git a/src/Spryker/Zed/Router/RouterConfig.php b/src/Spryker/Zed/Router/RouterConfig.php
@@ -19,10 +19,10 @@ class RouterConfig extends AbstractBundleConfig
      * Specification:
      * - Returns a Router configuration which makes use of a Router cache.
      *
-     * @see \Symfony\Component\Routing\Router::setOptions()
-     *
      * @api
      *
+     * @see \Symfony\Component\Routing\Router::setOptions()
+     *
      * @return array
      */
     public function getRouterConfiguration(): array
@@ -40,10 +40,10 @@ public function getRouterConfiguration(): array
      * - Returns a Router configuration which does not make use of a Router cache.
      * - Fallback for development which is executed when the cached Router is not able to match.
      *
-     * @see \Symfony\Component\Routing\Router::setOptions()
-     *
      * @api
      *
+     * @see \Symfony\Component\Routing\Router::setOptions()
+     *
      * @return array
      */
     public function getDevelopmentRouterConfiguration(): array

diff --git a/tests/SprykerTest/Shared/Router/Resolver/ControllerResolverTest.php b/tests/SprykerTest/Shared/Router/Resolver/ControllerResolverTest.php
@@ -13,6 +13,7 @@
 
 /**
  * Auto-generated group annotations
+ *
  * @group SprykerTest
  * @group Shared
  * @group Router

diff --git a/tests/SprykerTest/Shared/Router/_support/RouterTester.php b/tests/SprykerTest/Shared/Router/_support/RouterTester.php
@@ -16,6 +16,7 @@
 
 /**
  * Inherited Methods
+ *
  * @method void wantToTest($text)
  * @method void wantTo($text)
  * @method void execute($callable)

diff --git a/tests/SprykerTest/Yves/Router/Plugin/YvesRouterPluginRouteConverterTest.php b/tests/SprykerTest/Yves/Router/Plugin/YvesRouterPluginRouteConverterTest.php
@@ -14,6 +14,7 @@
 
 /**
  * Auto-generated group annotations
+ *
  * @group SprykerTest
  * @group Yves
  * @group Router

diff --git a/tests/SprykerTest/Yves/Router/Plugin/YvesRouterPluginTest.php b/tests/SprykerTest/Yves/Router/Plugin/YvesRouterPluginTest.php
@@ -14,6 +14,7 @@
 
 /**
  * Auto-generated group annotations
+ *
  * @group SprykerTest
  * @group Yves
  * @group Router

diff --git a/...rTest/Yves/Router/Plugin/YvesRouterPluginWithLanguageAndStorePrefixRouterEnhancerTest.php b/...rTest/Yves/Router/Plugin/YvesRouterPluginWithLanguageAndStorePrefixRouterEnhancerTest.php
@@ -19,6 +19,7 @@
 
 /**
  * Auto-generated group annotations
+ *
  * @group SprykerTest
  * @group Yves
  * @group Router

diff --git a/...s/SprykerTest/Yves/Router/Plugin/YvesRouterPluginWithLanguagePrefixRouterEnhancerTest.php b/...s/SprykerTest/Yves/Router/Plugin/YvesRouterPluginWithLanguagePrefixRouterEnhancerTest.php
@@ -17,6 +17,7 @@
 
 /**
  * Auto-generated group annotations
+ *
  * @group SprykerTest
  * @group Yves
  * @group Router

diff --git a/tests/SprykerTest/Yves/Router/Plugin/YvesRouterPluginWithQueryStringRouterEnhancerTest.php b/tests/SprykerTest/Yves/Router/Plugin/YvesRouterPluginWithQueryStringRouterEnhancerTest.php
@@ -16,6 +16,7 @@
 
 /**
  * Auto-generated group annotations
+ *
  * @group SprykerTest
  * @group Yves
  * @group Router

diff --git a/...rTest/Yves/Router/Plugin/YvesRouterPluginWithStoreAndLanguagePrefixRouterEnhancerTest.php b/...rTest/Yves/Router/Plugin/YvesRouterPluginWithStoreAndLanguagePrefixRouterEnhancerTest.php
@@ -19,6 +19,7 @@
 
 /**
  * Auto-generated group annotations
+ *
  * @group SprykerTest
  * @group Yves
  * @group Router

diff --git a/tests/SprykerTest/Yves/Router/Plugin/YvesRouterPluginWithStorePrefixRouterEnhancerTest.php b/tests/SprykerTest/Yves/Router/Plugin/YvesRouterPluginWithStorePrefixRouterEnhancerTest.php
@@ -17,6 +17,7 @@
 
 /**
  * Auto-generated group annotations
+ *
  * @group SprykerTest
  * @group Yves
  * @group Router

diff --git a/tests/SprykerTest/Yves/Router/_support/RouterYvesTester.php b/tests/SprykerTest/Yves/Router/_support/RouterYvesTester.php
@@ -11,6 +11,7 @@
 
 /**
  * Inherited Methods
+ *
  * @method void wantToTest($text)
  * @method void wantTo($text)
  * @method void execute($callable)

diff --git a/...unication/Plugin/Router/RouterEnhancer/BackwardsCompatibleUrlRouterEnhancerPluginTest.php b/...unication/Plugin/Router/RouterEnhancer/BackwardsCompatibleUrlRouterEnhancerPluginTest.php
@@ -13,6 +13,7 @@
 
 /**
  * Auto-generated group annotations
+ *
  * @group SprykerTest
  * @group Zed
  * @group Router

diff --git a/tests/SprykerTest/Zed/Router/Communication/Resolver/ControllerResolverTest.php b/tests/SprykerTest/Zed/Router/Communication/Resolver/ControllerResolverTest.php
@@ -11,6 +11,7 @@
 
 /**
  * Auto-generated group annotations
+ *
  * @group Spryker
  * @group Router
  * @group tests

diff --git a/tests/SprykerTest/Zed/Router/_support/RouterCommunicationTester.php b/tests/SprykerTest/Zed/Router/_support/RouterCommunicationTester.php
@@ -15,6 +15,7 @@
 
 /**
  * Inherited Methods
+ *
  * @method void wantToTest($text)
  * @method void wantTo($text)
  * @method void execute($callable)