Uploaded image for project: 'OFBiz'
  1. OFBiz
  2. OFBIZ-7108

Provide an option to specify the Facility Location while producing the inventory/stock once production run gets completed

    Details

    • Type: New Feature
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 16.11.01
    • Fix Version/s: 16.11.01
    • Component/s: manufacturing
    • Labels:
      None
    • Sprint:
      Community Day 2 - 2016

      Description

      Currently there is no option to specify where the produced inventory against finished good would be stocked in the facility.

      Let's try to add the option to select and specify the facility location under 'Production Run Produce' or 'Production Run Declare and Produce' section. Once user submit the quantity and location same should be set over produced InventoryItem.locationSeqId

      Please refer the attached screenshot for reference

      1. ProduceStock.png
        106 kB
        Swapnil Shah
      2. OFBIZ-7108.patch
        9 kB
        Anuj Jain
      3. OFBIZ-7110-A.patch
        3 kB
        Anuj Jain

        Activity

        Hide
        diveshdut Divesh Dutta added a comment -

        We can add autocompleter in another task. Closing this one.

        Show
        diveshdut Divesh Dutta added a comment - We can add autocompleter in another task. Closing this one.
        Hide
        diveshdut Divesh Dutta added a comment -

        Thanks Anuj Jain . I have committed the work in revision number 1768215 in trunk.

        Show
        diveshdut Divesh Dutta added a comment - Thanks Anuj Jain . I have committed the work in revision number 1768215 in trunk.
        Hide
        anuj.jain Anuj Jain added a comment - - edited

        I have added the patch 7110-A, where I have moved the code related to creation of product facility entry in ProductionRunServices.java

        Thanks,

        Show
        anuj.jain Anuj Jain added a comment - - edited I have added the patch 7110-A, where I have moved the code related to creation of product facility entry in ProductionRunServices.java Thanks,
        Hide
        swash78 Swapnil Shah added a comment - - edited

        I have tested it and works good as per expected outlined behavior except in following scenario:

        1. Start a routing task over production run and issue the components partially.
        2. Via "Production Run Declare And Produce" section we can still produce and stock the product but the UI doesn't display any place to specify the location. We can add the Location search autocompleter at this stage of production run processing as well to have consistent behavior.
        Show
        swash78 Swapnil Shah added a comment - - edited I have tested it and works good as per expected outlined behavior except in following scenario: Start a routing task over production run and issue the components partially. Via "Production Run Declare And Produce" section we can still produce and stock the product but the UI doesn't display any place to specify the location. We can add the Location search autocompleter at this stage of production run processing as well to have consistent behavior.
        Hide
        diveshdut Divesh Dutta added a comment -

        Anuj Jain I have assigned this ticket to you so that you can work on remaining part. Swapnil Shah meanwhile you can test the main functionality of this ticket.

        Show
        diveshdut Divesh Dutta added a comment - Anuj Jain I have assigned this ticket to you so that you can work on remaining part. Swapnil Shah meanwhile you can test the main functionality of this ticket.
        Hide
        diveshdut Divesh Dutta added a comment - - edited

        Thanks Anuj Jain for your patch . I have applied slightly modified patch. If product which is produced is not associated with facility then, ProductFacility record was created in service setLastInventoryCount, I skipped this part of the patch. I think this should be done in productionRunProduce service. So I am committing the main functionality to specify the facility location. Patch is committed in revision 1749109 in trunk

        Please work on creating ProductFacility part in productionRunProduce service.

        Show
        diveshdut Divesh Dutta added a comment - - edited Thanks Anuj Jain for your patch . I have applied slightly modified patch. If product which is produced is not associated with facility then, ProductFacility record was created in service setLastInventoryCount, I skipped this part of the patch. I think this should be done in productionRunProduce service. So I am committing the main functionality to specify the facility location. Patch is committed in revision 1749109 in trunk Please work on creating ProductFacility part in productionRunProduce service.
        Hide
        pfm.smits Pierre Smits added a comment -

        What are the version(s) and the component(s) this issue relates to?

        Show
        pfm.smits Pierre Smits added a comment - What are the version(s) and the component(s) this issue relates to?
        Hide
        anuj.jain Anuj Jain added a comment -

        Hello,

        I have attached a patch which would add a field with the lookup for locationSeqId on 'Production Run Produce' form.
        As in lookup user have options to set the locationSeqId of any facility for the manufactured product, Which is not quite correct. As user should only be able to assign locationSeqId to the product from the facility, where it is produced.

        I have also made changes in the LooupFacilityLocation. If the form for the lookup has facilityId in the parameters, the user then not able to select the facility from the drop down,

        Show
        anuj.jain Anuj Jain added a comment - Hello, I have attached a patch which would add a field with the lookup for locationSeqId on 'Production Run Produce' form. As in lookup user have options to set the locationSeqId of any facility for the manufactured product, Which is not quite correct. As user should only be able to assign locationSeqId to the product from the facility, where it is produced. I have also made changes in the LooupFacilityLocation. If the form for the lookup has facilityId in the parameters, the user then not able to select the facility from the drop down,
        Hide
        pfm.smits Pierre Smits added a comment -

        The default for this should be taken from the primary warehouse of the product.

        Show
        pfm.smits Pierre Smits added a comment - The default for this should be taken from the primary warehouse of the product.

          People

          • Assignee:
            swash78 Swapnil Shah
            Reporter:
            swash78 Swapnil Shah
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development

                Agile