ppi / framework

src/Module/AbstractModule.php

Unused Use Statements -4 removed lines

@@ -16,16 +16,12 @@
 use PPI\LaravelRouting\LaravelRouter;
 use PPI\FastRoute\Wrapper\FastRouteWrapper;
 use PPI\Framework\Router\Loader\YamlFileLoader;
-
 use Symfony\Component\Config\FileLocator;
 use Symfony\Component\Finder\Finder;
-
 use Zend\ModuleManager\Feature\ConfigProviderInterface;
 use Zend\Stdlib\ArrayUtils;
-
 use Aura\Router\Router as AuraRouter;
 use Aura\Router\RouterFactory as AuraRouterFactory;
-
 use Illuminate\Events\Dispatcher;
 
 /**

Indentation +7 added lines, -7 removed lines

@@ -1,12 +1,12 @@
 <?php
 /**
- * This file is part of the PPI Framework.
- *
- * @copyright  Copyright (c) 2011-2013 Paul Dragoonis <paul@ppi.io>
- * @license    http://opensource.org/licenses/mit-license.php MIT
- *
- * @link       http://www.ppi.io
- */
+     * This file is part of the PPI Framework.
+     *
+     * @copyright  Copyright (c) 2011-2013 Paul Dragoonis <paul@ppi.io>
+     * @license    http://opensource.org/licenses/mit-license.php MIT
+     *
+     * @link       http://www.ppi.io
+     */
 
 namespace PPI\Framework\View;
 

Spacing +5 added lines, -5 removed lines

@@ -137,7 +137,7 @@ 
         $dataGenerator = new \FastRoute\DataGenerator\GroupCountBased();
         $routeCollector = new \FastRoute\RouteCollector($routeParser, $dataGenerator);
 
-        if(!is_readable($path)) {
+        if (!is_readable($path)) {
             throw new \InvalidArgumentException('Invalid fast route routes path found: ' . $path);
         }
 
@@ -149,7 +149,7 @@ 
         };
 
         $routeCollector = $getRouteCollector();
