Как найти причину странного поведения скрипта?

Question

[Максим Тимофеев]

Да же не представляю где копать.
Есть проект на yii2
В afterAction всех контроллеров, кроме авторизационного есть
скрипт запоминающий урл пользователя, что бы работал return $this->goBack();

public function afterAction($action, $result)
    {
        Yii::$app->getUser()->setReturnUrl(Yii::$app->request->url);
        return parent::afterAction($action, $result);
    }

Все нормально работает кроме 1 случая.
Только у заказчика, во всех браузерах, только при восстановлении пароля (все стандартно для yii2), $this->goBack() перенаправляет его на страницу:
http://домен/тут-язык/css/https://fonts.googleapis...
Отчего такое может происходить?

Answer 1

[Никита]

Что за костыль?
return $this->redirect(Yii::$app->request->referrer);

GoBack нужен для возвращения из дебрей ссылок (например, запонение полей в несколько шагов) в какую-то одну выбранную.

Comments

[Максим Тимофеев]

Yii::$app->request->referrer не всегда то место куда нужно вернуть. Поэтому есть контролеры в которых afterAction задается url возврата, а есть где не задается, что бы остался прежний. Никаких костылей. Используется стандартный метод yii.

[Максим Тимофеев]

если переход не с сайта и Yii::$app->request->referrer пуст или ведет хрен знает куда?

[Максим Тимофеев]

При пустом прийдется делать проверку и делать путь по default и вот мы приходит к 1в1 реализации метода goback

[Никита]

Максим Тимофеев: то есть у вас goBack есть на страницах с GET запросом? Ой какая грязь.

[Никита]

И да, мой предложенный вариант решит вашу проблему.
Так как вы повесили ваш костыльный afterAction в SiteController. Чего категорически делать не рекомендуется.

[Максим Тимофеев]

Никита: Чего Вы решили что с get запросом? Конкретно в данной проблеме речь о POST запросе со сменой пароля.

[Максим Тимофеев]

Никита: Что костыльного в afterAction? Это соблюдение принципов ООП. Если бы я в экшен эту логику засунул - вот это было бы странно.

[Никита]

Максим Тимофеев:
>"если переход не с сайта"
>"Чего Вы решили что с get запросом"
Не с сайта POST запрос не поступит.
>"Что костыльного в afterAction"
Не afterAction, а использование goBack. Повторюсь - эта функция не для этого что вы делаете.
>"Это соблюдение принципов ООП"
Что? Вы жертва ряда статей про архитектуру, где её прославляют и ни капли похоже не понимаете в том что происходит. Я не против объектов, я сам ооп-шник, но вы упомянули архитектуру совсем не в том контексте.
>"Если бы я в экшен эту логику засунул - вот это было бы странно"
Ничего странного, написав строки в afterAction вы их обобщили, с тем же успехом можно было создать отдельный метод и вызывать в конце каждого экшена или не делать метод и писать этот код прям там. Обобщение это хорошо - это не принцип ООП, а принцип хорошего тона.

Уберите ваш костыль с SiteController и все заработает как надо. Угадайте почему?