Size: a a a

2020 March 30

А

Алексей R in Yii Framework 3
Dmitriy S
Да, там много недостатков. Завязка на ActionCaller (привязываем форматтер к роутеру). Стрим конвертер который управляет риспонсом. Это из разряда хвост виляет собакой. Дполнительный рендерер мидлтварь, которую всегда нужно задавать. Нет ленивого рендеринга в эмиттере. Нет возможности форматировать после чсистой мидлтвари в экшине. Еще что-то было но я уже не упомню все.
как же с тобой не просто ) ActionCaller пришлось переписать т.к. в роутере плохое решение; ленивый рендеринг - не цель и уже решено, что не нужен. Про заголовки конвертера в сопроводительном письме я сразу написал. Мидлварь на рендеринг - нормально, во много раз лучше, чем рендеринг и привязка заголовка по getBody() ответа (плюс сам решаешь, когда рендерить). И не рендеринг, а конвертирование данных в нужный формат )
источник

DS

Dmitriy S in Yii Framework 3
Алексей R
как же с тобой не просто ) ActionCaller пришлось переписать т.к. в роутере плохое решение; ленивый рендеринг - не цель и уже решено, что не нужен. Про заголовки конвертера в сопроводительном письме я сразу написал. Мидлварь на рендеринг - нормально, во много раз лучше, чем рендеринг и привязка заголовка по getBody() ответа (плюс сам решаешь, когда рендерить). И не рендеринг, а конвертирование данных в нужный формат )
Мне лень тебе по пятому кругу отвечать. Сэмдарк посмотрит - пусть скажет, что он думает.
источник

DS

Dmitriy S in Yii Framework 3
Я там кстати на форматтеры базовые тесты добаввил в yii-web
источник

А

Алексей R in Yii Framework 3
тот пр в yii-web очень сырой, надо больше тестов
источник

А

Алексей R in Yii Framework 3
особенно по части манипулирования декоратором запроса.
источник

DS

Dmitriy S in Yii Framework 3
Alexander Makarov
Да, можно.
Готово, запушил
источник

DS

Dmitriy S in Yii Framework 3
Алексей R
тот пр в yii-web очень сырой, надо больше тестов
Сырой и больше тестов - это разные вещи. Я его на реальном приложении тестил с кучей кейсов. Тесты еще добавлю, конечно.
источник

А

Алексей R in Yii Framework 3
как скажешь 😐
источник

DS

Dmitriy S in Yii Framework 3
Алексей R
ему не нравится. Он для этого даже ActonCaller удалил из роутера :D
Ну так он там ошибочно был, это внутрення кухня мидлтвари Route, он не нужен снаружи. Если б так изначально было, у вас бы с Дмитрием грешных мыслей не возникало)
источник

А

Алексей R in Yii Framework 3
Dmitriy S
Ну так он там ошибочно был, это внутрення кухня мидлтвари Route, он не нужен снаружи. Если б так изначально было, у вас бы с Дмитрием грешных мыслей не возникало)
а какая разница? тот же код переместился внутрь. Был снаружи )
источник

А

Алексей R in Yii Framework 3
ты ожидаешь от экшена/контроллера Response объект, хотя эта строгость ничем не оправдана
источник

DS

Dmitriy S in Yii Framework 3
Алексей R
ты ожидаешь от экшена/контроллера Response объект, хотя эта строгость ничем не оправдана
Я ожидаю, что можно форматировать риспонс после любой мидлтвари в качестве экшина, в том числе и чистой мидлтвари, реалиующей соответсвующий интерфейс. У тебя же работает только с оберткой вокруг экшина.
источник

А

Алексей R in Yii Framework 3
Dmitriy S
Я ожидаю, что можно форматировать риспонс после любой мидлтвари в качестве экшина, в том числе и чистой мидлтвари, реалиующей соответсвующий интерфейс. У тебя же работает только с оберткой вокруг экшина.
это ожидание обеспечивает интерфейс
источник

А

Алексей R in Yii Framework 3
если  у тебя внутренний ActionCaller стал боследним в пайплайне, то это он должен вернуть Response, а не Action
источник

DS

Dmitriy S in Yii Framework 3
Алексей R
это ожидание обеспечивает интерфейс
Route::get('/blog', MyMiddleware::class);
Как будешь форматировать своим решением?
источник

А

Алексей R in Yii Framework 3
Dmitriy S
Route::get('/blog', MyMiddleware::class);
Как будешь форматировать своим решением?
речь не о моём решении. И ты кое что упустил
источник

DS

Dmitriy S in Yii Framework 3
Алексей R
речь не о моём решении. И ты кое что упустил
По-любому что-то упустил, делай ревью.
источник

А

Алексей R in Yii Framework 3
коммит уже видел. вот это ты упустил 😐
источник

AM

Alexander Makarov in Yii Framework 3
источник

А

Алексей R in Yii Framework 3
перед тем, как пилить этот код, ты написал что-то вроде того, что lazy не нужен. А теперь он стал заботить? :)
источник