Почему иногда не срабатывает событие?

Question

[web_bitrix]

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

Сам код:

AddEventHandler('crm', 'OnBeforeCrmCompanyAdd',  'OnBeforeCrmCompanyAddHandler');
function OnBeforeCrmCompanyAddHandler(&$arFields) {
   $res = CCrmCompany::getList($arFields["ID"]);
$companyExists = false;

while ($result = $res->Fetch()) {
    if ($result["TITLE"] == $arFields["TITLE"]) {
        $companyExists = true;
        break;
    }
}

if ($companyExists) {
    $arFields["RESULT_MESSAGE"] = "Данная компания уже существует!";
    return false;
} else {
    return true;
}
}

Comments

[Михаил Ливач]

Код выглядит безумно. Вы вешаетесь на событие OnBeforeCrmCompanyAdd - то есть "до создания компании". Значит, на этом этапе ID не может существовать. Однако Вы его якобы передаёте в CCrmCompany::getList() .
Далее: первый аргумент этого метода - массив для сортировки результата. Вы же туда пихаете то, что, по Вашим предположениям, должно быть числом ( а по факту получается null, как понимаю).
Фильтр же пустой, так что метод просто вываливает вообще все компании, которые есть, хотя можно было бы отфильтровать по имени.

Я бы советовал Вам сначала переписать на адекватный вариант, используя $arFilter и $nPageTop=1, а все случаи, когда выборка оказалась пустой - логировать.
Тогда, при возникновении проблемы с дублем компанией, будет проще воспроизвести ситуацию.
Может, хитрый пользователь создаёт компанию с одним именем, а потом переименовывает. Или там пробелы. Или ещё какая-то дичь.

[web_bitrix]

Михаил Ливач, спасибо за разъяснение

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

Михаил Ливач, оформите в виде ответа на вопрос пожалуйста

[Михаил Ливач]

Андрей Николаев, оформил

Answer 1

[Михаил Ливач]

Вы неправильно используете аргументы в CCrmCompany::getList(), из-за этого получаете вообще все компании и вынуждены их перебирать. Плюс, для событий "OnBefore....Add" ключ ID не может существовать - ведь такие события происходят до создания новой записи ( в смысле элемента, раздела, компании ).

Должно быть что-то такое ( префиксный фильтр в arFilter должен работать, но я не проверял ):

function OnBeforeCrmCompanyAddHandler(&$arFields)
{
    $res = CCrmCompany::getList(['ID' => 'ASC'], ['=TITLE' => $arFields["TITLE"]], ['ID'], 1);
    if ($res->Fetch()) {
        $arFields["RESULT_MESSAGE"] = "Данная компания уже существует!";
        return false;
    }
    return true;
}

Список префиксов можно посмотреть в файле bitrix/modules/main/classes/general/sqlwhere.php , в свойстве $operations