kot13 / bootstrapi

app/dependencies.php

Unused Use Statements -3 removed lines

@@ -2,17 +2,14 @@
 use Illuminate\Database\Capsule\Manager as Capsule;
 use Illuminate\Events\Dispatcher;
 use Illuminate\Container\Container;
-
 use Monolog\Logger;
 use Monolog\Processor\UidProcessor;
 use Monolog\Handler\StreamHandler;
-
 use Illuminate\Translation\FileLoader;
 use Illuminate\Filesystem\Filesystem;
 use Illuminate\Translation\Translator;
 use Illuminate\Validation\DatabasePresenceVerifier;
 use Illuminate\Validation\Factory;
-
 use App\Common\Acl;
 use App\Common\Renderer;
 use App\Common\MailRenderer;

Spacing +15 added lines, -15 removed lines

@@ -22,20 +22,20 @@ 
 $container = $app->getContainer();
 
 // render
-$container['renderer'] = function($c){
+$container['renderer'] = function($c) {
     $renderer = new Renderer();
 
     return $renderer;
 };
 
-$container['mailRenderer'] = function($c){
+$container['mailRenderer'] = function($c) {
     $settings = $c->get('settings');
     $renderer = new MailRenderer($settings['mailTemplate']);
     return $renderer;
 };
 
 // monolog
-$container['logger'] = function ($c) {
+$container['logger'] = function($c) {
     $settings = $c->get('settings');
     $logger   = new Logger($settings['logger']['name']);
     $logger->pushProcessor(new UidProcessor());
@@ -45,8 +45,8 @@ 
 };
 
 // error handlers
-$container['errorHandler'] = function ($c) {
-    return function ($request, $response, $exception) use ($c) {
+$container['errorHandler'] = function($c) {
+    return function($request, $response, $exception) use ($c) {
         $details = (defined('DEBUG_MODE') && DEBUG_MODE == 1) ? $exception->getMessage() : 'Internal server error';
         $e = new JsonException(null, 500, 'Internal server error', $details);
 
@@ -54,14 +54,14 @@ 
     };
 };
 
-$container['notAllowedHandler'] = function ($c) {
-    return function ($request, $response, $methods) use ($c) {
-        throw new JsonException(null, 405, 'Method Not Allowed', 'Method must be one of: ' . implode(', ', $methods));
+$container['notAllowedHandler'] = function($c) {
+    return function($request, $response, $methods) use ($c) {
+        throw new JsonException(null, 405, 'Method Not Allowed', 'Method must be one of: '.implode(', ', $methods));
     };
 };
 
-$container['notFoundHandler'] = function ($c) {
-    return function ($request, $response) use ($c) {
+$container['notFoundHandler'] = function($c) {
+    return function($request, $response) use ($c) {
         throw new JsonException(null, 404, 'Not found', 'Entity not found');
     };
 };
@@ -69,7 +69,7 @@ 
 // database
 $setting = $container->get('settings');
 $capsule = new Capsule;
-foreach ($setting['database']['connections'] as $name => $connection){
+foreach ($setting['database']['connections'] as $name => $connection) {
     $capsule->addConnection($connection, $name);
 }
 $capsule->setEventDispatcher(new Dispatcher(new Container));
@@ -79,7 +79,7 @@ 
 $container['databaseManager'] = $capsule->getDatabaseManager();
 
 // ACL
-$container['acl'] = function ($c) {
+$container['acl'] = function($c) {
     $settings = $c->get('settings');
     $acl      = new Acl($settings['acl']);
 
@@ -87,7 +87,7 @@ 
 };
 
 // translation
-$container['translator'] = function($c){
+$container['translator'] = function($c) {
     $settings                = $c->get('settings');
     $translation_file_loader = new FileLoader(new Filesystem, $settings['translate']['path']);
     $translator              = new Translator($translation_file_loader, $settings['translate']['locale']);
@@ -96,7 +96,7 @@ 
 };
 
 // validation
-$container['validation'] = function($c){
+$container['validation'] = function($c) {
     $translator = $c->get('translator');
     $validation = new Factory($translator);
     $presence   = new DatabasePresenceVerifier($c->get('databaseManager'));
@@ -106,7 +106,7 @@ 
 };
 
 // mailer
-$container['mailer'] = function($c){
+$container['mailer'] = function($c) {
     $transport = \Swift_MailTransport::newInstance();
     $mailer    = \Swift_Mailer::newInstance($transport);
 

Braces +6 added lines, -6 removed lines

@@ -22,13 +22,13 @@ 
 $container = $app->getContainer();
 
 // render
-$container['renderer'] = function($c){
+$container['renderer'] = function($c) {
     $renderer = new Renderer();
 
     return $renderer;
 };
 
-$container['mailRenderer'] = function($c){
+$container['mailRenderer'] = function($c) {
     $settings = $c->get('settings');
     $renderer = new MailRenderer($settings['mailTemplate']);
     return $renderer;
@@ -69,7 +69,7 @@ 
 // database
 $setting = $container->get('settings');
 $capsule = new Capsule;
-foreach ($setting['database']['connections'] as $name => $connection){
+foreach ($setting['database']['connections'] as $name => $connection) {
     $capsule->addConnection($connection, $name);
 }
 $capsule->setEventDispatcher(new Dispatcher(new Container));
@@ -87,7 +87,7 @@ 
 };
 
 // translation
-$container['translator'] = function($c){
+$container['translator'] = function($c) {
     $settings                = $c->get('settings');
     $translation_file_loader = new FileLoader(new Filesystem, $settings['translate']['path']);
     $translator              = new Translator($translation_file_loader, $settings['translate']['locale']);
@@ -96,7 +96,7 @@ 
 };
 
 // validation
-$container['validation'] = function($c){
+$container['validation'] = function($c) {
     $translator = $c->get('translator');
     $validation = new Factory($translator);
     $presence   = new DatabasePresenceVerifier($c->get('databaseManager'));
@@ -106,7 +106,7 @@ 
 };
 
 // mailer
-$container['mailer'] = function($c){
+$container['mailer'] = function($c) {
     $transport = \Swift_MailTransport::newInstance();
     $mailer    = \Swift_Mailer::newInstance($transport);
 

app/src/Controller/CrudController.php

Doc Comments +5 added lines, -5 removed lines

@@ -15,7 +15,7 @@ 
      * @param Response $response
      * @param array    $args
      *
-     * @return mixed
+     * @return \Psr\Http\Message\ResponseInterface
      */
     public function actionIndex(Request $request, Response $response, $args)
     {
@@ -87,7 +87,7 @@ 
      * @param Response $response
      * @param array    $args
      *
-     * @return mixed
+     * @return \Psr\Http\Message\ResponseInterface
      * @throws JsonException
      */
     public function actionGet(Request $request, Response $response, $args)
@@ -110,7 +110,7 @@ 
      * @param Response $response
      * @param array    $args
      *
-     * @return mixed
+     * @return \Psr\Http\Message\ResponseInterface
      * @throws JsonException
      */
     public function actionCreate(Request $request, Response $response, $args)
@@ -133,7 +133,7 @@ 
      * @param Response $response
      * @param array    $args
      *
-     * @return mixed
+     * @return \Psr\Http\Message\ResponseInterface
      * @throws JsonException
      */
     public function actionUpdate(Request $request, Response $response, $args)
@@ -162,7 +162,7 @@ 
      * @param Response $response
      * @param array    $args
      *
-     * @return mixed
+     * @return \Psr\Http\Message\ResponseInterface
      * @throws JsonException
      */
     public function actionDelete(Request $request, Response $response, $args)

Unused Use Statements -1 removed lines

@@ -4,7 +4,6 @@
 use App\Common\Helper;
 use App\Common\JsonException;
 use App\Scopes\MaxPerPageScope;
-
 use Slim\Http\Request;
 use Slim\Http\Response;
 

Spacing +1 added lines, -1 removed lines

@@ -44,7 +44,7 @@
                         $query = $query->whereNotIn($filter['attribute'], $filter['value']);
                         break;
                     case 'like':
-                        $query = $query->where($filter['attribute'], 'like', '%' . $filter['value'] . '%');
+                        $query = $query->where($filter['attribute'], 'like', '%'.$filter['value'].'%');
                         break;
                     case '=':
                     case '!=':

Braces +3 added lines, -1 removed lines

@@ -34,7 +34,9 @@
                 $filter['operator']  = trim(strtolower($filter['operator']));
                 $filter['attribute'] = trim($filter['attribute']);
 
-                if (empty($filter['operator']) || empty($filter['attribute']) || empty($filter['value'])) continue;
+                if (empty($filter['operator']) || empty($filter['attribute']) || empty($filter['value'])) {
+                    continue;
+                }
 
                 switch ($filter['operator']) {
                     case 'in':

app/src/Controller/TokenController.php

Doc Comments +1 added lines, -1 removed lines

@@ -97,7 +97,7 @@
      * @param Response $response
      * @param array    $args
      *
-     * @return mixed
+     * @return \Psr\Http\Message\ResponseInterface
      * @throws JsonException
      */
     public function auth(Request $request, Response $response, $args)

Spacing +1 added lines, -1 removed lines

@@ -45,7 +45,7 @@
                 return false;
             }
             return true;
-        } catch (\Exception $e){
+        } catch (\Exception $e) {
             return false;
         }
     }

Braces +1 added lines, -1 removed lines

@@ -45,7 +45,7 @@
                 return false;
             }
             return true;
-        } catch (\Exception $e){
+        } catch (\Exception $e) {
             return false;
         }
     }

Unused Use Statements -2 removed lines

@@ -3,12 +3,10 @@
 
 use App\Model\User;
 use App\Common\JsonException;
-
 use App\Requests\RequestResetPasswordRequest;
 use App\Requests\ResetPasswordRequest;
 use App\Requests\UserCreateRequest;
 use App\Requests\UserUpdateRequest;
-
 use Slim\Http\Request;
 use Slim\Http\Response;
 

app/src/Controller/UserController.php

Doc Comments +4 added lines, -4 removed lines

@@ -19,7 +19,7 @@ 
      * @param Response $response
      * @param array    $args
      *
-     * @return mixed
+     * @return \Psr\Http\Message\ResponseInterface
      * @throws JsonException
      */
     public function actionCreate(Request $request, Response $response, $args)
@@ -48,7 +48,7 @@ 
      * @param Response $response
      * @param array    $args
      *
-     * @return mixed
+     * @return \Psr\Http\Message\ResponseInterface
      * @throws JsonException
      */
     public function actionUpdate(Request $request, Response $response, $args)
@@ -80,7 +80,7 @@ 
      * @param Response $response
      * @param array    $args
      *
-     * @return mixed
+     * @return \Psr\Http\Message\ResponseInterface
      * @throws JsonException
      */
     public function actionRequestResetPassword(Request $request, Response $response, $args)
@@ -126,7 +126,7 @@ 
      * @param Response $response
      * @param array    $args
      *
-     * @return mixed
+     * @return \Psr\Http\Message\ResponseInterface
      * @throws JsonException
      */
     public function actionResetPassword(Request $request, Response $response, $args)

Unused Use Statements -2 removed lines

@@ -3,12 +3,10 @@
 
 use App\Model\User;
 use App\Common\JsonException;
-
 use App\Requests\RequestResetPasswordRequest;
 use App\Requests\ResetPasswordRequest;
 use App\Requests\UserCreateRequest;
 use App\Requests\UserUpdateRequest;
-
 use Slim\Http\Request;
 use Slim\Http\Response;
 

Spacing +1 added lines, -1 removed lines

@@ -141,7 +141,7 @@
             $user->setPassword($params['data']['attributes']['password']);
             $user->removePasswordResetToken();
 
-            if($user->save()){
+            if ($user->save()) {
                 return $this->renderer->jsonApiRender($response, 204);
             };
         }

Braces +1 added lines, -1 removed lines

@@ -141,7 +141,7 @@
             $user->setPassword($params['data']['attributes']['password']);
             $user->removePasswordResetToken();
 
-            if($user->save()){
+            if($user->save()) {
                 return $this->renderer->jsonApiRender($response, 204);
             };
         }

public/index.php

Braces +2 added lines, -1 removed lines

@@ -5,7 +5,8 @@
 }
 
 // Super debug func
-function prr($value) {
+function prr($value)
+{
     if (defined('DEBUG_MODE') && DEBUG_MODE == 1) {
         echo '<pre>';
         print_r($value);

Spacing +5 added lines, -5 removed lines

@@ -13,20 +13,20 @@
     };
 }
 
-require __DIR__ . '/../vendor/autoload.php';
+require __DIR__.'/../vendor/autoload.php';
 
-$settings = require __DIR__ . '/../app/settings.php';
+$settings = require __DIR__.'/../app/settings.php';
 
 $app = new \Slim\App($settings);
 
 // Set up dependencies
-require __DIR__ . '/../app/dependencies.php';
+require __DIR__.'/../app/dependencies.php';
 
 // Register middleware
-require __DIR__ . '/../app/middleware.php';
+require __DIR__.'/../app/middleware.php';
 
 // Register routes
-require __DIR__ . '/../app/routes.php';
+require __DIR__.'/../app/routes.php';
 
 // Run!
 $app->run();

app/routes.php

Spacing +5 added lines, -5 removed lines

@@ -1,7 +1,7 @@ 
 <?php
 // Routes
-$app->group('/api', function () {
-    $this->options('[/{params:.*}]', function ($request, $response, $args) {
+$app->group('/api', function() {
+    $this->options('[/{params:.*}]', function($request, $response, $args) {
         return $response->withHeader('Access-Control-Allow-Origin', '*')
             ->withHeader('Access-Control-Allow-Methods', 'GET, PUT, PATCH, POST, DELETE, OPTIONS')
             ->withHeader('Access-Control-Allow-Credentials', 'true')
@@ -11,7 +11,7 @@ 
 
     $this->post('/token', 'App\Controller\TokenController:auth');
 
-    $this->group('/{entity:user}', function () {
+    $this->group('/{entity:user}', function() {
         $this->get('', 'App\Controller\UserController:actionIndex');
         $this->post('', 'App\Controller\UserController:actionCreate');
         $this->get('/{id:[0-9]+}', 'App\Controller\UserController:actionGet');
@@ -22,7 +22,7 @@ 
         $this->post('/reset-password', 'App\Controller\UserController:actionResetPassword');
     });
 
-    $this->group('/{entity:role|right}', function () {
+    $this->group('/{entity:role|right}', function() {
         $this->get('', 'App\Controller\CrudController:actionIndex');
         $this->get('/{id:[0-9]+}', 'App\Controller\CrudController:actionGet');
         $this->post('', 'App\Controller\CrudController:actionCreate');
@@ -30,7 +30,7 @@ 
         $this->delete('/{id:[0-9]+}', 'App\Controller\CrudController:actionDelete');
     });
 
-    $this->group('/{entity:log}', function () {
+    $this->group('/{entity:log}', function() {
         $this->get('', 'App\Controller\CrudController:actionIndex');
         $this->get('/{id:[0-9]+}', 'App\Controller\CrudController:actionGet');
     });

app/database/migrations/20160706232752_create_roles_table.php

Braces +1 added lines, -2 removed lines

@@ -8,8 +8,7 @@
      */
     public function up()
     {
-        Capsule::schema()->create('roles', function($table)
-        {
+        Capsule::schema()->create('roles', function($table) {
             $table->increments('id');
             $table->string('name')->unique();
             $table->string('description');

app/database/migrations/20161016143152_create_roles_to_right_table.php

Indentation +4 added lines, -4 removed lines

@@ -3,8 +3,8 @@ 
 
 class CreateRolesToRightTable{
     /**
-    * Do the migration
-    */
+     * Do the migration
+     */
     public function up()
     {
         Capsule::schema()->create('roles_to_rights', function($table)
@@ -17,8 +17,8 @@ 
     }
 
     /**
-    * Undo the migration
-    */
+     * Undo the migration
+     */
     public function down()
     {
         Capsule::schema()->drop('roles_to_rights');

Braces +1 added lines, -2 removed lines

@@ -8,8 +8,7 @@
     */
     public function up()
     {
-        Capsule::schema()->create('roles_to_rights', function($table)
-        {
+        Capsule::schema()->create('roles_to_rights', function($table) {
             $table->integer('role_id')->unsigned();
             $table->integer('right_id')->unsigned();
             $table->primary(['role_id', 'right_id']);

app/database/seeds/20160706233231_create_roles_rights_users.php

Braces +2 added lines, -1 removed lines

@@ -2,7 +2,8 @@
 use App\Model\Role;
 use App\Model\User;
 
-class CreateRolesRightsUsers {
+class CreateRolesRightsUsers
+{
     public function run()
     {
         Role::create([