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

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

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

Answer 2

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

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

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 中，将方法设置为 tasks 和 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);
    }
  }
}

这只是另一种方法，使您的路线更清晰，并将逻辑放入控制器中。

Answer 3

从@Matthias S 的回答中，这行得通吗？不要使用委托过滤器，而是像这样检查路由的权限：

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