複数のAdminMiddlewareのあいまいさ

Question

私は複数の管理システムを持っています：1つはスーパー管理者で、もう1つは通常の管理者で、ユーザテーブルのis_admin属性で区別されます。

そして、これら二つのミドルウェア： SuperAdminMiddleware.php

public function handle($request, Closure $next, $guard = null) 
{ 
    if(Auth::check()) 
    { 
     if($request->user()->is_admin==1) 
     { 
      return $next($request); 
     } 
     return redirect('/login');  
    } 
    else 
    { 
     return redirect('/login'); 
    } 
} 

と、NormalAdminMiddleware.php

public function handle($request, Closure $next, $guard = null) 
{ 
    if(Auth::check()) 
    { 
     if($request->user()->is_admin==2) 
     { 
      return $next($request); 
     } 
     return redirect('/login');  
    } 
    else 
    { 
     return redirect('/login'); 
    } 
} 

とloginControllerで：

protected function authenticated() 
{ 
    if (auth()->user()->is_admin==1) { 
    return redirect('/super-admin'); 
    } 

    else if(auth()->user()->is_admin==2){ 
    return redirect('/normal-admin'); 
    } 

    else { 
    return redirect('/home'); 
    } 
} 

ここで、削除と読み取りは、スーパー管理者がすべてのユーザーの詳細を削除して見ることができるように設計されていますが、通常の管理者は自分の市のユーザーしか見ることができません。

id name city is_admin 
1 Non Maety 1 
3 Pom Lorey 2 
4 Rom Lorey 0 
2 Yer Easter 0 

ノンはすべて見ることができるはずです。 PomはID3と4だけを表示するはずです。

SuperAdminMiddlewareでルートを表示して削除すると、通常の管理者は都市のレコードを見ることができませんでした。

Route::group(['middleware' => ['App\Http\Middleware\SuperAdminMiddleware']], function() { 
    Route::get('/show/{id}', 'MyController@show');  
    Route::post('/delete', 'MyController@delete'); 
    }); 

と私はSuperAdminMiddlewareとNormalAddminMiddlewareの両方でこれらのルートを置く場合。 NormalAdminMiddlewareは他の都市のレコードも見ることができます。

Route::group(['middleware' => ['App\Http\Middleware\NormalAdminMiddleware']], function() { 
     Route::get('/show/{id}', 'MyController@show');  
     Route::post('/delete', 'MyController@delete'); 
     }); 

この状況をどのように克服できますか？

Answer 1

あなたは政策とそれを解決することができます。ユーザーが認証されたユーザーモデルになり

class UserPolicy 
{ 

    /** 
    * Determine if the given user can be viewed by the user. 
    * 
    * @param \App\User $user 
    * @param \App\User $account 
    * @return bool 
    */ 
    public function view(User $user, User $account) 
    { 
     switch($user->is_admin) { 
      case 1: 
       return true; 
      case 2: 
       return $user->city == $account->city; 
      default: 
      return 0; 

     } 
    } 

    /** 
    * Determine if the given user can be updated by the user. 
    * 
    * @param \App\User $user 
    * @param \App\User $account 
    * @return bool 
    */ 
    public function update(User $user, User $account) 
    { 
     switch($user->is_admin) { 
      case 1: 
       return true; 
      case 2: 
       return $user->city == $account->city; 
      default: 
      return 0; 

     } 
    } 
} 

、アカウントが表示されなければならないユーザモデルになります。

あなたのポリシー（https://laravel.com/docs/5.4/authorization#registering-policies）を登録した後、あなたはあなたのようなコントローラーの機能でそれを呼び出すことができます。

public function show(User $user) { 
    $this->can('view', $user); 
} 

Answer 2

私は質問を読むことによって明確にあなたの目的を理解していません。 SuperAdminMiddlewareの下にメソッドを置くと、通常の管理者は拒否され、 '/ login'にリダイレクトされます。管理者の操作を制御したいのであれば、ミドルウェアは問題を解決できないと思います。前回の解答と同様に、データベースのデータを操作するときは、superAdminかどうかを判断してください。