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

Bilgin Ibryam commented on OFBIZ-1453:
--------------------------------------

Jacques,

I fixed the bugs you described (Could you confirm this, please). You were right 
about them!

The only bug that i cannot reproduce is "create an order from this existing 
order (in Order View) " It works for me w/o errors.
In line 348 of ShoppingCartServices the product type *should be * 
AGGREGATED_CONF, because during order creation AGGREGATED products are replaced 
by AGGREGATED_CONF typed products, and orderItems are AGGREGATED_CONF. So in 
ShoppingCartServices.loadCartFromOrder() the product can not be 
AGGREGATED_CONF.  I do not know how you got an AGGREGATED product in OrderItems.

Also fixed the "you end with the default configuration in any cases" bug. But 
after the product is configured, before adding to shopping list is required to 
use "Verify Configuration" button. This should be replaced by an ajax event in 
the future.

Thanks for your help.

> store/load possibility for AGGREGATED product configurations
> ------------------------------------------------------------
>
>                 Key: OFBIZ-1453
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-1453
>             Project: OFBiz
>          Issue Type: Improvement
>          Components: order, product
>    Affects Versions: SVN trunk
>            Reporter: Bilgin Ibryam
>            Priority: Minor
>             Fix For: SVN trunk
>
>         Attachments: aggregated_full_for_review.patch, 
> aggregated_full_for_review.patch, aggregated_full_for_review.patch, 
> aggregated_full_for_review.patch, aggregated_full_for_review.patch, 
> aggregated_full_for_review.patch
>
>


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

Reply via email to