[jira] Issue Comment Edited: (OFBIZ-2115) OrderServices.updateApprovedOrderItems should update OrderItemAttributes

["Stephen Rufle (JIRA)" <ji...@apache.org>]

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

srufle edited comment on OFBIZ-2115 at 1/23/09 7:21 AM:
---------------------------------------------------------------

I think OFBIZ-2109 will also be effected. Should I try to create two
patches?



      was (Author: srufle):
    I think OFBIZ-2109 will also be effected. Should I try to create two
patches?


-- 
Stephen P Rufle
srufle@salmonllc.com
H1:480-626-8022
H2:480-802-7173
Yahoo IM: stephen_rufle
AOL IM: stephen1rufle


  
> OrderServices.updateApprovedOrderItems should update OrderItemAttributes
> ------------------------------------------------------------------------
>
>                 Key: OFBIZ-2115
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-2115
>             Project: OFBiz
>          Issue Type: Improvement
>          Components: order
>    Affects Versions: SVN trunk
>         Environment: Windows XP
>            Reporter: Stephen Rufle
>            Assignee: Jacques Le Roux
>             Fix For: SVN trunk
>
>         Attachments: 2009-01-08_001_OrderItemAtrributes.patch, 2009-01-14_001_shoppingcart.patch
>
>
> I wanted updateApprovedOrderItems to be able to update any OrderItemAttributes there may be. added a new map (itemAttributesMap) to /applications/order/servicedef/services.xml#updateOrderItems the key value of the map should be in the form of
> attibuteName+ ":" + orderItemSeqId
> At the top of
> OrderServices.updateApprovedOrderItems(DispatchContext, Map) I create a new map with only the attribute names, later in the same method we repopulate the cartItem's OrderItemAttributes with any non-empty values. In saveUpdatedCartToOrder we call my modified ShoppingCart.makeAllOrderItemAttributes(String, int) I added the ability to return just empty , just filled or both empty and filled. 
> For previously filled attributes that are now empty I remove them from the database. for either new or chnaged ones they are just added to the toStore that already existed 

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