fix(control): 简化关联代码

Author: lisgroup

laravel/app/Http/Controllers/Api/RoleController.php

@@ -138,14 +138,19 @@ public function update(Update $request, $id)
             //     $role->revokePermissionTo($p); // 将会移除与角色关联的所有权限
             // }
             // Fix: 一并移除了权限的错误
-            foreach ($role->permissions as $permission) {
-                $role->permissions()->detach($permission->id); // 仅移除 permission_role 表与角色关联的权限
-            }
-
-            foreach ($permissions as $permission) {
-                $p = Permission::where('id', '=', $permission)->first(); //从数据库中获取相应权限
-                $p && $role->givePermissionTo($p);  // 分配权限到角色
-            }
+            /**
+             * 多对多关联
+             * 附加 / 移除 attach 和 detach
+             * @link https://learnku.com/docs/laravel/5.5/eloquent-relationships/1333#a93d4c
+             */
+            $role->permissions()->detach(); // 仅移除 permission_role 表与角色关联的权限
+
+            $role->permissions()->attach($permissions);
+
+            // foreach ($permissions as $permission) {
+            //     $p = Permission::where('id', '=', $permission)->first(); //从数据库中获取相应权限
+            //     $p && $role->givePermissionTo($p);  // 分配权限到角色
+            // }
 
             return $this->out(200, ['data' => ['id' => $id]]);
         } else {

laravel/app/Http/Controllers/Api/UserController.php

@@ -140,7 +140,7 @@ public function update(Update $request, $id)
             if (isset($roles)) {
                 $user->roles()->sync($roles);  // 如果有角色选中与用户关联则更新用户角色
             } else {
-                $user->roles()->detach(\Auth::id()); // 如果没有选择任何与用户关联的角色则将之前关联角色解除
+                $user->roles()->detach(); // 如果没有选择任何与用户关联的角色则将之前关联角色解除
             }
 
             return $this->out(200, ['data' => ['id' => $id]]);