locomotivemtl / charcoal-app

src/Charcoal/App/ServiceProvider/LoggerServiceProvider.php

Spacing +7 added lines, -7 removed lines

@@ -54,7 +54,7 @@ 
          * @param  Container $container A container instance.
          * @return LoggerConfig
          */
-        $container['logger/config'] = function (Container $container) {
+        $container['logger/config'] = function(Container $container) {
             $appConfig    = isset($container['config']) ? $container['config'] : [];
             $loggerConfig = isset($appConfig['logger']) ? $appConfig['logger'] : null;
             return new LoggerConfig($loggerConfig);
@@ -63,7 +63,7 @@ 
         /**
          * @return FactoryInterface
          */
-        $container['logger/processor/factory'] = function () {
+        $container['logger/processor/factory'] = function() {
             return new Factory([
                 'map' => [
                     'memory-usage' => MemoryUsageProcessor::class,
@@ -76,7 +76,7 @@ 
          * @param  Container $container A container instance.
          * @return StreamHandler|null
          */
-        $container['logger/handler/stream'] = function (Container $container) {
+        $container['logger/handler/stream'] = function(Container $container) {
             $loggerConfig  = $container['logger/config'];
             $handlerConfig = $loggerConfig['handlers.stream'];
 
@@ -92,7 +92,7 @@ 
          * @param  Container $container A container instance.
          * @return BrowserConsoleHandler|null
          */
-        $container['logger/handler/browser-console'] = function (Container $container) {
+        $container['logger/handler/browser-console'] = function(Container $container) {
             $loggerConfig  = $container['logger/config'];
             $handlerConfig = $loggerConfig['handlers.console'];
 
@@ -108,14 +108,14 @@ 
          * @param  Container $container A container instance.
          * @return Container Collection of defined record handlers, in a service container.
          */
-        $container['logger/handlers'] = function (Container $container) {
+        $container['logger/handlers'] = function(Container $container) {
             $loggerConfig   = $container['logger/config'];
             $handlersConfig = $loggerConfig['handlers'];
             $handlerFactory = $container['logger/handler/factory'];
 
             $handlers = new Container();
             foreach ($handlersConfig as $handler) {
-                $handlers[$handler['type']] = function () use ($handler, $handlerFactory) {
+                $handlers[$handler['type']] = function() use ($handler, $handlerFactory) {
                     $type    = $handler['type'];
                     $handler = $handlerFactory->create($type);
                     return $handler;
@@ -131,7 +131,7 @@ 
          * @param  Container $container A container instance.
          * @return LoggerInterface
          */
-        $container['logger'] = function (Container $container) {
+        $container['logger'] = function(Container $container) {
             $loggerConfig = $container['logger/config'];
 
             if ($loggerConfig['active'] !== true) {

src/Charcoal/App/ServiceProvider/FilesystemServiceProvider.php

Spacing +3 added lines, -3 removed lines

@@ -53,7 +53,7 @@ 
          * @param  Container $container Pimple DI Container.
          * @return FlysystemConfig
          */
-        $container['filesystem/config'] = function (Container $container) {
+        $container['filesystem/config'] = function(Container $container) {
             $appConfig = isset($container['config']) ? $container['config'] : [];
             $fsConfig  = isset($appConfig['filesystem']) ? $appConfig['filesystem'] : null;
             return new FilesystemConfig($fsConfig);
@@ -63,11 +63,11 @@ 
          * @param Container $container Pimple DI Container.
          * @return MountManager
          */
-        $container['filesystem/manager'] = function () {
+        $container['filesystem/manager'] = function() {
             return new MountManager();
         };
 
-        $container['filesystems'] = function (Container $container) {
+        $container['filesystems'] = function(Container $container) {
             $filesystemConfig = $container['filesystem/config'];
             $filesystems = new Container();
 

src/Charcoal/App/Config/LoggerConfig.php

Doc Comments +2 added lines, -2 removed lines

@@ -135,7 +135,7 @@ 
     /**
      * Add a record handler to use.
      *
-     * @param  array       $handler The record handler structure.
+     * @param  string       $handler The record handler structure.
      * @param  string|null $key     The handler's key.
      * @throws InvalidArgumentException If the handler is invalid.
      * @return self
@@ -197,7 +197,7 @@ 
     /**
      * Add a record processor to use.
      *
-     * @param  array       $processor The record processor structure.
+     * @param  string       $processor The record processor structure.
      * @param  string|null $key       The processor's key.
      * @throws InvalidArgumentException If the processor is invalid.
      * @return self

src/Charcoal/App/ServiceProvider/DatabaseServiceProvider.php

Spacing +6 added lines, -6 removed lines

@@ -43,12 +43,12 @@ 
          * @param Container $container A container instance.
          * @return Container The Collection of DatabaseSourceConfig, in a Container.
          */
-        $container['databases/config'] = function (Container $container) {
+        $container['databases/config'] = function(Container $container) {
             $config = $container['config'];
             $databases = $config['databases'];
             $configs = new Container();
             foreach ($databases as $dbIdent => $dbOptions) {
-                $configs[$dbIdent] = function () use ($dbOptions) {
+                $configs[$dbIdent] = function() use ($dbOptions) {
                     return new DatabaseConfig($dbOptions);
                 };
             }
@@ -59,7 +59,7 @@ 
          * @param Container $container A container instance.
          * @return Container
          */
-        $container['databases'] = function (Container $container) {
+        $container['databases'] = function(Container $container) {
             $config = $container['config'];
             $databases = $config['databases'];
             $dbs = new Container();
@@ -71,7 +71,7 @@ 
                  * @param Container $container A container instance.
                  * @return PDO
                  */
-                $dbs[$dbIdent] = function () use ($dbIdent, $origContainer) {
+                $dbs[$dbIdent] = function() use ($dbIdent, $origContainer) {
                     $dbConfigs = $origContainer['databases/config'];
                     $dbConfig = $dbConfigs[$dbIdent];
 
@@ -112,7 +112,7 @@ 
          * @param Container $container A container instance.
          * @return DatabaseSourceConfig
          */
-        $container['database/config'] = function (Container $container) {
+        $container['database/config'] = function(Container $container) {
             $config = $container['config'];
             $databaseIdent = $config['default_database'];
             return $container['databases/config'][$databaseIdent];
@@ -125,7 +125,7 @@ 
          * @throws Exception If the database configuration is invalid.
          * @return PDO
          */
-        $container['database'] = function (Container $container) {
+        $container['database'] = function(Container $container) {
             $config = $container['config'];
             $databaseIdent = $config['default_database'];
             $databases = $container['databases'];

src/Charcoal/App/App.php

Spacing +2 added lines, -2 removed lines

@@ -163,7 +163,7 @@ 
     {
         if (!isset($this->routeManager)) {
             $config = $this->config();
-            $routesConfig = (isset($config['routes']) ? $config['routes'] : [] );
+            $routesConfig = (isset($config['routes']) ? $config['routes'] : []);
 
             $this->routeManager = new RouteManager([
                 'config' => $routesConfig,
@@ -223,7 +223,7 @@ 
         // For now, need to rely on a catch-all...
         $this->get(
             '{catchall:.*}',
-            function (
+            function(
                 RequestInterface $request,
                 ResponseInterface $response,
                 array $args

Unused Use Statements -1 removed lines

@@ -18,7 +18,6 @@
 
 // From 'charcoal-app'
 use Charcoal\App\Route\RouteManager;
-use Charcoal\App\Route\RouteFactory;
 
 /**
  * Charcoal App

src/Charcoal/App/Route/RouteManager.php

Spacing +9 added lines, -9 removed lines

@@ -51,17 +51,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);
             }
@@ -87,12 +87,12 @@ 
 
         $methods = isset($templateConfig['methods'])
             ? $templateConfig['methods']
-            : [ 'GET' ];
+            : ['GET'];
 
         $routeHandler = $this->app->map(
             $methods,
             $routePattern,
-            function (
+            function(
                 RequestInterface $request,
                 ResponseInterface $response,
                 array $args = []
@@ -163,12 +163,12 @@ 
 
         $methods = isset($actionConfig['methods'])
             ? $actionConfig['methods']
-            : [ 'POST' ];
+            : ['POST'];
 
         $routeHandler = $this->app->map(
             $methods,
             $routePattern,
-            function (
+            function(
                 RequestInterface $request,
                 ResponseInterface $response,
                 array $args = []
@@ -239,12 +239,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/Script/AbstractScript.php

Unused Use Statements -1 removed lines

@@ -24,7 +24,6 @@
 use Charcoal\Config\AbstractEntity;
 
 // From 'charcoal-app'
-use Charcoal\App\AppInterface;
 use Charcoal\App\Script\ScriptInterface;
 
 /**

src/Charcoal/App/Route/TemplateRoute.php

Spacing +1 added lines, -1 removed lines

@@ -101,7 +101,7 @@
         $response->getBody()->write($templateContent);
 
         if (!empty($config['headers'])) {
-            foreach($config['headers'] as $name => $val) {
+            foreach ($config['headers'] as $name => $val) {
                 $response = $response->withHeader($name, $val);
             }
         }

src/Charcoal/App/Route/ActionRoute.php

Spacing +1 added lines, -1 removed lines

@@ -80,7 +80,7 @@
 
         // Set headers if necessary.
         if (!empty($config['headers'])) {
-            foreach($config['headers'] as $name => $val) {
+            foreach ($config['headers'] as $name => $val) {
                 $response = $response->withHeader($name, $val);
             }
         }