[W1][Codeunit][7312][Create Pick] Add integration event OnBeforeOnCreateTempLine in Procedure CreateTempLine

[mavohra]

Why do you need this change?

We need to skip pick line creation entirely for specific items at the very start of CreateTempLine in Codeunit 7312 "Create Pick", before any quantities are reset or location data is loaded.

The standard procedure begins immediately with TotalQtyPickedBase := 0 and GetLocation(LocationCode), with no hook before this block. There is no way for an extension to evaluate item-level conditions and exit early without those side effects having already occurred.

No event exists before TotalQtyPickedBase := 0 in CreateTempLine that exposes ItemNo, LocationCode, and VariantCode together with an IsHandled flag to allow a subscriber to suppress the entire procedure.

Describe the request

Add an integration event OnBeforeOnCreateTempLine in CreateTempLine in Codeunit 7312 "Create Pick" before TotalQtyPickedBase is reset and before GetLocation is called, so that a subscriber can exit early and suppress pick line creation for a given item.

procedure CreateTempLine(LocationCode: Code[10]; ItemNo: Code[20]; VariantCode: Code[10]; UnitofMeasureCode: Code[10]; FromBinCode: Code[20]; ToBinCode: Code[20]; QtyPerUnitofMeasure: Decimal; QtyRoundingPrecision: Decimal; QtyRoundingPrecisionBase: Decimal; var TotalQtytoPick: Decimal; var TotalQtytoPickBase: Decimal)
var
    ...
    IsHandled: Boolean;
begin
    IsHandled := false;
    OnBeforeOnCreateTempLine(LocationCode, ItemNo, VariantCode, IsHandled); // <---- New Event
    if IsHandled then
        exit;

    TotalQtyPickedBase := 0;
    GetLocation(LocationCode);

    InitCalculationSummary(LocationCode, ItemNo, VariantCode, UnitofMeasureCode, FromBinCode, TotalQtytoPick, TotalQtytoPickBase);

    if CheckReservationAndUpdateQtyToPick(LocationCode, ItemNo, VariantCode, UnitofMeasureCode, FromBinCode, QtyPerUnitofMeasure, TotalQtytoPick, TotalQtytoPickBase) then begin
        FinalizeCalculationSummary();
        exit;
    end;

Event Signature:

[IntegrationEvent(false, false)]
local procedure OnCreateTempLineOnBeforeDoNotPickItem(LocationCode: Code[10]; ItemNo: Code[20]; VariantCode: Code[10]; var IsHandled: Boolean)
begin
end;

Alternatives evaluated: No event currently exists before TotalQtyPickedBase := 0 in the public CreateTempLine overload. OnCreateTempLineOnBeforeCheckReservation fires later inside CheckReservationAndUpdateQtyToPick after TotalQtyPickedBase has already been reset and GetLocation has already executed, making it too late to suppress the full procedure cleanly. No other event in CreateTempLine is positioned before the quantity reset and location load.
Internal work item: AB#636049

[github-actions]

Hi @mavohra,

Thank you for your extensibility request. To help us process it efficiently, please update the issue to include the required information.

📚 Extensibility Guidelines

Missing Information:

• performance_considerations: Describe the expected execution frequency of this IsHandled event and any anticipated performance impact.
• data_sensitivity_review: Confirm that the event does not expose sensitive or security-relevant data.
• multi_extension_interaction: Acknowledge possible conflicts or expected behavior when multiple extensions subscribe to this new IsHandled event.

Please update the issue with the missing details. We'll automatically re-analyze once updated.

Note

Automatically generated by the Triage Agent — advisory only. An engineer will review before any action is taken.

Tip

Not accurate? Reply with /not-accurate (optional: add a short explanation). Your feedback improves future responses.

[mavohra]

Performance considerations: The public CreateTempLine overload runs once per pick source line. The event fires once at the very start of the procedure, before TotalQtyPickedBase is reset and before GetLocation is called. When IsHandled is set, the subscriber exits immediately and none of the pick calculation executes, so this is a net performance improvement for items that need to be skipped. When IsHandled is not set, the only cost is one boolean evaluation at the entry point.

Data sensitivity review: The event signature exposes LocationCode, ItemNo, VariantCode, and IsHandled. These are routine pick calculation identifiers, the same values already passed into the public procedure. Nothing sensitive is exposed.

Multi-extension interaction: Unlikely

[AleksandricMarko]

✅ Analysis complete - approved for implementation

---

Implementation:

Add a new IsHandled integration event OnBeforeCreateTempLine in the public CreateTempLine procedure of Codeunit 7312 "Create Pick" before TotalQtyPickedBase := 0 and GetLocation, to allow suppressing pick line creation for specific items.

Location in CreateTempLine procedure (around line 126):

procedure CreateTempLine(LocationCode: Code[10]; ItemNo: Code[20]; VariantCode: Code[10]; UnitofMeasureCode: Code[10]; FromBinCode: Code[20]; ToBinCode: Code[20]; QtyPerUnitofMeasure: Decimal; QtyRoundingPrecision: Decimal; QtyRoundingPrecisionBase: Decimal; var TotalQtytoPick: Decimal; var TotalQtytoPickBase: Decimal)
var
    ...
    IsHandled: Boolean;
begin
    IsHandled := false;
    OnBeforeCreateTempLine(LocationCode, ItemNo, VariantCode, IsHandled);  // <-- New event
    if IsHandled then
        exit;

    TotalQtyPickedBase := 0;
    GetLocation(LocationCode);

    InitCalculationSummary(LocationCode, ItemNo, VariantCode, UnitofMeasureCode, FromBinCode, TotalQtytoPick, TotalQtytoPickBase);

Event Publisher (add at the end of the codeunit with other events):

[IntegrationEvent(false, false)]
local procedure OnBeforeCreateTempLine(LocationCode: Code[10]; ItemNo: Code[20]; VariantCode: Code[10]; var IsHandled: Boolean)
begin
end;

---

File: CreatePick.Codeunit.al

Note

Automatically generated by the Triage Agent — advisory only. An engineer will review before any action is taken.

Tip

Not accurate? Reply with /not-accurate (optional: add a short explanation). Your feedback improves future responses.