Ошибка в AX2009 EE: SalesFormLetter_xxx

[EVGL]

Коллега сообщает: в указанных классах восточноевропейцы добавили генерацию RecId вручную. К сожалению, забыли пару мест:

SalesFormLetter_PackingSlip\chooseLinesFromWMSArrival:

    void createParmTable()
    {
        if (this.allowCreateParmTable())
        {
            if (!localSalesParmTable.ParmId)
            {
                this.createParmTable(localSalesParmTable,
                                      localSalesTable,
                                      inventSiteId,
                                      localSalesTable.SalesName);

                // PAHA, Error when invoicing from shipment -->
                //RecId is needed
                localSalesParmTable.RecId  = systemSequence.reserveValues(1, tablenum(SalesParmTable));
                // PAHA, Error when invoicing from shipment <--

                recordInsertSalesParmTable.add(localSalesParmTable);
                salesParmTableMap.insert(this.salesParmTableMapKey(localSalesParmTable.SalesId, inventSiteId, '', '', ''), localSalesParmTable);

                salesParmSubTable = SalesParmSubTable::createFromSalesParmTable(localSalesParmTable,false);
                recordInsertSalesParmSubTable.add(salesParmSubTable);

                tableRefUsed = true;
                this.allowCreateParmTable(this.allowEmptyTable());
            }
        }
    }

SalesFormLetter_Invoice\chooseLinesFromWMSShipment:

    void createParmTable()
    {
        if (this.allowCreateParmTable())
        {
            localSalesParmTable.clear();
            localSalesParmTable.TableRefId  = tableRef;

            this.createParmTable(localSalesParmTable, SalesTable::find(previousSalesId));

            // PAHA, Error when invoicing from shipment -->
            //RecId is needed
            localSalesParmTable.RecId  = systemSequence.reserveValues(1, tablenum(SalesParmTable));
            // PAHA, Error when invoicing from shipment <--

            recordInsertSalesParmTable.add(localSalesParmTable);
            salesParmTableMap.insert(this.salesParmTableMapKey(localSalesParmTable.SalesId, '', '', '', ''), localSalesParmTable);
            localSalesParmSubTable = SalesParmSubTable::createFromSalesParmTable(localSalesParmTable,false);
            recordInsertSalesParmSubTable.add(localSalesParmSubTable);

            tableRefUsed = true;
            this.allowCreateParmTable(this.allowEmptyTable());
        }
    }

[Jabberwocky]

А Ваш коллега не задумался, для чего была добавлена генерация RecId?

Просто везде после строчки:

localSalesParmTable.RecId  = systemSequence.reserveValues(1, tablenum(SalesParmTable));

стоят вызовы:

AddressTrans_RU::createFromSalesParmTable(localSalesParmTable);
PlInventPackageParm::updateFromSalesParmTable(localSalesParmTable);

Для корректной работы этих методов необходимо, чтобы буфер localSalesParmTable уже содержал "реальный" RecId.

В вышеприведенных методах

SalesFormLetter_PackingSlip\chooseLinesFromWMSArrival, SalesFormLetter_Invoice\chooseLinesFromWMSShipment

эти вызовы не используются.

[EVGL]

Спасибо за разъяснение, передал коллеге!

[Logger]

Хм, а что при вставке записи recId не перезатирается ? Или там добавлен код подавляющий это ?

[SRF]

Цитата:

Сообщение от EVGL

...
SalesFormLetter_PackingSlip\chooseLinesFromWMSArrival:

SalesFormLetter_Invoice\chooseLinesFromWMSShipment:
...

Цитата:

Сообщение от Jabberwocky

А Ваш коллега не задумался, для чего была добавлена генерация RecId?

Просто везде после строчки:

localSalesParmTable.RecId  = systemSequence.reserveValues(1, tablenum(SalesParmTable));

стоят вызовы:

AddressTrans_RU::createFromSalesParmTable(localSalesParmTable);
PlInventPackageParm::updateFromSalesParmTable(localSalesParmTable);

Для корректной работы этих методов необходимо, чтобы буфер localSalesParmTable уже содержал "реальный" RecId.

В вышеприведенных методах

SalesFormLetter_PackingSlip\chooseLinesFromWMSArrival, SalesFormLetter_Invoice\chooseLinesFromWMSShipment

эти вызовы не используются.

Цитата:

Сообщение от Logger

Хм, а что при вставке записи recId не перезатирается ? Или там добавлен код подавляющий это ?

На самом деле все-таки видимо это ошибка(не проверял на реальных данных), в выше означенных методах, как и в других chooseLines... идет вызов

this.recordInsertListInit()

провалившись в него можно увидеть метод suspendRecIds()

systemSequence = new systemSequence();
systemSequence.suspendRecIds(tableNum(SalesParmLine));
systemSequence.suspendRecIds(tableNum(SalesParmTable));

Отсюда можно сделать вывод, что генерацией recId будут заниматься вручную, т.е. по идее не важно идут вызовы

AddressTrans_RU::createFromSalesParmTable(localSalesParmTable);
PlInventPackageParm::updateFromSalesParmTable(localSalesParmTable);

или нет, recId выделять все равно нужно.

[EVGL]

Коллега Пауль комментирует:

Цитата:

kewl, also ist die sql-fehlermeldung die kommt nicht by design

Цитата:

Похоже, сообщение об ошибке SQL [при создании накладной из отгрузки] все-таки не было предусмотрено проектом

[b_nosoff]

Вот вроде бы все ничего, но только при вызове методов reSelect и reArrange порой пересоздаются записи SalesParmTable. А вот создание AddressTrans_RU для них никто не предусмотрел...

И еще - при создании записи AddressTrans_RU методом createFromIds почему-то никто не подумал, что с недавних пор некоторые таблицы связаны с таблицей Address посредством DirPartyTable.