[Event Request] codeunit 88 "Sales Post via Job Queue" trigger OnRun()

[kmistrytnp]

Why do you need this change?

I want to be able to call my own custom sales posting routine which is a wrapper really around the standard sales posting - it has various logic including the ability to split the posting and call the standard posting routine multiple times.

I can achieve this in all the places calling the standard sales post (except one), i.e. in the following codeunits I have an IsHandled event available...

codeunit 79	"Sales-Post and Send" - OnCodeOnBeforePostSalesHeader(SalesHeader, TempDocumentSendingProfile, HideDialog, IsHandled);
codeunit 81 "Sales-Post (Yes/No)" - OnBeforeRunSalesPost(SalesHeader, IsHandled, SuppressCommit);
codeunit 82	"Sales-Post + Print" - OnBeforeConfirmPost(SalesHeader, HideDialog, IsHandled, SendReportAsEmail, DefaultOption);
codeunit 89 "Sales-Post + Email" - OnBeforePostAndEMail(SalesHeader, HideDialog, IsHandled, HideMailDialog);

but there is 1 codeunit where I am not able to do this because there is no IsHandled parameter like there is on all the other sales posting related codeunits...

codeunit 88 "Sales Post via Job Queue" - OnRunOnBeforeRunSalesPost(SalesHeader);

Describe the request

Please can you extend the event as follows...

Current code

        OnRunOnBeforeRunSalesPost(SalesHeader);
        if not Codeunit.Run(Codeunit::"Sales-Post", SalesHeader) then begin

New code

        IsSuccess := false;
        IsHandled := false;
        OnRunOnBeforeRunSalesPost(SalesHeader, IsSuccess, IsHandled);
        if not isHandled then
            IsSuccess := Codeunit.Run(Codeunit::"Sales-Post", SalesHeader);
        if not IsSuccess then begin

This will not break anything because...

1. The event is an IntegrationEvent not a BusinessEvent, so it can be changed
2. Added extra params does NOT break existing subscribers as subscriber parameters are optional
3. The default behaviour remains unchanged - i.e. it doesn't break anything

Adding an IsHandled parameter will make it consistent with all the other codeunits which call Sales Post as they all have an IsHandled parameter which allows you to bypass calling the standard Sales Post codeunit, this is the only codeunit which doesn't.

[kmistrytnp]

Performance considerations:
There is no performance impact

Data sensitivity review:
No sensitive data is exposed

Multi-extension interaction:
This event does not bypass any code in this codeunit except for the call to CU 80 "Sales Post". All the other codeunits which call "Sales Post" can already bypass calling that codeunit so doing the same for this one doesn't introduce any additional risk

[github-actions]

Hi @kmistrytnp,

Thank you for your extensibility request. Unfortunately, this request cannot be implemented.

Reason: This request cannot be approved. Adding an IsHandled pattern to an existing event is not allowed because it fundamentally changes the event's purpose and contract. Existing subscribers rely on the current behavior, and introducing IsHandled would break them by allowing the base logic to be bypassed — something the original subscribers were never designed to account for.

If you need bypass/handled functionality, please request a NEW separate event with IsHandled instead of modifying the existing one.

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.

[kmistrytnp]

/not-accurate - this doesn't break current behaviour and I explained why in my original request. I'm happy to have a new separate event with an IsHandled instead, but I really don't see the point of this when the existing event can be easily modified.

Please reconsider adding the IsHandled to the existing event as per the original request, OR add a new similar one at the same place working in the same way

[github-actions]

✅ Feedback recorded
Thanks — your /not-accurate feedback has been logged and will be reviewed.