Проблема с добавлением чек листа по событию б24. Что подскажете?

Question

[Nurvana_lite]

добавил в файл init.php такой код. при создании задачи добавляется новый чек лист и тут же удаляется. (видно по логам самой задачи). Не могу понять почему происходит удаление

use \Bitrix\Main;
use \Bitrix\Tasks\CheckList\Task\TaskCheckListFacade;

AddEventHandler("tasks", "OnTaskAdd", ["TaskEvent", "addCheckList"]);

class TaskEvent {
    public static function addCheckList($arTask,&$field)
    {
        CModule::IncludeModule('tasks');
        global $USER;
        $userId = $USER->getId();

        $result = TaskCheckListFacade::add(
            $field['ID'],
            $userId,
            [
                'TITLE' => 'чек-лист',
                'PARENT_ID' => 0,
                'SORT_INDEX' => 0,
                'IS_COMPLETE' => 'N',
            ]
        );
        $item = TaskCheckListFacade::add(
            $field['ID'],
            $userId,
            [
                'TITLE' => 'пункт 1',
                'PARENT_ID' => $result->getData()['ITEM']->getFields()['ID'],
                'SORT_INDEX' => 0,
                'IS_COMPLETE' => 'N',
            ]
        );
return $field;
    }
}

Answer 1

[Андрей Николаев]

Не могу понять почему происходит удаление

Это происходит потому что процесс создания задачи не заканчивается на создании записи о задаче.

Добавление задачи из публичной части сводится к методу: `Bitrix\Tasks\Manager\Task::add`, где вначале происходит создание записи и инициирование события (строчка 139), а затем объединение чек-листов (строчка 177). Таким образом получается что изначально при создании задачи передается пустой набор чек-листов, потом вы создаете элементы чек-листов, а затем merge производит удаление существующих чек-листов, которые не были переданы в изначальном запросе, а так как их не было - просто удаляет ваши созданные чек-листы.

Как можно избежать этой ситуации?

1. Наилучшим вариантом будет попробовать использовать другое событие: before.
Т.е. не создавать элементы чек-листов, а дополнить исходный массив данных структурой, которая будет создавать чек-листы.
Попробуйте отловить прочку событий before с чек-листами чтобы понять что нужно делать

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

3. Так же, неплохим хаком будет являться использование background job - т.к. он выполняется в конце страницы уже после выполнения всех правок.

Обращу ваше внимание - событие является независимым от контекста, поэтому вполне может быть такая ситуация, что при создании задачи на агентах (или консольном скрипте) у вас `$USER` будет null, таким образом ваш скрипт упадет с фатальной ошибкой. Но наличие `$USER` еще не являетися поводом для радости, сам GetId может вернуть `null`, таким образом фатальная ошибка прилетит уже от контроллера прав.
Рекомендую использовать выполнить проверки и сделать fallback механим - подставить ID = 1 (или самому бросить фатальную ошибку) в случае непредвиденной ситуации

Comments

[Nurvana_lite]

1. Наилучшим вариантом будет попробовать использовать другое событие: before.

проблема в том что для создания фасада для чек листа требуется id задачи. а в событии before еще нет созданного id

[Андрей Николаев]

проблема в том что для создания фасада для чек листа требуется id задачи. а в событии before еще нет созданного id

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

[Nurvana_lite]

Андрей Николаев, поставил логгер. воспроизвел структуру но все так же мимо.

$field['CHECKLIST'] = [
            [
                'NODE_ID' => 'test1',
                'PARENT_NODE_ID' => 0,
                'COPIED_ID' => '',
                'PARENT_ID' => null,
                'TITLE' => 'чек-лист 1',
                'SORT_INDEX' => 0,
                'IS_COMPLETE' => 0,
                'IS_IMPORTANT' => false,
                'ACTION' => [
                    'MODIFY' => true,
                    'REMOVE' => true,
                    'TOGGLE' => true,
                ]
            ],
            [
                'NODE_ID' => 'sub_test',
                'PARENT_NODE_ID' => 'test1',
                'COPIED_ID' => '',
                'PARENT_ID' => null,
                'TITLE' => 'пункт 1',
                'SORT_INDEX' => 0,
                'IS_COMPLETE' => 0,
                'IS_IMPORTANT' => false,
                'ACTION' => [
                    'MODIFY' => true,
                    'REMOVE' => true,
                    'TOGGLE' => true,
                ]
            ],
        ];

[Андрей Николаев]

поставил логгер. воспроизвел структуру но все так же мимо.

Согласен, я тоже попробовал и мимо :(
Значит остаются другие варианты

[Андрей Николаев]

поставил логгер. воспроизвел структуру но все так же мимо.

Согласен, я тоже попробовал и мимо :(
Значит остаются другие варианты

[Nurvana_lite]

3. Так же, неплохим хаком будет являться использование background job - т.к. он выполняется в конце страницы уже после выполнения всех правок.

не очень понял что имеется в виду тут. если речь идет про крон. то это уж как то странно

[Андрей Николаев]

Nurvana_lite, https://dev.1c-bitrix.ru/api_d7/bitrix/main/applic...

[Nurvana_lite]

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

[Андрей Николаев]

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

Почему бы тогда не сделать свой rest метод?

[Nurvana_lite]

Андрей Николаев, попробовал вариантом на d7

$eventManager = \Bitrix\Main\EventManager::getInstance();
$eventManager->addEventHandler(
    'tasks',
    '\Bitrix\Tasks\Task::onAfterAdd',
    ['TaskEvent', 'addCheckList']
);

но видимо что то не так понял. потому как не реагирует вообще

[Nurvana_lite]

так же создал новое событие. Но везде мимо

AddEventHandler('rest', 'OnRestServiceBuildDescription', ['\MyRestApi', 'OnRestServiceBuildDescription']);

class MyRestApi
{
    public static function OnRestServiceBuildDescription()
    {
        return [
            'task' => [
                'task.item.add' => [
                    'callback' => [__CLASS__, 'taskAdd'],
                    'options' => [],
                ],
            ],
        ];
    }

    public static function taskAdd($query, $nav, \CRestServer $server)
    {
        if ($query['error'])
        {
            throw new \Bitrix\Rest\RestException(
                'Message',
                'ERROR_CODE',
                \CRestServer::STATUS_PAYMENT_REQUIRED
            );
        }

        if (!isset($query['id']))
        {
            return [
                'error' => 'ERROR_METHOD_NOT_FOUND',
                'error_description' => 'Method not found! ' . __LINE__,
            ];
        }

        if (!isset($query['fields']))
        {
            return [
                'error' => 'ERROR_METHOD_NOT_FOUND',
                'error_description' => 'Method not found! ' . __LINE__,
            ];
        }

        $id = intval($query['id']);

        if ($id <= 0)
        {
            return [
                'error' => 'ERROR_METHOD_NOT_FOUND',
                'error_description' => 'Method not found! ' . __LINE__,
            ];
        }
        $file = $_SERVER['DOCUMENT_ROOT'] . '/logg.log';
        $data = serialize($query);
        file_put_contents($file, $data);
        return array('result' => 'ok');
    }
}

[Андрей Николаев]

Nurvana_lite, во-первых это уже совсем другой вопрос, во-вторых невнимательно читали документацию