[ https://issues.apache.org/jira/browse/OFBIZ-5568?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13924986#comment-13924986 ]
Christian Carlow commented on OFBIZ-5568: ----------------------------------------- Thanks for the feedback Pierre, Based on the reusable yeast residue by-product mentioned in your ML brewery example, I see how my suggestion to employ ProductAssoc with "PRODUCT_DEFECT" to limit the possible products that can be declared is insufficient. But shouldn't a rule be applied to limit what can be declared other than the needed logic to prevent at least the finished good and materials being declared using that form? To handle your yeast residue by-product scenario, instead a "BYPRODUCT" type could be added and be assigned as the parent type of "PRODUCT_DEFECT." Then the assignment of yeast-residue as "BYPRODUCT" to the finished good could be used to limit the adding some other non-yeast residue that should never apply to that production run. Can you think of any scenarios where "BYPRODUCT" suggestion would be insufficient also? > Production Run inventory declaration form doesn't prevent products other than > either defective materials or finished goods from being declared > ---------------------------------------------------------------------------------------------------------------------------------------------- > > Key: OFBIZ-5568 > URL: https://issues.apache.org/jira/browse/OFBIZ-5568 > Project: OFBiz > Issue Type: Bug > Components: manufacturing > Reporter: Christian Carlow > > The production run inventory declaration form should be limited to defective > products associated with either the finished good or raw materials. > Currently is allows any product to be produced which is incorrect. > ProductAssoc with type "PRODUCT_DEFECT" needs to be added to handle the > limitation logic in the service. Then the reason field of ProductAssoc could > be used to indicate defective variants more clearly to the user. -- This message was sent by Atlassian JIRA (v6.2#6252)