DS
эм... зачем?
Есть возможность сломать приложение вызвав его в каком-то middleware до роутера. Следующие middleware не смогут добавить свои роуты.
Size: a a a
2020 February 17
DS
Вернее в коллектор роуты они добавят, а в фастроуте их не будет
DS
PR норм. Если ничего туда добавлять не планируется, солью.
Ладно, можешь сливать. Тестирование с фс пусть потом @xepozz глянет.
DS
Поправил роутер, добавил методы в интерфейс матчера, удалил RouterFactory
https://github.com/yiisoft/router/pull/33
https://github.com/yiisoft/router/pull/33
АН
было бы хорошо актуализировать README к роутам
DS
Александр Некрасов
было бы хорошо актуализировать README к роутам
У нас нет сертифицированного специалиста по актуализации доков?
AM
Как вы файловую систему мокаете? Хочу тесты на кеш добить. vfsStream можно использовать?
никак...
AM
Есть возможность сломать приложение вызвав его в каком-то middleware до роутера. Следующие middleware не смогут добавить свои роуты.
ОК, а как тогда юзать это в Router middleware?
AM
У нас нет сертифицированного специалиста по актуализации доков?
Пока нет.
DS
Сделать метод протектед и шарить его через абстрактный роутер
AM
Д
Или я опять вне контекста?)
DS
Или я опять вне контекста?)
Чем поможет интерфейс? Нам надо запретить вызов метода
match() всем классам кроме middlware класса Router.AM
хм...
AM
это потянет за собой переезд UrlMatcherInterface в абстрактный класс
AM
мне что-то не очень нравится идея
DS
Почему?
AM
ну потому что cohesion начинаем убивать в противном случае