[ 
https://issues.apache.org/jira/browse/OFBIZ-10831?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16765752#comment-16765752
 ] 

Jacques Le Roux commented on OFBIZ-10831:
-----------------------------------------

HI Vijay,

Sure a patch is always welcom, please follow 
https://cwiki.apache.org/confluence/display/OFBIZ/OFBiz+Contributors+Best+Practices,
 thanks

> Re-reservation of Inventory Items when Ship Group Destination Address or 
> Shipment Method is updated in Sales Order
> ------------------------------------------------------------------------------------------------------------------
>
>                 Key: OFBIZ-10831
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-10831
>             Project: OFBiz
>          Issue Type: Bug
>          Components: order
>    Affects Versions: Release Branch 16.11
>            Reporter: Vijay Ganesh
>            Priority: Major
>             Fix For: Release Branch 16.11
>
>
> Hi,
> We came across this issue recently and also have fixed the same. 
> After creating a sales order and approving it, we created a shipment and 
> receive all the items and change the status to Packed. 
> Moving to the sales order edit screen. We are able to update the Ship Group 
> Destination Address or Shipment Method. Even without changing anything if we 
> simply click the update button, Inventory re-reservation service is being 
> called and getting re-reserved. Since there are no records in the 
> [OrderItemShipGrpInvRes|https://localhost:8443/webtools/control/FindGeneric?entityName=OrderItemShipGrpInvRes]
>  Entity, the inventory values are getting re-reserved again and this is a 
> major issue.
> For a Sales Order of quantity 10, after approving and creating a shipment, we 
> issue 10 items and change status to packed. Now we go to the Order View/Edit 
> Screen and update the shipment method/Address or simply click the update 
> button and the quantity in the Shipment for that order changes to 20 which 
> should not happen.
>  
> The fix we made is passing a hidden variable from the shipping Info Section 
> and when clicking the update button, we pass that value to the Inventory 
> Re-reservation event and check the value in that method.
> In the loadCartForUpdate method in OrderService.java, we pass the checkValue 
> to see whether this request is coming from the Shipping Information update 
> section and prevent the cancelOrderInventoryReservation service from calling.
> In the saveUpdatedCartToOrder method in OrderService.java using the same 
> checkValue, we prevent the reserveInventory service from calling. 
> This occurs when an existing shipment is in packed status for an order and 
> that order's shipping information needs to be edited.
> This was an issue in our case and we fixed this way.
> If it is ok we will provide the patch here?.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to