Why do you need this change?
I need to be able to change how customer related checks are financially voided. I need other types of checks processed normally.
I've tried using the OnBeforeFinancialVoidCheck() event at the start but can't achieve what I want because I would need to call local functions which then call other local functions so I end up having to replicate a lot of code. Also, I would need a reference to the GenJnlPostLine codeunit to ensure all journal lines are posted through the same instance of this codeunit but I can get access to it. There are other events that pass this codeunit but they are called too late.
Describe the request
Please can you extend the OnFinancialVoidCheckOnBeforeCheckBalAccountType to have an IsHandled parameter and if this is set to true by the subscriber then the case statement immediately following this should be skipped. This will also require the IsHandled variable to be set to false just before this event.
Adding this extra parameter will not break any existing subscribers, because subscribers do not need to have all the parameters included in the publisher, and the default value for this (false) will result in the case statement being executed as it currently does.
Performance considerations:
The event already exists so no performance issues
Data sensitivity review:
No sensitive data is exposed, there is just 1 additional paratemeter - the IsHandled boolean field
Multi-extension interaction:
There is already an IsHandled event at the start of this procedure in OnBeforeFinancialVoidCheck() so adding a new one has no additional risk. In fact there is less risk with this change as more of the standard code in this procedure would be called compared to subscribing to the OnBeforeFinancialVoidCheck() event and setting IsHandled there.
Why do you need this change?
I need to be able to change how customer related checks are financially voided. I need other types of checks processed normally.
I've tried using the OnBeforeFinancialVoidCheck() event at the start but can't achieve what I want because I would need to call local functions which then call other local functions so I end up having to replicate a lot of code. Also, I would need a reference to the GenJnlPostLine codeunit to ensure all journal lines are posted through the same instance of this codeunit but I can get access to it. There are other events that pass this codeunit but they are called too late.
Describe the request
Please can you extend the OnFinancialVoidCheckOnBeforeCheckBalAccountType to have an IsHandled parameter and if this is set to true by the subscriber then the case statement immediately following this should be skipped. This will also require the IsHandled variable to be set to false just before this event.
Adding this extra parameter will not break any existing subscribers, because subscribers do not need to have all the parameters included in the publisher, and the default value for this (false) will result in the case statement being executed as it currently does.
Performance considerations:
The event already exists so no performance issues
Data sensitivity review:
No sensitive data is exposed, there is just 1 additional paratemeter - the IsHandled boolean field
Multi-extension interaction:
There is already an IsHandled event at the start of this procedure in OnBeforeFinancialVoidCheck() so adding a new one has no additional risk. In fact there is less risk with this change as more of the standard code in this procedure would be called compared to subscribing to the OnBeforeFinancialVoidCheck() event and setting IsHandled there.