Как автоматически удалять пользователя из группы, если у него закончился период активности?

Question

[SpeakeazyYT2]

Здравствуйте. Делаю автоматическое добавление пользователей в группу на определенный срок. В документации Bitrix'а обнаружил, что если ставить фиксированный период активности от ... и до ... и если этот период активности заканчивался, то по идее система должна автоматически выписывать пользователя из группы, но почему-то этого не происходит и пользователь продолжает находится в группе. Что можно сделать?

Answer 1

[Kudis]

Вот тут нужно не забыть поставить дату окончания (2) и тогда после этой даты пользователь будет исключен из группы

Comments

[Sp1keazyYT]

Об этом я то знаю, что нужно поставить до какого числа будет функционировать группа для пользователя, но проблема в том, что когда я даже ставлю дату окончания, то галочка с группой не снимается, когда заканчивается время.

[Kudis]

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

[Sp1keazyYT]

Kudis, А как сделать чтобы именно галочки с группой снималась автоматически? Потому что у меня это имеет большую роль, во-первых если срок группы заканчивается, то название группы той всё равно остаётся.

[Kudis]

вот таким условием можно исключать группы с просроченным членством
strtotime($arGroup['DATE_ACTIVE_TO']) < time()

[Sp1keazyYT]

Kudis, А можно какой-то пример привести, используя этот код? Просто не очень понятно, в каком месте его можно использовать. Условие само понятно, но не ясно как убирать человека из группы по истечению срока группы. Нужно убирать галочку.

[Kudis]

Запускайте периодически через агентов битрикса или вручную вот такой код:

$arSort = [];
$arOrder = [];
$arFilter = [
    'ACTIVE' => 'Y', // работаем только с активными пользователями
    'GROUPS_ID' => [3,4,5] // список групп, чьих членов касается данная процедура (можно не использовать, тогда для всех)
];

$obUsers = CUser::GetList($arSort, $order, $filter);

while ($arUser = $obUsers->Fetch()) {

    $arGroups_db = CUser::GetUserGroupList($arUser['ID']);

    $counter = 0;
    $arGroups = [];
    while ($arGroup = $arGroups_db->Fetch()){
        $counter++;
        if (empty($arGroup['DATE_ACTIVE_TO']) || strtotime($arGroup['DATE_ACTIVE_TO']) >= time()) {
            $arGroups[] = $arGroup;
        }
    }

    if (count($arGroups) !== $counter) {
        CUser::SetUserGroup($arUser['ID'], $arGroups);
    }

}

галки у просрочников снимутся

или в моём коде из вопроса
замените цикл:

while ($arGroup = $obGroups->Fetch() && !$isAdmin) {
    if ($maxGroupId < $arGroup['GROUP_ID']) {
        $maxGroupId = $arGroup['GROUP_ID'];
    }
    if ($arGroup['GROUP_ID'] == 1) {
        $isAdmin = true;
    }
}

на:

while ($arGroup = $obGroups->Fetch() && !$isAdmin) {
    if (empty($arGroup['DATE_ACTIVE_TO']) || strtotime($arGroup['DATE_ACTIVE_TO']) >= time()) {
        if ($maxGroupId < $arGroup['GROUP_ID']) {
            $maxGroupId = $arGroup['GROUP_ID'];
        }
        if ($arGroup['GROUP_ID'] == 1) {
            $isAdmin = true;
        }
    }
}

[Sp1keazyYT]

Kudis,
"или в моём коде из вопроса
замените цикл:"
А тут какая операция будет выполнятся? Там проверка на активность должная быть.

[Kudis]

Sp1keazyYT, это и есть проверка на активность

empty($arGroup['DATE_ACTIVE_TO']) || strtotime($arGroup['DATE_ACTIVE_TO']) >= time()

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

[Sp1keazyYT]

Это я уже понял. А как сделать перебор не по всем группам, а по конкретным? (GetUserGroupList)

[Kudis]

Sp1keazyYT, создать массив $arAimGroups = [1,2,3];
и засунуть в это условие

empty($arGroup['DATE_ACTIVE_TO']) || strtotime($arGroup['DATE_ACTIVE_TO']) >= time()

