Size: a a a

2020 February 09

DS

Dmitriy S in Yii Framework 3
Alexander Makarov
Да сделайте уже git bisect...
Им лень, проще в группу написать)
источник

AM

Alexander Makarov in Yii Framework 3
Ну так-то мы все ленивые по-своему 🙂 Это нормально. Но может кто никогда bisect не пробовал. Вот повод есть теперь...
источник

Д

Дмитрий in Yii Framework 3
Dmitriy S
Тесты, что ты написал в целом норм, хотя есть моменты, но пока лучше повременить с этим PR, я сейчас меняю немного архитектуру.
А что не так с архитектурой роутера?
источник

AM

Alexander Makarov in Yii Framework 3
Можно лучше.
источник

AM

Alexander Makarov in Yii Framework 3
В этом случае имеет смысл поменять сразу потому что затрагивает интерфейсы.
источник

Д

Дмитрий in Yii Framework 3
Разделить роутер интерфейс?
источник

Д

Дмитрий in Yii Framework 3
Раз менять надумали, то давай запретим создание группы через new и сделаем всё через метод create, а так же запретить null как значение префикса в аргументе
источник

AM

Alexander Makarov in Yii Framework 3
@yiiliveext норм вроде идеи. Запилишь заодно или @xepozz оставишь?
источник

Д

Дмитрий in Yii Framework 3
Пусть делает :)
источник

DS

Dmitriy S in Yii Framework 3
Alexander Makarov
@yiiliveext норм вроде идеи. Запилишь заодно или @xepozz оставишь?
Не спешите, у меня уже все продумано. Но выкладывать буду по частям. Сегодня я  отдыхаю, а вот завтра доделаю и сделаю ПР с вынесенным генератором. Потом напишу суть дальнейших изменений.
источник

DS

Dmitriy S in Yii Framework 3
Дмитрий
Раз менять надумали, то давай запретим создание группы через new и сделаем всё через метод create, а так же запретить null как значение префикса в аргументе
null там нужен для корневой группы
источник

AM

Alexander Makarov in Yii Framework 3
null или пустая строка?
источник

DS

Dmitriy S in Yii Framework 3
Alexander Makarov
null или пустая строка?
фастроуту пофиг, для него это одно и то же
источник

DS

Dmitriy S in Yii Framework 3
Лучше бы по префиксам имен маршрутов высказались, это важная вещь
источник

А

Алексей R in Yii Framework 3
@xepozz , зачем null пользователь там, где он обязан быть?
источник

Д

Дмитрий in Yii Framework 3
Dmitriy S
Лучше бы по префиксам имен маршрутов высказались, это важная вещь
Как вариант делать матчинг с генерацию с возможностью указать группу опционально. По умолчанию использовать какую-нибудь системную. Идея прикольная, только хз будет ли удобно
источник

Д

Дмитрий in Yii Framework 3
Алексей R
@xepozz , зачем null пользователь там, где он обязан быть?
Ну дык в конструкторе нет юзера. Нет конструктора - нет стабильности :)
источник

AM

Alexander Makarov in Yii Framework 3
Dmitriy S
Лучше бы по префиксам имен маршрутов высказались, это важная вещь
потенциально удобная для разруливания конфликтов имён
источник

AM

Alexander Makarov in Yii Framework 3
но можно решить по-разному
источник
2020 February 10

AM

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