Porto-SAP / Hello-API

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/UI/API/Tests/Functional/AssignUserToRoleTest.php

Unused Use Statements -1 removed lines

@@ -4,7 +4,6 @@
 
 use App\Containers\User\Models\User;
 use App\Port\Tests\PHPUnit\Abstracts\TestCase;
-use Vinkla\Hashids\Facades\Hashids;
 
 /**
  * Class AssignUserToRoleTest.

app/Containers/Authorization/UI/API/Tests/Functional/AttachPermissionsToRoleTest.php

Unused Use Statements -1 removed lines

@@ -4,7 +4,6 @@
 
 use App\Containers\Authorization\Models\Permission;
 use App\Containers\Authorization\Models\Role;
-use App\Containers\User\Models\User;
 use App\Port\Tests\PHPUnit\Abstracts\TestCase;
 
 /**

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);
         }
 

app/Containers/Authorization/Tasks/AttachPermissionsToRoleTask.php

Spacing +3 added lines, -3 removed lines

@@ -42,13 +42,13 @@
     {
         $role = $this->getRoleAction->run($roleName);
 
-        if(is_array($permissionNames)){
+        if (is_array($permissionNames)) {
 
-            foreach ($permissionNames as $permissionName){
+            foreach ($permissionNames as $permissionName) {
                 $role->givePermissionTo($permissionName);
             }
 
-        }else{
+        } else {
             $role->givePermissionTo($permissionNames);
         }