You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ofbiz.apache.org by "Scott Gray (JIRA)" <ji...@apache.org> on 2009/08/14 15:47:14 UTC

[jira] Updated: (OFBIZ-2753) Cart that is loaded from an order has its orderId set to null

     [ https://issues.apache.org/jira/browse/OFBIZ-2753?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Scott Gray updated OFBIZ-2753:
------------------------------

    Issue Type: Improvement  (was: Bug)

Sounds more like an improvement than a bug

> Cart that is loaded from an order has its orderId set to null
> -------------------------------------------------------------
>
>                 Key: OFBIZ-2753
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-2753
>             Project: OFBiz
>          Issue Type: Improvement
>          Components: order
>    Affects Versions: SVN trunk
>            Reporter: Bob Morley
>
> ShoppingCartEvents.loadCartFromOrder has the following near the bottom of the method:
>         // Since we only need the cart items, so set the order id as null
>         cart.setOrderId(null); 
> I believe this is ok with standard Ofbiz applications because the only time a shopping cart is loaded from an order i when you wish to initiate the order entry process from an existing order (you will be creating a new order).  In our system however, you can start an order, persist it, and then pick it up later to make more adjustments.  In this scenario the actual orderId should be available (and accurate) to the order that was used for loading.
> Recommendation - create a new service method "initializeCartFromOrder" or provide a parameter that indicates if the intent is to load into the cart for initialization (vs. for existing order manipulation).

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.