АН
и там уже пытается инжектор отработать, но бросает исключение, если не находит что нужно, к примеру ResponseInterface
Size: a a a
2020 September 28
АН
Но, с другой стороны, а чем плохи кложи с __invoke?
не плохи, но если реализует интерфейс MiddlewareInterface, нужно ее как кложу или process отработать ?
DS
Александр Некрасов
не плохи, но если реализует интерфейс MiddlewareInterface, нужно ее как кложу или process отработать ?
Они все равно в обертку заворачиваются. Можешь сделать тест, который ломает текущее поведение с помощью таких клож?
АН
Но, с другой стороны, а чем плохи кложи с __invoke?
и тогда в чем разница между классом который реализует интерфейс MiddlewareInterface и клссом где __invoke ?
АН
Они все равно в обертку заворачиваются. Можешь сделать тест, который ломает текущее поведение с помощью таких клож?
АН
такая миддлеваре не работает
АН
пытается заинжектить из __invoke, но там ResponseInterface которая не определена
DS
Мне помнится, мы уже разбирали этот кейс и ты даже пр делал, чтобы все работало
АН
Мне помнится, мы уже разбирали этот кейс и ты даже пр делал, чтобы все работало
это в MiddlewareDispacher в yii-web
АН
такой же фикс, который я сделал в route
DS
Ладно, давай я попробую в роут эту мидлтварь добавить и посмотрю как лучше решить.
АН
если передан объект, и реализует MiddlewareInterface, то отрабатывает его как Middleware, если нет, уже дальше обарачивается и отрабатывает
DS
Александр Некрасов
если передан объект, и реализует MiddlewareInterface, то отрабатывает его как Middleware, если нет, уже дальше обарачивается и отрабатывает
Я понимаю где проблема, но это скорее редкий случай, чтобы одновременно и __invoke и MiddlewareInterface. И оно будет работать если передать ->addMIddleware(fn () => CorsMiddlware(...))
АН
Я понимаю где проблема, но это скорее редкий случай, чтобы одновременно и __invoke и MiddlewareInterface. И оно будет работать если передать ->addMIddleware(fn () => CorsMiddlware(...))
согласен, но попался такой случай ), на сколько fn() => new CorsMiddlewae() дешевле, чем сразу обьект передать ?, если в конструкторе ничего не тянится
DS
Александр Некрасов
согласен, но попался такой случай ), на сколько fn() => new CorsMiddlewae() дешевле, чем сразу обьект передать ?, если в конструкторе ничего не тянится
Ненамного, но не у всех в конструкторе ничего не тянется. А если у тебя несколько сотен роутов, да по пару-тройке мидлтварей, это уже прилично может набежать.
АН
ну если в конструкторе есть зависимости, согласен, но если нету, то разницы не думаю что будет, тогда запретить передачу объекта вообще, и указать в доке, что стринг, или кложа, и уже нужно сетапить в контейнере (если стринг)
DS
Александр Некрасов
ну если в конструкторе есть зависимости, согласен, но если нету, то разницы не думаю что будет, тогда запретить передачу объекта вообще, и указать в доке, что стринг, или кложа, и уже нужно сетапить в контейнере (если стринг)
Та я думаю, что объект-кложа - это норм. Можно этот кейс запретить просто
АН
Та я думаю, что объект-кложа - это норм. Можно этот кейс запретить просто
я же не говорю что плохо )
АН
можно запретить объекты с реализацией интерфейса MiddlewareInterface
DS
Ну вот и запретить кложи, которые реализуют MiddlewareInterface