обязательное заполнение дополнительных полей / Вопросы, проблемы и их решения / Alto CMS

обязательное заполнение дополнительных полей (comment #16177)

fordistantjob@gmail.com — Tue, 04 Nov 2014 22:30:44 +0300

неужели только я столкнулся с тем что дополнительные поля нужны обязательно заполненные?

обязательное заполнение дополнительных полей (comment #16201)

braindep@gmail.com — Thu, 06 Nov 2014 03:12:47 +0300

На уровне движка есть класс validator. Не разобрался пока как это навесить на доп поля.

обязательное заполнение дополнительных полей (comment #16204)

fordistantjob@gmail.com — Thu, 06 Nov 2014 12:01:41 +0300

В общем вчера сидел над этим весь день. Там проблема немного глубже чем навесить валидатор. Движок сначала добавляет топик, а уже потом обрабатывает и добавляет дополнительные поля в Topic_processfields(). В итоге я разбил processfields() на две функции в первой только обработка (processfields) а во второй уже непосредственное добавление (addfields). Как прикрутить валидатор я тоже не разобрался поэтому сделал просто проверку на наличие реквеста и если его нет хотя бы по одному полю, то processfields возвращает false.если все ок то возвращает массив Data который потом передается в addfields. Получилось естественно велосипедисто, но хотя бы работает.

а проблема с проверкой загрузки фото в моем случае решилась просто.

if($this->oType->isAllow('photoset') && !$oTopic->getPhotosetMainPhotoId()) { 
			$this->Message_AddErrorSingle($this->Lang_Get('topic_create_error_no_main_photo'),$this->Lang_Get('error'));
			return false; 
		}

обязательное заполнение дополнительных полей (comment #16206)

vshemarov@gmail.com — Thu, 06 Nov 2014 13:13:12 +0300

Думаю, понятно, что делать это надо отдельным плагином.

И я бы cделал примерно так: создается плагин для конкретного сайта (напр., PluginMine, где переопределяется ActionContent, в котором расширяем метод checkTopicFields($oTopic):

class PluginMine_ActionContent extends PluginMine_Inherits_ActionContent
    protected function checkTopicFields($oTopic) {
        // сначала вызываем стандартную проверку
        $bResult = parent::checkTopicFields($oTopic);
        // а теперь проверяем заполнение дополнительных полей
        if (empty($_REQUEST['fields'][2])) {
            $this->Message_AddError('Поле не заполнено', $this->Lang_Get('error'));
            $bResult = false;
        }
        return $bResult;
    }
}

обязательное заполнение дополнительных полей (comment #16213)

fordistantjob@gmail.com — Thu, 06 Nov 2014 13:48:32 +0300

Спасибо, попробую привести к такому виду.