Merge pull request #54 from jeremykenedy/analysis-zEwZKA

Apply fixes from StyleCI
jeremykenedy · May 28, 2019 · 8dcb76e · 8dcb76e
2 parents d71dc1e + ebbdf2e
commit 8dcb76e
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 17 deletions.
diff --git a/src/App/Http/Controllers/Api/LaravelRolesApiController.php b/src/App/Http/Controllers/Api/LaravelRolesApiController.php
@@ -3,10 +3,7 @@
 namespace jeremykenedy\LaravelRoles\App\Http\Controllers\Api;
 
 use App\Http\Controllers\Controller;
-use Illuminate\Http\Request;
 use jeremykenedy\LaravelRoles\App\Http\Requests\StoreRoleRequest;
-use jeremykenedy\LaravelRoles\App\Http\Requests\UpdateRoleRequest;
-use jeremykenedy\LaravelRoles\App\Services\RoleFormFields;
 use jeremykenedy\LaravelRoles\Traits\RolesAndPermissionsHelpersTrait;
 use jeremykenedy\LaravelRoles\Traits\RolesUsageAuthTrait;
 
@@ -39,9 +36,9 @@ public function index()
      */
     public function store(StoreRoleRequest $request)
     {
-        $roleData           = $request->roleFillData();
-        $rolePermissions    = $request->get('permissions');
-        $role               = $this->storeRoleWithPermissions($roleData, $rolePermissions);
+        $roleData = $request->roleFillData();
+        $rolePermissions = $request->get('permissions');
+        $role = $this->storeRoleWithPermissions($roleData, $rolePermissions);
 
         return response()->json([
             'code'      => 201,
@@ -50,5 +47,4 @@ public function store(StoreRoleRequest $request)
             'role'      => $role,
         ], 201);
     }
-
 }
diff --git a/src/App/Http/Controllers/LaravelpermissionsDeletedController.php b/src/App/Http/Controllers/LaravelpermissionsDeletedController.php
@@ -36,8 +36,8 @@ public function index()
      */
     public function show($id)
     {
-        $item           = $this->getDeletedPermissionAndDetails($id);
-        $typeDeleted    = 'deleted';
+        $item = $this->getDeletedPermissionAndDetails($id);
+        $typeDeleted = 'deleted';
 
         return view('laravelroles::laravelroles.crud.permissions.show', compact('item', 'typeDeleted'));
     }

diff --git a/src/Traits/RolesAndPermissionsHelpersTrait.php b/src/Traits/RolesAndPermissionsHelpersTrait.php
@@ -360,9 +360,9 @@ public function getRolePermissionsIds($id)
     /**
      * Gets the role users.
      *
-     * @param int $roleId   The role identifier
+     * @param int $roleId The role identifier
      *
-     * @return array        The role users.
+     * @return array The role users.
      */
     public function getRoleUsers($roleId)
     {
@@ -387,25 +387,25 @@ public function getRoleUsers($roleId)
     /**
      * Gets the deleted permission and details (Roles and Users).
      *
-     * @param int $id     The identifier
+     * @param int $id The identifier
      *
      * @return Permission The permission and details.
      */
     public function getDeletedPermissionAndDetails($id)
     {
-        $permission             = $this->getDeletedPermission($id);
-        $users                  = $this->getAllUsersForPermission($permission);
-        $permission['users']    = $users;
+        $permission = $this->getDeletedPermission($id);
+        $users = $this->getAllUsersForPermission($permission);
+        $permission['users'] = $users;
 
         return $permission;
     }
 
     /**
      * Gets all users for permission.
      *
-     * @param collection $permission    The permission
+     * @param collection $permission The permission
      *
-     * @return collection               All users for permission.
+     * @return collection All users for permission.
      */
     public function getAllUsersForPermission($permission)
     {