DS
Дефолтных мидлтварей под это нет
Size: a a a
2020 March 31
AM
В демо разве нельзя юзнуть в двух местах JsonWebResponseFormatter?
AM
Будет же ровно тот же результат (да, отрендерится в разные моменты).
А
Ок, я уберу эту возможность, будет только оборачиванием в коллейбл
а как ты уберёшь эту возможность, если
$wrap(new Instance(123), 'method') как раз и вернёт кложуру?DS
В демо разве нельзя юзнуть в двух местах JsonWebResponseFormatter?
Нет, тогда ответ апи инфо обернется в обертку в другой мидлтвари
AM
Чем оно тебе мешает?
Тем, что чёрт ногу сломит в большом числе штук, которые выглядят одинаково.
AM
Нет, тогда ответ апи инфо обернется в обертку в другой мидлтвари
Ну и хорошо. Незачем ломать пайплайны...
DS
Тем, что чёрт ногу сломит в большом числе штук, которые выглядят одинаково.
Я уберу возможность передавать объект в массиве, будет тогда только через кложу для специальных случаев
А
Я уберу возможность передавать объект в массиве, будет тогда только через кложу для специальных случаев
будет так:
$wrap(new Instance(123), 'method'):ClosureAM
Я уберу возможность передавать объект в массиве, будет тогда только через кложу для специальных случаев
Какая closure? Я про [new WebResponseFormatter(new JsonResponseFormatter(), true), 'process'] - вот это...
DS
Какая closure? Я про [new WebResponseFormatter(new JsonResponseFormatter(), true), 'process'] - вот это...
Именно, запушил в демо
А
зачем ты костыли демонстрируешь?
AM
Именно, запушил в демо
жесть какая
А
удали и претензия @samdark'а исчезнет (правда проблема останется, но никто не узнает)
DS
будет так:
$wrap(new Instance(123), 'method'):ClosureОбъект будет создаватся после матча роута только для этого роута. Если у тебя 500 роутов и ты во все передаешь инстансы, то у тебя каждые раз будет создаваться 500 инстансов. С кложей - только один.
DS
жесть какая
Правильно, это редкий случай, оно оправдано
А
Объект будет создаватся после матча роута только для этого роута. Если у тебя 500 роутов и ты во все передаешь инстансы, то у тебя каждые раз будет создаваться 500 инстансов. С кложей - только один.
не у всех 500 роутов, а у кого 500, тот сам знает как лучше. Особенно если по перформансу подожмёт
AM
Правильно, это редкий случай, оно оправдано
Ну я не вижу нафига вырываться из middleware-стэка... почему не убрать "лишний" middleware с группы?
DS
Ну я не вижу нафига вырываться из middleware-стэка... почему не убрать "лишний" middleware с группы?
Ок, я удалю эту возможность. Посмотрим как скоро создадут ишью после релиза.
AM
хорошо