AM
я только за, только заранее расскажи планы и мелкими кусочками pull request-ы чтобы можно было воспринять, переварить и быстро вмёржить
Size: a a a
2020 February 09
DS
я только за, только заранее расскажи планы и мелкими кусочками pull request-ы чтобы можно было воспринять, переварить и быстро вмёржить
Дык так же ж и планирую. Дабы сразу не травмировать психику :)
DS
Не очень в том плане, что сейчас имя роута должно быть уникальным по всей коллекции. А нужно чтобы было униклаьным в пределах группы. Для этого нужен префикс имени для группы. Он либо делается автоматом из обычного префикса, либо надо вводить в группу дополнительное свойство, которое можно будет устанавливать опционально (по дефолту авто режим).
А
думаешь так удобно будет? имя по логике должно быть уникальным среди всех маршрутов
DS
думаешь так удобно будет? имя по логике должно быть уникальным среди всех маршрутов
Это пока у тебя модулей нет, то еще возможно. А с модулями угадать - проблемно будет.
А
тык это группа для имен а не для маршрутов
DS
тык это группа для имен а не для маршрутов
Это о чем?
А
в смысле если и группировать, то не по группам маршрутов
А
а вводить отдельные группы на имена
DS
Группа одна, префиксы могут быть разные, я выше писал.
DS
При желании можно еще режим без префикса сделать, будет как сейчас, только ошибку генерить при совпадении имени.
AM
А что мешает выбрать одинаковые префиксы в двух модулях?
А
https://github.com/yiisoft/yii-demo/pull/49 го ревьювить )
DS
А что мешает выбрать одинаковые префиксы в двух модулях?
Модули будут загружаться каждый в свою группу с префиксом по имени/айди модуля по дефолту. Эти имена/префиксы можно будет мапить на случай совпадения. Но это я так делаю в автозагрузчике.
AM
https://github.com/yiisoft/yii-demo/pull/49 го ревьювить )
Я чуть позже. После поезда отдохнуть надо... Спал мало, норм ревью не выйдет так.
AM
Модули будут загружаться каждый в свою группу с префиксом по имени/айди модуля по дефолту. Эти имена/префиксы можно будет мапить на случай совпадения. Но это я так делаю в автозагрузчике.
Хм... Ну логично вообще, да.
А
я там чёт не могу отправить запрос на ревью группе
yiisoft/reviewersА
@thenotsoft присоединяйся тоже, вдруг чего не понравится по части репозиториев, контроллеров и т.д. )
DS
Еще вопрос по composer-config-plugin, а зачем он загружает все пакеты, даже не конфигурируемые? Ну и можно было бы немного оптимизировать, сделать тип пакета library-configurable, тогда можно не грузить конфиг, а сразу отбросить его. Да и я свои наработки в плане автопостроения конфига мог бы портировать и использовать, как альтернативу этому плагину.
А
это какие не конфигурируемые?