NO
короче
Size: a a a
2020 August 09
NO
боль (
RT
Но это не значит что решение норм:
1. Его не понимают.
2. Оно форсит наследование.
1. Его не понимают.
2. Оно форсит наследование.
1. Согласен, не понятно. Вот так было бы на много понятнее.
RT
final class Provider implements ListenerProviderInterface
{
public function attach(callable $listener, string $eventClassName = ''): void
{
if ($this->applicationState->isRunning()) {
throw new \Exception('you can\'t configure it because the app is running');
}
...
}RT
Рассказывал коллегам, тоже не поняли, большинство даже не знают про функционал дружественных классов. А когда узнают, не горят желанием использовать.
NO
магия
AM
боль (
Умгу. Не переделаю никак. Бегает на Yii 1.1 и вёрстка по-старинке float-ами :)
AM
final class Provider implements ListenerProviderInterface
{
public function attach(callable $listener, string $eventClassName = ''): void
{
if ($this->applicationState->isRunning()) {
throw new \Exception('you can\'t configure it because the app is running');
}
...
}Хм... а может сделать у провайдера метод аля lock()?
AM
И типа когда залочен, то всё?
RT
или так да
RT
при запуске лочить
AM
@yiiliveext как тебе?
RT
ну или везде пробрасывать applicationState
RT
где он нужен
RT
одно и тоже впринципе
AM
Цель достигнута будет + проще, не нужны конфигураторы. В сервис-провайдере в конце лок вызвать и всё.
RT
намного проще
AM
Единственное, я не знаю что там с модулями у @yiiliveext. Давно их пилит приватно.
DS
@yiiliveext как тебе?
Я тебе выше это писал, лок надо делать в конструкторе и туда же передавать сконфиженный конфиг или обычний конфиг и конфигуратор
DS
Цель достигнута будет + проще, не нужны конфигураторы. В сервис-провайдере в конце лок вызвать и всё.
Да и нафига тогда вообще публичный attach()?