1. OFBiz
  2. OFBIZ-3724

Copying WorkEffortCostCalc as part of a ProductionRun


    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: Trunk
    • Fix Version/s: Trunk
    • Component/s: manufacturing
    • Labels:


      Currently when a ProductionRun is created the WorkEffortCostCalc relations are not cloned from the routing tasks. This results in not being able to change the CostCalc as part of a ProductionRun unless you were to change the routing task template for all production runs. As part of reviewing this patch I would like two things to be considered –

      I created this patch which includes an enhancement to the production run unit tests to test for the creation of these newly cloned relationships.

      1) createWorkEffortCostCalc service requires ACCOUNTING_CREATE permission; this does not feel right – it caused me to add that permission into the sample user/security group to execute the services. I wonder if these services should be in the WorkEffort component with appropriate work effort permission.

      2) migration concerns – if people have existing production runs in place, the new createProductionRunTaskCosts will not pickup the cost calcs that were not copied. Rather than put "backwards compatible" code in, I would prefer (if we want to handle this) to create a service that performs this clone on non-completed/closed production runs that is executed as part of a migration. Would like community discussion on this.


        Bob Morley created issue -
        Bob Morley made changes -
        Field Original Value New Value
        Attachment OFBIZ-3724_CloneWorkEffortCostCalc.patch [ 12442206 ]
        Bob Morley made changes -
        Status Open [ 1 ] Patch Available [ 10002 ]
        Jacopo Cappellato made changes -
        Assignee Jacopo Cappellato [ jacopoc ]
        Jacopo Cappellato made changes -
        Status Patch Available [ 10002 ] Closed [ 6 ]
        Resolution Fixed [ 1 ]


          • Assignee:
            Jacopo Cappellato
            Bob Morley
          • Votes:
            0 Vote for this issue
            1 Start watching this issue


            • Created: