[ https://issues.apache.org/jira/browse/OFBIZ-1953?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12645720#action_12645720 ]
Dimitri Unruh commented on OFBIZ-1953: -------------------------------------- Bilgin, sorry for my late answer... The problem is the for-loop in ShoppingCart.cleanUpShipGroups() {code} for (int i = 0; i < this.shipInfo.size(); i++) { CartShipInfo csi = this.getShipInfo(i); .... if (csi.shipItemInfo.size() == 0) { this.shipInfo.remove(csi); } } {code} Example: If you have two shipGroups and the both are empty, the first one will be removed from the list. The second one id still in the list. The loop-condition is wrong, so the second ahipGroup will not be removed from the list.... I hope you could follow me :) Dimitri > Cleanup ship groups doesn't work > -------------------------------- > > Key: OFBIZ-1953 > URL: https://issues.apache.org/jira/browse/OFBIZ-1953 > Project: OFBiz > Issue Type: Bug > Components: order > Affects Versions: SVN trunk > Reporter: Dimitri Unruh > Attachments: ShoppingCart.java.patch > > > Hi everyone, > I guess the method cleanUpShipGroups in the ShoppingCart.java is buggy. > If you have many CartShipInfo objects in your cart and some of them are empty > (shipItemInfo.size() == 0) the method don't remove all of them. > -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.