Bug 629779: [Inventory] Show a single Export/Import item data action pair on Cost Adjustment and Item Card

[alexei-dobriansky]

Fixes AB#629779

Summary

The Inventory Cost Adjustment page and the Item Card each showed two overlapping data export/import actions when the Manufacturing app was installed: a standard Export/Import Item Data pair plus a production pair (Export/Import Production Item Data / a second Export Item Data on the Item Card). This created ambiguity about which one to use.

Change

The system now detects at runtime whether Manufacturing is enabled (ApplicationAreaMgmtFacade.IsManufacturingEnabled()) and shows exactly one pair:

• Manufacturing enabled -> the base pair is hidden (Visible = not ManufacturingEnabled) and the Manufacturing pair (ApplicationArea = Manufacturing) is shown, routing to the Mfg. Export Item Data XMLPort (production data).
• Manufacturing not enabled -> the base pair is shown, routing to the Export Item Data XMLPort.

In both cases the actions are captioned "Export item data" / "Import item data" - no "Production"/"manufacturing" wording is visible to the user.

Files

• CostAdjustmentOverview.Page.al - base pair re-captioned + gated on ManufacturingEnabled.
• MfgCostAdjustmentOverview.PageExt.al - Mfg pair re-captioned, tooltips aligned, promoted refs added, fixed a stray Xmlport. line break.
• ItemCard.Page.al - base Export item data action re-captioned + gated on ManufacturingEnabled.
• MfgItemCard.PageExt.al - Mfg action re-captioned, tooltip aligned.
• Propagated the Item Card change to the APAC/GB/IT/NA/RU BaseApp layers via miapp.

Notes

• Pure UI caption/visibility change; no schema or public API change.

[github-actions]

Copilot PR Review

Iteration 1 · Outcome: completed

Knowledge source: https://github.com/microsoft/BCQuality@822cae1b2771ac25f665f73369f69093bd4fd630

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

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

_{Findings produced by the Copilot CLI agent against BCQuality at 822cae1b2771ac25f665f73369f69093bd4fd630. Reply 👎 on any inline comment to flag false positives.}

[alexei-dobriansky]

Agentic PR Review - Round 1

Recommendation: Accept

What this PR does

This PR hides the standard item data actions when Manufacturing is enabled and lets the Manufacturing page extensions provide the visible actions instead. When Manufacturing is not enabled, the standard actions remain visible.

The changed action registrations look consistent. Cost Adjustment keeps the base Export/Import actionrefs but hides the underlying base actions when Manufacturing is enabled, and the Manufacturing extension adds matching promoted actionrefs. Item Card hides the base export action in W1 and propagated country-layer pages, while the W1 Manufacturing extension provides the Manufacturing export action. I did not find an orphaned or double-registered action path in the changed files.

Suggestions

None.

Risk assessment and necessity

Risk: Low. The change is limited to page action captions and visibility on Item Card and Cost Adjustment Overview, plus matching Manufacturing page extension action refs. No schema, public API, or data-processing behavior changes.

Necessity: The change is needed because Manufacturing currently exposes overlapping standard and Manufacturing export/import actions. The PR keeps one visible action set per Manufacturing state and routes to the matching XMLPort.

---

[AI-PR-REVIEW] version=1 system=github pr=9212 round=1 by=alexei-dobriansky sha=790cb3d15f829427258a1c28fbe5d77c5a3e8de4