Почему ActionFilter не рендерит нужный файл?

Question

[Сергей Семенко]

Вот создал фильтр, но он не работает, просто выводит главную страницу (/site/index)

class FilterBannedIp extends ActionFilter {
    public function beforeAction($action) {
        $bannedIp = BannedIp::findByIp(Yii::$app->request->getUserIP());

        // die(var_dump(Yii::$app->request->getUserIP()));
        // die($bannedIp->ip);

        if ($bannedIp && $bannedIp->isBanned(Yii::$app->request->getUserIP())) {
            if (Yii::$app->request->isAjax) {
                return 'Error: Ваш IP заблокирован';
            } else {
                return $action->controller->render('/site/banned_ip', [
                    'bannedIp' => $bannedIp,
                ]);
            }
        }

        return parent::beforeAction($action);
    }
}

Условие срабатывает, я проверил, и если указать несуществующий файл в $action->controller->render получаю ошибку, но какой шаблон не укажи, всегда выводит главную страницу.

И как вообще правильно сделать фильтр по IP ко всем контроллерам?

Answer 1

[Виталий Озерский]

Потому что задача фильтра проверить есть ли доступ к действию или нет.

Он может вернуть true или false, рендерить ответ - задача контроллера.

Внутри beforeAction можно вызвать исключение, которое перехватит errorHandler, ну а там в errorHandler->errorAction вы можете зарендерить ошибку

Comments

[Сергей Семенко]

В beforeAction фильтра или контроллера?

[Виталий Озерский]

Сергей Семенко: Это одно и тоже))) Просто фильтр можно прикреплять к разным контроллерам...

[Сергей Семенко]

Виталий Озерский: логично ) А как прикрепить фильтр ко всем контролерам?

[Виталий Озерский]

Сергей Семенко: Прикрепить фильтр к базовому контроллеру, от которого наследуются ваши контроллеры, но в этом случае лучше просто в базовом контроллере реализовать beforeAction()

[Виталий Озерский]

Сергей Семенко: Прикрепить фильтр к базовому контроллеру, от которого наследуются ваши контроллеры, но в этом случае лучше просто в базовом контроллере реализовать beforeAction()

[Сергей Семенко]

Виталий Озерский: ясно, думал есть другие методы, спасибо за ответ