АН
😂
Size: a a a
2020 February 06
AM
@yiiliveext https://github.com/yiisoft/router/pull/26 - всмотрелся ещё раз, есть моменты, которые хорошо-бы поправить. Сама идея норм.
DS
@yiiliveext https://github.com/yiisoft/router/pull/26 - всмотрелся ещё раз, есть моменты, которые хорошо-бы поправить. Сама идея норм.
Исправил
DS
Как лучше сделать генерацию аболютных URL, добавить параметр в generate или добавить отдельный метод в интерфейс?
АН
Как лучше сделать генерацию аболютных URL, добавить параметр в generate или добавить отдельный метод в интерфейс?
я за отдельный метод
А
отдельно
АН
Как лучше сделать генерацию аболютных URL, добавить параметр в generate или добавить отдельный метод в интерфейс?
ты прям своими PR-ми выручил )))
DS
Название метода generateAbsoluteUrl пойдет?
АН
Название метода generateAbsoluteUrl пойдет?
вроде норм
А
относительный же без Url?
DS
ну можно и без
DS
а можно и generateUrl() и generateAbsoluteUrl сделать
АН
без url лучше, и так $urlGenerator
DS
Как по мне, то логичнее c url, потому как у нас общий интерфейс роутера, и мы инжектим как RouterInterface $router и выходит $router->generate()
АН
вынести бы этот генератор отдельно, было бы хорошо
DS
Этоесли инжектить как UrlGeneratorInterface то тогда да
DS
Александр Некрасов
вынести бы этот генератор отдельно, было бы хорошо
Тогда надо делать три отдельных класса наверное
АН
как будет больше методов, то начнется каша
АН
и неясно, что откуда и куда
А
посмотри возможность того, чтобы роут использовал urlGenerator в методе generate