Создание новых записей и номерные серии

[Владимир Максимов]

Проблема жива и в Ax2012, хотя тянется еще с ранних версий

Форма, источником данных которой является таблица, одно из полей которой формируется из номерной серии. Записи отображены списком в Grid и создание новой записи происходит там же в Grid без вызова дополнительной формы. Выполняем следующие действия

1. Создаем новую запись, заполняем все поля, обязательные к заполнению
2. Не сохраняя запись, пытаемся создать новую запись

Получаем исключение с сообщением об ошибке

Цитата:

Не удается назначить новый номер журнала, так как %1 не был сохранен или удален.

Причина ошибки очевидная. При создании новой записи сначала выполняется метод DataSource.create(), и только внутри этого метода (внутри super()) выполняются все остальные методы по сохранению текущей записи. Но! Стандартный код метода DataSource.create() на форме, если используется номерная серия имеет вид

// Form\DataSource\MyTable\create
void create(boolean append = false)
{
    ;

    element.numberSeqFormHandler().formMethodDataSourceCreatePre();

    super(append);

    element.numberSeqFormHandler().formMethodDataSourceCreate();
}

И вот в методе formMethodDataSourceCreatePre() и стоит проверка

void formMethodDataSourceCreatePre()
{
    ;
    validateWriteCalled = false;
    validateWriteFailed = false;

    if (lastNumber)
    {
        throw(error(strfmt("@SYS85169", lastNumber)));
    }
}

Но поскольку DataSource.create() оказался вызван до DataSource.write(), то номерная серия не была сохранена и значение переменной lastNumber не очищено.

В качестве решения проблемы напрашивается перемещение вызова formMethodDataSourceCreatePre() после вызова super()

void create(boolean append = false)
{
    ;

// Переносим после super()
//    element.numberSeqFormHandler().formMethodDataSourceCreatePre();

    super(append);

    element.numberSeqFormHandler().formMethodDataSourceCreatePre();

    element.numberSeqFormHandler().formMethodDataSourceCreate();
}

Насколько данное решение корректно? Есть ли другие варианты решения проблемы, кроме "забить"?

[S.Kuskov]

Цитата:

Сообщение от Владимир Максимов

Есть ли другие варианты решения проблемы, кроме "забить"?

Сохранять предыдущую запись перед созданием новой
Запись не сохраняется при переходе на другую в гриде

[Владимир Максимов]

Цитата:

Сообщение от S.Kuskov

Сохранять предыдущую запись перед созданием новой
Запись не сохраняется при переходе на другую в гриде

Это не то. Запись в описанном сценарии сохраняется. Просто нарушена последовательность срабатывания методов сохранения. Точнее, нарушена не сама последовательность срабатывания, а последовательность срабатывания в перекрытых методах

Попробую по другому.

Если явным образом нажать "Сохранить", то выполнится метод DataSource.write(), где произойдет сохранение номерной серии и обнуление переменной lastNumber в классе-обработчике номерных серий

Если же нажать кнопку "Создать", то будет вызван метод DataSource.create(), а уже из него (внутри super()) будет вызван DataSource.write(). Но! Анализ значения переменной lastNumber в классе-обработчике номерных серий выполняется ДО вызова super(). Т.е. ДО фактического сохранения записи. Из-за чего и получаем проблему

Вы предлагаете вручную запустить сохранение записи? Повторить все то, что и так будет выполнено внутри super() но явным образом в коде?

-----

Чтобы совсем было понятно

DataSource.forceWrite() - это установка некоего флага, говорящего Axapta о том, что данная запись была изменена. Но анализ этого флага будет выполнен, очевидно, внутри super(). А проблема возникает ДО вызова super(). Т.е. до анализа данного признака дело еще не дошло

[S.Kuskov]

Цитата:

Сообщение от Владимир Максимов

Если же нажать кнопку "Создать", то будет вызван метод DataSource.create(), а уже из него (внутри super()) будет вызван DataSource.write().

Удивительно.

Я честно говоря, думал что речь про ситуацию, при которой уход с записи DataSource.leaveRecord() не вызывает её сохранение.

По вашим же словам проблема возникает из-за того что leaveRecord первой записи вызывается позже чем Сreate для второй.

В любом случае, ситуация при которой мы можем получить несколько не сохранённых строк не будет вписываться в логику работы номерных серий, которые вообще говоря могут быть непрерывными.

[Владимир Максимов]

Цитата:

Сообщение от S.Kuskov

По вашим же словам проблема возникает из-за того что leaveRecord первой записи вызывается позже чем Сreate для второй.

Да. Именно так.

Цитата:

Сообщение от S.Kuskov

В любом случае, ситуация при которой мы можем получить несколько не сохранённых строк не будет вписываться в логику работы номерных серий, которые вообще говоря могут быть непрерывными.

А вот это не так

Внутри метода Create() сначала будут выполнены все методы обработки первой записи и только потом выполняются методы по созданию второй записи. Здесь все корректно и никаких проблем нет.

Проблема в том, что обработчик номерной серии - это некая внешняя "приблуда" не имеющая вообще никакого отношения к стандартным методам ядра. По факту, метод formMethodDataSourceCreatePre() выполняется слишком рано. Он должен был бы быть выполнен после DataSource.write(). Но влезть внутрь super() невозможно. Из-за чего и получаем проблему.

Как другой "костыль" - это "взведение" дополнительного флага внутри Create и его обработка в write

[S.Kuskov]

Цитата:

Сообщение от Владимир Максимов

Внутри метода Create() сначала будут выполнены все методы обработки первой записи и только потом...

Ну как же? В оригинале вызов numberSeqFormHandler().formMethodDataSourceCreatePre() выполняется в самом начале метода Create(), т.е. до данных обработок. Другое дело, что это всего-лишь CreatePre, в котором по идее не должно быть критических операций.

[Владимир Максимов]

Цитата:

Сообщение от S.Kuskov

Ну как же? В оригинале вызов numberSeqFormHandler().formMethodDataSourceCreatePre() выполняется в самом начале метода Create(), т.е. до данных обработок. Другое дело, что это всего-лишь CreatePre, в котором по идее не должно быть критических операций.

По факту, вызов методов leaveRecord(), validateWrite(), write() для старой записи происходит внутри super() в методе create().

А это значит, что, с одной стороны, недопустимо в CreatePre() выполнять контроль факта завершения процесса сохранения записи (поскольку он просто еще не начался), а, с другой стороны, бессмысленно изменять значений флагов, которые как раз и устанавливаются внутри методов контроля и сохранения записи.

В результате, метод numberSeqFormHandler().formMethodDataSourceCreatePre() в существующем наполнении - не просто не имеет смысла (занимается не своим делом), но приводит к описанной проблеме.

Другими словами, вызов данного метода следует вообще исключить из процедуры create() на DataSource формы. В идеале, удалить этот метод из класса NumberSeqFormHandler, поскольку в существующей реализации он вводит в заблуждение разработчика. Ведь по факту, этот метод выполняется не перед созданием новой записи, а перед сохранением текущей записи. Что совсем не одно и то же...

Хотя, конечно, есть вероятность того, что новое значение номерной серии было создано вне стандартного функционала создания записей. Например, через программный вызов. Но контролировать эту ситуацию в методе CreatePre() явно избыточно. Избыточный контроль, который добавляет проблем, а не решает их...