-        if(!($routeCollector instanceof \FastRoute\RouteCollector)) {
+        if (!($routeCollector instanceof \FastRoute\RouteCollector)) {
             throw new \Exception('Invalid return value from '
                 . pathinfo($path, PATHINFO_FILENAME)
                 . ' expected instance of RouteCollector'
@@ -175,7 +175,7 @@ 
     protected function loadAuraRoutes($path)
     {
 
-        if(!is_readable($path)) {
+        if (!is_readable($path)) {
             throw new \InvalidArgumentException('Invalid aura routes path found: ' . $path);
         }
 
@@ -184,14 +184,14 @@ 
         // The included file must return the aura router
         $router = include $path;
 
-        if(!($router instanceof AuraRouter)) {
+        if (!($router instanceof AuraRouter)) {
             throw new \Exception('Invalid return value from '
                 . pathinfo($path, PATHINFO_FILENAME)
                 . ' expected instance of AuraRouter'
             );
         }
 
-        foreach($router->getRoutes() as $route) {
+        foreach ($router->getRoutes() as $route) {
             $route->addValues(array('_module' => $this->getName()));
         }
 

src/Console/Command/ModuleCreateCommand.php

Indentation +7 added lines, -7 removed lines

@@ -1,12 +1,12 @@
 <?php
 /**
- * This file is part of the PPI Framework.
- *
- * @copyright  Copyright (c) 2011-2013 Paul Dragoonis <paul@ppi.io>
- * @license    http://opensource.org/licenses/mit-license.php MIT
- *
- * @link       http://www.ppi.io
- */
+     * This file is part of the PPI Framework.
+     *
+     * @copyright  Copyright (c) 2011-2013 Paul Dragoonis <paul@ppi.io>
+     * @license    http://opensource.org/licenses/mit-license.php MIT
+     *
+     * @link       http://www.ppi.io
+     */
 
 namespace PPI\Framework\View;
 

Switch Indentation +30 added lines, -30 removed lines

@@ -190,39 +190,39 @@
 
         // Copy files relative to the selected templating engine
         switch ($this->tplEngine) {
-            case self::TPL_ENGINE_PHP:
-            case self::TPL_ENGINE_TWIG:
-            case self::TPL_ENGINE_SMARTY:
-                // Copy templating files over
-                $tplFiles = $this->tplEngineFilesMap[$this->tplEngine];
-                $this->copyFiles($this->skeletonModuleDir, $moduleDir, $tplFiles);
-                // Setting up templating tokens
-                foreach ($tplFiles as $tplFile) {
-                    $tokenizedFiles[] = $tplFile;
-                }
-                $tokens['[MODULE_NAME]'] = $moduleName;
-                $tokens['[TPL_ENGINE_EXT]'] = $this->tplEngine;
-                break;
+        case self::TPL_ENGINE_PHP:
+        case self::TPL_ENGINE_TWIG:
+        case self::TPL_ENGINE_SMARTY:
+            // Copy templating files over
+            $tplFiles = $this->tplEngineFilesMap[$this->tplEngine];
+            $this->copyFiles($this->skeletonModuleDir, $moduleDir, $tplFiles);
+            // Setting up templating tokens
+            foreach ($tplFiles as $tplFile) {
+                $tokenizedFiles[] = $tplFile;
+            }
+            $tokens['[MODULE_NAME]'] = $moduleName;
+            $tokens['[TPL_ENGINE_EXT]'] = $this->tplEngine;
+            break;
         }
         // Routing
         switch ($this->routingEngine) {
-            case self::ROUTING_ENGINE_SYMFONY:
-            case self::ROUTING_ENGINE_AURA:
-            case self::ROUTING_ENGINE_LARAVEL:
-            case self::ROUTING_ENGINE_FASTROUTE:
-                // Copy routing files over
-                $routingFiles = $this->routingEngineFilesMap[$this->routingEngine];
-                $this->copyFiles($this->skeletonModuleDir, $moduleDir, $routingFiles);
-
-                // Setting up routing tokens
-                foreach ($routingFiles as $routingFile) {
-                    $tokenizedFiles[] = $routingFile;
-                }
-                $routingTokensMap = $this->routingEngineTokenMap[$this->routingEngine];
-                foreach ($routingTokensMap as $routingTokenKey => $routingTokenVal) {
-                    $tokens[$routingTokenKey] = $routingTokenVal;
-                }
-                break;
+        case self::ROUTING_ENGINE_SYMFONY:
+        case self::ROUTING_ENGINE_AURA:
+        case self::ROUTING_ENGINE_LARAVEL:
+        case self::ROUTING_ENGINE_FASTROUTE:
+            // Copy routing files over
+            $routingFiles = $this->routingEngineFilesMap[$this->routingEngine];
+            $this->copyFiles($this->skeletonModuleDir, $moduleDir, $routingFiles);
+
+            // Setting up routing tokens
+            foreach ($routingFiles as $routingFile) {
+                $tokenizedFiles[] = $routingFile;
+            }
+            $routingTokensMap = $this->routingEngineTokenMap[$this->routingEngine];
+            foreach ($routingTokensMap as $routingTokenKey => $routingTokenVal) {
+                $tokens[$routingTokenKey] = $routingTokenVal;
+            }
+            break;
         }
 
         // Replace tokens in all files

Spacing +3 added lines, -3 removed lines

@@ -351,16 +351,16 @@
     private function checkRouters(InputInterface $input, OutputInterface $output)
     {
         // Aura Check
-        if($this->routingEngine == self::ROUTING_ENGINE_AURA && !class_exists('\Aura\Router\Router')) {
+        if ($this->routingEngine == self::ROUTING_ENGINE_AURA && !class_exists('\Aura\Router\Router')) {
             $output->writeln("<comment>Aura Router doesn't appear to be loaded. Run: <info>composer require ppi/aura-router</info></comment>");
         }
 
         // Laravel check
-        if($this->routingEngine == self::ROUTING_ENGINE_LARAVEL && !class_exists('\PPI\LaravelRouting\LaravelRouter')) {
+        if ($this->routingEngine == self::ROUTING_ENGINE_LARAVEL && !class_exists('\PPI\LaravelRouting\LaravelRouter')) {
             $output->writeln("<comment>Laravel Router doesn't appear to be loaded. Run: <info>composer require ppi/laravel-router</info></comment>");
         }
 
-        if($this->routingEngine == self::ROUTING_ENGINE_FASTROUTE && !class_exists('\PPI\FastRoute\FastRouteWrapper')) {
+        if ($this->routingEngine == self::ROUTING_ENGINE_FASTROUTE && !class_exists('\PPI\FastRoute\FastRouteWrapper')) {
             $output->writeln("<comment>Laravel Router doesn't appear to be loaded. Run: <info>composer require ppi/fast-route</info></comment>");
         }
 

src/Module/Controller/ControllerNameParser.php

Indentation +7 added lines, -7 removed lines

@@ -1,12 +1,12 @@
 <?php
 /**
- * This file is part of the PPI Framework.
- *
- * @copyright   Copyright (c) 2011-2013 Paul Dragoonis <paul@ppi.io>
- * @license     http://opensource.org/licenses/mit-license.php MIT
- *
- * @link        http://www.ppi.io
- */
+     * This file is part of the PPI Framework.
+     *
+     * @copyright   Copyright (c) 2011-2013 Paul Dragoonis <paul@ppi.io>
+     * @license     http://opensource.org/licenses/mit-license.php MIT
+     *
+     * @link        http://www.ppi.io
+     */
 
 namespace PPI\Framework\ServiceManager;
 

src/ServiceManager/Factory/RouterFactory.php

Indentation +7 added lines, -7 removed lines

@@ -1,12 +1,12 @@
 <?php
 /**
- * This file is part of the PPI Framework.
- *
- * @copyright   Copyright (c) 2011-2015 Paul Dragoonis <paul@ppi.io>
- * @license     http://opensource.org/licenses/mit-license.php MIT
- *
- * @link        http://www.ppi.io
- */
+     * This file is part of the PPI Framework.
+     *
+     * @copyright   Copyright (c) 2011-2015 Paul Dragoonis <paul@ppi.io>
+     * @license     http://opensource.org/licenses/mit-license.php MIT
+     *
+     * @link        http://www.ppi.io
+     */
 
 namespace PPI\Framework\ServiceManager;
 

Switch Indentation +28 added lines, -28 removed lines

@@ -60,37 +60,37 @@
 
             switch(true) {
                 // @todo - move this to a separate method()
-                case $moduleRoutingResponse instanceof SymfonyRouteCollection:
-                    $sfRouter = new SymfonyRouter($requestContext, $moduleRoutingResponse, $routerOptions, $logger);
-                    $sfRouterWrapper = new SymfonyRouterWrapper($sfRouter);
-                    $chainRouter->add($sfRouterWrapper);
-                    break;
-
-                // @todo - move this to a separate method()
-                case $moduleRoutingResponse instanceof AuraRouter:
-                    $auraRouterWrapper = new AuraRouterWrapper($moduleRoutingResponse);
-                    $chainRouter->add($auraRouterWrapper);
-                    break;
-
-                // @todo - move this to a separate method()
-                case $moduleRoutingResponse instanceof LaravelRouter:
-                    $laravelRequest = new LaravelRequest();
-                    $laravelUrlGenerator = new LaravelUrlGenerator($moduleRoutingResponse->getRoutes(), $laravelRequest);
-                    $laravelRouterWrapper = new LaravelRouterWrapper(
-                        $moduleRoutingResponse, $laravelRequest, $laravelUrlGenerator
-                    );
-                    // @todo - solve this problem
+            case $moduleRoutingResponse instanceof SymfonyRouteCollection:
+                $sfRouter = new SymfonyRouter($requestContext, $moduleRoutingResponse, $routerOptions, $logger);
+                $sfRouterWrapper = new SymfonyRouterWrapper($sfRouter);
+                $chainRouter->add($sfRouterWrapper);
+                break;
+
+            // @todo - move this to a separate method()
+            case $moduleRoutingResponse instanceof AuraRouter:
+                $auraRouterWrapper = new AuraRouterWrapper($moduleRoutingResponse);
+                $chainRouter->add($auraRouterWrapper);
+                break;
+
+            // @todo - move this to a separate method()
+            case $moduleRoutingResponse instanceof LaravelRouter:
+                $laravelRequest = new LaravelRequest();
+                $laravelUrlGenerator = new LaravelUrlGenerator($moduleRoutingResponse->getRoutes(), $laravelRequest);
+                $laravelRouterWrapper = new LaravelRouterWrapper(
+                    $moduleRoutingResponse, $laravelRequest, $laravelUrlGenerator
+                );
+                // @todo - solve this problem
 //                    $laravelRouterWrapper->setModuleName($this->getName());
-                    $chainRouter->add($laravelRouterWrapper);
-                    break;
+                $chainRouter->add($laravelRouterWrapper);
+                break;
 
-                case $moduleRoutingResponse instanceof FastRouteWrapper:
-                    $chainRouter->add($moduleRoutingResponse);
-                    break;
+            case $moduleRoutingResponse instanceof FastRouteWrapper:
+                $chainRouter->add($moduleRoutingResponse);
+                break;
 
-                default:
-                    throw new \Exception('Unexpected routes value return from module: ' . $moduleName .
-                        '. found value of type: ' . gettype($moduleRoutingResponse));
+            default:
+                throw new \Exception('Unexpected routes value return from module: ' . $moduleName .
+                    '. found value of type: ' . gettype($moduleRoutingResponse));
             }
         }