Porto-SAP / Hello-API

app/Port/Middleware/Loaders/MiddlewaresLoaderTrait.php

Unused Use Statements -2 removed lines

@@ -4,8 +4,6 @@
 
 use App;
 use App\Port\Middleware\PortKernel;
-use DB;
-use File;
 
 /**
  * Class MiddlewaresLoaderTrait.

app/Port/Foundation/Traits/FractalTrait.php

Spacing +1 added lines, -1 removed lines

@@ -34,7 +34,7 @@
 
         // if DataArray `\League\Fractal\Serializer\DataArraySerializer` do noting since it's set by default by the Dingo API
         if ($serializerName !== 'DataArray') {
-            app('Dingo\Api\Transformer\Factory')->setAdapter(function () use ($serializerName) {
+            app('Dingo\Api\Transformer\Factory')->setAdapter(function() use ($serializerName) {
                 switch ($serializerName) {
                     case 'JsonApi':
                         $serializer = new \League\Fractal\Serializer\JsonApiSerializer(Config::get('api.domain'));

Unused Use Statements -3 removed lines

@@ -4,10 +4,7 @@
 
 use App;
 use App\Port\Exception\Exceptions\UnsupportedFractalSerializerException;
-use DB;
-use File;
 use Illuminate\Support\Facades\Config;
-use Log;
 
 /**
  * Class FractalTrait.

app/Port/Broadcast/Providers/BroadcastServiceProvider.php

Spacing +1 added lines, -1 removed lines

@@ -25,7 +25,7 @@
         /*
          * Authenticate the user's personal channel...
          */
-        Broadcast::channel('App.User.*', function ($user, $userId) {
+        Broadcast::channel('App.User.*', function($user, $userId) {
             return (int)$user->id === (int)$userId;
         });
     }

app/Containers/User/Data/Factories/UserFactory.php

Spacing +1 added lines, -1 removed lines

@@ -3,7 +3,7 @@
 use App\Containers\User\Models\User;
 
 // User
-$factory->define(User::class, function (Faker\Generator $faker) {
+$factory->define(User::class, function(Faker\Generator $faker) {
     return [
         'name'              => $faker->name,
         'email'             => $faker->email,

app/Containers/Socialauth/Tasks/UpdateUserSocialProfileTask.php

Doc Comments +4 added lines, -4 removed lines

@@ -33,12 +33,12 @@
     /**
      * @param      $userId
      * @param null $token
-     * @param null $expiresIn
-     * @param null $refreshToken
-     * @param null $tokenSecret
+     * @param boolean|null $expiresIn
+     * @param boolean|null $refreshToken
+     * @param boolean|null $tokenSecret
      * @param null $provider
      * @param null $avatar
-     * @param null $avatar_original
+     * @param boolean|null $avatar_original
      * @param null $socialId
      * @param null $nickname
      * @param null $name

app/Containers/Authorization/Data/Migrations/2016_12_29_201047_create_permission_tables.php

Spacing +5 added lines, -5 removed lines

@@ -14,7 +14,7 @@ 
     {
         $config = config('laravel-permission.table_names');
 
-        Schema::create($config['roles'], function (Blueprint $table) {
+        Schema::create($config['roles'], function(Blueprint $table) {
             $table->increments('id');
             $table->string('name')->unique();
             $table->string('display_name')->nullable();
@@ -22,7 +22,7 @@ 
             $table->timestamps();
         });
 
-        Schema::create($config['permissions'], function (Blueprint $table) {
+        Schema::create($config['permissions'], function(Blueprint $table) {
             $table->increments('id');
             $table->string('name')->unique();
             $table->string('display_name')->nullable();
@@ -30,7 +30,7 @@ 
             $table->timestamps();
         });
 
-        Schema::create($config['user_has_permissions'], function (Blueprint $table) use ($config) {
+        Schema::create($config['user_has_permissions'], function(Blueprint $table) use ($config) {
             $table->integer('user_id')->unsigned();
             $table->integer('permission_id')->unsigned();
 
@@ -47,7 +47,7 @@ 
             $table->primary(['user_id', 'permission_id']);
         });
 
-        Schema::create($config['user_has_roles'], function (Blueprint $table) use ($config) {
+        Schema::create($config['user_has_roles'], function(Blueprint $table) use ($config) {
             $table->integer('role_id')->unsigned();
             $table->integer('user_id')->unsigned();
 
@@ -64,7 +64,7 @@ 
             $table->primary(['role_id', 'user_id']);
         });
 
-        Schema::create($config['role_has_permissions'], function (Blueprint $table) use ($config) {
+        Schema::create($config['role_has_permissions'], function(Blueprint $table) use ($config) {
             $table->integer('permission_id')->unsigned();
             $table->integer('role_id')->unsigned();
 

app/Containers/User/Actions/CreateUserAction.php

Indentation +1 added lines, -1 removed lines

@@ -67,7 +67,7 @@
         // be default give all users the client role (normal user)
         $this->assignRoleTask->run($user, ['client']);
 
-       //  add Client as role for normal users
+        //  add Client as role for normal users
         $this->fireUserCreatedEventTask->run($user);
 
         return $user;

app/Containers/Authorization/Actions/AssignRoleAction.php

Doc Comments +1 added lines, -1 removed lines

@@ -21,7 +21,7 @@
     /**
      * GetAdminRoleAction constructor.
      *
-     * @param \App\Containers\Authorization\Data\Repositories\AssignRoleTask $assignRoleTask
+     * @param AssignRoleTask $assignRoleTask
      */
     public function __construct(AssignRoleTask $assignRoleTask)
     {

app/Containers/Authorization/Tasks/AssignRoleTask.php

Spacing +2 added lines, -2 removed lines

@@ -45,7 +45,7 @@ 
      */
     public function run($user, $roles)
     {
-        if(!$user instanceof User){
+        if (!$user instanceof User) {
             $user = $this->findUserByIdTask->run($user);
         }
 
@@ -54,7 +54,7 @@ 
                 $this->assignRole($user, $role);
             }
 
-        }else{
+        } else {
             $this->assignRole($user, $roles);
         }
 

Braces +1 added lines, -1 removed lines

@@ -54,7 +54,7 @@
                 $this->assignRole($user, $role);
             }
 
-        }else{
+        } else{
             $this->assignRole($user, $roles);
         }