Доступность всех групп при редактировании записи Для InstantCMS 2.X. 1 250 просмотров

Сейчас администратор при редактировании чьей-то записи не может указать группу в которой не состоит сам. Соответственно если запись была привязана к чужой группе, то для группы она теряется. Если указание группы для записи обязательно, то и вовсе группа этой записи меняется.
Не ахти сформулировал, но кто сталкивался поймет.

В идеале если пользователь имеет права на редактирование всех записей, то он должен иметь возможность указать любую группу. Наверное...

Короче, для себя я пока прикрутил костыль, который дает такую возможность администраторам и модераторам. Когда столкнусь с этим вопросом снова и буду чуток повыспавшись, постараюсь придумать, что-то более кошерное. Если кто-то не придумает это раньше.

Сам костыль:

Неправильный костыль

~~В файле /system/controllers/groups/model.php меняем строку 299:~~

$this->filterEqual('user_id', $user_id);

~~на:~~

		$user = cmsUser::getInstance();
		if(!$user->is_admin && !in_array('5', $user->groups)){
			$this->filterEqual('user_id', $user_id);
		}

* Указанный выше код оставлен для тех кто возможно успел уже его внести в свой проект, чтобы было проще исправить обратно.

При указанном в спойлере выше решении возникают проблемы в других местах. Нпример, при удалении пользователя администратором во всех группах декрементится (-1) счетчик участников, а не только в группах где пользователь в действительности является участником.

Экшен для персчета количества участников в группах

	public function run(){
 
		if(!$this->cms_user->is_admin){ cmsCore::error404(); }
 
		$this->model->selectOnly('id');
		$groups = $this->model->get('groups');
 
		foreach($groups as $group){
			$ids = $this->model_groups->getMembersIds($group['id']);
			$this->model->update('groups', $group['id'], ['members_count' => count($ids)]);
		}
 
		die('ok');
 
	}

Следуя совету @SmartControl, полез в экшен content/item_edit и content/item_add. К сожалению через хук внести нужные изменения для всех типов контента не получается, т.к. объект формы передается для изменения только в "content_{$ctype['name']}_form". Так что пока по прежнему только костыль. В тех экшенах где нам это требуется, меняем:

$groups = $groups_model->getUserGroups($this->cms_user->id);

на:

if($this->cms_user->is_admin){
	$groups_model->select('g.id', 'id');
	$groups_model->select('g.*');
	$groups_model->joinInner('groups', 'g', 'g.id = i.group_id');
	$groups_model->orderBy('g.title');
	$groups = $groups_model->get('groups_members', function($group, $model){
 
		$group['slug'] = $group['slug'] ? $group['slug'] : $group['id'];
 
		return $group;
 
	});
}else{
	$groups = $groups_model->getUserGroups($this->cms_user->id);
}

Или если вам достаточно, чтобы просто не сбрасывались привязки к группам, то можно подтягивать только группы автора записи. Для этого в экшене content/item_edit меняем все ту же строку:

$groups = $groups_model->getUserGroups($this->cms_user->id);

на:

if($this->cms_user->is_admin){
	$groups = $groups_model->getUserGroups($item['user_id']);
}else{
	$groups = $groups_model->getUserGroups($this->cms_user->id);
}

[/code]

Редирект менеджер обновлен

3 комментария

Реклама #

SmartControl 6 лет назад #

Иван Шкута:

В идеале если пользователь имеет права на редактирование всех записей, то он должен иметь возможность указать любую группу. Наверное...

Но надо понимать, что желая дать возможность редактировать записи групп (в которых модератор не состоит), вы ломаете функцию получения групп пользователя для админа и модераторов. А эта функция используется много где еще.

SmartControl 6 лет назад #

Аккуратнее было бы поправить action редактирования записи (item_edit), поменяв там "получение групп текущего пользователя" на "получение всех групп" (для админа и модераторов). Это бы не повлияло на другие процессы.

Иван Шкута 6 лет назад #

Да, вы вы правы. Жаль я раньше не обратил на это внимание. Теперь пришлось писать объект для пересчета количества пользователей в группах.
Эх, не люблю я вносить изменения в ядро и не зря!!!

Доступность всех групп при редактировании записи

Реклама #

SmartControl 6 лет назад #

SmartControl 6 лет назад #

Иван Шкута 6 лет назад #

Еще от автора

Редирект менеджер обновлен

Поле "Точка на карте"

Изображение с возможностью обрезки на стороне клиента

InstantCMS Team

О проекте

Поддержка

Дополнения