locomotivemtl / charcoal-app

src/Charcoal/App/AbstractManager.php

Indentation +6 added lines, -6 removed lines

@@ -19,18 +19,18 @@
     use LoggerAwareTrait;
 
     /**
-    * @var array
-    */
+     * @var array
+     */
     private $config = [];
 
     /**
-    * @var SlimApp
-    */
+     * @var SlimApp
+     */
     private $app;
 
     /**
-    * @param array $data The dependencies container
-    */
+     * @param array $data The dependencies container
+     */
     public function __construct(array $data)
     {
         $this->set_config($data['config']);

src/Charcoal/App/Action/AbstractAction.php

Indentation +21 added lines, -21 removed lines

@@ -20,19 +20,19 @@ 
     const DEFAULT_MODE = self::MODE_REDIRECT;
 
     /**
-    * @var string $mode
-    */
+     * @var string $mode
+     */
     private $mode = self::DEFAULT_MODE;
 
     /**
-    * @var boolean $success
-    */
+     * @var boolean $success
+     */
     private $success = false;
 
     /**
-    * @param array $data
-    * @return AbstractAction Chainable
-    */
+     * @param array $data
+     * @return AbstractAction Chainable
+     */
     public function set_data(array $data)
     {
         foreach ($data as $prop => $val) {
@@ -53,10 +53,10 @@ 
     }
 
     /**
-    * @param string $mode
-    * @throws InvalidArgumentException if mode is not a string
-    * @return ActionInterface Chainable
-    */
+     * @param string $mode
+     * @throws InvalidArgumentException if mode is not a string
+     * @return ActionInterface Chainable
+     */
     public function set_mode($mode)
     {
         if (!is_string($mode)) {
@@ -69,18 +69,18 @@ 
     }
 
     /**
-    * @return string
-    */
+     * @return string
+     */
     public function mode()
     {
         return $this->mode;
     }
 
     /**
-    * @param bool $success
-    * @throws InvalidArgumentException if success is not a boolean
-    * @return ActionInterface Chainable
-    */
+     * @param bool $success
+     * @throws InvalidArgumentException if success is not a boolean
+     * @return ActionInterface Chainable
+     */
     public function set_success($success)
     {
         if (!is_bool($success)) {
@@ -93,15 +93,15 @@ 
     }
 
     /**
-    * @return bool
-    */
+     * @return bool
+     */
     public function success()
     {
         return $this->success;
     }
 
     /**
-    * @return string
-    */
+     * @return string
+     */
     abstract public function response();
 }

src/Charcoal/App/Action/ActionInterface.php

Indentation +12 added lines, -12 removed lines

@@ -6,29 +6,29 @@
 {
 
     /**
-    * @param string $mode
-    * @return ActionInterface Chainable
-    */
+     * @param string $mode
+     * @return ActionInterface Chainable
+     */
     public function set_mode($mode);
 
     /**
-    * @return string
-    */
+     * @return string
+     */
     public function mode();
 
     /**
-    * @param boolean $success
-    * @return ActionInterface Chainable
-    */
+     * @param boolean $success
+     * @return ActionInterface Chainable
+     */
     public function set_success($success);
 
     /**
-    * @return boolean
-    */
+     * @return boolean
+     */
     public function success();
 
     /**
-    * @return array
-    */
+     * @return array
+     */
     public function response();
 }

src/Charcoal/App/App.php

Indentation +48 added lines, -48 removed lines

@@ -45,36 +45,36 @@ 
     use ConfigurableTrait;
 
     /**
-    * @var SlimApp $app
-    */
+     * @var SlimApp $app
+     */
     private $app;
 
     /**
-    * @var ModuleManager
-    */
+     * @var ModuleManager
+     */
     private $module_manager;
 
     /**
-    * @var RouteManager
-    */
+     * @var RouteManager
+     */
     private $route_manager;
 
     /**
-    * @var MiddlewareManager
-    */
+     * @var MiddlewareManager
+     */
     private $middleware_manager;
 
     /**
-    * @var LanguageManager
-    */
+     * @var LanguageManager
+     */
     private $language_manager;
 
     /**
-    * # Required dependencies
-    * - `logger` A PSR-3 logger.
-    *
-    * @param array $data
-    */
+     * # Required dependencies
+     * - `logger` A PSR-3 logger.
+     *
+     * @param array $data
+     */
     public function __construct(array $data)
     {
         $this->set_logger($data['app']->logger);
@@ -85,9 +85,9 @@ 
     }
 
     /**
-    * @param SlimApp $app
-    * @return App Chainable
-    */
+     * @param SlimApp $app
+     * @return App Chainable
+     */
     public function set_app(SlimApp $app)
     {
         $this->app = $app;
@@ -95,8 +95,8 @@ 
     }
 
         /**
-    * @return ModuleManager
-    */
+         * @return ModuleManager
+         */
     public function module_manager()
     {
         if ($this->module_manager == null) {
@@ -112,8 +112,8 @@ 
     }
 
     /**
-    * @return RouteManager
-    */
+     * @return RouteManager
+     */
     public function route_manager()
     {
         if ($this->route_manager === null) {
@@ -129,8 +129,8 @@ 
     }
 
     /**
-    * @return MiddlewareManager
-    */
+     * @return MiddlewareManager
+     */
     public function middleware_manager()
     {
         if ($this->middleware_manager === null) {
@@ -146,8 +146,8 @@ 
     }
 
     /**
-    * @return LanguageManager
-    */
+     * @return LanguageManager
+     */
     public function language_manager()
     {
         if (!isset($this->language_manager)) {
@@ -174,8 +174,8 @@ 
     }
 
     /**
-    * @return App Chainable
-    */
+     * @return App Chainable
+     */
     public function setup()
     {
         $this->setup_languages();
@@ -187,8 +187,8 @@ 
     }
 
     /**
-    * @return void
-    */
+     * @return void
+     */
     protected function setup_languages()
     {
         $language_manager = $this->language_manager();
@@ -196,8 +196,8 @@ 
     }
 
     /**
-    * @return void
-    */
+     * @return void
+     */
     protected function setup_middlewares()
     {
         $middleware_manager = $this->middleware_manager();
@@ -205,10 +205,10 @@ 
     }
 
     /**
-    * Set up the app's "global" routes, via a RouteManager
-    *
-    * @return void
-    */
+     * Set up the app's "global" routes, via a RouteManager
+     *
+     * @return void
+     */
     protected function setup_routes()
     {
         $route_manager = $this->route_manager();
@@ -216,11 +216,11 @@ 
     }
 
     /**
-    * Set up the app's "global" routables.
-    * Routables can only be defined globally (app-level) for now.
-    *
-    * @return void
-    */
+     * Set up the app's "global" routables.
+     * Routables can only be defined globally (app-level) for now.
+     *
+     * @return void
+     */
     protected function setup_routables()
     {
         $charcoal = $this;
@@ -252,8 +252,8 @@ 
     }
 
     /**
-    * @return void
-    */
+     * @return void
+     */
     protected function setup_modules()
     {
         $module_manager = $this->module_manager();
@@ -261,11 +261,11 @@ 
     }
 
     /**
-    * ConfigurableTrait > create_config()
-    *
-    * @param array $data
-    * @return AppConfig
-    */
+     * ConfigurableTrait > create_config()
+     *
+     * @param array $data
+     * @return AppConfig
+     */
     public function create_config(array $data = null)
     {
         return new AppConfig($data);

Spacing +4 added lines, -4 removed lines

@@ -101,7 +101,7 @@ 
     {
         if ($this->module_manager == null) {
             $config  = $this->config();
-            $modules = (isset($config['modules']) ? $config['modules'] : [] );
+            $modules = (isset($config['modules']) ? $config['modules'] : []);
             $this->module_manager = new ModuleManager([
                 'config' => $modules,
                 'app'    => $this->app,
@@ -118,7 +118,7 @@ 
     {
         if ($this->route_manager === null) {
             $config = $this->config();
-            $routes = (isset($config['routes']) ? $config['routes'] : [] );
+            $routes = (isset($config['routes']) ? $config['routes'] : []);
             $route_manager = new RouteManager([
                 'config' => $routes,
                 'app'    => $this->app,
@@ -135,7 +135,7 @@ 
     {
         if ($this->middleware_manager === null) {
             $config = $this->config();
-            $middlewares = (isset($config['middlewares']) ? $config['middlewares'] : [] );
+            $middlewares = (isset($config['middlewares']) ? $config['middlewares'] : []);
             $middleware_manager = new MiddlewareManager([
                 'config' => $middlewares,
                 'app'    => $this->app,
@@ -227,7 +227,7 @@ 
         // For now, need to rely on a catch-all...
         $this->app->get(
             '{catchall:.*}',
-            function (
+            function(
                 RequestInterface $request,
                 ResponseInterface $response,
                 $args

src/Charcoal/App/Middleware/MiddlewareManager.php

Indentation +2 added lines, -2 removed lines

@@ -10,8 +10,8 @@
 class MiddlewareManager extends AbstractManager
 {
     /**
-    * Set up the middlewares
-    */
+     * Set up the middlewares
+     */
     public function setup_middlewares()
     {
         // ...

src/Charcoal/App/Module/AbstractModule.php

Indentation +8 added lines, -8 removed lines

@@ -27,8 +27,8 @@ 
     
     // ...
     /**
-    * @return void
-    */
+     * @return void
+     */
     public function setup()
     {
         $this->setup_middlewares();
@@ -36,8 +36,8 @@ 
     }
 
     /**
-    * @return void
-    */
+     * @return void
+     */
     protected function setup_middlewares()
     {
         $middlewares = $this->config['middlewares'];
@@ -53,10 +53,10 @@ 
 
 
     /**
-    * Set up the app's "global" routes, via a RouteManager
-    *
-    * @return void
-    */
+     * Set up the app's "global" routes, via a RouteManager
+     *
+     * @return void
+     */
     protected function setup_routes()
     {
         $routes = $this->config['routes'];

src/Charcoal/App/Module/ModuleInterface.php

Indentation +2 added lines, -2 removed lines

@@ -5,7 +5,7 @@
 interface ModuleInterface
 {
     /**
-    * @return void
-    */
+     * @return void
+     */
     public function setup();
 }

src/Charcoal/App/Module/ModuleManager.php

Indentation +13 added lines, -13 removed lines

@@ -8,19 +8,19 @@ 
 class ModuleManager extends AbstractManager
 {
     /**
-    * @var array $modules
-    */
+     * @var array $modules
+     */
     private $modules = [];
 
     /**
-    * @var \Slim\App $app
-    */
+     * @var \Slim\App $app
+     */
     private $app;
 
     /**
-    * @param array $modules
-    * @return ModuleManager Chainable
-    */
+     * @param array $modules
+     * @return ModuleManager Chainable
+     */
     public function set_modules(array $modules)
     {
         foreach ($modules as $module_ident => $module_config) {
@@ -30,10 +30,10 @@ 
     }
 
     /**
-    * @param string $module_ident
-    * @param array|ConfigInterface $module_config
-    * @return ModuleManager Chainable
-    */
+     * @param string $module_ident
+     * @param array|ConfigInterface $module_config
+     * @return ModuleManager Chainable
+     */
     public function add_module($module_ident, $module_config)
     {
         $this->modules[$module_ident] = $module_config;
@@ -41,8 +41,8 @@ 
     }
 
     /**
-    * @return void
-    */
+     * @return void
+     */
     public function setup_modules()
     {
         $modules = $this->config();

src/Charcoal/App/Routable/RoutableFactory.php

Indentation +2 added lines, -2 removed lines

@@ -11,8 +11,8 @@
 class RoutableFactory extends ResolverFactory
 {
     /**
-    * @param array $data
-    */
+     * @param array $data
+     */
     public function base_class()
     {
         return '\Charcoal\App\Routable\RoutableInterface';

Doc Comments -1 removed lines

@@ -11,7 +11,6 @@
 class WidgetFactory extends ResolverFactory
 {
     /**
-    * @param array $data
     */
     public function base_class()
     {