microsoft / ALAppExtensions

Repository for collaboration on Microsoft AL application add-on and localization extensions for Microsoft Dynamics 365 Business Central.
MIT License
773 stars 610 forks source link

[Change Request] table 36 "Sales Header" #26693

Open innonav-dev opened 3 months ago

innonav-dev commented 3 months ago

Describe the request

In table 36 "Sales Header" we could use IsHandled parameter in 2 existing publisher.

First one is OnAfterCheckSellToCust - we suggest following improvement:

                IsHandled := false;
                OnAfterCheckSellToCust(Rec, xRec, Customer, CurrFieldNo, IsHandled); //TODO Event Req Param+
                If IsHandled then begin
                    exit;
                end;

                [IntegrationEvent(false, false)]
                local procedure OnAfterCheckBillToCust(var SalesHeader: Record "Sales Header"; xSalesHeader: Record "Sales Header"; Customer: Record Customer; var IsHandled: Boolean)
                begin
                end;

Second one is OnAfterCheckSellToCust:

                IsHandled := false;
                OnAfterCheckBillToCust(Rec, xRec, Customer, IsHandled); //TODO Event Req
                If IsHandled then begin
                    exit;
                end;

                [IntegrationEvent(false, false)]
                local procedure OnAfterCheckSellToCust(var SalesHeader: Record "Sales Header"; xSalesHeader: Record "Sales Header"; Customer: Record Customer; CurrentFieldNo: Integer; var IsHandled: Boolean)
                begin
                end;

Additional context

We use it, to handle some customer specific requirements. Internal work item: AB#538788

tomasevicst commented 3 months ago

Hi,

These two publishers are the same?

tomasevicst commented 3 months ago

Can you explain what is the reason for skipping all the code that are comming after mentioned publisher?