А
был бы день, я бы мем не поленился нарисовать )
Size: a a a
2020 May 01
RT
ок, давай закончим дисскусию) Спасибо.
RT
на вкус и цвет и все такое...
RT
ну да, в этом и смысл, я хочу все, и ничего не делать.
А
на вкус и цвет и все такое...
ну тут вопрос не вкуса и цвета. Важное дело обсуждаем. Мы свои позиции высказали
А
скоро @samdark и @yiiliveext проснутся и решат в пользу декоратора. А может и нет
RT
DS
остаётся только контроллер )
Хелпер зафигачить. RequestHelper::isAjax($request);
NO
Мне кажется, надо исходить из ожидаемого поведения) Все будут ожидать $request->isAjax(), значит так и надо сделать...
А
а почему не $this->isAjax в контроллере?
NO
Ну контроллер не аяксовый
NO
А запрос аяксовый )
А
у меня так на проде одном сделано и норм
NO
Ну я видел и глобальные переменные на проде ) И норм ) Но в Yii 3 не предлагаю )
А
добавь Request если хочешь ) $this->isAjaxRequest
NO
Тогда нелогично. Контроллер не может быть аяксреквестом)
А
в контроллере даже RequestInterface передавать не придётся, т.к. он уже по пути мидлвари передан
NO
Возможный вариант $this->isAjax($request)
Но он всё равно будет путать, потому что искать будут не там.
Но он всё равно будет путать, потому что искать будут не там.
А
Тогда нелогично. Контроллер не может быть аяксреквестом)
метод то приватненький. В контексте контроллера не запутаешься
А
Ну контроллер не аяксовый
ах да, тут же не любят коротких названий (на примере addMiddleware). Тогда можно
$this->checkAjaxHeaderInRequest()