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

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

I slept on it and now I wonder if we should not actually change the calsl to 
the deprecated methods but keep them to avoid the issue David reported. Then we 
have best of both world: issue fixed but still backwards compatible, opinions?
                
> Customer's Shipment Address Not Assigned to Dropship Purchase Orders
> --------------------------------------------------------------------
>
>                 Key: OFBIZ-3883
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-3883
>             Project: OFBiz
>          Issue Type: Bug
>          Components: order
>    Affects Versions: Release Branch 10.04, SVN trunk
>         Environment: demo-trunk.ofbiz.apache.org
>            Reporter: Martin Kreidenweis
>            Assignee: Jacques Le Roux
>             Fix For: Release Branch 10.04, Release Branch 11.04, SVN trunk, 
> Release Branch 12.04
>
>         Attachments: Jira-3883-ShoppingCart.java-patch.txt
>
>
> When buying drop ship products from the demo store the customer's shipping 
> address is not saved with the purchase order, so the shipping address cannot 
> be provided to the drop ship supplier. 
> Steps to Reproduce:
> * From the DropShip Category add "DropShip from BigSupplier" and "DropShip 
> from DemoSupplier" to your cart. 
> * Login as admin
> * Use the one page checkout to order the products
> Resulting Order: 
> https://demo-trunk.ofbiz.apache.org:8443/ordermgr/control/orderview?orderId=WSCO10002
> The purchase orders WS10003 and WS10004 are associated correctly with the 
> order items, but as the shipping groups don't have addresses, the purchase 
> orders don't have either.
> Expectation:
> All Shipment groups should have the shipping address assigned.
> Actual:
> Only the first shipment group has the shipping address assigned. 
> Possibly offending line of Code: 
> ShoppingCart.java line 2235:
>         this.setShippingContactMechId(0, shippingContactMechId);
> Should probably not add the contactMechId only to the first CartShipInfo. 
> When we tried to change that, though, drop shipping broke somehow completely. 

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to