locomotivemtl / charcoal-app

src/Charcoal/App/Route/TemplateRouteConfig.php

Spacing +1 added lines, -1 removed lines

@@ -177,7 +177,7 @@
     public function setRedirectMode($redirectMode)
     {
         $redirectMode = (int)$redirectMode;
-        if ($redirectMode < 300 || $redirectMode  >= 400) {
+        if ($redirectMode < 300 || $redirectMode >= 400) {
             throw new InvalidArgumentException(
                 'Invalid HTTP status for redirect mode'
             );

Unused Use Statements -1 removed lines

@@ -8,7 +8,6 @@
 use Charcoal\App\App;
 
 // Dependencies from 'charcoal-translator'
-use Charcoal\Translator\Translation;
 
 // Local namespace dependencies
 use Charcoal\App\Route\RouteConfig;

src/Charcoal/App/AppConfig.php

Doc Comments +2 added lines, -2 removed lines

@@ -467,7 +467,7 @@ 
 
     /**
      * @param array $databases The avaiable databases config.
-     * @return Config Chainable
+     * @return AppConfig Chainable
      */
     public function setDatabases(array $databases)
     {
@@ -550,7 +550,7 @@ 
 
     /**
      * @throws Exception If trying to access this method before a setter.
-     * @return mixed
+     * @return string
      */
     public function defaultDatabase()
     {

Spacing +1 added lines, -1 removed lines

@@ -436,7 +436,7 @@
             $this->routes = [];
         }
 
-        $toIterate = [ 'templates', 'actions', 'scripts' ];
+        $toIterate = ['templates', 'actions', 'scripts'];
         foreach ($routes as $key => $val) {
             if (in_array($key, $toIterate) && isset($this->routes[$key])) {
                 $this->routes[$key] = array_merge($this->routes[$key], $val);

src/Charcoal/App/Handler/AbstractHandler.php

Doc Comments +1 added lines, -1 removed lines

@@ -141,7 +141,7 @@
      *
      * @see    ConfigurableTrait::createConfig()
      * @param  mixed|null $data Optional config data.
-     * @return ConfigInterface
+     * @return HandlerConfig
      */
     public function createConfig($data = null)
     {

Unused Use Statements -2 removed lines

@@ -7,7 +7,6 @@ 
 use Psr\Log\LoggerAwareTrait;
 
 // Dependencies from PSR-7 (HTTP Messaging)
-use Psr\Http\Message\ResponseInterface;
 use Psr\Http\Message\ServerRequestInterface;
 use Psr\Http\Message\UriInterface;
 
@@ -28,7 +27,6 @@ 
 
 // Intra-module (`charcoal-app`) dependencies
 use Charcoal\App\AppConfig;
-use Charcoal\App\Template\TemplateInterface;
 use Charcoal\App\Handler\HandlerInterface;
 use Charcoal\App\Handler\HandlerConfig;
 

src/Charcoal/App/Handler/NotAllowed.php

Doc Comments +1 added lines, -1 removed lines

@@ -81,7 +81,7 @@
     /**
      * Set the HTTP methods allowed by the current request.
      *
-     * @param  array $methods Case-sensitive array of methods.
+     * @param  string[] $methods Case-sensitive array of methods.
      * @return NotAllowed Chainable
      */
     protected function setMethods(array $methods)

src/Charcoal/App/Route/RouteManager.php

Spacing +9 added lines, -9 removed lines

@@ -30,17 +30,17 @@ 
         $routes = $this->config();
 
         if (PHP_SAPI == 'cli') {
-            $scripts = ( isset($routes['scripts']) ? $routes['scripts'] : [] );
+            $scripts = (isset($routes['scripts']) ? $routes['scripts'] : []);
             foreach ($scripts as $scriptIdent => $scriptConfig) {
                 $this->setupScript($scriptIdent, $scriptConfig);
             }
         } else {
-            $templates = ( isset($routes['templates']) ? $routes['templates'] : [] );
+            $templates = (isset($routes['templates']) ? $routes['templates'] : []);
             foreach ($templates as $routeIdent => $templateConfig) {
                 $this->setupTemplate($routeIdent, $templateConfig);
             }
 
-            $actions = ( isset($routes['actions']) ? $routes['actions'] : [] );
+            $actions = (isset($routes['actions']) ? $routes['actions'] : []);
             foreach ($actions as $actionIdent => $actionConfig) {
                 $this->setupAction($actionIdent, $actionConfig);
             }
@@ -66,12 +66,12 @@ 
 
         $methods = isset($templateConfig['methods'])
             ? $templateConfig['methods']
-            : [ 'GET' ];
+            : ['GET'];
 
         $routeHandler = $this->app()->map(
             $methods,
             $routePattern,
-            function (
+            function(
                 RequestInterface $request,
                 ResponseInterface $response,
                 array $args = []
@@ -144,12 +144,12 @@ 
 
         $methods = isset($actionConfig['methods'])
             ? $actionConfig['methods']
-            : [ 'POST' ];
+            : ['POST'];
 
         $routeHandler = $this->app()->map(
             $methods,
             $routePattern,
-            function (
+            function(
                 RequestInterface $request,
                 ResponseInterface $response,
                 array $args = []
@@ -222,12 +222,12 @@ 
 
         $methods = isset($scriptConfig['methods'])
             ? $scriptConfig['methods']
-            : [ 'GET' ];
+            : ['GET'];
 
         $routeHandler = $this->app()->map(
             $methods,
             $routePattern,
-            function (
+            function(
                 RequestInterface $request,
                 ResponseInterface $response,
                 array $args = []

src/Charcoal/App/Module/AbstractModule.php

Spacing +1 added lines, -1 removed lines

@@ -118,7 +118,7 @@
     {
         if (!isset($this->routeManager)) {
             $config = $this->config();
-            $routes = (isset($config['routes']) ? $config['routes'] : [] );
+            $routes = (isset($config['routes']) ? $config['routes'] : []);
 
             $this->routeManager = new RouteManager([
                 'config' => $routes,

src/Charcoal/App/Script/CronScriptTrait.php

Indentation +3 added lines, -3 removed lines

@@ -51,9 +51,9 @@
         $lockFile = sys_get_temp_dir().'/'.$lockName;
         $this->lockFilePointer = fopen($lockFile, 'w');
         if (!$this->lockFilePointer) {
-             throw new Exception(
-                 sprintf('Can not run action. Lock file "%s" not available.', $lockFile)
-             );
+                throw new Exception(
+                    sprintf('Can not run action. Lock file "%s" not available.', $lockFile)
+                );
         }
         if (flock($this->lockFilePointer, LOCK_EX)) {
             return true;

src/Charcoal/App/CallableResolverAwareTrait.php

Spacing +4 added lines, -4 removed lines

@@ -87,22 +87,22 @@
                 $method = $matches[2];
 
                 if (is_object($context)) {
-                    $callable = [ $context, $method ];
+                    $callable = [$context, $method];
                 }
 
                 if (!is_callable($callable)) {
                     switch ($class) {
                         case '':
                         case 'self':
-                            $callable = [ $this, $method ];
+                            $callable = [$this, $method];
                             break;
 
                         case 'static':
-                            $callable = [ static::class, $method ];
+                            $callable = [static::class, $method];
                             break;
 
                         case 'parent':
-                            $callable = [ $this, 'parent::'.$method ];
+                            $callable = [$this, 'parent::'.$method];
                             break;
                     }
                 }

src/Charcoal/App/Script/ArgScriptTrait.php

Spacing +1 added lines, -1 removed lines

@@ -54,7 +54,7 @@
         foreach ($params as $key => $param) {
             $setter = $this->setter($key);
 
-            if (!is_callable([ $this, $setter ])) {
+            if (!is_callable([$this, $setter])) {
                 continue;
             }