AM
Это потенциально может выявить фатальный недостаток.
Size: a a a
2020 April 01
AM
Что-то ещё подбивает прибраться... Может всё что касается DataResponse сунуть в соответствующий namespace?
AM
Именование меня начало устраивать.
DS
Именование меня начало устраивать.
Ты поломал что-то. Или нет, ща попереименовываю все в демке и скажу
AM
тесты проходят
AM
если поломал, то в них этого не было
AM
Что насчёт "Может всё что касается DataResponse сунуть в соответствующий namespace"?
DS
Что насчёт "Может всё что касается DataResponse сунуть в соответствующий namespace"?
Это вообще отдельным пакетом можно сделать было бы, оно в принципе и вне фреймворка может использоваться
DS
если поломал, то в них этого не было
Норм все, обновил демо
DS
Именование меня начало устраивать.
Форматтеры не будешь переименовывать?
DS
Именование меня начало устраивать.
Смерж этот пока Алексей спит😉
https://github.com/yiisoft/router/pull/45
https://github.com/yiisoft/router/pull/45
AM
Смерж этот пока Алексей спит😉
https://github.com/yiisoft/router/pull/45
https://github.com/yiisoft/router/pull/45
Только за обновление README ;)
AM
Это вообще отдельным пакетом можно сделать было бы, оно в принципе и вне фреймворка может использоваться
Сейчас тогда ещё попереименовываю. На этот раз namespace. Извиняюсь, что надо после этого править демку, могу сам потом... или сразу.
DS
Сейчас тогда ещё попереименовываю. На этот раз namespace. Извиняюсь, что надо после этого править демку, могу сам потом... или сразу.
Давай потом сразу, я пока ридми поправлю
AM
потом сразу? :)
DS
потом сразу? :)
потом за один раз я поправлю)
DS
Поправил ридми, глянь еще там внизу, где генерация урл, полноценный пример, его наверное вверх нужно перекинуть.
DS
Кажется, мы сделали https://docs.laminas.dev/laminas-diactoros/v2/custom-responses/ только интересней.
Глянул - однозначно у нас лучше)
DS
Легко. Тысячи - это об инстансах а не о роутах. Один инстанс мидлтвари может тянуть из контейнере еще полдесятка зависимостей. В итоге создается полдесятка инстансов. И так в двух сотнях роутов, вот тебе и тысяча)
AM
Так. Отставить про производительность :) Без Blackfire холивар продолжать нельзя ;)