[Event Request] codeunit 99000855 "Planning-Get Parameters"

[Simon110394]

Why do you need this change?

We need an event before evaluating "Safety Lead Time", in order to be able to give it a value, according to custom logics.
The actual event OnAtSKUOnBeforeSetSafetyLeadTime is not suffient because the aim is to assign a value to the field "Safety Lead Time". So if I exploit the event OnAtSKUOnBeforeSetSafetyLeadTime, the value I assign to the field can be overwritten by the standard procedure. That's why I need to manage the IsHandled variable.

Describe the request

In procedure AtSKU of codeunit 99000855 "Planning-Get Parameters" we need an event:

[IntegrationEvent(false, false)]
local procedure OnAtSKUOnBeforeSetSafetyLeadTime2(var Ishandled: Boolean; var GlobalSKU: Record "Stockkeeping Unit"; ManualScheduling: Boolean)
begin
end;

Changes between **:

procedure AtSKU(var SKU: Record "Stockkeeping Unit"; ItemNo: Code[20]; VariantCode: Code[10]; LocationCode: Code[10])
var
**IsHandled: Boolean;**
    begin
        GetInventorySetUp();
        if (ItemNo <> GlobalSKU."Item No.") or
            (VariantCode <> GlobalSKU."Variant Code") or
            (LocationCode <> GlobalSKU."Location Code")
        then begin
            Clear(GlobalSKU);
            GlobalSKU.SetRange("Item No.", ItemNo);
            GlobalSKU.SetRange("Variant Code", VariantCode);
            GlobalSKU.SetRange("Location Code", LocationCode);
            if not GlobalSKU.FindFirst() then begin
                GetItem(ItemNo);
                GlobalSKU."Item No." := ItemNo;
                GlobalSKU."Variant Code" := VariantCode;
                GlobalSKU."Location Code" := LocationCode;
                GlobalSKU.CopyFromItem(Item);
                OnAtSKUOnAfterCopyFromItem(GlobalSKU, Item, ItemNo, VariantCode, LocationCode);
                if LotForLot then begin
                    GlobalSKU."Reorder Point" := 0;
                    GlobalSKU."Maximum Inventory" := 0;
                    GlobalSKU."Reorder Quantity" := 0;
                    case Item."Reordering Policy" of
                        GlobalSKU."Reordering Policy"::" ":
                            GlobalSKU."Reordering Policy" := GlobalSKU."Reordering Policy"::" ";
                        GlobalSKU."Reordering Policy"::Order:
                            GlobalSKU."Reordering Policy" := GlobalSKU."Reordering Policy"::Order;
                        GlobalSKU."Reordering Policy"::"Lot-for-Lot":
                            GlobalSKU."Reordering Policy" := GlobalSKU."Reordering Policy"::"Lot-for-Lot";
                        else begin
                            GlobalSKU."Reordering Policy" := GlobalSKU."Reordering Policy"::"Lot-for-Lot";
                            GlobalSKU."Plan Minimal Supply" := true;
                        end;
                    end;
                    if GlobalSKU."Reordering Policy" = GlobalSKU."Reordering Policy"::"Lot-for-Lot" then
                        GlobalSKU."Include Inventory" := true
                    else
                        GlobalSKU."Include Inventory" := Item."Include Inventory";
                    GlobalSKU."Maximum Order Quantity" := 0;
                    GlobalSKU."Safety Stock Quantity" := 0;
                    GlobalSKU."Order Multiple" := 0;
                    GlobalSKU."Overflow Level" := 0;
                end;
            end;
            SetComponentsAtLocation(LocationCode);
        end;

        OnAtSKUOnBeforeSetSafetyLeadTime(GlobalSKU, ManualScheduling);
		
		**IsHandled := false;
		OnAtSKUOnBeforeSetSafetyLeadTime2(GlobalSKU, ManualScheduling);
		if not IsHandled then**
			if not ManualScheduling then
				if Format(GlobalSKU."Safety Lead Time") = '' then
					if Format(InventorySetup."Default Safety Lead Time") <> '' then
						GlobalSKU."Safety Lead Time" := InventorySetup."Default Safety Lead Time"
					else
						Evaluate(GlobalSKU."Safety Lead Time", '<0D>');


        AdjustInvalidSettings(GlobalSKU);
        SKU := GlobalSKU;

        OnAfterAtSKU(SKU, GlobalSKU);
    end;

[github-actions]

Hi @Simon110394,

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:

• Alternatives evaluated: Explain which existing events or patterns were considered, including why the current OnAtSKUOnBeforeSetSafetyLeadTime event is insufficient.
• Justification for IsHandled: Explain why bypass/handled behavior is required instead of using a regular event.
• Performance considerations: Provide brief notes on expected execution frequency and expected performance impact.
• Data sensitivity review: Confirm whether any sensitive data exposure is involved.
• Multi-extension interaction: Describe possible conflicts if multiple extensions subscribe and affect control flow.

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.

[Simon110394]

Hi, I've updated the request info.
Thanks

[github-actions]

Hi @Simon110394,

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: Provide brief notes on expected execution frequency and expected performance impact.
• Data sensitivity review: Confirm whether any sensitive data exposure is involved.
• Multi-extension interaction: Describe possible conflicts if multiple extensions subscribe and affect control flow.

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.