Size: a a a

2020 February 14

AM

Alexander Makarov in Yii Framework 3
ну и хорошо
источник

DS

Dmitriy S in Yii Framework 3
Alexander Makarov
ну тогда норм, только есть всё-таки подозрение что можно будет убить сборный Router
Глянул код сегодня, в адаптерах вроде фастроута он вообще не нужен, поскольку он не работает с особенностями того или иного роутера и  будет одинаковым для всех адаптеров. Если нужен класс, который реализует сборный RouterInterface, то его нужно создать в пакете router.
источник

AM

Alexander Makarov in Yii Framework 3
А он нужен?
источник

AM

Alexander Makarov in Yii Framework 3
По мне так нет...
источник

DS

Dmitriy S in Yii Framework 3
Alexander Makarov
По мне так нет...
Разве что для удобства
источник

DS

Dmitriy S in Yii Framework 3
Можно сделать, но в той же демке не биндить в конфиге. Кому надо - сам добавит.
источник

Д

Дмитрий in Yii Framework 3
Alexander Makarov
А он нужен?
Уже нет
источник

DS

Dmitriy S in Yii Framework 3
В общем я делаю в этом же PR, да?
источник

AM

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

AM

Alexander Makarov in Yii Framework 3
давай грохнем лишнее
источник

AM

Alexander Makarov in Yii Framework 3
оно только смущает
источник

AM

Alexander Makarov in Yii Framework 3
никакого удобства по факту не даёт
источник

DS

Dmitriy S in Yii Framework 3
ok
источник

AM

Alexander Makarov in Yii Framework 3
Я верно понимаю что https://github.com/yiisoft/router/pull/29 станет после этого неактуален?
источник

DS

Dmitriy S in Yii Framework 3
Я прямо в него собираюсь добавить все
источник

DS

Dmitriy S in Yii Framework 3
Только отредактирую описание
источник

AM

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

AM

Alexander Makarov in Yii Framework 3
@roxblnfk по https://github.com/yiisoft/yii-demo/pull/49 осталась мелочёвка 👍
источник

DS

Dmitriy S in Yii Framework 3
затупил, этот да, наверное неактуален, а добавлю я в свой
хотя может и пригодится частично
источник

AM

Alexander Makarov in Yii Framework 3
ок, пока пусть будет
источник