А
Тебе бы в симфони контрибьютить)
если даже здесь мои наркоманские идеи не приживаются, то там то и подавно...
Size: a a a
2020 April 23
a
да, теперь там два огромных метода
Уважаю) выпили это по репкам, зафигачь множество сервисов, и побей их на команды) а вообще правильный подход
DS
если даже здесь мои наркоманские идеи не приживаются, то там то и подавно...
Вот буквально недавно прикидывал адаптер под симфониевский роутер. Как тебе метод на 150 строк?🤨
https://github.com/symfony/routing/blob/master/Generator/UrlGenerator.php#L170
https://github.com/symfony/routing/blob/master/Generator/UrlGenerator.php#L170
А
Вот буквально недавно прикидывал адаптер под симфониевский роутер. Как тебе метод на 150 строк?🤨
https://github.com/symfony/routing/blob/master/Generator/UrlGenerator.php#L170
https://github.com/symfony/routing/blob/master/Generator/UrlGenerator.php#L170
да там комментариев много просто
А
Вот буквально недавно прикидывал адаптер под симфониевский роутер. Как тебе метод на 150 строк?🤨
https://github.com/symfony/routing/blob/master/Generator/UrlGenerator.php#L170
https://github.com/symfony/routing/blob/master/Generator/UrlGenerator.php#L170
а прикинь, если бы они ещё строки умели переносить
a
да там комментариев много просто
Та нет. Это откровенно говно. Я топлю за симфу. Что такое в массиве 5й элемент (земля, вода, воздух, огонь)?
a
Не делайте так
AM
Ну, тут не всё так однозначно.
AM
Если blame сделать, будет Grekas?
AM
Если да, то есть вероятность что он проходил по этому профайлером.
AM
И так сделано сознательно.
a
Если да, то есть вероятность что он проходил по этому профайлером.
Если это код от Николаса (уже запутался где он) то очень не похоже
a
Вот буквально недавно прикидывал адаптер под симфониевский роутер. Как тебе метод на 150 строк?🤨
https://github.com/symfony/routing/blob/master/Generator/UrlGenerator.php#L170
https://github.com/symfony/routing/blob/master/Generator/UrlGenerator.php#L170
Там даже psr1 нет. Это нечитабельно
DS
Если да, то есть вероятность что он проходил по этому профайлером.
a
Ну вот не делайте так. Ладно на проекте рефакторинг под 15 попугаев, тут не нужно
2020 April 24
RT
@romkatsu
> у я думаю можно FilterableDataInterface и SortableDataInterface реализовать в коллекции.
Трейтами?
> DataWriterInterface - непонятно для чего может пригодится.
Данные писать: экспорт, редактирование грида.
> Мне кажется это должны быть 2 разные реализации.
Или так или трейты.
> меня смущает DEFAULT_LIMIT
@roxblnfk верно сказал. Он для того, чтобы умолчания выставлять единообразно.
> в KeysetPaginator не типизированы свойства
Косяк. issue заведёшь?
> Можно сделать коллекцию, чтобы покрывала 2 частных случая CoockieCollection в yii-web и rbac. И посмотреть что получится на выходе.
Я за.
> у я думаю можно FilterableDataInterface и SortableDataInterface реализовать в коллекции.
Трейтами?
> DataWriterInterface - непонятно для чего может пригодится.
Данные писать: экспорт, редактирование грида.
> Мне кажется это должны быть 2 разные реализации.
Или так или трейты.
> меня смущает DEFAULT_LIMIT
@roxblnfk верно сказал. Он для того, чтобы умолчания выставлять единообразно.
> в KeysetPaginator не типизированы свойства
Косяк. issue заведёшь?
> Можно сделать коллекцию, чтобы покрывала 2 частных случая CoockieCollection в yii-web и rbac. И посмотреть что получится на выходе.
Я за.
> Трейтами?
Как вариант
> Косяк. issue заведёшь?
вроде завели один и на тесты и на типы https://github.com/yiisoft/data/issues/35
> Я за.
Тогда займусь.
Как вариант
> Косяк. issue заведёшь?
вроде завели один и на тесты и на типы https://github.com/yiisoft/data/issues/35
> Я за.
Тогда займусь.
Д
Когда там уже пр с мидлваром по авторизации закроете?)
AM
По remember me?
Д
По авторизации
AM
Который?