#10878 Ruleset hardening: promote five low-risk rules to Error
Removes the Warning overrides for AL0254, AL0523, AA0242, AA0224 and AW0007 from the base ruleset so they apply as errors. The findings are fixed in code across many localization layers, with narrow suppressions kept for accepted existing behavior.
- 52files
- 50AL files
- 35objects
Pull request · merged into main (BC30) on 2026-09-03 by JesperSchulz · 52 files (+280 -59), 50 AL · refactor · tier official · reviewed (checked by Opus)
What changed
- Five rules (AL0254, AL0523, AA0242, AA0224, AW0007) now apply at Error level in the repo ruleset
- Partial-record loads fixed by adding fields to SetLoadFields calls
- Missing captions added and prepayment test report request options separated from the internal option
- New Employee Ledger Entry keys added for Closed by Entry No. and NL proposal sorting
- 46 ID-qualified suppressions kept; overrides drop from 81 to 76
AL objects touched
- Report 18468 “Stock Register for Job Work”, INGST (IN layer): modified
- Report 18933 “Voucher Register”, INVoucherInterface (IN layer): modified
- Page 46863 “BC14 Migration Error Overview”, HybridBC14: modified
- Report 28023 “Deposit Slip”, Base Application (AU layer): modified
- Report 2000007 “File SEPA 001.001.09 Pmts”, Base Application (BE layer): modified
- Report 2000005 “File SEPA Payments”, Base Application (BE layer): modified
- Report 3010541 “DTA File”, Base Application (CH layer): modified
- Report 3010545 “DTA Payment Journal”, Base Application (CH layer): modified
- Report 3010543 “DTA Payment Order”, Base Application (CH layer): modified
- Report 3010542 “EZAG File”, Base Application (CH layer): modified
- Report 3010544 “EZAG Payment Order”, Base Application (CH layer): modified
- Report 10707 “Make 347 Declaration”, Base Application (ES layer): modified
- Report 10814 “Vendor Journal”, Base Application (FR layer): modified
- Report 10813 “Customer Journal”, Base Application (FR layer): modified
- Codeunit 5708 “Release Transfer Document”, Base Application: modified
- Page 10144 “Posted Deposit Subform”, Base Application (CA layer): modified
- Report 10161 “Serial Number Status/Aging”, Base Application (CA layer): modified
- Table 5222 “Employee Ledger Entry”, Base Application: modified
- Report 11400 “CBG Posting - Test”, Base Application (NL layer): modified
- Report 15000002 “Remittance Test Report”, Base Application (NO layer): modified
- Codeunit 22 “Item Jnl.-Post Line”, Base Application: modified
- Report 12441 “Customer Accounting Card”, Base Application (RU layer): modified
- Report 14910 “Customer - Reconciliation Act”, Base Application (RU layer): modified
- Page 615 “IC Inbox Transactions”, Base Application: modified
- Codeunit 108 “Net Cust/Vend Balances Mgt.”, Base Application: modified
- Page 6404 “Purchase Document Entity”, Base Application: modified
- Table 5405 “Production Order”, Base Application: modified
- Report 6698 “Move Negative Purchase Lines”, Base Application: modified
- Report 412 “Purchase Prepmt. Doc. - Test”, Base Application: modified
- Report 6699 “Move Negative Sales Lines”, Base Application: modified
- Report 212 “Sales Prepmt. Document Test”, Base Application: modified
- Page extension 6491 “Serv. Workflow Customer Entity”, Base Application: modified
- Page 6408 “Workflow - Customer Entity”, Base Application: modified
- Page 6409 “Workflow - Item Entity”, Base Application: modified
- Page 6410 “Workflow - Vendor Entity”, Base Application: modified
src/Layers/APAC/BaseApp/Inventory/Posting/ItemJnlPostLine.Codeunit.al: modified, not in the BC30 snapshotsrc/Layers/APAC/BaseApp/Purchases/Reports/PurchasePrepmtDocTest.Report.al: modified, not in the BC30 snapshotsrc/Layers/APAC/BaseApp/Sales/Reports/CustomerLabels.Report.al: modified, not in the BC30 snapshotsrc/Layers/CH/BaseApp/Inventory/Posting/ItemJnlPostLine.Codeunit.al: modified, not in the BC30 snapshotsrc/Layers/ES/BaseApp/Inventory/Posting/ItemJnlPostLine.Codeunit.al: modified, not in the BC30 snapshotsrc/Layers/ES/BaseApp/Purchases/Reports/PurchasePrepmtDocTest.Report.al: modified, not in the BC30 snapshotsrc/Layers/ES/BaseApp/Sales/Reports/SalesPrepmtDocumentTest.Report.al: modified, not in the BC30 snapshotsrc/Layers/IT/BaseApp/Inventory/Posting/ItemJnlPostLine.Codeunit.al: modified, not in the BC30 snapshotsrc/Layers/IT/BaseApp/Purchases/Reports/PurchasePrepmtDocTest.Report.al: modified, not in the BC30 snapshotsrc/Layers/IT/BaseApp/Sales/Reports/SalesPrepmtDocumentTest.Report.al: modified, not in the BC30 snapshotsrc/Layers/NA/BaseApp/Purchases/Reports/PurchasePrepmtDocTest.Report.al: modified, not in the BC30 snapshotsrc/Layers/NA/BaseApp/Sales/Reports/SalesPrepmtDocumentTest.Report.al: modified, not in the BC30 snapshot
Context
- Work items: AB#640773
- Labels: AL: Apps (W1), Ownership: Needs Review, Team: Other
- Systems: finance, development, inventory, purchasing, sales, hr, administration, integration, manufacturing
Source: metadata of a merged pull request in microsoft/BCApps (bcapps): title, labels, changed paths and counts, with a summary in our words. No source text and no description beyond one short quote (D10, D61).