Битрикс не заполнено обязательное поле?

Question

[JastaFly]

Доброго всем времени суток! Есть форма обратной связи, после отправки которой получаю сообщение об ошибке мол одно из полей не заполнено. Данное поле скрытое, атрибут value точно заполнен и данные из данного поля точно отправляются, но в любом случае возвращается массив $arResult["ERRORS"] с ошибкой. Как мне найти обработчик данной формы который, проверяет валидность формы и формирует массив $arResult["ERRORS"]?!??

Comments

[PetrPo]

Посмотри есть ли какие-нибудь обработчики добавления результата формы

$module = 'form';
$event = 'onBeforeResultAdd';
$eventManager = \Bitrix\Main\EventManager::getInstance();
$eventsList = $eventManager->findEventHandlers($module, $event);
print_r($eventsList);

[JastaFly]

PetrPo, поискал.... ничего такого в проекте нет

[Ярослав Александров]

Посмотреть нет ли кастомного скрипта обработчика на аяксе. Сайт случайно не на решении с маркетплейс?

[PetrPo]

JastaFly, что значит поискал в проекте? в битриксовой консоли набери код, который написан выше

[JastaFly]

PetrPo, набрал получаю пустой массив. Уточните пожалуйста что должно быть в переменной $module!???

[JastaFly]

Ярослав Александров, нет..... скрипта такого вроде не нашел. Но может н правильно искал. Подскажите как это сделать наверняка??!?

[PetrPo]

JastaFly, то что сейчас там и написано, название модуля. Я правильно понимаю, что речь про модуль веб-формы?

[JastaFly]

PetrPo, нет судя по всему это комплексный компонент техподдержки: support.ticket.form

[PetrPo]

JastaFly, именно support.ticket.form? в системе есть support.ticket.edit

[JastaFly]

PetrPo, да именно код подключения компонента выглядит следующим образом:

<? $APPLICATION->IncludeComponent(
        "altasib:support.ticket.form",
        "question",
        Array(
            "ID" => $_REQUEST['ID'],
            "SHOW_FULL_FORM" => "Y",
            "TICKET_LIST_URL" => SITE_DIR . "support/",
            "HAVE_CREATE" => 1,
        )
    ); ?>

[PetrPo]

JastaFly, вообщем раз это саппорт, то все в том же коде измени переменные

$module = 'support';
$event = 'OnBeforeTicketAdd';

Еще проверь $event = 'OnBeforeTicketUpdate';

[PetrPo]

JastaFly, пространство имен видишь? altasib, не bitrix, наверняка там события навешаны

[JastaFly]

PetrPo, support.ticket.edit ничего такого вроде не нашел. Уточните где именно стоит искать??!?

[PetrPo]

JastaFly, не надо его искать, я к тому что компонент кастомный, конторы под названием altasib. Смотри обработчики, код я тебе дал

[JastaFly]

PetrPo, массив всё так же пуст

[Ярослав Александров]

JastaFly, altasib:support.ticket.form - компонент не от битрикса, а altasib если у вас есть поддержка вы можете к ним обратиться. Вполне возможно что компонент косячный.

Подскажите как это сделать наверняка??!?

В консоли браузера кнопка F12, посмотреть какие js скрипты отрабатывают во время работы формы, далее в js скриптах можно увидеть пути на используемые РНР файлы обработчики если есть такие.

[JastaFly]

PetrPo, да это кастомный шаблон формы. Вроде кроме вёрстки там ничего больше нет

[JastaFly]

Ярослав Александров, да вряд-ли это JS. Массив с ошибкой возвращается после обновления страницы

[PetrPo]

JastaFly, так не бывает) Это все к тому что написал Ярослав выше. Или в саппорт altasib или еще как-нибудь... У меня нет такого компонента, а не видя код можно только гадать. А так смотри /btirix/components/altasib/support.ticket.form/component.php или class.php

[Ярослав Александров]

JastaFly, вам виднее, очень часто самодельные формы работают на аяксе, а аякс это как раз JS, там обычно цепляют РНР файлы обработчики. есть несколько вариантов проблемы:
1) Ошибка в шаблоне компонента (проверить работу на дефолтном)
2) Косячный самодельный компонент (смотреть и делать отладку кода)
3) Обработчики событий
4) РНР обработчики на аяксе.

[JastaFly]

Ярослав Александров, сменил шаблон на .default.... таже ошибка

[JastaFly]

PetrPo, вот така структура данного компонента:

И там я уже вроде всё просмотрел. Но всё таки подскажите на какой файл обратить внимание??!?

[PetrPo]

JastaFly, я же тебе прямо путь написал /btirix/components/altasib/support.ticket.form/component.php

[Ярослав Александров]

JastaFly, ну так код шаблона полностью приложите

[Ярослав Александров]

JastaFly, добавьте архив компонента в яндекс диск и пришлите ссылкой

[Ярослав Александров]

Вы кстати не ответили ваш сайт из маркетплейса или индивидуальная разработка?

[JastaFly]

Ярослав Александров, индивидуальная разработка

[JastaFly]

PetrPo, по указанному Вами пути нашел нужный файл и в нём нашел участок кода который добавляет ошибку:

$result = Support\TicketTable::add($arTicket);
$TICKET_ID = $result->getId();
 if (!$result->isSuccess()) {
                $arResult = $arTicket;
                if ($arParams['ALLOW'] && (int)$request['PARRENT_MESSAGE_ID'] > 0) {
                    $arResult['PARRENT_MESSAGE_ID'] = $request['PARRENT_MESSAGE_ID'];
                }
                $arResult["ERRORS"] = $result->getErrorMessages();

Метод $result->isSuccess() возвращает false
Как теперь добраться до данного метода чтобы посмотреть что там происходит???!

[JastaFly]

Вот ссылка на архив с файлам шаблона в папке local и компонентом который я вытащил из папки bitrix

[PetrPo]

JastaFly, твой вопрос

Как мне найти обработчик данной формы который, проверяет валидность формы и формирует массив $arResult["ERRORS"]?!??

Дальше ты пишешь

нашел участок кода который добавляет ошибку:

И зачем-то мне предлагаешь скачать компонент? что к чему...

[JastaFly]

PetrPo, да участок нашёл, но ответа на суть проблемы это не дало. Теперь мне нужно найти метод: $result->isSuccess(), который как я понял пришёл из пространства имён ALTASIB\Support, но как найти файл с этим пространством имён?!??

[Ярослав Александров]

Нужно в самом компоненте проверить что приходит в $request

[PetrPo]

JastaFly, есть 3 варианта
1. Прочитать про структуру модулей битрикса
2. Через рефлексию
3. установить ide и найти через нее

[JastaFly]

Ярослав Александров, в $request лежит объект HttpRequest
в котором есть массив:

[values:protected] => Array
                (
                    [TICKET_ID] => 0
                    [PARRENT_MESSAGE_ID] => 
                    [SEND_MESSAGE] => Y
                    [t_submit_go] => 
                    [PLATFORM_TECHNOLOG] => technologist
                    [OWNER_ID] => 684
                    [sessid] => cc527ce93748c7a8981c48180090c88b
                    [CATEGORY_ID] => 4
                    [KindAction] => 1387
                    [TITLE] => Сколько ты зарабатываешь?!??
                    [MESSAGE] => ryryryr
                )

Как видите поле [PLATFORM_TECHNOLOG] на которое ругается валидатор, мол оно пустое имеет значение. если нужно могу прислать объект целиком

[JastaFly]

Так короче удалось избавиться от ошибки. В компоненте есть массив :

$arTicket = Array(
            'TITLE' => $request["TITLE"],
            'CATEGORY_ID' => (int)$request["CATEGORY_ID"],
            'PRIORITY_ID' => (int)$request["PRIORITY_ID"],
            'MESSAGE' => $_POST['MESSAGE'],
            'FILES' => $request["FILES"],
        );

Как можно догадаться в нём хранятся данные тикета в службу поддержки. Он же фигурирует в проверке которую я выкладывал выше. Короче я прото добавил в него новый элемент PLATFORM_TECHNOLOG:

$arTicket = Array(
            'TITLE' => $request["TITLE"],
            'CATEGORY_ID' => (int)$request["CATEGORY_ID"],
            'PRIORITY_ID' => (int)$request["PRIORITY_ID"],
            'MESSAGE' => $_POST['MESSAGE'],
            'FILES' => $request["FILES"],
            'PLATFORM_TECHNOLOG' => $request["PLATFORM_TECHNOLOG"]
        );

И сообщение отправилось без ошибок. Но что-то мне такое решение не нравится. Как думаете?!??

[PetrPo]

JastaFly, толку-то что ты так добавил это поле? как оно у тебя в базу попадет?

Вот эта запись ни о чем не говорит?
$result = Support\TicketTable::add($arTicket);

[JastaFly]

эта запись по коду ниже самого $arTicket и мой новый элемент так же добавится как и все остальные если Вы об этом

[JastaFly]

пробовал добраться до пространства имён с помощью:
new ReflectionClass('Support\TicketTable')
В результате получаю:
Class Support\TicketTable does not exist
Я всё верно сделал или имя класса введено не так?!??
Ставить IDE не варик.... у меня на локалке только часть файлов, большая часть, включая ядро системы поднята на виртуалке.... туда IDE не поставишь..... а весит всё добро 12 гигов

[JastaFly]

кстати после добавления моего элемента в массив, тикет приходит в поддержку

[PetrPo]

JastaFly, да тикет-то приходит, вот это поле PLATFORM_TECHNOLOG для чего тебе? если конечно сущность Support\TicketTable описана по канонам Битрикс ORM, то это поле должно присутствовать там, а так оно просто игнорируется и никуда не сохраняется, можешь с тем же успехом его из шаблона выпилить и не будет у тебя ошибки.

Про рефлексию - ну серьезно, ты не знаешь что такое namespace? смотри вверху должно быть
use что_то_тут\Support\TicketTable; всю строку и подставь, писать так надо

$a = new ReflectionClass('что_то_тут\Support\TicketTable');
echo $a->getFileName();

[JastaFly]

Про namespace знаю.... но не всё.... а с рефлексией затупил и указал неполное имя класса
Короче в итоге я добрался до нужного класса он оказался наследником этого класса ядра Битрикса для добавления информации использует его метод add:
$result = parent::add($data)
В конце работы метода возвращается $result. В $data лежит $arTicket в который я и добавил новую строку.
Лезть и править что-то в ядре битрикса я конечно же не буду, но посмотреть как работает метод add любопытно. Но насколько я понял внутри пространства имен можно использовать только классы которые в нём определены и таже рефлексия работать уже не будет?!?? Как мне тогда найти файл с классом Bitrix\Main\Entity\DataManager?!??

[JastaFly]

За проверку полей отвечает вот эта функция:

public static function checkFields(Result $result, $primary, array $data)
    {
        $entity = static::getEntity();
        //checks required fields
        foreach ($entity->getFields() as $field)
        {
            if ($field instanceof ScalarField && $field->isRequired())
            {
                $fieldName = $field->getName();
                if (
                    (empty($primary) && (!isset($data[$fieldName]) || $field->isValueEmpty($data[$fieldName])))
                    || (!empty($primary) && isset($data[$fieldName]) && $field->isValueEmpty($data[$fieldName]))
                )
                {
                    $result->addError(new FieldError(
                        $field,
                        Loc::getMessage("MAIN_ENTITY_FIELD_REQUIRED", array("#FIELD#"=>$field->getTitle())),
                        FieldError::EMPTY_REQUIRED
                    ));
                }
            }
        }

        // checks data - fieldname & type & strlen etc.
        foreach ($data as $k => $v)
        {
            if ($entity->hasField($k))
            {
                $field = $entity->getField($k);

            }
            else
            {
                throw new Main\ArgumentException(sprintf(
                    'Field `%s` not found in entity when trying to query %s row.',
                    $k, $entity->getName()
                ));
            }

            $field->validateValue($v, $primary, $data, $result);
        }
    }

То есть если я верно понял, она получает сущность в которой есть имена всех полей, затем она проверяет есть ли в $data элемент с аналогичным именем, если нет то записывает ошибку. Получается я всё верно сделал добавив в массив недостающий элемент

[Дмитрий Игошев]

Что в public static function getMap()?
Там должны быть описаны поля таблицы.
Ну и саму таблицу можете посмотреть через /bitrix/admin/perfmon_table.php?lang=ru&table_name=b_file (b_file поменяйте на нужную таблицу)
Если поля "PLATFORM_TECHNOLOG" нет то и писать его некуда.

[JastaFly]

getMap() выдаёт следующий массив:

Array
(
    [ID] => Array
        (
            [data_type] => integer
            [primary] => 1
            [autocomplete] => 1
            [title] => №
        )

    [SITE_ID] => Array
        (
            [data_type] => string
            [required] => 1
            [title] => Сайт
        )

    [KindAction] => Array
        (
            [data_type] => string
            [required] => 
        )

    [PLATFORM_TECHNOLOG] => Array
        (
            [data_type] => string
            [required] => 1
        )

    [DATE_CREATE] => Array
        (
            [data_type] => datetime
            [required] => 1
            [title] => Дата создания
        )

    [DATE_CREATE_X] => Array
        (
            [data_type] => datetime
            [expression] => Array
                (
                    [0] => DATE(%s)
                    [1] => DATE_CREATE
                )

        )

    [TIMESTAMP] => Array
        (
            [data_type] => datetime
            [title] => Дата изменения
        )

    [TIMESTAMP_X] => Array
        (
            [data_type] => datetime
            [expression] => Array
                (
                    [0] => DATE(%s)
                    [1] => TIMESTAMP
                )

        )

    [LAST_MESSAGE_DATE] => Array
        (
            [data_type] => datetime
            [title] => Дата последнего сообщения
        )

    [LAST_MESSAGE_USER_ID] => Array
        (
            [data_type] => integer
            [title] => Автор последнего сообщения
        )

    [LAST_MESSAGE_USER] => Array
        (
            [data_type] => ALTASIB\Support\User
            [reference] => Array
                (
                    [=this.LAST_MESSAGE_USER_ID] => ref.ID
                )

        )

    [LAST_MESSAGE_BY_SUPPORT] => Array
        (
            [data_type] => boolean
            [values] => Array
                (
                    [0] => N
                    [1] => Y
                )

        )

    [IS_CLOSE] => Array
        (
            [data_type] => boolean
            [title] => Закрыто
            [values] => Array
                (
                    [0] => N
                    [1] => Y
                )

        )

    [DATE_CLOSE] => Array
        (
            [data_type] => datetime
            [title] => Дата закрытия
        )

    [DATE_CLOSE_X] => Array
        (
            [data_type] => datetime
            [expression] => Array
                (
                    [0] => DATE(%s)
                    [1] => DATE_CLOSE
                )

        )

    [OWNER_USER_ID] => Array
        (
            [data_type] => integer
            [required] => 1
            [title] => Автор
        )

    [OWNER_USER] => Array
        (
            [data_type] => ALTASIB\Support\User
            [reference] => Array
                (
                    [=this.OWNER_USER_ID] => ref.ID
                )

        )

    [CREATED_USER_ID] => Array
        (
            [data_type] => integer
            [required] => 1
            [title] => Создано
        )

    [CREATED_USER] => Array
        (
            [data_type] => ALTASIB\Support\User
            [reference] => Array
                (
                    [=this.CREATED_USER_ID] => ref.ID
                )

        )

    [MODIFIED_USER_ID] => Array
        (
            [data_type] => integer
            [title] => Изменено
        )

    [MODIFIED_USER] => Array
        (
            [data_type] => ALTASIB\Support\User
            [reference] => Array
                (
                    [=this.MODIFIED_USER_ID] => ref.ID
                )

        )

    [RESPONSIBLE_USER_ID] => Array
        (
            [data_type] => integer
            [required] => 1
            [title] => Ответственный
        )

    [RESPONSIBLE_USER] => Array
        (
            [data_type] => ALTASIB\Support\User
            [reference] => Array
                (
                    [=this.RESPONSIBLE_USER_ID] => ref.ID
                )

        )

    [CATEGORY_ID] => Array
        (
            [data_type] => integer
            [required] => 1
            [title] => Категория
        )

    [CATEGORY] => Array
        (
            [data_type] => ALTASIB\Support\CategoryTable
            [reference] => Array
                (
                    [=this.CATEGORY_ID] => ref.ID
                )

        )

    [PRIORITY_ID] => Array
        (
            [data_type] => integer
            [title] => Приоритет
        )

    [STATUS_ID] => Array
        (
            [data_type] => integer
            [required] => 1
            [title] => Статус
        )

    [STATUS] => Array
        (
            [data_type] => ALTASIB\Support\StatusTable
            [reference] => Array
                (
                    [=this.STATUS_ID] => ref.ID
                )

        )

    [SLA_ID] => Array
        (
            [data_type] => integer
            [title] => Уровень поддержки
        )

    [SLA] => Array
        (
            [data_type] => ALTASIB\Support\SlaTable
            [reference] => Array
                (
                    [=this.SLA_ID] => ref.ID
                )

        )

    [TITLE] => Array
        (
            [data_type] => string
            [required] => 1
            [title] => Заголовок обращения
        )

    [IP] => Array
        (
            [data_type] => string
            [required] => 1
            [title] => IP
        )

    [MESSAGE] => Array
        (
            [data_type] => string
            [required] => 1
            [title] => Текст обращения
        )

    [MESSAGE_CNT] => Array
        (
            [data_type] => integer
            [expression] => Array
                (
                    [0] => (SELECT COUNT(altasib_support_ticket_message.ID) from altasib_support_ticket_message WHERE altasib_support_ticket_message.TICKET_ID=%s)
                    [1] => ID
                )

            [title] => Сообщений
        )

    [MESSAGES] => Array
        (
            [data_type] => ALTASIB\Support\TicketMessageTable
            [reference] => Array
                (
                    [=this.ID] => ref.TICKET_ID
                )

        )

    [GROUP_ID] => Array
        (
            [data_type] => integer
        )

    [GROUP] => Array
        (
            [data_type] => \Bitrix\Socialnetwork\WorkgroupTable
            [reference] => Array
                (
                    [=this.GROUP_ID] => ref.ID
                )

        )

    [TASK_ID] => Array
        (
            [data_type] => integer
        )

    [DEAL_ID] => Array
        (
            [data_type] => integer
        )

    [IS_OVERDUE] => Array
        (
            [data_type] => boolean
            [values] => Array
                (
                    [0] => N
                    [1] => Y
                )

        )

    [IS_NOTIFY_SEND] => Array
        (
            [data_type] => boolean
            [values] => Array
                (
                    [0] => N
                    [1] => Y
                )

        )

    [IS_DEFERRED] => Array
        (
            [data_type] => boolean
            [values] => Array
                (
                    [0] => N
                    [1] => Y
                )

        )

    [SUM_ELAPSED_TIME] => Array
        (
            [data_type] => integer
            [expression] => Array
                (
                    [0] => (SELECT SUM(altasib_support_ticket_message.ELAPSED_TIME) from altasib_support_ticket_message WHERE altasib_support_ticket_message.TICKET_ID=%s)
                    [1] => ID
                )
        )
    [COMMENT] => Array
        (
            [data_type] => string
        )
)

Таблица судя по всему называется altasib_support_ticket_message?!??
Если да то столбца "PLATFORM_TECHNOLOG" тут нет:

[JastaFly]

Дмитрий Игошев, добавил столбец PLATFORM_TECHNOLOG, ошибка не пропала

[Дмитрий Игошев]

JastaFly,

$result = parent::add($data)

Что в $data покажите принтовку.
Таблица указана в вашем классе в методе getTableName()
Покажите таблицу, строковое поле добавили?
Ошибка не поменялась, так и считает что поле пустое?

[JastaFly]

Дмитрий Игошев, в $data всегда массив переданный в component.php:
$result = Support\TicketTable::add($arTicket);
Формируется он из значений части полей вот так:

$arTicket = Array(
            'TITLE' => $request["TITLE"],
            'CATEGORY_ID' => (int)$request["CATEGORY_ID"],
            'PRIORITY_ID' => (int)$request["PRIORITY_ID"],
            'MESSAGE' => $_POST['MESSAGE'],
            'FILES' => $request["FILES"],
        );

[JastaFly]

Вот принтовка $data:

Array
(
    [TITLE] => Сколько ты зарабатываешь?!??
    [CATEGORY_ID] => 4
    [PRIORITY_ID] => 0
    [MESSAGE] => цкцукцкцукцу
    [OWNER_USER_ID] => 684
    [DATE_CREATE] => Bitrix\Main\Type\DateTime Object
        (
            [value:protected] => DateTime Object
                (
                    [date] => 2021-01-26 01:59:54.000000
                    [timezone_type] => 3
                    [timezone] => America/New_York
                )

        )

    [CREATED_USER_ID] => 684
    [RESPONSIBLE_USER_ID] => 1
    [SITE_ID] => s1
    [IP] => 192.168.56.1
    [STATUS_ID] => 1
)

Нашел таблицу. Она называется: altasib_support_ticket. Столбец PLATFORM_TECHNOLOG в ней имеется:

[Дмитрий Игошев]

JastaFly,

Вот принтовка $data:

В этой принтовке должно быть поле "PLATFORM_TECHNOLOG".
В вашем принте этого поля нет и следовательно в parent::add() оно не попадает.

[JastaFly]

Дмитрий Игошев, вы все верно говорите. Смотрите как формируется массив $data. В начале был массив $arrTicket:

$arTicket = Array(
            'TITLE' => $request["TITLE"],
            'CATEGORY_ID' => (int)$request["CATEGORY_ID"],
            'PRIORITY_ID' => (int)$request["PRIORITY_ID"],
            'MESSAGE' => $_POST['MESSAGE'],
            'FILES' => $request["FILES"],
        );

Далее он попадает в статический метод класса TicketTable:

$result = Support\TicketTable::add($arTicket);
$TICKET_ID = $result->getId();
 if (!$result->isSuccess()) {
                $arResult = $arTicket;
                if ($arParams['ALLOW'] && (int)$request['PARRENT_MESSAGE_ID'] > 0) {
                    $arResult['PARRENT_MESSAGE_ID'] = $request['PARRENT_MESSAGE_ID'];
                }
                $arResult["ERRORS"] = $result->getErrorMessages();

Класс TicketTable является наследником вот этого класса из ядра и используя его метод
$result = parent::add($data)
Отправляет массив $data дальше. Проверка полей осуществляется в данной методе класса DataManager:

public static function checkFields(Result $result, $primary, array $data)
    {
        $entity = static::getEntity();
        //checks required fields
        foreach ($entity->getFields() as $field)
        {
            if ($field instanceof ScalarField && $field->isRequired())
            {
                $fieldName = $field->getName();
                if (
                    (empty($primary) && (!isset($data[$fieldName]) || $field->isValueEmpty($data[$fieldName])))
                    || (!empty($primary) && isset($data[$fieldName]) && $field->isValueEmpty($data[$fieldName]))
                )
                {
                    $result->addError(new FieldError(
                        $field,
                        Loc::getMessage("MAIN_ENTITY_FIELD_REQUIRED", array("#FIELD#"=>$field->getTitle())),
                        FieldError::EMPTY_REQUIRED
                    ));
                }
            }
        }

        // checks data - fieldname & type & strlen etc.
        foreach ($data as $k => $v)
        {
            if ($entity->hasField($k))
            {
                $field = $entity->getField($k);

            }
            else
            {
                throw new Main\ArgumentException(sprintf(
                    'Field `%s` not found in entity when trying to query %s row.',
                    $k, $entity->getName()
                ));
            }

            $field->validateValue($v, $primary, $data, $result);
        }
    }

Когда я добавил проблемное поле в самый первый массив $arrTicket, то ошибка пропала:

$arTicket = Array(
            'TITLE' => $request["TITLE"],
            'CATEGORY_ID' => (int)$request["CATEGORY_ID"],
            'PRIORITY_ID' => (int)$request["PRIORITY_ID"],
            'MESSAGE' => $_POST['MESSAGE'],
            'FILES' => $request["FILES"],
            'PLATFORM_TECHNOLOG' => $request["PLATFORM_TECHNOLOG"]
        );

Но мне как-то подобное решение не нравится. Хотя файл компонента можно без проблем перенести в local и не трогая файлов ядра решить вопрос. Что думаете?!??

[Дмитрий Игошев]

JastaFly, Вообще я бы не рекомендовал обновлять данный модуль после модификации. Вы вносите правку в компонент, маппинг класса и в таблицу БД. Все эти изменения могут перетереться при очередном апдейте.
Либо не обновляйте модуль, либо переносите весь в свое пространство имен, вместе с таблицами БД. Но особого смысла в этом нет ибо и так и так вы лишаетесь обновлений модуля.
Файлы ядра не трогайте ни при каких условиях - битрикс потом не обновите.

[JastaFly]

Нууу я перенес файл component.php в local и там всё поправил. А почему могут возникнуть проблемы с обновлением??!? Я всегда считал что файлы ядра нельзя править, потому что исправления могут потеряться при обновлении, а значит рухнет завязанный на них функционал