ещё проверку на принадлежность к этому массиву

in_array($arGroup['GROUP_ID'], $arAimGroups) && (empty($arGroup['DATE_ACTIVE_TO']) || strtotime($arGroup['DATE_ACTIVE_TO']) >= time())

[Sp1keazyYT]

Kudis, Kudis,
Так?

global $USER;

	$user_id=$USER->GetID();
	$obGroups = CUser::GetUserGroupList($user_id);
	$arAimGroups = [6,7,8,9,10];

	$maxGroupId = 0;
	
	while ($arGroup = $obGroups->Fetch()) {
		if (in_array($arGroup['GROUP_ID'], $arAimGroups) && (empty($arGroup['DATE_ACTIVE_TO']) || strtotime($arGroup['DATE_ACTIVE_TO']) >= time())) {
			if ($maxGroupId < $arGroup['GROUP_ID']) {
				$maxGroupId = $arGroup['GROUP_ID'];
			}
		}
	}
	
	$obMaxGroup = CGroup::GetByID($maxGroupId);
	if ($arMaxGroup = $obMaxGroup->Fetch()) {
		
		$name_pex_without = str_replace(['[',']'],['',''],$arMaxGroup['NAME']);
		$class1 = str_replace(['6','7','8','9','10'],['player1','vip1','premium','deluxe1','ultra1'],$maxGroupId);
		$class2 = str_replace(['6','7','8','9','10'],['player2','vip2','premium','deluxe2','ultra2'],$maxGroupId);
		$name_pex = str_replace(['[',']',$name_pex_without],['<pex class="'.$class1.'">[</pex>','<pex class="'.$class1.'">]</pex>','<pex class="'.$class2.'">'.$name_pex_without.'</pex>'],$arMaxGroup['NAME']); 

		echo $name_pex;
	}

И ещё, как такую проверку организовать в этом коде (ниже):

$pex = str_replace(['pex user [user] group add ',' * 2592000'],['',''],$cur_data["command"]);
				
				$arGroups_db = CUser::GetUserGroupList($uid);

				if($pex=="vip") {
					$aimGroupId = 7; //id группы, куда добавляем
				}

				if($pex=="premium") {
					$aimGroupId = 8; //id группы, куда добавляем
				}

				if($pex=="deluxe") {
					$aimGroupId = 9; //id группы, куда добавляем
				}

				if($pex=="ultra") {
					$aimGroupId = 10; //id группы, куда добавляем
				}

				$aimGroupUpdated = false;

				while ($arGroup = $arGroups_db->Fetch()){

					if ($arGroup['GROUP_ID'] == $aimGroupId) {
						$arGroup['DATE_ACTIVE_FROM'] = $cur_date;
						$arGroup['DATE_ACTIVE_TO'] = $to_date;
						$aimGroupUpdated = true;
					}

					$arGroups[] = $arGroup;
				}

				if (!$aimGroupUpdated) {
					$arGroups[] = [
						'GROUP_ID' => $aimGroupId,
						'DATE_ACTIVE_FROM' => $cur_date,
						'DATE_ACTIVE_TO' => $to_date
					];
				}

				$add = CUser::SetUserGroup($uid);

[SpeakeazyYT2]

Kudis, Вот ещё не понимаю как сделать: Как для определённых групп пользователей разрешить загружать файлы, больше заданного размера в пользовательских полях?

Answer 2

[Александр]

удалять не надо.
при окончании даты битрикс не учитывает эту группу

Comments

[SpeakeazyYT2]

Я проверял - не работает! Я к примеру давал пользователю группу Администраторы на 1 минуту, сначала у человека группа появлялась, но когда срок завершался, то группа и права все оставались у человека. Поэтому как сделать чтобы при окончании срока группы галочка на эту группа в настройках Пользователя автоматически снималась?

[Александр]

надо перелогиниться чтобы группы обновились

[SpeakeazyYT2]

idruweb, Такой способ к сожалению не подходит. Как-то можно это автоматизировать, чтобы всё работало?

[Александр]

только проверять группы на каждом хите у авторизованных пользователей

[SpeakeazyYT2]

idruweb, Можно код?

[Александр]

поищите по форумам битрикса, проскакивала эта тема