[Bug Fix] #519: Move Overhead Rate next to Indirect Cost % on Item Card - #12011
Conversation
Fixes AB#651342 Promoted from microsoft/BCAppsBugFix#526 (https://github.com/microsoft/BCAppsBugFix/pull/526)
Good Sense Reviewer - Round 1Recommendation: AcceptWhat this PR doesThis change moves Problem-solution fitFit: Strong
SuggestionsNo suggestions. Risk assessment and necessityRisk: The change is limited to page layout metadata and does not alter costing logic, stored data, or public APIs. Existing personalizations may need visual confirmation because the control moves between page objects, but the control properties and all repository layer variants are preserved. Necessity: The move is needed so purchase-item users can find a general costing field beside the related percentage field. Automated tests cannot distinguish the page hierarchy or sibling order, so source placement and build validation are appropriate for this change.
|
Bug Reference
Fixes microsoft/BCAppsBugFix#519
Summary
Move Overhead Rate next to Indirect Cost % in the base Item Card's Costs & Posting > Cost Details group, making both indirect-cost components discoverable together for purchased and manufactured items.
Root Cause
Overhead Rate was defined inside the manufacturing Item Card extension's Production group, although it is general inventory costing functionality with
ApplicationArea = Basic, Suite. The base Item Card's Cost Details group contained only the related percentage field.Changes Made
src\Layers\W1\BaseApp\Inventory\Item\ItemCard.Page.al: place the existing Overhead Rate control immediately after Indirect Cost %, before Last Direct Cost.src\Layers\W1\BaseApp\Manufacturing\Inventory\Item\MfgItemCard.PageExt.al: remove the original control from Production.Implementation Process
Validation Evidence
No-Test Validation Evidence
ready, with compiler symbols, runtime dependencies and live endpoint all verified. Exact installed PackageId:e3080ae4-79a2-4dd2-8998-6a4140b5ac36; artifact SHA256:4BAB98583E2C544E2CA916D0922735FDF2CCE1BB4AA13486D7974C3AD2DB5105. Mandatory inventory health was healthy.Validation Coverage
Miapp Propagation
AL Review
al-performance-review.mdaction skill.Test-LocalReviewEvidence.ps1 -AllowEmptyReportindependently classified the absent report asskipped-empty. The first and only invocation's output and artifact-absence evidence are archived; no earlier blocking findings existed.Review Notes
Please confirm the Essential-experience visual placement and existing personalization/extension behavior. The W1 fix and five localized copies retain the original Additional importance and inventory-only enabled state. Country-layer compilation is owned by CI.
Generated by the bc-fixbug skill.
Promoted from microsoft/BCAppsBugFix#526: https://github.com/microsoft/BCAppsBugFix/pull/526
Fixes AB#651342