[Subcontracting] Bug 634720: Production Routing factbox count must match drill-down operation filter#8157
Open
ChethanT wants to merge 2 commits into
Open
[Subcontracting] Bug 634720: Production Routing factbox count must match drill-down operation filter#8157ChethanT wants to merge 2 commits into
ChethanT wants to merge 2 commits into
Conversation
…tch drill-down operation filter Aligned CalcNoOfProductionOrderRoutings in codeunit "Subc. ProdO. Factbox Mgmt." with ShowProductionOrderRouting by adding the missing "Routing No." and "Operation No." filters. Previously, the factbox count returned all routing operations of the production order line (e.g. 4), while the drill-down filtered to the single linked operation (always 1) — so the count and drill-down were inconsistent. Both Subc. Purchase Line Factbox and Subc. Transfer Line Factbox benefit from this single-point fix. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
4) that didn't match the single routing line opened by its drill-down. Same issue on the Transfer Order factbox.CalcNoOfProductionOrderRoutingsin codeunit 99001559Subc. ProdO. Factbox Mgmt.filtered theProd. Order Routing Linetable byStatus+Prod. Order No.+Routing Reference No.only, while its drill-down counterpartShowProductionOrderRoutingadditionally filters byRouting No.andOperation No.. The count therefore returned all operations of the production order line; the drill-down always returns the single linked routing operation.CalcNoOfProductionOrderRoutingsso both methods apply the same filter set. Single-point fix shared by bothSubc. Purchase Line FactboxandSubc. Transfer Line Factbox.Test
Added
CalcNoOfProductionOrderRoutingsReturnsOneForSubcontractingPurchaseLine([SCENARIO 634720]) in codeunit 139989Subc. Subcontracting Test(src/Apps/W1/Subcontracting/Test/src/Codeunits/Tests/SubcSubcontractingTest.Codeunit.al). The test:Count() > 1as a precondition to detect the bug),CalcNoOfProductionOrderRoutings(PurchaseLine) = 1, matching the drill-down.Without the fix, the assertion would observe the total number of routing operations of the prod-order line (e.g.
4).Fixes AB#634720
🤖 Generated with Claude Code