[Subcontracting] [28.x] Item Charge for Last-Operation Subcontracting Receipt Linked to Wrong Ledger Entry

[stevengrossGOB]

Bug: Item Charge for Last-Operation Subcontracting Receipt Linked to Wrong Ledger Entry

Scenario ID

AB#640042

Affected Objects

Type	Name	ID
Codeunit (App)	Subc. Purch. Post Ext	99001535
Codeunit (Base App)	Item Jnl.-Post Line	22

Reproduction Steps

1. Manufacturing item with a single-operation routing (subcontracting work center = last operation). Certified routing + BOM.
2. Released production order → subcontracting purchase order.
3. Post purchase receipt (Receive = Yes, Invoice = No).
4. Create a separate purchase invoice with an item charge assigned to the receipt line and post it.

False Behavior

ValueEntry."Item Ledger Entry No." points to an arbitrary, unrelated ILE whose entry number coincidentally equals the Capacity Ledger Entry number from the receipt — or posting fails with "The Item Ledger Entry does not exist". The production order reference and serial number on the linked ILE are wrong.

Expected Behavior

Tracked items (serial / lot)

ValueEntry."Item Ledger Entry No." = Output ILE, "Capacity Ledger Entry No." = 0.
Order Type = Production, correct Order No. and serial number.

Non-tracked items

ValueEntry."Capacity Ledger Entry No." = Capacity Ledger Entry from the service receipt, "Item Ledger Entry No." = 0.
Order Type = Production, correct Order No.

Root Cause

CopySubcontractingProdOrderFieldsToItemJnlLine unconditionally set Entry Type = Output and Subcontracting = true on the item journal line — including for last-operation item charges. This routed posting through PostOutput / InsertCapValueEntry regardless of item tracking, producing wrong value entry linkages.

Tracked items

PurchRcptLine."Item Rcpt. Entry No." = 0 for tracked items. PostItemChargePerRcpt uses CollectItemEntryRelation to distribute the charge across each Output ILE. With Entry Type = Output still set, PostOutput was called per ILE using the Output ILE entry number as a Capacity Ledger Entry key → error or wrong entry.

Non-tracked items

PurchRcptLine."Item Rcpt. Entry No." = CapLedgEntryNo. With Entry Type = Purchase (default from Purch.-Post, before our code ran), PostItem called GlobalItemLedgEntry.Get(CapLedgEntryNo) — loading whichever ILE happened to carry that number.

Fix

CopySubcontractingProdOrderFieldsToItemJnlLine now distinguishes last-operation from non-last-operation and tracked from non-tracked:

if PurchRcptLineIsLastOperation(PurchRcptLine) then begin
    if Item."Item Tracking Code" <> '' then begin
        // Tracked: PostItem path — Output ILE found via Item Entry Relation.
        // Order info comes from the ILE in InitValueEntry; no explicit setting needed.
        ItemJournalLine.Subcontracting := false;
        ItemJournalLine."Entry Type" := "Item Ledger Entry Type"::Purchase;
    end else begin
        // Non-tracked: PostOutput path — links to the Capacity Ledger Entry.
        // Order info must be set explicitly; InsertCapValueEntry reads it from the journal line.
        ItemJournalLine.Subcontracting := true;
        ItemJournalLine."Order Type"     := "Inventory Order Type"::Production;
        ItemJournalLine."Order No."      := PurchRcptLine."Prod. Order No.";
        ItemJournalLine."Order Line No." := PurchRcptLine."Prod. Order Line No.";
        ItemJournalLine."Entry Type"     := "Item Ledger Entry Type"::Output;
        // ... Type, No., Routing, Work Center fields ...
    end;
    exit;
end;
// Non-last-op: Subcontracting = true, Entry Type = Output (unchanged)

"Inventory Posting Group" and "Subc. Item Charge Assign." are set before the check for both paths.

Related Work Items / PRs

• Work item: AB#640042