[Event Request] [W1][Report][7000096][Redraw Receivable Bills] trigger OnPreReport

[jcarrascolbt]

Why do you need this change?

We need this event to be able to assign the requestpage variables without needing to show it to the user and also, if necessary, to be able to skip the control that retrieves the journal batch to automate it.

Describe the request

Our cliente want to be able to assign variables to the request page and be able to skip the error message to make a custom manage for that.

trigger OnPreReport()
begin
    if NewDueDate = 0D then
        Error(Text1100000);

>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>
IsHandled := false;
OnBeforeOnPreReport(PostingDate, NewDueDate, NewPmtMethod, IncludeDiscCollExpenses, IncludeRejExpenses, IncludeFinanceCharges, TemplName, BatchName, IsHandled,);
    if not IsHandled then
        if not GenJnlBatch.Get(TemplName, BatchName) then
            Error(Text1100001);
<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<
end;

[IntegrationEvent(false, false)]
local procedure OnBeforeOnPreReport(var PostingDate: Date; var NewDueDate: Date; var NewPmtMethod: Code[10]; var IncludeDiscCollExpenses: Boolean; var IncludeRejExpenses: Boolean; var IncludeFinanceCharges: Boolean; var TemplName: Code[10]; var BatchName: Code[10]; var IsHandled: Boolean)
begin
end;

Alternatives Evaluated
We have tried to use alternatives to avoid making the change, but we don't see a way to assign the variables from the request page.

Performance Considerations
This code will not negatively affect performance, we just need it to assign some variables to make this automatic.

Data Sensitivity Review
This event is just asked to skip a dialog window, it will not expose sensitive data.

Multi-Extension Interaction
The only risk is to reactivate the dialog window when it should be deactivated for an automatic process. Also there are a control that if u put the data in the GenJnlBatch ant try to make a get, BC doesnt get the right journal batch, so we need to be able to skip the comprobation
Internal work item: AB#618758

[AleksandricMarko]

Hello @jcarrascolbt
why IsHandled even is needed here?
If you want just to assign variables, would regular one be enough?

[jcarrascolbt]

Hello @AleksandricMarko ,
We need the ishandled because when we assigned variables, Get was not taking the right GenJnlBatch.
I think maybe is because this is into OnPreReport

[AleksandricMarko]

What about adding regular even in OnInit trigger?
Or better in trigger OnOpenPage for request page

[jcarrascolbt]

Meanwhile we can assign all of this variables i think yes, maybe in the OnOpenPage is enough to us.
The thing we want is that we can do this automatic and if u think in those trigger we can do it equal that i said to you its fine.

[AleksandricMarko]

We will add new event in OnOpenPage trigger

trigger OnOpenPage()
begin
      OnBeforeOnOpenRequestPagePage(PostingDate, NewDueDate, NewPmtMethod, IncludeDiscCollExpenses, IncludeRejExpenses, IncludeFinanceCharges, TemplName, BatchName);
end;

[IntegrationEvent(false, false)]
local procedure OnBeforeOnOpenRequestPagePage(var PostingDate: Date; var NewDueDate: Date; var NewPmtMethod: Code[10]; var IncludeDiscCollExpenses: Boolean; var IncludeRejExpenses: Boolean; var IncludeFinanceCharges: Boolean; var TemplName: Code[10]; var BatchName: Code[10])
begin
end;

Object: report 7000096 "Redraw Receivable Bills" (ES layer)

[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: 47814.