Проверка permission в процессорах #9

Activity

1. biz87 commented on Feb 15, 2026

   

   biz87

   on Feb 15, 2026

   Member

   More actions

   Анализ актуальности

   Проблема подтверждена и по-прежнему актуальна.

   Текущее состояние

   56 процессоров содержат дублирующую проверку прав — и в checkPermissions() (наследуется от ModelProcessor), и в initialize():

   public $permission = 'mssetting_save';
   
   public function initialize()
   {
       if (!$this->modx->hasPermission($this->permission)) {
           return $this->modx->lexicon('access_denied');
       }
       return parent::initialize();
   }

   Базовый ModelProcessor::checkPermissions() уже проверяет $this->permission автоматически и вызывается в Processor::run() до initialize().

   Нюанс: разные сообщения об ошибке

   • checkPermissions() возвращает permission_denied_processor
   • initialize() возвращает access_denied

   Нужно решить, какое сообщение оставить.

   Три паттерна в кодовой базе

   Паттерн	Кол-во	Описание
   Дублирующая проверка в initialize()	56	Избыточная, можно удалить
   Кастомный checkPermissions()	12	Легитимная кастомная логика (например, Product/Delete.php) — оставить
   Без переопределения (наследуется)	~95	Корректный подход, ничего не менять

   План работ

   1. Удалить метод initialize() с проверкой прав из 56 процессоров
   2. Убедиться, что свойство $permission задано корректно в каждом
   3. Оставить процессоры с кастомным checkPermissions() без изменений
   4. Проверить, что access_denied и permission_denied_processor не нуждаются в унификации

   Объём: ~56 файлов, механическая правка, низкий риск. Потребуется тестирование прав доступа после изменений.
2. biz87

   added

   priority: lowНизкий приоритет, когда будет времяНизкий приоритет, когда будет время

   on Feb 15, 2026
3. Ibochkarev

   self-assigned this

   on Feb 15, 2026
4. biz87

   closed this as completedin #95on Feb 16, 2026