[Master]-Rounding Differences in ARAP with Deferrals and Non-Deductible VAT

[DeepsShukla]

Workitem Bug 642527: [Main]-Rounding Differences in ARAP with Deferrals and Non-Deductible VAT
Fixes AB#64527

[github-actions]

$\textbf{🟠\ High\ Severity\ —\ Events} \quad \color{gray}{\texttt{\small Iteration\ 1}}$

The local IntegrationEvent OnBeforeInsertDeferralNonDeductibleVATGLEntries in codeunit 12 ("Gen.

Jnl.-Post Line", W1 layer) removed its existing var parameter VATAmountRounding and replaced it with two new parameters PositiveVATAmountRounding and NegativeVATAmountRounding. Even though the event publisher is local, subscribers can still bind to it by parameter name; removing/renaming an existing parameter breaks any existing subscriber that references VATAmountRounding by name, per AS0063/AS0077. New parameters may be added at any position on a local/internal event, but the existing parameter name, type, and var mode must be preserved.

Knowledge:

• microsoft/knowledge/events/treat-local-and-internal-events-as-subscriber-contracts.md

_{👍 useful · ❤️ especially valuable · 👎 wrong - reply with why}

[github-actions]

Copilot PR Review

Iteration 1 · Outcome: completed

Knowledge source: https://github.com/microsoft/BCQuality@186d8a131465475c79244d994acb872cd5c0d4bf

Findings by domain

Findings split into Knowledge-backed (cite a BCQuality article) and Agent (the agent's own judgement, no matching BCQuality rule).

Domain	Findings	Knowledge-backed	Agent	Inline	Fallback
Events	1	1	0	1	0

Totals: 1 knowledge-backed · 0 agent findings.

Orchestrator pre-filter (2 file(s) excluded)

• layer-disabled (knowledge) : 2 file(s)

_{Findings produced by the AL review agent v1.7.3. Reply 👎 on any inline comment to flag false positives.}