AM
Да:
RenderDataStream
SetStreamConverter
RenderDataStream
SetStreamConverter
Size: a a a
2020 March 30
А
Он за рендерер говорит. У тебя для форматирования задействованы две мидлтвари
вторая опциональная. Первая определяет место, где переопределение формата уже невозможно и применяется конвертер
А
мне показалось это решение лучше, чем если засовывать форматтер/ковертер в стрим
А
более прозрачное и понятное
А
и консервативное )
AM
@samdark, пофиксил баг с билдом роутов непосредственно при получении урлматчера зависимостью
https://github.com/yiisoft/router/pull/42
https://github.com/yiisoft/router/pull/42
Тесты упали.
DS
Тесты упали.
А блин, форспушнул, забыл тест исправить еще\
DS
Тесты упали.
Исправил
AM
@roxblnfk ок, допустим это ещё норм. Теперь про конвертер. У нас получается что используемый конвертер по факту должен и стрим модифицировать и заголовки добавить. Заголовки добавляются в респонс, а не в стрим. RenderDataStream работает фактически тоже с респонсом, а не со стримом. Стоит ли нам показывать юзеру столько деталей про стрим, если по факту мы работаем на уровень выше?
А
> и заголовки добавить
я разве не запушил, изменения, что крнвертер не работает с заголовками?
я разве не запушил, изменения, что крнвертер не работает с заголовками?
AM
Запушил, но необходимость не прошла.
AM
То есть как-бы странно будет назначить но группу роутов JSON, но не получить заголовок в итоге и выставлять его руками.
А
тип данных неразлучен с данными. Если мидлварь хочет добавить заголовки - она добавит и разрешение у стрима ен спросит
А
То есть как-бы странно будет назначить но группу роутов JSON, но не получить заголовок в итоге и выставлять его руками.
какие-то странные заключения
AM
Ну вот, собственно, да. Мы приходим к тому, что стримы - это деталь реализации, а реализация очень сходна с тем, что у @yiiliveext. По факту формат определяет middleware и конвертит она же. Реализация внутри отличается только. У тебя больше технических деталей торчит наружу.
AM
И профита я не вижу, если честно :(
А
ну берите реализацию Дмитрия, если нравится. Я всё-равно буду при мнении, что формат не должен быть привязан к респонсу
А
у респонса можно поменять стрим, а что станет с форматом?
DS
Давайте я подожду с часик, а вы определитесь с решением. Если с моим ок, то я допишу еще полдесятка тестов и смержим. Время не стоит на сесте.
DS
ну берите реализацию Дмитрия, если нравится. Я всё-равно буду при мнении, что формат не должен быть привязан к респонсу
А к чему он должен быть привязан, если форматируется как раз риспонс?