Какова основная задача моего скрипта, который предназначен для добавления нового пользователя в список наблюдателей закрытых задач определенного пользователя? У меня установлен фильтр для задач, где пользователь выступает либо в роли ответственного, либо в качестве постановщика, однако возникает проблема с фильтрацией: работая с двумя субфильтрами, второй субфильтр не применяется должным образом. Можете ли вы помочь мне с этой проблемой? Вот весь код: <br/> <br/> <pre><code class="php">global $USER;
if (empty($_POST["OLDUSER"]) || empty($_POST["NEWUSER"])) {
echo "Не указаны ID пользователей.";
exit;
}
$oldUserId = intval($_POST["OLDUSER"]);
$newUserId = intval($_POST["NEWUSER"]);
$creationDateFrom = $_POST['CREATION_DATE_FROM'];
$creationDateTo = $_POST['CREATION_DATE_TO'];
$oldUser = CUser::GetByID($oldUserId)->Fetch();
$newUser = CUser::GetByID($newUserId)->Fetch();
if (!$oldUser) {
echo "Не найден старый пользователь с таким ID";
exit;
} elseif (!$newUser) {
echo "Не найден новый пользователь с таким ID";
exit;
}
$creationDateFrom = !empty($creationDateFrom) ? ConvertTimeStamp(strtotime($creationDateFrom), "FULL") : null;
$creationDateTo = !empty($creationDateTo) ? ConvertTimeStamp(strtotime($creationDateTo) + 86400, "FULL") : null;
if (!$creationDateFrom || !$creationDateTo) {
echo "Не указаны даты создания задач.";
exit;
}
$filter = array(
'::LOGIC' => 'AND',
'>=CREATED_DATE' => $creationDateFrom,
'<=CREATED_DATE' => $creationDateTo,
'::SUBFILTER-1' => array(
'::LOGIC' => 'OR',
'RESPONSIBLE_ID' => $oldUserId,
'REAL_STATUS' => array(CTasks::STATE_COMPLETED),
),
'::SUBFILTER-2' => array(
'CREATED_BY' => $oldUserId,
'REAL_STATUS'
Для создания эффективного фильтра задач в Box-версии Битрикс24, ключевой момент заключается в правильной настройке фильтрации по всем нужным параметрам, включая использование субфильтров. Ваша проблема, связанная с использованием двух субфильтров, возможно, может быть решена следующим образом.
Проблема, с которой вы столкнулись, может быть связана с тем, что условие между двумя субфильтрами должно быть задано правильно, а также не забывайте об использовании логики `AND` или `OR` между условиями. Предполагаю, что ваши два субфильтра должны быть объединены с использованием логики `OR`, так как вы хотите объединить задачи, где пользователь выставлен ответственно или является создателем задачи.
Ваш код может быть переписан следующим образом для корректного применения субфильтров:
```php
global $USER;
if (empty($_POST["OLDUSER"]) || empty($_POST["NEWUSER"])) {
echo "Не указаны ID пользователей.";
exit;
}
$oldUserId = intval($_POST["OLDUSER"]);
$newUserId = intval($_POST["NEWUSER"]);
$creationDateFrom = $_POST['CREATION_DATE_FROM'];
$creationDateTo = $_POST['CREATION_DATE_TO'];
$oldUser = CUser::GetByID($oldUserId)->Fetch();
$newUser = CUser::GetByID($newUserId)->Fetch();
if (!$oldUser) {
echo "Не найден старый пользователь с таким ID";
exit;
} elseif (!$newUser) {
echo "Не найден новый пользователь с таким ID";
exit;
}
$creationDateFrom = !empty($creationDateFrom) ? ConvertTimeStamp(strtotime($creationDateFrom), "FULL") : null;
$creationDateTo = !empty($creationDateTo) ? ConvertTimeStamp(strtotime($creationDateTo) + 86400, "FULL") : null;
if (!$creationDateFrom || !$creationDateTo) {
echo "Не указаны даты создания задач.";
exit;
}
$filter = array(
'::LOGIC' => 'AND',
'>=CREATED_DATE' => $creationDateFrom,
'<=CREATED_DATE' => $creationDateTo,
'::SUBFILTER' => array(
'::LOGIC' => 'OR',
array(
'RESPONSIBLE_ID' => $oldUserId,
'REAL_STATUS' => array(CTasks::STATE_COMPLETED)
),
array(
'CREATED_BY' => $oldUserId,
'REAL_STATUS' => array(CTasks::STATE_COMPLETED)
)
)
);
// Здесь добавьте код для выборки задач с помощью $filter
```
### Основные изменения:
1. **Упрощение субфильтров:** Вместо двух отдельных субфильтров (`SUBFILTER-1` и `SUBFILTER-2`), создается один общий субфильтр с использованием логики `OR`, объединяющей условия для `RESPONSIBLE_ID` и `CREATED_BY`. Это обеспечит правильную фильтрацию задач.
2. **Обработка ошибок:** Убедитесь, что все проверки на валидные вводимые данные выполняются до использования фильтра.
3. **Тестирование результатов:** После выполнения фильтра проверьте, что задачи возвращаются корректно и содержат нужные данные.
Не забудьте протестировать код и убедиться, что фильтрация работает правильно в зависимости от ваших требований.
В общем решил сделать другим путём, сначала запускаю поиск по ответственным, потом по наблюдателям, насколько это правильно или производительней незнаю.
КОД
global $USER; if (empty($_POST["OLDUSER"]) || empty($_POST["NEWUSER"])) { echo "Не указаны ID пользователей."; exit; } $oldUserId = intval($_POST["OLDUSER"]); $newUserId = intval($_POST["NEWUSER"]); $creationDateFrom = $_POST['CREATION_DATE_FROM']; $creationDateTo = $_POST['CREATION_DATE_TO']; $oldUser = CUser::GetByID($oldUserId)->Fetch(); $newUser = CUser::GetByID($newUserId)->Fetch(); if (!$oldUser) { echo "Не найден старый пользователь с таким ID"; exit; } elseif (!$newUser) { echo "Не найден новый пользователь с таким ID"; exit; } $creationDateFrom = !empty($creationDateFrom) ? ConvertTimeStamp(strtotime($creationDateFrom), "FULL") : null; $creationDateTo = !empty($creationDateTo) ? ConvertTimeStamp(strtotime($creationDateTo) + 86400, "FULL") : null; if (!$creationDateFrom || !$creationDateTo) { echo "Не указаны даты создания задач."; exit; } // Первый этап: ищем задачи, где старый пользователь является ответственным $filterForResponsible = array( '::LOGIC' => 'AND', '>=CREATED_DATE' => $creationDateFrom, '<=CREATED_DATE' => $creationDateTo, 'RESPONSIBLE_ID' => $oldUserId, 'REAL_STATUS' => array(CTasks::STATE_COMPLETED), ); // Обновляем наблюдателей $messages = []; $tasksUpdated = processTasks($filterForResponsible, $newUserId, $USER->GetID(), $messages); if ($tasksUpdated === 0) { // Второй этап: ищем задачи, где старый пользователь является постановщиком $filterForCreator = array( '::LOGIC' => 'AND', '>=CREATED_DATE' => $creationDateFrom, '<=CREATED_DATE' => $creationDateTo, 'CREATED_BY' => $oldUserId, 'REAL_STATUS' => array(CTasks::STATE_COMPLETED), ); $tasksUpdated += processTasks($filterForCreator, $newUserId, $USER->GetID(), $messages); } if ($tasksUpdated === 0) { $messages[] = "Не найдено задач для обновления."; } echo implode("<br>", $messages); function processTasks($filter, $newUserId, $currentUserId, &$messages) { $res = CTasks::GetList([], $filter); $tasksUpdated = 0; while ($arTask = $res->GetNext()) { $taskId = $arTask["ID"]; $taskItem = new CTaskItem($taskId, $currentUserId); $taskData = $taskItem->getData(); $auditors = $taskData["AUDITORS"] ?? []; if (!in_array($newUserId, $auditors)) { $auditors[] = $newUserId; $obTask = new CTasks; $updateFields = [ "AUDITORS" => $auditors ]; $success = $obTask->Update($taskId, $updateFields); if ($success) { $messages[] = "Задача с ID $taskId: Новый наблюдатель добавлен."; $tasksUpdated++; } else { if ($e = $APPLICATION->GetException()) { $messages[] = "Ошибка при обновлении задачи ID $taskId: " . htmlspecialchars($e->GetString()); } } } else { $messages[] = "Задача с ID $taskId: Наблюдатель с ID $newUserId уже существует."; } } return $tasksUpdated; }