А
хотя странно иметь protectd streamFactory через конструктор, но не использовать (пусть и в абстрактном классе)
Size: a a a
2020 March 23
А
в общем StreamFactoryInterface можно выпилить от туда и ничего не поменяется
А
например, если это будет ленивый рендеринг, то там тупо создаёшь стрим из генератора и не нужна фабрика )
А
вот JsonResponseFactory в конструкторе требует StreamFactory. Ему надо, он его юзает
Д
угу, спилил
А
в общем StreamFactoryInterface можно выпилить от туда и ничего не поменяется
что думаешь?
А
а зачем ты
$stream->rewind(); делаешь в AbstractResponseFactory ?А
я думаю это ненужное действие. Эмиттер сам перематывает на начало, если что. И нельзя так просто делать rewind, не проверив, может ли это делать стрим
А
если
$this->jsonSerializer->serialize($data); по умолчанию не кодирует UTF-8 символы, то в return 'application/json'; наверное следует параметр кодировки добавитьД
а зачем ты
$stream->rewind(); делаешь в AbstractResponseFactory ?потому что указатель стоит в конце. тесты это подтверждают
Д
я думаю это ненужное действие. Эмиттер сам перематывает на начало, если что. И нельзя так просто делать rewind, не проверив, может ли это делать стрим
если стрим не умеет rewind - это его проблема. есть интерфейс - соответствуй заложенному поведению
А
потому что указатель стоит в конце. тесты это подтверждают
это нормально
А
если стрим не умеет rewind - это его проблема. есть интерфейс - соответствуй заложенному поведению
стрим может изначально быть на нужной позиции и при этом не уметь перематываться
А
перематывать в начало или читать с установленного места - решает эмиттер
Д
если
$this->jsonSerializer->serialize($data); по умолчанию не кодирует UTF-8 символы, то в return 'application/json'; наверное следует параметр кодировки добавитьнадо бы это решать в сериалайзере
Д
перематывать в начало или читать с установленного места - решает эмиттер
что предлагаешь?
Д
в тестах отматывать?
А
да, это было бы норм
А
в тестах ты знаешь, что не юзаешь не перематываемые стримы, поэтому можешь смело перематывать в начало, если надо контент проверить )
Д
убрал