Почему не работает редирект в Zend Framework 2?

Question

[Eugene Vdovenko]

В контроллере метод есть метод manageAction().
В нем вызываю метод fillForm().
В методе fillForm() есть блок try...catch, в разделе catch которого добавляю сообщение об ошибке и редирект.

$this->flashMessenger()->addErrorMessage("Ошибка при работе с API v1.5.");
return $this->redirect()->toRoute('home');

Это согласно документации. И это не работает. Т.е. отрабатывает ViewModel экшена manageAction(), а не indexAction() как задано в роуте "home". Однако, если написать так:

$this->redirect()->toRoute('home');
$this->flashMessenger()->addErrorMessage("Ошибка при работе с API v1.5.");

все работает, как надо. И самое интересное, что если непосредственно в manageAction() вызвать

$this->flashMessenger()->addErrorMessage("Ошибка при работе с API v1.5.");
return $this->redirect()->toRoute('home');

тоже все работает.

Answer 1

[Кирилл Арутюнов]

А вы уверены, что catch срабатывает?
И то, что в $this у вас есть метод redirect()?

Проверьте дебагом, что у вас отрабатывает, а что нет.

Comments

[Eugene Vdovenko]

Проверял. В catch попадает, плагин redirect() отрабатывает. Только переход не срабатывает, если сначала сообщение в стек добавить, а потом редирект делать. А если наоборот - все работает.

Answer 2

[Андрей Мохов]

очевидно же, вы в методе fillForm возвращаете Responce, а делать return в manageAction кто будет?

public function manageAction()
{
    // ...
    $ret = $this->fillForm();
    if ($ret instanceof Response) {
        return $ret;
    }
    // ...
}

Comments

[Eugene Vdovenko]

В целом, конечно, решение... Но делать проверку каждого метода, в котором я обрабатываю исключение - по-моему это не самый хороший вариант.

[Котик Антон]

Eugene Vdovenko: Тогда ловите исключения на уровне выше: в manageAction() или даже в onDispatch().

[Eugene Vdovenko]

Вопрос, на самом деле не в том, как написать, чтобы работало. А почему не работает так, как написал я. По идее при вызове redirect() должен сработать Location, а он не срабатывает.

[Андрей Мохов]

Eugene Vdovenko: метод редирект просто возвращает объект типа Response, он не добавляет в header какой-либо Location

[Eugene Vdovenko]

ОК. Почему, когда я меняю местами redirect и flashmesseges - перенаправление происходит?

[Котик Антон]

Eugene Vdovenko: А после catch что-то дальше идёт? Вы же не только местами поменяли, но ещё и return убрали.

[Eugene Vdovenko]

27cm:

public function fillForm($form)
{
    try {
        // ...
    } catch (\Exception $e) {
        $this->redirect()->toRoute('home');
        $this->flashMessenger()->addErrorMessage("Ошибка при работе с API v1.5.");
    }
    return $form;
}

[Котик Антон]

Eugene Vdovenko: Что-то ту не так, не должно быть редиректа. Попробуйте сделать полный минимальный пример, на котором можно будет проверить это поведение: конфигурацию, код контроллера с парой методов. И написать разработчикам https://github.com/zendframework/zend-mvc/issues

[Андрей Мохов]

27cm: да что далеко ходить, можно в отладке увидеть кто чего возвращает и где происходит отсылка хидера

[Eugene Vdovenko]

В общем, есть у меня подозрение, что объект Response с заголовком location проходить по стеку до экшена в контроллере и там уже выполняет redirect. Пока решил проблему прокидыванием исключения и перехвата его в экшене.

public function manageAction()
{
    try {
        $result = $this->fillForm();
    } catch (\Exception $e) {
        return $this->redirect()->toRoute('home');
    }
}

public function fillForm()
{
    try {
        // ...
    } catch (\Exception $e) {
        $this->flashMessenger()->addErrorMessage("Ошибка при работе с API v1.5.");
        throw $e;
    }
}

Вопрос теперь - как прервать стек вызовов методов контроллера и отослать ответ прям в месте вызова редиректа, а не только из экшена?

[Андрей Мохов]

Eugene Vdovenko: это плохая практика, т.к. есть другие слушатели, которые также ждут своей очереди

[Андрей Мохов]

Eugene Vdovenko: с тем же успехом кто вам мешает сделать header('Location: utl'); exit();

[Eugene Vdovenko]

Андрей Мохов: В тем же успехом я могу все на нативном PHP написать. :) Вопрос на самом деле чисто исследовательский: есть же какая-то функция, которая начинает обрабатывать объект Response и выполнять какие-то действия. Думается мне ее можно запускать не только автоматом по завершению экшена.

В плане плохой практики - в чем? Т.е. я вот наткнулся на место в коде, где мне не важно что происходит по коду дальше. Надо перезапустить скрипт с новыми входными данными. По моему вся логика работы PHP на этом строится: запуститься - отработать - завершиться - запуститься отработать-завершиться... Я ошибаюсь?