Skip to content

[master] NORWAY - Test Report (ID 2) cannot be run and throw validation errors about field reference that is not more included in the dataset of the Norway version#9530

Open
neeleshsinghal wants to merge 1 commit into
mainfrom
bugs/Bug-642372-Test-Report-cannot-be-run-and-throw-validation-errors
Open

[master] NORWAY - Test Report (ID 2) cannot be run and throw validation errors about field reference that is not more included in the dataset of the Norway version#9530
neeleshsinghal wants to merge 1 commit into
mainfrom
bugs/Bug-642372-Test-Report-cannot-be-run-and-throw-validation-errors

Conversation

@neeleshsinghal

@neeleshsinghal neeleshsinghal commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

Issue: Report fails validation/rendering on missing dataset fields.
Cause: NO layout referenced old field names (VATCaption/BalVATCodeCaption/VATCode/BalVATCode_GenJnlLine) after they were renamed to VATNumber....
Solution: Updated the 4 references to the new field names so the layout matches the dataset.

Workitem: Bug 642372: [master][all-e]NORWAY - Test Report (ID 2) cannot be run and throw validation errors about field reference that is not more included in the dataset of the Norway version

Fixes AB#642372

@github-actions github-actions Bot added this to the Version 29.0 milestone Jul 16, 2026
@neeleshsinghal neeleshsinghal added the Finance GitHub request for Finance area label Jul 16, 2026
@neeleshsinghal neeleshsinghal marked this pull request as ready for review July 16, 2026 09:33
@neeleshsinghal neeleshsinghal requested a review from a team July 16, 2026 09:33
@github-actions

Copy link
Copy Markdown
Contributor

Copilot PR Review

Iteration 1 · Outcome: no-knowledge

The only changed file is a Report layout (GeneralJournalTest.rdlc), an RDLC XML resource, not AL source. Its four edits correct dataset-column references (VATCode_GenJnlLine -> VATNumber_GenJnlLine, etc.) to match the already-renamed columns in GeneralJournalTest.Report.al; no stale references to the old column names remain. No AL sub-skill's knowledge domain (performance, security, privacy, upgrade, style, UI, error-handling, events, interfaces, breaking-changes, web-services, testing, data-modeling, query, appsource, telemetry) targets RDLC layout markup, so every sub-skill is not applicable to this diff. The cross-cutting self-review pass also found no defect: the change is a mechanical, internally-consistent rename fix with no remaining inconsistency.

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

No findings were posted for this iteration.

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.

@neeleshsinghal neeleshsinghal enabled auto-merge July 16, 2026 12:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Finance GitHub request for Finance area

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants