Крон: фикс

[psnet]

https://github.com/livestreet/livestreet-framework/blob/master/classes/modules/cron/entity/Task.entity.class.php#L87

зачем?

$this->setState($this->getState() == ModuleCron::TASK_STATE_ACTIVE ? ModuleCron::TASK_STATE_ACTIVE : ModuleCron::TASK_STATE_NOT_ACTIVE);

[mzhelskiy]

что именно зачем?

[psnet]

Зачем писать такую конструкцию? Она дублирует сама себя

[mzhelskiy]

Как нужно написать?

[psnet]

Если getState и так 1 или 0, которые и есть константы

[mzhelskiy]

Если с формы передать state = 2 ?

[psnet]

А почему тогда не через встроенный валидатор enum? Просто с ним будет яснее чем с таким кодом инлайнового валидатора.