[Event Request] - Page 5871 "Item Availability by BOM Level" - OnOpenPage

[auzhhv]

Why do you need this change?

Set IncludeSender to true.

    //[IntegrationEvent(false, false)]
    [IntegrationEvent(true, false)]
    local procedure OnOpenPageOnBeforeRefreshPage(var BOMBuffer: Record "BOM Buffer")
    begin
    end;

Describe the request

We want to override the LocationFilter on the page. The variabel is protected var, but it's not available in the OnOpenPageOnBeforeRefreshPage event. Because of this we can set det filter on OnOpenPage, but then we have to run RefreshPage again for the change to take effect.

Internal work item: [AB#620964](https://dynamicssmb2.visualstudio.com/1fcb79e7-ab07-432a-a3c6-6cf5a88ba4a5/_workitems/edit/620964)

[AleksandricMarko]

Hi @auzhhv,

According to our implementation rules, we recommend using the this keyword instead of IncludeSender=true as it provides a more explicit and intentional approach.

Recommended approach:

Pass the page instance using the this keyword, which would allow you to access public members:

[IntegrationEvent(false, false)]
local procedure OnOpenPageOnBeforeRefreshPage(var BOMBuffer: Record "BOM Buffer"; Sender: Page "Item Availability by BOM Level")
begin
end;

And in the OnOpenPage trigger, call it with:

OnOpenPageOnBeforeRefreshPage(Rec, this);

Please confirm if this alternative approach works for your scenario, or explain why you specifically need IncludeSender=true instead of the this keyword approach.

[auzhhv]

Hi @auzhhv,

According to our implementation rules, we recommend using the this keyword instead of IncludeSender=true as it provides a more explicit and intentional approach.

Recommended approach:

Pass the page instance using the this keyword, which would allow you to access public members:

[IntegrationEvent(false, false)]
local procedure OnOpenPageOnBeforeRefreshPage(var BOMBuffer: Record "BOM Buffer"; Sender: Page "Item Availability by BOM Level")
begin
end;
And in the OnOpenPage trigger, call it with:

OnOpenPageOnBeforeRefreshPage(Rec, this);
Please confirm if this alternative approach works for your scenario, or explain why you specifically need IncludeSender=true instead of the this keyword approach.

If "this" can be included in the event as reference, that will work :)

[AleksandricMarko]

Hi @auzhhv did you close the issue intentionally or by mistake?

[auzhhv]

Hi @auzhhv did you close the issue intentionally or by mistake?

That was by mistake. My bad

[AleksandricMarko]

✅ Analysis complete - approved for implementation

---

Implementation:

Add a Sender parameter to the existing event OnOpenPageOnBeforeRefreshPage to allow access to the page instance and its protected members (e.g., LocationFilter).

Event Publisher (modify at line 656-659):

Current:

[IntegrationEvent(false, false)]
local procedure OnOpenPageOnBeforeRefreshPage(var BOMBuffer: Record "BOM Buffer")
begin
end;

New:

[IntegrationEvent(false, false)]
local procedure OnOpenPageOnBeforeRefreshPage(Sender: Page "Item Availability by BOM Level"; var BOMBuffer: Record "BOM Buffer")
begin
end;

Event Call in OnOpenPage trigger (modify at line 473):

Current:

OnOpenPageOnBeforeRefreshPage(Rec);

New:

OnOpenPageOnBeforeRefreshPage(this, Rec);

---

File: ItemAvailabilitybyBOMLevel.Page.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: 45983.