[Event Request] page 7001 "Price List Lines" - OnAfterUpdateColumnVisibility

[auzhhv]

Why do you need this change?

Can you change IncludeSender to true, so i can get information from the page and be able to set visibility for my own fields?

    //[IntegrationEvent(false, false)]
    [IntegrationEvent(true, false)]
    local procedure OnAfterUpdateColumnVisibility(PriceListHeader: Record "Price List Header"; var SourceTypeVisible: Boolean; var JobSourceTypeVisible: Boolean)
    begin
    end;

Describe the request

..
Internal work item: AB#615080

[AleksandricMarko]

Thank you for your request. To process this event modification, please provide the following missing information:

1. Complete Description:
The "Describe the request" section is incomplete. Please provide:

• What specific problem you're trying to solve
• What information from the page you need to access
• Why the current event parameters are insufficient
• How you plan to use the page sender reference

2. Detailed Justification for IncludeSender=true:
Changing IncludeSender to true is a critical modification that requires careful justification. Please explain:

• Why subscribers MUST have access to the page instance - what specific page properties, methods, or state do you need to access?
• Why the current event parameters (PriceListHeader, SourceTypeVisible, JobSourceTypeVisible) are insufficient - what can't you accomplish with these?
• What alternatives you evaluated - why can't you achieve your goal by adding parameters to the event instead?
• Specific use case - describe the exact scenario where your extension needs page instance access

Please update the issue with this information so we can evaluate the request properly.

[auzhhv]

Thank you for your request. To process this event modification, please provide the following missing information:

1. Complete Description: The "Describe the request" section is incomplete. Please provide:

• What specific problem you're trying to solve
• What information from the page you need to access
• Why the current event parameters are insufficient
• How you plan to use the page sender reference

2. Detailed Justification for IncludeSender=true: Changing IncludeSender to true is a critical modification that requires careful justification. Please explain:

• Why subscribers MUST have access to the page instance - what specific page properties, methods, or state do you need to access?
• Why the current event parameters (PriceListHeader, SourceTypeVisible, JobSourceTypeVisible) are insufficient - what can't you accomplish with these?
• What alternatives you evaluated - why can't you achieve your goal by adding parameters to the event instead?
• Specific use case - describe the exact scenario where your extension needs page instance access

Please update the issue with this information so we can evaluate the request properly.

Hi Alexander,

1. Complete Description:
We have added custom fields that should behave simular to other fields in the same page regarding visibility. These fields have custom criterias for visibility, but the points where our visibility variables should be updated is the same as the existing fields. The event is called OnAfterUpdateColumnVisibility, but it's not possible to update visibility for other fields because of access. As we cannot add custom variables to events, we need access to find and change other events on pages. We will add procedures in the a page extension to access our visibility variables. Ex. SetField1Visibility(NewVisibility) where Field1 is the name of our field.

2. Detailed Justification for IncludeSender=true:
We must have access to the page to be able to set the our own visibility variables.
It makes no sense to put it other places as the name of the event applies that its the correct place to change visibility. And I have not found other places it makes sense to do the same. And this is probably the reason why this is the place base application has the same code.

[AleksandricMarko]

✅ Analysis complete - approved for implementation

---

Implementation:

Change IncludeSender from false to true on the existing OnAfterUpdateColumnVisibility event in Page 7001 "Price List Lines" to allow subscribers to access the page instance.

Current code (around line 490):

[IntegrationEvent(false, false)]
local procedure OnAfterUpdateColumnVisibility(PriceListHeader: Record "Price List Header"; var SourceTypeVisible: Boolean; var JobSourceTypeVisible: Boolean)
begin
end;

Proposed change:

[IntegrationEvent(true, false)]
local procedure OnAfterUpdateColumnVisibility(PriceListHeader: Record "Price List Header"; var SourceTypeVisible: Boolean; var JobSourceTypeVisible: Boolean)
begin
end;

---

File: PriceListLines.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: 43889.