Size: a a a

2021 July 15

KS

Konstantine Shtikov in symfony
да я расширил, не меняя логику старого функционала) помоему контекст и так понятен, что будет проверять все аргументы)
источник

KS

Konstantine Shtikov in symfony
👍🏿
источник

OK

Oleg Krasavin in symfony
Сие чудо выглядит слабо полезным, если честно.

На мой взгляд, если и делать, то добавляя новый HasAll() как сказали выше, не трогая текущее апи.
источник

KS

Konstantine Shtikov in symfony
а че дизлайкнул то?)
источник

KS

Konstantine Shtikov in symfony
можно аргументы?)
источник

AK

Alexander Kompaneyts... in symfony
Single responsibilty
источник

KS

Konstantine Shtikov in symfony
в плане не пойму чем будет лучше новый метод
источник

СВ

Сергей Вершинин... in symfony
потому что это похоже на историю с is_granted() с массивом ролей, где точно не ясно, как будет работать в режиме and или or
источник

KS

Konstantine Shtikov in symfony
это же параметрбаг, принимает параметры из запроса, зачем вообще на or проверять?)
источник

MM

Maxim Mesilov in symfony
Вы работу не ищите? Нам нужны адекватные разработчики хD
источник

VK

Vladyslav Kopaihorod... in symfony
ну спрашивать замудреное апи фреймворка или либы на собесе это ж дичь)
источник

VK

Vladyslav Kopaihorod... in symfony
очевидно что интервьювер лопух
источник

KS

Konstantine Shtikov in symfony
так я и не наршаю это своими изменениями, у этого метода своя ответственность: проверить наличие параметра по ключу, добавив несколько аргументов, он проверит все ключи
источник

OK

Oleg Krasavin in symfony
В вендорных штуках код добавить всегда проще, чем менять текущий. Потому что BC,  дока и прочее(если это паблик апи какое-то).

Вот, например твои изменения меняют сигнатуру метода, теперь туда можно инт передать или еще чего.
источник

KS

Konstantine Shtikov in symfony
Ну тут ты прав, согласен, надо подумать о строгой типизации
источник

OK

Oleg Krasavin in symfony
Плюс еще куча анрилейтед мусора с \. Такие фиксы лучше делать отдельным пр, чтобы не усложнять жизнь ревьюверам.
источник

✨Basic_Instinct✨ in symfony
в has - типизацию?
источник

KS

Konstantine Shtikov in symfony
с этим согласен
источник

KS

Konstantine Shtikov in symfony
хотел пиплайн починить
источник

OK

Oleg Krasavin in symfony
Ну и про то, что теперь неочевидно семантически что метод делает без ныряния в код: AND или OR уже выше написали.
источник