You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@ofbiz.apache.org by "Nicolas Malin (JIRA)" <ji...@apache.org> on 2019/04/15 22:17:00 UTC

[jira] [Commented] (OFBIZ-10926) Update an order linked to an other order lost relation

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

Nicolas Malin commented on OFBIZ-10926:
---------------------------------------

Commited at revision :
 * trunk 1857605
 * 18.12 1857606
 * 17.12 1857607
 * 16.10 1857608

> Update an order linked to an other order lost relation
> ------------------------------------------------------
>
>                 Key: OFBIZ-10926
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-10926
>             Project: OFBiz
>          Issue Type: Bug
>          Components: order
>    Affects Versions: Trunk, 16.11.05, Release Branch 17.12, Release Branch 18.12
>            Reporter: Nicolas Malin
>            Assignee: Nicolas Malin
>            Priority: Minor
>         Attachments: OFBIZ-10926.patch
>
>
> When you edit an order linked to an other order like drop shipment process, shopping cart lost the connexion.
> The problem came from ShoppingCart.makeAllOrderItemAssociations() function that prepared all generic value OrderItemAssoc without toOrderId
> {code:java}
> Index: applications/order/src/main/java/org/apache/ofbiz/order/shoppingcart/ShoppingCart.java
> ===================================================================
> --- applications/order/src/main/java/org/apache/ofbiz/order/shoppingcart/ShoppingCart.java (révision 1857601)
> +++ applications/order/src/main/java/org/apache/ofbiz/order/shoppingcart/ShoppingCart.java (copie de travail)
> @@ -4122,6 +4122,7 @@
>    orderItemAssociation.set("orderId", commitment.getString("orderId"));
>    orderItemAssociation.set("orderItemSeqId", commitment.getString("orderItemSeqId"));
>    orderItemAssociation.set("shipGroupSeqId", "_NA_");
> +  orderItemAssociation.set("toOrderId", this.getOrderId());
>    orderItemAssociation.set("toOrderItemSeqId", item.getOrderItemSeqId());
>    orderItemAssociation.set("toShipGroupSeqId", "_NA_");
>    orderItemAssociation.set("orderItemAssocTypeId", "PURCHASE_ORDER");
> @@ -4136,6 +4137,7 @@
>    orderItemAssociation.set("orderId", item.getAssociatedOrderId());
>    orderItemAssociation.set("orderItemSeqId", item.getAssociatedOrderItemSeqId());
>    orderItemAssociation.set("shipGroupSeqId", csi.getAssociatedShipGroupSeqId() != null ? csi.getAssociatedShipGroupSeqId() : "_NA_");
> +  orderItemAssociation.set("toOrderId", this.getOrderId());
>    orderItemAssociation.set("toOrderItemSeqId", item.getOrderItemSeqId());
>    orderItemAssociation.set("toShipGroupSeqId", csi.getShipGroupSeqId() != null ? csi.getShipGroupSeqId() : "_NA_");
>    orderItemAssociation.set("orderItemAssocTypeId", item.getOrderItemAssocTypeId());{code}



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