Пересчет с/с в проводках с заполненным InventRefTransId

[kashperuk]

Цитата:

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

Есть такое в 3.0. Если запустить закрытие, то такие ссылки чистятся.

Для одного клиента мы с mazzy в процедуру пересчета вставляли такой же кусок кода (чистящий ссылки), как и в закрытии.

Чистятся то они чистятся, для закрытия, как я и написал. Но почему-то не для контрактов. Приведу код, на который дал ссылку в первом посте:
Это из метода, который вызывается при пересчете себестоимости и закрытии склада.

    // clear unused references

    if (inventClosing.AdjustmentType == inventAdjustmentType::Closing && (mapQtyIssue || mapQtyReceipt))
    {
        while select forupdate inventTrans
            index hint openItemIdx
            where inventTrans.ValueOpen         == inventTransOpen::Yes &&
                  inventTrans.ItemId            == _itemId              &&
                  inventTrans.InventRefTransId  != ''
        {
            if (! setPhysicalMarked.in(inventTrans.inventTransId) && this.financialOpenQty(inventTrans))
            {
                if ((inventTrans.Qty > 0 && (! mapQtyIssue   || ! mapQtyIssue.exists(inventTrans.InventTransId)    || abs(mapQtyIssue.lookup(inventTrans.InventTransId)) <  inventAdj::settleQtyDiff())) ||
                    (inventTrans.Qty < 0 && (! mapQtyReceipt || ! mapQtyReceipt.exists(inventTrans.InventTransId)  || abs(mapQtyReceipt.lookup(inventTrans.InventTransId)) <  inventAdj::settleQtyDiff())))
                {
                    if (inventTrans.QtySettled)
                        inventTrans.updateSplit(this.financialOpenQty(inventTrans));

                    if (!InventCostItemDim::isBlanketOrderReference(inventTrans))
                    {
                        inventTrans.InventRefTransId = '';
                        inventTrans.doUpdate();

                        if (! inventTrans.isUpdatedFinancial())
                        {
                            movement = inventTrans.inventMovement(true);
                            if (movement && movement.inventRefTransId() != '')
                            {
                                movement.setInventRefFields(inventRefType::None,'','');
                                movement.updateDoBuffer();
                            }
                        }
                    }
                }
            }
        }
    }

Обратите внимание на строку:

if (!InventCostItemDim::isBlanketOrderReference(inventTrans))

Вот из-за нее пропускаются строки, ссылающиеся на строки заказов типа Контракт

[kashperuk]

Посмотрел только что всю эту функциональность в 4.0 SP2

При создании заказа из контракта они связываются уже по новому полю - BlanketRefTransId. То есть для контрактов InventRefTransId не заполняется.
А соответственно при пересчете уже все нормально отрабатывает.
Кстати, класс для создания заказа такого уже по другому называется - SalesAutoCreate_ReleaseOrder (был SalesAutoCreate_tmpSalesLine).
Приведенный код для пересчета в 4.0 тоже немного подкорректировали.

Кстати, опять же про 3.0. Не проверял, но скорее всего так:
Проверка

if (!InventCostItemDim::isBlanketOrderReference(inventTrans))

вообще некорректной получается, если стоит галка "Удаление строк заказа после отгрузки" в параметрах Расчетов с Клиентами.
В методе делается поиск строки контракта по номеру лота. И этой строки может уже не существовать.