具有相同名称的laravel过滤器组路由

Question

我在laravel 4项目中遇到了这个烦人的问题，该项目有三种类型的用户，即学生，教师和主持人（我使用Entrust作为角色管理解决方案）。

他们每个人都可以浏览相同的路线，但是根据用户类型，应调用另一种方法。 所以我的route.php文件的结构如下：

Route::group(array('before' => 'auth'), function(){


Route::group(array('before' => 'teacher'), function(){
        Route::get('/tasks',array('as'=>'tasks','uses'=>'TasksController@tasksAsTeacher'));
        Route::get('/task/{id}',array('as'=>'showTask','uses'=>'TasksController@showTaskAsTeacher'));



});
Route::group(array('before' => 'moderator'), function(){
        Route::get('/tasks',array('as'=>'tasks','uses'=>'TasksController@tasksAsModerator'));
        Route::get('/task/{id}',array('as'=>'showTask','uses'=>'TasksController@showTaskAsModerator'));

});     
Route::group(array('before' => 'student'), function(){

        Route::get('/tasks',array('as'=>'tasks','uses'=>'TasksController@tasksAsStudent'));
        Route::get('/task/{id}',array('as'=>'showTask','uses'=>'TasksController@showTaskAsStudent'));




});

});

但是，使用教师或主持人帐户浏览这些路由始终会返回404错误。 我发现这是因为在其他两个过滤器组中重新定义了路由。

因此，如果我将教师用户重定向到“ showTask”，则laravel会将任务作为学生的路由返回，因为那是最后一次重新定义“ showTask”路由，并且会出现404错误。

我现在的问题是：处理此错误的最佳方法是什么？

我希望这不会太混乱。 提前致谢！

Answer 1

取自@Matthias S的答案，这行得通吗？ 不要使用entrust过滤器，而是检查路由的权限，如下所示：

//routes.php
 if(Entrust::hasRole('teacher')) {
     Route::get('/tasks',array('as'=>'tasks','uses'=>'TasksController@tasksAsTeacher'));
     Route::get('/task/{id}',array('as'=>'showTask','uses'=>'TasksController@showTaskAsTeacher'));
 }

重复不同的角色

编辑：同样，如果您在会话中存储了用户角色，则可以使用如下自动路由：

//routes.php
 if(Entrust::hasRole(Session::get('role'))) {
     Route::get('/tasks',array('as'=>'tasks','uses'=>'TasksController@tasksAs'.Session::get('role')));
     Route::get('/task/{id}',array('as'=>'showTask','uses'=>'TasksController@showTaskAs'.Session::get('role')));
 }

这样，一旦为角色添加了正确的控制器功能，便可以根据需要添加任意数量的角色。

编辑＃2：

或者我想更好

//routes.php - UPDATED, verify role inside controller instead of defining routes based on role
     Route::get('/tasks',array('as'=>'tasks','uses'=>'TasksController@tasks'));
     Route::get('/task/{id}',array('as'=>'showTask','uses'=>'TasksController@showTask'));


//TasksController.php
public function __construct(){
    if(!Session::get('role')){ //Make sure the user has a role assigned
        return Redirect::to('login'); // Redirect to login or permission screen if not
    }
}

public function tasks(){
    if(Entrust::hasRole(Session::get('role')){
        $tasks = Tasks::where('role_id', '=', $role->id); // Get tasks based on role
        View::make('tasks.index')->with('tasks', $tasks);
    } else{
        // Show permissions error for user
    }
}

public function showTask($task_id){
    if(Entrust::hasRole(Session::get('role')){
        $task = Tasks::where('role_id', '=', $role->id)->where('id', '=', $task_id)->first();
        View::make('tasks.view')->with('task', $task);
    }
}

Answer 2

将辅助路由组移出主要身份验证组，然后使用pipe命令在每个组之前运行身份验证，例如

Route::group(array('before' => 'auth|teacher'), function(){
Route::group(array('before' => 'auth|moderator'), function(){

Answer 3

我不确定您的方法是否是完成此任务的好方法。 我认为两次定义相同的路线不是一个好习惯，但是我不知道这是否是正确的。

解决此问题的一种方法可能是，您仅定义两个路由，然后让控制器根据用户的角色决定要执行的操作。 这不是直接解决问题的方法，而是处理具有不同角色执行不同控制器操作的问题的另一种方法。

Route::group(array('before' => 'auth'), function(){

     Route::get('/tasks',array('as'=>'tasks','uses'=>'TasksController@tasks'));
     Route::get('/task/{id}',array('as'=>'showTask','uses'=>'TasksController@showTask'));

});

然后在TasksController中，使方法task和showTask像这样

class TasksController extends BaseController {
  public function tasks() {
    if(Entrust::hasRole('teacher')) {
      return $this->tasksAsTeacher();
    } else if(Entrust::hasRole('moderator')) {
      return $this->tasksAsModerator();
    } else if(Entrust::hasRole('student')) {
      return $this->tasksAsStudent();
    }
  }

  public function showTask($id) {
    if(Entrust::hasRole('teacher')) {
      return $this->showTaskAsTeacher($id);
    } else if(Entrust::hasRole('moderator')) {
      return $this->showTaskAsModerator($id);
    } else if(Entrust::hasRole('student')) {
      return $this->showTaskAsStudent($id);
    }
  }
}

这样做的另一种方法是使您的路由更整洁，并将逻辑放入控制器中。