AM
мне это не нравится
Size: a a a
2020 March 26
AM
А как она может не всплывать?
А
сейчас у нас можно юзать мидлвари откуда угодно
тут не понял, о чём ты
AM
ща, погоди
А
А как она может не всплывать?
есть несколько исключительных случаев. Эти исключения только для завершения работы роутера, а не генерирования ответа
AM
а разве сейчас не так?
AM
https://github.com/yiisoft/router/blob/master/src/Middleware/Router.php#L36 - вот это у нас вроде и есть выход без выполнения стека роута
AM
или ты хочешь прям особый стек в Route, который по "плохому" статусу прерывает цепочку?
А
https://github.com/yiisoft/router/blob/master/src/Middleware/Router.php#L36 - вот это у нас вроде и есть выход без выполнения стека роута
посмотрел, это немного не тот случай. Это сработает, если не сматчился роут или метод
А
или ты хочешь прям особый стек в Route, который по "плохому" статусу прерывает цепочку?
ну там вроде и так свой стек. Просто не надо продолжать эту цепочку в WebActionsCaller если экшен не найден
AM
ну, в принципе, норм идея
AM
поправишь?
А
но по сути, если дело дошло до выполнения WebActionsCaller, то это значит, что предыдущие мидлвари роута уже начали работу. И кроме исключения для того, чтобы откатиться назад, вариантов то и нет
А
Теперь посмотрим на
\Yiisoft\Router\Middleware\ActionCaller. У меня возникают сомнения в том, что $handler надо передавать параметром в InjectorА
Почему я должен убрать хендлер? У меня еще более вложенная мидлтварь может быть для роутов. И в экшине я могу, например, только установить некоторые реквизиты риквеста и сделать $response = $handler->handle($request)
вот что пишет Дмитрий
А
ну, в принципе, норм идея
кроме того, думаю, что надо дать такую возможность контроллерам, которые самодостаточны и не используют ActionCaller'ы. Кроме как ввести
RoutingException, мне варианты не приходятAM
но по сути, если дело дошло до выполнения WebActionsCaller, то это значит, что предыдущие мидлвари роута уже начали работу. И кроме исключения для того, чтобы откатиться назад, вариантов то и нет
Есть. Ничего не делать.
AM
Предыдущие, конечно, отработают, да
AM
запили PR
AM
станет понятней