AM
ОК, ясно.
Size: a a a
2020 August 24
DS
Александр Некрасов
я класс не беру из контейнера, как контейнер повлияет ?
Ты константу берешь, а она только дефолтное значение, его можно изменить при конфигурировании
АН
Ты константу берешь, а она только дефолтное значение, его можно изменить при конфигурировании
хмм, в принципе да, сломать можно )
АН
ну и $request->getAttribute('_csrf') можно так сломать
DS
Александр Некрасов
ну и $request->getAttribute('_csrf') можно так сломать
Да, можно, никто ж не спорит. Но при замене реализации этот вариант более живучий, я просто пишу в тот же атрибут
А
@viperawa если не можешь выделить время на тот PR yii-cycle, то напиши, ктонить другой переведёт
Д
А
куда?
А
я чёт провтыкал?
DS
куда?
В бочку с пивом)
АН
Да, можно, никто ж не спорит. Но при замене реализации этот вариант более живучий, я просто пишу в тот же атрибут
ну если меняется название атрибута, то придется везде это изменить, как вариянт создать свой класс с константами
АН
В бочку с пивом)
🤣🤣🤣
А
точно, провтыкал 😐 сорян )
AM
Я за то, чтобы не давать менять.
AM
К ночи надеюсь закончу причёсывать auth и сделаю жирный pull request.
DS
ОК, ясно.
Для фреймворка я бы подумал за отдельный атрибут и все в нем передавал
withAttribute('@yiiframework', ['_csrf' => 'irj440938rj']
withAttribute('@system.core', ['_csrf' => 'irj440938rj']AM
По мне так это лишняя конфигурация.
AM
Не ясно ради чего.
AM
Раз конфликта не будет при ::class, то и конфижить незачем.
AM
Кейс @roxblnfk валидный, но слишком редкий.