AM
\Yiisoft\Router\Middleware\ActionCaller. У меня возникают сомнения в том, что $handler надо передавать параметром в InjectorSize: a a a
2020 March 26
DS
Теперь посмотрим на
Обоснуй почему нет.
А
что интересно. Если аргумент без указания типа, то
->allowNull() вернёт trueDS
но по сути, если дело дошло до выполнения WebActionsCaller, то это значит, что предыдущие мидлвари роута уже начали работу. И кроме исключения для того, чтобы откатиться назад, вариантов то и нет
Исключение не откатит уже произошедший в цепочке действия, оно не даст выполнить остальные, то есть нарушит логику пайплайна. Когда ты вместо исключения передаешь управление NotFoundHandler, то он создает и запускает в обратную сторону цепочки ошибку 404, при этом у мидлтварей есть возможность на это среагировать.
А
Исключение не откатит уже произошедший в цепочке действия, оно не даст выполнить остальные, то есть нарушит логику пайплайна. Когда ты вместо исключения передаешь управление NotFoundHandler, то он создает и запускает в обратную сторону цепочки ошибку 404, при этом у мидлтварей есть возможность на это среагировать.
есть разная логика. Та, которую надо завершить, должна всегда юзать try{}catch{}finally{}
А
т.е. ловим исключение, завершаем действие, пропускаем исключение дальше
DS
И вообще, выбрось это слово пайплайн, я уже от тебя заразился. Мидлтвари - это не пайплайн!
А
пипелин
DS
т.е. ловим исключение, завершаем действие, пропускаем исключение дальше
Ну так и сейчас есть, только без исключения. Ловим 404 ответ, завершаем действие, передаем 404 ответ дальше по цепочке и так к самому началу
А
модернизируя респонс так, будто бы он нормальный
DS
модернизируя респонс так, будто бы он нормальный
Почему? Потому что у разраба кривые руки?
А
будешь в каждой мидлвари проверять статус ответа?
DS
будешь в каждой мидлвари проверять статус ответа?
Не в каждой мидлтвари это нужно, далеко не в каждой. А вот в твоем случае в каждой мидлтвари надо все делать в try|catch|finally
А
Не в каждой мидлтвари это нужно, далеко не в каждой. А вот в твоем случае в каждой мидлтвари надо все делать в try|catch|finally
точно также не в каждой )
DS
точно также не в каждой )
Ну и чем лучше с исключениями?
А
а тем, что запрос не модифицированный косячным роутом дальше пойдёт
А
в твоём случае все аргументы роута засядут в запросе
DS
Сделал понятное представление работы стека мидл тварей. Я думаю так будет понятнее тем, кто с ними не сталкивался.
https://3v4l.org/ZWnJ3
https://3v4l.org/ZWnJ3
DS
а тем, что запрос не модифицированный косячным роутом дальше пойдёт
Запрос модифицируется по пути туда, а не обратно
А
дак вот он у тебя и будет модифицирован роутером