0

我注意到重定向控制器方法部分时间不起作用。当我设置 debug > 0 时不会出现任何消​​息。在调用重定向方法之前我不会回显任何代码,所以它不应该是因为“标头已发送”。

让我们看一下我的 ArticlesController 添加操作,其中重定向在一个实例中起作用,但在另一个实例中不起作用。

public function add($page = null) {
    // Custom component to get if user has required access level
    // of page to write an article. If not, setflash to an error message
    // specific to user's access level and redirect.

    $access_message = $this->CustomPage->AccessMessage(4, $this->viewVars['access']);

    if($access_message){

        // Flash works but redirect does not
        $this->Session->setFlash(__($access_message));
        $this->redirect(array('action' => 'index', 'page' => $page));
        // Also tried
        // $this->redirect(array('controller'=>'articles', 'action' => 'index', 'page' => $page), null, true);
    } else
    {

    if ($this->request->is('post')) {

        $this->Article->create();
                if ($this->Article->save($this->request->data)) {

                    // BLAH BLAH save post, do other stuff
                    // BLAH BLAH save post, do other stuff

                    // This flash and redirect works
                    $this->Session->setFlash(__('The article has been saved'));
                    $this->redirect(array('action' => 'view', 'id' => $article_id, 'page' => $page));

                    } else {
                    $this->Session->setFlash(__('The article could not be saved. Please, try again.'));
                    } // end else if article cannot be saved
        } // if method is post
    } // end if user has access
} // end add action

它肯定与组件有关,但我不确定是什么。也许由于在使用组件后立即调用了重定向,因此“$this”试图在组件而不是控制器上执行重定向方法。我尝试了 $this->Article->redirect 并在重定向之前重新加载了 Article 模型,但这些都不起作用。

我的组件代码是:

public function AccessMessage($required_level, $user_level) {

    if(!$user_level && $this->_View->viewVars['access']){
        $user_level = $this->_View->viewVars['access'];
    }

    if(!$required_level || !$user_level || $user_level != $required_level){
        $accessModel = ClassRegistry::init('Access');
        $access_message = $accessModel->field('access_message', array('Access.id' => $required_level));
     }

      return $access_message;
}

编辑1:好的,所以我进行了一些挖掘,以找出问题出在哪里。组件的使用不是我之前认为的问题。如果我的组件中只有

public function AccessMessage($required_level, $user_level) {

    if(!$user_level && $this->_View->viewVars['access']){
        $user_level = $this->_View->viewVars['access'];
    }

    if(!$required_level || !$user_level || $user_level != $required_level){
        $access_message = 1;
     }

      return $access_message;
}

然后它工作。问题在于正确实现的这两行,因为它们返回了我期望的 $access_message 的值,但是它们干扰了重定向的能力。也许标头已经发出?

        $accessModel = ClassRegistry::init('Access');
        $access_message = $accessModel->field('access_message', array('Access.id' => $required_level));

请注意,我也尝试过:

        $access_message = ClassRegistry::init('Access')->field('access_message', array('Access.id' => $required_level));

        $this->loadModel('Access');
        $access_message = $this->Access->field('access_message', array('Access.id' => $required_level));

要点:

组件要点:https ://gist.github.com/970a951715205c222348

控制器要点:https ://gist.github.com/2b90e5af2518a81672fb

访问模型要点:https ://gist.github.com/bhndbrweyes/f333a93f0a21302d832f

4

4 回答 4

2

你可能有空间before/after php Opening/Closing tags in controller and models。在打开标签之前从所有控制器和模型中删除所有结束标签以及任何空格。然后检查结果。

于 2013-01-31T05:19:09.520 回答
1

还没有答案,但至少您的代码中有一些变量可能未定义,或者可能发生错误:

public function AccessMessage($required_level, $user_level) {

    if(!$user_level && $this->_View->viewVars['access']){
        $user_level = $this->_View->viewVars['access'];
    }

    if(!$required_level || !$user_level || $user_level != $required_level){
        $access_message = 1;
    }

    return $access_message;
}
  • 只有在不允许用户访问页面时才会定义变量 $access_message
  • 如果根本没有设置 'access' viewVar 可能会发生错误

将其更改为:

public function AccessMessage($required_level, $user_level) {
    $access_message = 0;

    if(!$user_level && $this->_View->get('access')){
        $user_level = $this->_View->get('access');
    }

    if(!$required_level || !$user_level || $user_level != $required_level){
        $access_message = 1;
    }

    return $access_message;
}

[更新] 看到你确实在你的要点上定义了 $access_message ( https://gist.github.com/970a951715205c222348 )

但是:这行不通

App::uses('Component', 'Controller', 'ClassRegistry', 'Utility');

App::uses() 接受两个参数;您想使用的“类”以及可以找到的位置。上面的行应该写成:

App::uses('Component',     'Controller/Component');
App::uses('Controller',    'Controller');
App::uses('ClassRegistry', 'Utility');

但我想知道 ClassRegistry 是否需要手动加载

[更新 2] 你的应用程序中确实发生了很多“奇怪”的事情,所以我想知道我们是否能够解决这个问题:

public function add($page = null) {
    $access = $this->viewVars['access'];

    if($this->CustomPage->AccessMessage(4, $access)){
        $this->Session->setFlash(__($this->CustomPage->AccessMessage(4, $access)));
        $this->redirect(array('action' => 'index', 'page' => $page));
    }

    // ......
}
  • 'viewVars['access']' 设置在哪里?
  • 您将 'viewVars['access']' 作为第二个参数 ($user_level) 传递给 AccessMessage(),但AccessMessage() 内部,如果未设置参数 '$user_level',您将尝试再次使用相同的 viewVar ?
  • $this->CustomPage->AccessMessage() 被调用两次以检查它是否返回任何内容,然后使用它。效率不是很高

.

public function add($page = null) {
    // Where does is $this->viewVars['access'] come from? Where is it set?
    $access  = empty($this->viewVars['access'])? null : $this->viewVars['access'];
    $message = $this->CustomPage->AccessMessage(4, $access);

    if ($message) {
        $this->Session->setFlash(__($message));
        $this->redirect(array('action' => 'index', 'page' => $page));
    }

    // ......
}

进一步说明。仅当找到“消息”且不为空时才重定向用户,而不是基于当前用户权限,您可以考虑将两者分开;

在您的组件中:

public function HasAccessLevel($required_level, $user_level) {
    if(!$user_level || $user_level != $required_level){
        return false;
    }

    return true;
}

public function AccessMessage($required_level) {
    return ClassRegistry::init('Access')->field('access_message', array('Access.id' => $required_level));
}

在您的控制器中:

public function add($page = null) {
    // Where does is $this->viewVars['access'] come from? Where is it set?
    $access  = empty($this->viewVars['access'])? null : $this->viewVars['access'];

    if($this->CustomPage->HasAccessLevel(4, $access)){
        $this->Session->setFlash(__($this->CustomPage->AccessMessage(4)));
        $this->redirect(array('action' => 'index', 'page' => $page));
    }

    // ......
}
于 2013-02-03T14:11:01.887 回答
1

Cake 2:在组件顶部:添加

  public function initialize(Controller $controller) {
        $this->Controller = $controller;
    }

在函数内部,您可以像这样重定向:

$this->Controller->redirect(array('plugin' => false, 'controller' => 'users', 'action' => 'index'));
于 2015-11-14T18:06:59.597 回答
0

如果调用了重定向,但您没有被重定向,我猜您在您的index()操作中进行了一些权限检查,以阻止访问。你能确认或发布整个控制器代码吗?

于 2013-01-27T10:52:13.610 回答