[Subcontracting] Bug 648937: Transfer WIP Item is lost when production orders are created from Planning Worksheet - #11644
Chethan Thopaiah (ChethanT) wants to merge 2 commits into
Conversation
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: cb368c63-4b66-4ace-824e-f59c37adf44f
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: cb368c63-4b66-4ace-824e-f59c37adf44f
Good Sense Reviewer - Round 1Recommendation: Request ChangesWhat this PR doesThis PR adds storage on Planning Routing Line for Transfer WIP Item and transfer descriptions, then copies those values from routing lines through Planning Worksheet carry-out into production order routing lines. The data flow matches the reported bug, and the selected BaseApp events fire at the right points: before the planning routing line is inserted and after the production order routing line has copied the planning routing values. Problem-solution fitFit: Strong The change directly addresses the missing planning-routing storage and propagation path. It covers both routing versions and base routings, which matches the two ways the affected value can be sourced. SuggestionsS1 (🔴 High): Fix the failing base-routing test Risk assessment and necessityRisk: The product code is narrow, but it changes subcontracting planning-to-production data flow. If it is wrong, production orders created from Planning Worksheet can still miss WIP transfer behavior or carry wrong transfer descriptions. The current validation failure is the main remaining merge risk. Necessity: The change is required so orders created from Planning Worksheet preserve Transfer WIP Item the same way direct production order refresh does. Without it, the transfer-WIP process can silently be skipped for planned subcontracting orders.
|
Summary
Testing
Subc. Planning TestFixes AB#648937