W1 codeunit 99000846 "Trans. Get Demand To Reserve" OnSyncTransferOrderLinesOnBeforeInsertReservationWkshLine

[RobertatK3]

Why do you need this change?

We like to add additional information from the TempTransferLine to the Reservation Wksh. Line and in addition change the conditions when to insert the record.

Describe the request

The new publisher should include the Reservation Wksh. Line record and the TempTransferLine record by var and a boolean parameter by var which can used to handle the insertion of the Reservation Wksh. Line record.
Example:
DoInsertReservationWkshLine := (ReservationWkshLine."Remaining Qty. to Reserve" > 0) and (ReservationWkshLine."Available Qty. to Reserve" > 0); OnSyncTransferOrderLinesOnBeforeInsertReservationWkshLine(ReservationWkshLine, TempTransferLine, DoInsertReservationWkshLine); if DoInsertReservationWkshLine then ReservationWkshLine.Insert(true);

local procedure OnSyncTransferOrderLinesOnBeforeInsertReservationWkshLine(var ReservationWkshLine: Record "Reservation Wksh. Line"; var TempTransferLine: Record "Transfer Line" temporary; var DoInsertReservationWkshLine: Boolean) begin end;

A similar publisher can be found in codeunit 99000839 "Sales Get Demand To Reserve" (which I requested some time ago).
Internal work item: AB#625778

[AleksandricMarko]

Hi Robert de Roos (@RobertatK3),

Thanks for submitting this extensibility request. To help us process it efficiently, please update the issue to include the required information.

📚 Extensibility Guidelines

Missing Information:

• Alternatives Evaluated: Please describe which existing events or patterns you tried and why they were insufficient for your scenario.

• Justification for IsHandled: Please explain specifically why a standard event is insufficient and why bypass functionality (IsHandled) is necessary. What business logic must be skipped/replaced? Why won't alternative approaches work?

• Performance Considerations: Please provide the expected execution frequency (e.g., per transaction, per batch) and assess the potential performance impact.

• Data Sensitivity Review: Please confirm whether the event exposes any sensitive data, or provide rationale if exposure is necessary.

• Multi-Extension Interaction: Please describe the risks if multiple extensions subscribe to this event and how conflicts would be mitigated.

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

[RobertatK3]

Hi Marko Aleksandric (@AleksandricMarko),

Here the requested information:

• Alternatives Evaluated: There are no alternative events or patterns available. Using the OnBeforeInsert trigger of the Reservation Wksh. Line will not work if the record is not inserted when there is nothing available to reserve, also I will not have the information from the local TempTransferLine available. Subscribing to the same publisher, OnCalculateDemandOnAfterSync, and copying the code and add our functionality will not work, because we could end up having two reservation worksheet Lines for the same transferline when both our conditions and the standard BC conditions are met.
• Justification for IsHandled: We want to enhance the reservation worksheet with the possibility to reserve from purchase order. We calculate the available quantity to reserve from purchase and want to add the condition available quantity to reserve from purchase > 0 to insert the reservation worksheet line. There is no alternative way (as answered alternitives evaluated).
• Performance Considerations: For every TempTransferLine we will calculate the quantity available to reserve from purchase and the performance impact will depend on the number of purchases.
• Data Sensitivity Review: The event does not expose any sensitive date.
• Multi-Extension Interaction: We ourself only add a condition when to insert the record and set values to fields we added in our extension. The only conflict possible is when an extension does not take the setting of the condition by others into account.

[AleksandricMarko]

✅ Analysis complete - approved for implementation

---

Implementation:

Add a new integration event in the SyncTransferOrderLines procedure of Codeunit 99000846 "Trans. Get Demand To Reserve" before ReservationWkshLine.Insert(true) in the loop.

Location in SyncTransferOrderLines procedure (around line 197):

DoInsertReservationWkshLine := (ReservationWkshLine."Remaining Qty. to Reserve" > 0) and (ReservationWkshLine."Available Qty. to Reserve" > 0);
OnSyncTransferOrderLinesOnBeforeInsertReservationWkshLine(ReservationWkshLine, TempTransferLine, DoInsertReservationWkshLine);
if DoInsertReservationWkshLine then
    ReservationWkshLine.Insert(true);

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

[IntegrationEvent(false, false)]
local procedure OnSyncTransferOrderLinesOnBeforeInsertReservationWkshLine(var ReservationWkshLine: Record "Reservation Wksh. Line"; var TempTransferLine: Record "Transfer Line" temporary; var DoInsertReservationWkshLine: Boolean)
begin
end;

---

File: TransGetDemandToReserve.Codeunit.al

[JesperSchulz]

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.

Build ID: 48589.