636017 Move Purchases report action tooltips to report objects#9549
636017 Move Purchases report action tooltips to report objects#9549V-CWurtzen wants to merge 2 commits into
Conversation
Move the report action ToolTips for 18 Purchases reports onto the report objects and remove the now-duplicate ToolTip from the W1 page actions that run them (purchase document lists, vendor pages and the Purchasing Agent role center, plus the Administrator role center and Data Administration). Covers Purchase Statistics, Purchase Reservation Avail., the Vendor - Balance to Date / Detail Trial Balance / Trial Balance / Labels / Order Detail / Order Summary / Payment Receipt / Purchase List set, Vendor Item Catalog, Vendor Register, Vendor/Item Purchases, Date Compress Vendor Ledger, and the Delete Invoiced Purch. Orders / Invd Blnkt Purch Orders / Invd Purch. Ret. Orders set. Where only a "Run the X report." stub existed (Day Book Vendor Ledger Entry) it is hoisted as the sole option. Reports with ObsoleteState = Pending/Removed are skipped (5).
Integrate the W1 Purchases tooltip move into the forked country layers, as required by the Miapp sync gate (VerifyMiappSync): - 55 page forks across 18 country layers: removed the duplicate ToolTip from the covered-report actions (305 lines). - 8 report forks (APAC VendorLabels/VendorPaymentReceipt; BE VendorPaymentReceipt/VendorPurchaseList; ES DateCompressVendorLedger/ VendorDetailTrialBalance; RU VendorOrderDetail/VendorOrderSummary): added the W1 report-level ToolTip. - 1 no-op integration (NA PurchasingAgentRoleCenter): the sync gate requires every fork of a changed W1 file in the changelist; it carries no covered action, so there is nothing to remove.
Copilot PR ReviewIteration 1 · Outcome: no-knowledge
Knowledge source: https://github.com/microsoft/BCQuality@186d8a131465475c79244d994acb872cd5c0d4bf No findings were posted for this iteration. Orchestrator pre-filter (2 file(s) excluded)
Findings produced by the AL review agent v1.7.3. Reply 👎 on any inline comment to flag false positives. |
1 participant
What & why
Moves the report action ToolTips for the Purchases reports off the page actions that run them and onto the report objects themselves, and integrates that move into every forked country layer. The tooltip is now defined once on the report and applies wherever the report is invoked, and the now-duplicate
ToolTipis removed from the page actions in W1 and its country forks.18 reports covered —
Purchase Statistics,Purchase Reservation Avail., theVendor - Balance to Date/Detail Trial Balance/Trial Balance/Labels/Order Detail/Order Summary/Payment Receipt/Purchase Listset,Vendor Item Catalog,Vendor Register,Vendor/Item Purchases,Date Compress Vendor Ledger, and theDelete Invoiced Purch. Orders/Invd Blnkt Purch Orders/Invd Purch. Ret. Ordersset.Where only a
Run the X report.stub existed (Day Book Vendor Ledger Entry), it is hoisted as the sole option — happy to replace with better wording if a reviewer suggests it. Reports withObsoleteState = Pending/Removedare skipped (5).This is part of the "move report action tooltips to report objects" cleanup (CP0529-331 / slice 636017).
Changes
ToolTipadded to the 18 reports; duplicate removed from the W1 actions that run them (purchase document lists, vendor pages and the Purchasing Agent role center, plus the Administrator role center and Data Administration). 31 files.VerifyMiappSync):ToolTipremoved from the covered actions (305 lines).APAC VendorLabels/VendorPaymentReceipt;BE VendorPaymentReceipt/VendorPurchaseList;ES DateCompressVendorLedger/VendorDetailTrialBalance;RU VendorOrderDetail/VendorOrderSummary): report-levelToolTipadded to match W1.NA PurchasingAgentRoleCenter): the sync gate requires every fork of a changed W1 file in the changelist; it carries no covered action, so there is nothing to remove.Total: 95 files (31 W1 + 64 country), tooltip-only.
Linked work
Fixes AB#636017
How I validated this
What I tested and the outcome
Metadata-only change (
ToolTipproperty). The diff is tooltip-only (reportToolTipadditions + page-actionToolTipremovals), with BOM/EOL preserved and no other lines changed; the only non-tooltip edit is a single trailing newline onNA PurchasingAgentRoleCenter.Page.al(no-op sync touch). Country propagation was produced by runningInvoke-Miappto identify the fork set, then applying the move surgically (Invoke-Miapp's auto-merge mis-aligned on the structurally-divergent role centers, so its output was not used directly). The set of changed forks matches exactly whatVerifyMiappSyncrequires — no missing forks, no extras.No tests added:
ToolTipis a design-time metadata property with no runtime behavior to assert; there is no existing tooltip test harness for these objects.Risk & compatibility
Low. No code, schema, table, or behavior change — only the
ToolTipdesign property moves from page action to report, in W1 and its country forks. No upgrade/data impact, no permission/telemetry impact, no breaking change. Same cleanup pattern already merged for Warehouse, Manufacturing, Service, Sales and Inventory, landed as the Pricing pilot (#9433), and applied to CashFlow (#9528), Finance (#9526), Bank (#9540), CRM (#9544), Cost Accounting (#9545), Fixed Assets (#9546) and Projects (#9548) on the new BCApps pipeline.