ppi / framework

src/App.php

Doc Comments +3 added lines, -3 removed lines

@@ -403,7 +403,7 @@ 
     /**
      * @note Added for compatibility with Symfony's HttpKernel\Kernel.
      *
-     * @return null|ServiceManager\ServiceManager
+     * @return ServiceManager\ServiceManager
      */
     public function getContainer()
     {
@@ -601,7 +601,7 @@ 
     /**
      * Creates and initializes a ServiceManager instance.
      *
-     * @return ServiceManager The compiled service manager
+     * @return ServiceManagerBuilder The compiled service manager
      */
     protected function buildServiceManager()
     {
@@ -651,7 +651,7 @@ 
     /**
      * Logs with an arbitrary level.
      *
-     * @param mixed  $level
+     * @param string  $level
      * @param string $message
      * @param array  $context
      */

src/MicroApp.php

Unused Use Statements -1 removed lines

@@ -14,7 +14,6 @@
 use Symfony\Component\Routing\Matcher\UrlMatcher as UrlMatcher;
 use Symfony\Component\Routing\Route as Route;
 use Symfony\Component\Routing\RouteCollection;
-use Symfony\Component\Routing\Router;
 
 /**
  * The PPI MicroApp bootstrap class.

src/Router/Loader/LaravelRoutesLoader.php

Unused Use Statements -1 removed lines

@@ -10,7 +10,6 @@
 
 namespace PPI\Framework\Router\Loader;
 
-use Illuminate\Routing\Router;
 use Illuminate\Routing\Router as LaravelRouter;
 
 /**

src/Module/ModuleManager.php

Spacing +1 added lines, -1 removed lines

@@ -122,7 +122,7 @@
     protected function getResourcesPath($module)
     {
         if (is_callable(array($module, 'getResourcesPath'))) {
-            $resourcesPath  = $module->getResourcesPath();
+            $resourcesPath = $module->getResourcesPath();
         } else {
             $resourcesPath = $module->getPath();
         }

src/Module/AbstractModule.php

Spacing +1 added lines, -1 removed lines

@@ -144,7 +144,7 @@
         }
 
         // The included file must return the laravel router
-        $getRouteCollector = function () use ($routeCollector, $path) {
+        $getRouteCollector = function() use ($routeCollector, $path) {
             $r = $routeCollector;
             include $path;
 

src/ServiceManager/Factory/RouterFactory.php

Switch Indentation +26 added lines, -26 removed lines

@@ -58,37 +58,37 @@
         foreach ($allModuleRoutes as $moduleName => $moduleRoutingResponse) {
             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;
+            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 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
+            // @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));
             }
         }
 

src/Http/Request.php

Spacing +6 added lines, -6 removed lines

@@ -173,7 +173,7 @@ 
             $value = array($value);
         }
 
-        if (! is_array($value) || ! $this->arrayContainsOnlyStrings($value)) {
+        if (!is_array($value) || !$this->arrayContainsOnlyStrings($value)) {
             throw new InvalidArgumentException(
                 'Invalid header value; must be a string or array of strings'
             );
@@ -212,13 +212,13 @@ 
             $value = array($value);
         }
 
-        if (! is_array($value) || ! $this->arrayContainsOnlyStrings($value)) {
+        if (!is_array($value) || !$this->arrayContainsOnlyStrings($value)) {
             throw new InvalidArgumentException(
                 'Invalid header value; must be a string or array of strings'
             );
         }
 
-        if (! $this->hasHeader($header)) {
+        if (!$this->hasHeader($header)) {
             return $this->withHeader($header, $value);
         }
 
@@ -246,7 +246,7 @@ 
      */
     public function withoutHeader($header)
     {
-        if (! $this->hasHeader($header)) {
+        if (!$this->hasHeader($header)) {
             return clone $this;
         }
 
@@ -599,14 +599,14 @@ 
         if (null === $method) {
             return true;
         }
-        if (! is_string($method)) {
+        if (!is_string($method)) {
             throw new InvalidArgumentException(sprintf(
                 'Unsupported HTTP method; must be a string, received %s',
                 (is_object($method) ? get_class($method) : gettype($method))
             ));
         }
         $method = strtoupper($method);
-        if (! in_array($method, $this->validMethods, true)) {
+        if (!in_array($method, $this->validMethods, true)) {
             throw new InvalidArgumentException(sprintf(
                 'Unsupported HTTP method "%s" provided',
                 $method

src/Http/Response.php

Spacing +9 added lines, -9 removed lines

@@ -163,7 +163,7 @@ 
             $value = array($value);
         }
 
-        if (! is_array($value) || ! $this->arrayContainsOnlyStrings($value)) {
+        if (!is_array($value) || !$this->arrayContainsOnlyStrings($value)) {
             throw new InvalidArgumentException(
                 'Invalid header value; must be a string or array of strings'
             );
@@ -202,13 +202,13 @@ 
             $value = array($value);
         }
 
-        if (! is_array($value) || ! $this->arrayContainsOnlyStrings($value)) {
+        if (!is_array($value) || !$this->arrayContainsOnlyStrings($value)) {
             throw new InvalidArgumentException(
                 'Invalid header value; must be a string or array of strings'
             );
         }
 
-        if (! $this->hasHeader($header)) {
+        if (!$this->hasHeader($header)) {
             return $this->withHeader($header, $value);
         }
 
@@ -236,7 +236,7 @@ 
      */
     public function withoutHeader($header)
     {
-        if (! $this->hasHeader($header)) {
+        if (!$this->hasHeader($header)) {
             return clone $this;
         }
 
@@ -257,7 +257,7 @@ 
      */
     public function setBody($body = 'php://memory')
     {
-        if (! is_string($body) && ! is_resource($body) && ! $body instanceof StreamInterface) {
+        if (!is_string($body) && !is_resource($body) && !$body instanceof StreamInterface) {
             throw new InvalidArgumentException(
                 'Stream must be a string stream resource identifier, '
                 . 'an actual stream resource, '
@@ -499,15 +499,15 @@ 
     {
         $headerNames = $headers = array();
         foreach ($originalHeaders as $header => $value) {
-            if (! is_string($header)) {
+            if (!is_string($header)) {
                 continue;
             }
 
-            if (! is_array($value) && ! is_string($value)) {
+            if (!is_array($value) && !is_string($value)) {
                 continue;
             }
 
-            if (! is_array($value)) {
+            if (!is_array($value)) {
                 $value = array($value);
             }
 
@@ -542,7 +542,7 @@ 
      */
     private function validateStatus($code)
     {
-        if (! is_numeric($code)
+        if (!is_numeric($code)
             || is_float($code)
             || $code < 100
             || $code >= 600