А
есть другие вкусняшки
отревьювить https://github.com/yiisoft/yii-cycle/pull/17
отревьювить https://github.com/yiisoft/yii-cycle/pull/17
поздно )
Size: a a a
2020 February 05
AM
Там ошибочное исправление в цикле, его не нужно применять
разве ошибочное?
DS
Да, поменяй эту строку у себя и запусти тесты в пакете router
А
а у нас база под виджеты уже готова?
AM
DS
разве ошибочное?
Ну и кроме того, я все равно уберу этот цикл завтра.
А
роутеры в мастере работают? можно обновляться? :)
AM
Ну и кроме того, я все равно уберу этот цикл завтра.
хм, да... откатил то изменение
AM
выпилил нерабочее, влил рабочее
AM
роутеры в мастере работают? можно обновляться? :)
да, норм работает
DS
Как лучше обозвать класс хелпера в роутере?
AM
а что он делает?
AM
@yiiliveext https://github.com/yiisoft/di/pull/100 ещё актуален, кстати? Фабрику саму по себе мы вроде причесали...
DS
Из текстового массива роутов создает массив объектов (Route и Group) для RouterFactory или корневую круппу для FastRoute
DS
@yiiliveext https://github.com/yiisoft/di/pull/100 ещё актуален, кстати? Фабрику саму по себе мы вроде причесали...
Не, можно убирать. У меня есть новая реализация фабрики.
RM
Там ошибочное исправление в цикле, его не нужно применять
это было предложение PHP Inspections :(
DS
Предлагаю убрать старый синтаксис для групп вроде
['/v1', function (Group $g) use ($container) {
$g->addGroup('/blog', function (Group $g) use ($container) {
$g->addRoute(Route::get('/test', new ActionCaller(SiteController::class, 'test2', $container))
->name('v1-blog-test'));
});
А оставить два вида синтаксиса. Один - массив с объектами Route и Group, а другой - объект корневой группы, в которую по иерархии добавляются роуты и группы.
['/v1', function (Group $g) use ($container) {
$g->addGroup('/blog', function (Group $g) use ($container) {
$g->addRoute(Route::get('/test', new ActionCaller(SiteController::class, 'test2', $container))
->name('v1-blog-test'));
});
А оставить два вида синтаксиса. Один - массив с объектами Route и Group, а другой - объект корневой группы, в которую по иерархии добавляются роуты и группы.
DS
Первый вариант удобнее, второй - быстрее.
AM
Предлагаю убрать старый синтаксис для групп вроде
['/v1', function (Group $g) use ($container) {
$g->addGroup('/blog', function (Group $g) use ($container) {
$g->addRoute(Route::get('/test', new ActionCaller(SiteController::class, 'test2', $container))
->name('v1-blog-test'));
});
А оставить два вида синтаксиса. Один - массив с объектами Route и Group, а другой - объект корневой группы, в которую по иерархии добавляются роуты и группы.
['/v1', function (Group $g) use ($container) {
$g->addGroup('/blog', function (Group $g) use ($container) {
$g->addRoute(Route::get('/test', new ActionCaller(SiteController::class, 'test2', $container))
->name('v1-blog-test'));
});
А оставить два вида синтаксиса. Один - массив с объектами Route и Group, а другой - объект корневой группы, в которую по иерархии добавляются роуты и группы.
ОК. Учитывая как всё у нас мёржится конфиг-плагином, это плюс будет.