Д
можно просто сделать экшен, который называется
beforeActionhttps://github.com/yiisoft/yii-demo/pull/73
это можно посмотреть?
Size: a a a
2020 March 28
NO
можно просто сделать экшен, который называется
beforeActionЯ не в одном контроллере делаю, а для всего приложения. Для мониторинга времени выполнения экшена через таймеры пинбы )
Мне удобнее в бутстрапе на события хендлеры прицепить. Код приложения не меняется )
Мне удобнее в бутстрапе на события хендлеры прицепить. Код приложения не меняется )
AS
В бутстрапе ж нет инстанса приложения
AS
Надо отдельный бехавиор делать
AS
И в конфиге к аппу цеплять
Д
видимо, мне не развиделось.
@yiiliveext видел мой PR? там просто возвращаем что угодно из контроллера и оно потом конвертится. сейчас твой ПР не делает ничего, что достойно того подхода, который сделал я.
из минусов:
1. свой респонс. кому он нужен? никто не будет это использовать, кроме малого круга лиц. кто-то будет использовать чужие реализации респонса - nyholm, например.
решение: подружить форматтер с интерфейсом, а не с кастомной реализацией.
2. под каждый экшен создавать инстанс форматтера - очень накладно.
решение: стейтлес мидлвар.
3. ну и по мелочи: файлы не структурированы, нет стриктов (@roxblnfk привет), нет тестов.
я бы советовал слить моё творение и поверх него сделать форматтер с кастомным конфигурированием.
опрос: что удобнее?
1.
2.
@yiiliveext видел мой PR? там просто возвращаем что угодно из контроллера и оно потом конвертится. сейчас твой ПР не делает ничего, что достойно того подхода, который сделал я.
из минусов:
1. свой респонс. кому он нужен? никто не будет это использовать, кроме малого круга лиц. кто-то будет использовать чужие реализации респонса - nyholm, например.
решение: подружить форматтер с интерфейсом, а не с кастомной реализацией.
2. под каждый экшен создавать инстанс форматтера - очень накладно.
решение: стейтлес мидлвар.
3. ну и по мелочи: файлы не структурированы, нет стриктов (@roxblnfk привет), нет тестов.
я бы советовал слить моё творение и поверх него сделать форматтер с кастомным конфигурированием.
опрос: что удобнее?
1.
return $this->responseFactory->createResponse()->withData($items); (и еще нужно заинъектить ResponseFactory)2.
return $items; (ничего не нужно инъектить)AM
@xepozz там немного не так:
1. Свой респос - это обёртка над не своим. Оборачивает любой кастомный.
2. Экшн за один запрос выполняется один. Один new - не накладно.
3. Это proof of concept.
Твой тоже надо посмотреть. Знаю, что он был раньше. Я обязательно до него доберусь. Скорее всего сегодня.
1. Свой респос - это обёртка над не своим. Оборачивает любой кастомный.
2. Экшн за один запрос выполняется один. Один new - не накладно.
3. Это proof of concept.
Твой тоже надо посмотреть. Знаю, что он был раньше. Я обязательно до него доберусь. Скорее всего сегодня.
А
видимо, мне не развиделось.
@yiiliveext видел мой PR? там просто возвращаем что угодно из контроллера и оно потом конвертится. сейчас твой ПР не делает ничего, что достойно того подхода, который сделал я.
из минусов:
1. свой респонс. кому он нужен? никто не будет это использовать, кроме малого круга лиц. кто-то будет использовать чужие реализации респонса - nyholm, например.
решение: подружить форматтер с интерфейсом, а не с кастомной реализацией.
2. под каждый экшен создавать инстанс форматтера - очень накладно.
решение: стейтлес мидлвар.
3. ну и по мелочи: файлы не структурированы, нет стриктов (@roxblnfk привет), нет тестов.
я бы советовал слить моё творение и поверх него сделать форматтер с кастомным конфигурированием.
опрос: что удобнее?
1.
2.
@yiiliveext видел мой PR? там просто возвращаем что угодно из контроллера и оно потом конвертится. сейчас твой ПР не делает ничего, что достойно того подхода, который сделал я.
из минусов:
1. свой респонс. кому он нужен? никто не будет это использовать, кроме малого круга лиц. кто-то будет использовать чужие реализации респонса - nyholm, например.
решение: подружить форматтер с интерфейсом, а не с кастомной реализацией.
2. под каждый экшен создавать инстанс форматтера - очень накладно.
решение: стейтлес мидлвар.
3. ну и по мелочи: файлы не структурированы, нет стриктов (@roxblnfk привет), нет тестов.
я бы советовал слить моё творение и поверх него сделать форматтер с кастомным конфигурированием.
опрос: что удобнее?
1.
return $this->responseFactory->createResponse()->withData($items); (и еще нужно заинъектить ResponseFactory)2.
return $items; (ничего не нужно инъектить)привет
DS
видимо, мне не развиделось.
@yiiliveext видел мой PR? там просто возвращаем что угодно из контроллера и оно потом конвертится. сейчас твой ПР не делает ничего, что достойно того подхода, который сделал я.
из минусов:
1. свой респонс. кому он нужен? никто не будет это использовать, кроме малого круга лиц. кто-то будет использовать чужие реализации респонса - nyholm, например.
решение: подружить форматтер с интерфейсом, а не с кастомной реализацией.
2. под каждый экшен создавать инстанс форматтера - очень накладно.
решение: стейтлес мидлвар.
3. ну и по мелочи: файлы не структурированы, нет стриктов (@roxblnfk привет), нет тестов.
я бы советовал слить моё творение и поверх него сделать форматтер с кастомным конфигурированием.
опрос: что удобнее?
1.
2.
@yiiliveext видел мой PR? там просто возвращаем что угодно из контроллера и оно потом конвертится. сейчас твой ПР не делает ничего, что достойно того подхода, который сделал я.
из минусов:
1. свой респонс. кому он нужен? никто не будет это использовать, кроме малого круга лиц. кто-то будет использовать чужие реализации респонса - nyholm, например.
решение: подружить форматтер с интерфейсом, а не с кастомной реализацией.
2. под каждый экшен создавать инстанс форматтера - очень накладно.
решение: стейтлес мидлвар.
3. ну и по мелочи: файлы не структурированы, нет стриктов (@roxblnfk привет), нет тестов.
я бы советовал слить моё творение и поверх него сделать форматтер с кастомным конфигурированием.
опрос: что удобнее?
1.
return $this->responseFactory->createResponse()->withData($items); (и еще нужно заинъектить ResponseFactory)2.
return $items; (ничего не нужно инъектить)У форматтера есть свой интерфейс. Если ты не заметил - это твои проблемы. Остальное сэмдарк объяснил.
DS
Что-то стало хуже?
Нельзя заголовки выставить при отложенном рендериге в эмиттере. Обращение к стриму идет уже после того, как отправлны заголовки в эмиттере, соответственно форматтер отрабатывает вхолостую. Но даже если дергать любой безобидный метод стрима, то все равно заголовки не сделаются, потому как заголовки устанавливаются в клоне.
AM
То есть форматтер не может выставить content type?
AM
DS
То есть форматтер не может выставить content type?
Может но поздно. Единственный вариант отказаться от мидлтвари DeferredResponseFormatter. Но так мы много теряем.
DS
В общем для посмотреть ветка с DeferredStream вот
https://github.com/yiiliveext/yii-demo/tree/def-stream
https://github.com/yiiliveext/yii-demo/tree/def-stream
DS
В общем я еще раз обновил основной пр по форматтеру, переименовал классы и откатил некоторые изменения,@samdark глянь еще раз, да я буду переносить в yii-web
Д
У форматтера есть свой интерфейс. Если ты не заметил - это твои проблемы. Остальное сэмдарк объяснил.
Причем тут интерфейс?
Д
@xepozz там немного не так:
1. Свой респос - это обёртка над не своим. Оборачивает любой кастомный.
2. Экшн за один запрос выполняется один. Один new - не накладно.
3. Это proof of concept.
Твой тоже надо посмотреть. Знаю, что он был раньше. Я обязательно до него доберусь. Скорее всего сегодня.
1. Свой респос - это обёртка над не своим. Оборачивает любой кастомный.
2. Экшн за один запрос выполняется один. Один new - не накладно.
3. Это proof of concept.
Твой тоже надо посмотреть. Знаю, что он был раньше. Я обязательно до него доберусь. Скорее всего сегодня.
2. Создание мидваров происходит после матчинга или при бутстрапе приложения?
1. Это лишнее даже при обёртке. Нужно работать без этого. Это уже не дебагер, и некоторые вещи не нужны в проде. Например, проксирование респонса
1. Это лишнее даже при обёртке. Нужно работать без этого. Это уже не дебагер, и некоторые вещи не нужны в проде. Например, проксирование респонса
DS
2. Создание мидваров происходит после матчинга или при бутстрапе приложения?
1. Это лишнее даже при обёртке. Нужно работать без этого. Это уже не дебагер, и некоторые вещи не нужны в проде. Например, проксирование респонса
1. Это лишнее даже при обёртке. Нужно работать без этого. Это уже не дебагер, и некоторые вещи не нужны в проде. Например, проксирование респонса
2. Смотря каких. Основного группового будет после матчинга.
1. Что лишнее? Это декорирование и оно используется по назначению. Твое решение - это откровенная лажа, оно не решает вообще ничего, это временный костыль.
1. Что лишнее? Это декорирование и оно используется по назначению. Твое решение - это откровенная лажа, оно не решает вообще ничего, это временный костыль.