Author: diveshdutta
Date: Sat Jun 20 16:01:54 2015
New Revision: 1686645

URL: http://svn.apache.org/r1686645
Log:
Applied fix from trunk for revision: 1686635 
===

[OFBIZ-5347] 1) Fixed issue: Shipping costs not recalculated after changing 
sales order shipment method. For this introduced new service as per the design 
in this task. 2) This service will loads the cart save new shipping method in 
cart, re-do all the calculations for tax and promotion, and saves the cart 
again in order. 3) Also adding new SECA rule which will trigger resetGrandTotal 
service on new service updateShipGroupShipInfo. 

Modified:
    ofbiz/branches/release13.07/   (props changed)
    ofbiz/branches/release13.07/applications/order/servicedef/secas.xml
    ofbiz/branches/release13.07/applications/order/servicedef/services.xml
    
ofbiz/branches/release13.07/applications/order/src/org/ofbiz/order/order/OrderReadHelper.java
    
ofbiz/branches/release13.07/applications/order/src/org/ofbiz/order/order/OrderServices.java
    
ofbiz/branches/release13.07/applications/order/webapp/ordermgr/WEB-INF/actions/order/OrderView.groovy
    
ofbiz/branches/release13.07/applications/order/webapp/ordermgr/WEB-INF/controller.xml
    
ofbiz/branches/release13.07/applications/order/webapp/ordermgr/order/ordershippinginfo.ftl

Propchange: ofbiz/branches/release13.07/
------------------------------------------------------------------------------
--- svn:mergeinfo (original)
+++ svn:mergeinfo Sat Jun 20 16:01:54 2015
@@ -5,4 +5,4 @@
 /ofbiz/branches/jquery:952958-1044489
 /ofbiz/branches/multitenant20100310:921280-927264
 /ofbiz/branches/release12.04:1557118
-/ofbiz/trunk:1506269,1506499,1506504,1506828,1509164,1510042,1511279,1512376,1512573,1516094,1517629,1517702,1517780,1517947,1518336,1518950,1519245,1519999,1520319,1520321,1520326,1524361,1524676,1524704,1524769,1524835,1524950,1525523,1526276,1526387,1526463,1527212,1527254,1527609,1527626,1527810,1528144,1528146,1528149,1528298,1529412,1529418,1529588,1530273,1530634,1530876,1530972,1530976,1531848,1532342,1532366,1533542,1533839,1535961,1536170,1536656,1537023,1537086,1537179,1537996,1538096,1539147,1539156,1539781,1542264,1542388,1542442,1543744,1543766,1543781,1543994,1544444,1547548,1548143,1549015,1550255,1550258,1550515,1551585,1551744,1552149,1552290,1552500,1552901,1552908,1554064,1554242,1554265,1554290,1554373,1554536,1554681,1554685,1554706,1554764,1554787,1555142,1557409-1557410,1557427,1557440,1557443,1557447,1557456-1557457,1557462,1557593,1558145,1558241,1558373,1558774,1559814,1560048,1560056,1560176,1560203,1560205,1560699,1561286,1561290,1561305,1561311,1561327,
 
1561467,1562767,1563238,1563683,1563958,1564111,1564113,1564463,1564473,1564493,1566096,1566273,1569078,1569743,1570611,1570622,1570639-1570640,1571207,1571219,1571247,1573161,1573498,1573639,1574019,1574201,1574404,1575508,1576259,1576331,1576378,1576506,1576511,1576757,1576839,1577268,1577744,1579155,1579161,1579277,1579309,1580455,1580850,1581386,1581972,1581997,1582762,1583040,1583427-1583428,1583551,1583674,1583681,1583689,1583696,1584873,1585033,1585574,1585958-1585959,1586987,1587841,1587843,1588733,1589589,1589602,1589606,1589612,1589625,1589669,1592530,1592588,1592745,1592977,1593902,1593908,1593952,1597239,1597464,1598113,1598327,1598475,1598544,1598913,1603439,1603732,1603739,1604357,1604363,1604522,1604554,1605029,1605269,1605348,1605354,1605707,1607457,1608355,1608495,1608526,1608698,1609047,1609065,1609076,1609087,1609149,1609155,1609167,1609184,1609389,1609394,1609398,1609406,1609418,1609885,1609889,1610355,1610420,1610425,1610685,1610918,1610925,1611321,1612190,16122
 
02,1613121,1614019,1614025,1614280,1614355,1614366,1614556,1615296-1615297,1616272,1616684,1616940,1617229,1617473,1617480,1617936,1617938,1618395,1618397,1618570,1618831,1619087,1619098,1619300,1621335,1621363,1621413,1621436,1621438-1621439,1621442,1621599,1621683,1622050,1622170,1622672,1623370,1624538,1624767,1624809,1624817,1626797,1627230,1627763,1628096,1628130,1628937,1628940,1629382,1629391,1629426,1631203,1631299-1631300,1632764,1632793,1633100,1633188,1633550,1636864,1637883,1639606,1639835,1639840-1639841,1639846,1639863,1639887,1640288,1640299,1640515,1640717,1641066,1641131,1641165,1641548,1641804,1642423,1643341,1644904,1645950,1646204,1646935,1646977,1646984,1647266,1647338,1647559,1647606,1648668,1649239,1649393,1650240,1650583,1650642,1650678,1650882,1650938,1651593,1652361,1652725,1652731,1652739,1653456,1654175,1654273,1655046,1655795,1656983,1660389,1660444,1661358,1661612,1661778,1661853,1661873,1661940,1661951,1661977,1662361,1662500,1663912,1663979,1664602,16
 
64604,1666633,1666836,1666939,1666949,1667055,1667253,1668214,1668246,1668258,1668265,1668314,1669317,1670952,1672862,1672873,1673764,1674447,1674491,1674496,1674908,1676674,1677123,1677597,1677769-1677770,1678294,1678882,1678911,1679689,1679697,1680155,1680304,1680672,1680733,1680840,1680873,1680881,1682272,1682415,1683998,1686566,1686569,1686574,1686583
+/ofbiz/trunk:1506269,1506499,1506504,1506828,1509164,1510042,1511279,1512376,1512573,1516094,1517629,1517702,1517780,1517947,1518336,1518950,1519245,1519999,1520319,1520321,1520326,1524361,1524676,1524704,1524769,1524835,1524950,1525523,1526276,1526387,1526463,1527212,1527254,1527609,1527626,1527810,1528144,1528146,1528149,1528298,1529412,1529418,1529588,1530273,1530634,1530876,1530972,1530976,1531848,1532342,1532366,1533542,1533839,1535961,1536170,1536656,1537023,1537086,1537179,1537996,1538096,1539147,1539156,1539781,1542264,1542388,1542442,1543744,1543766,1543781,1543994,1544444,1547548,1548143,1549015,1550255,1550258,1550515,1551585,1551744,1552149,1552290,1552500,1552901,1552908,1554064,1554242,1554265,1554290,1554373,1554536,1554681,1554685,1554706,1554764,1554787,1555142,1557409-1557410,1557427,1557440,1557443,1557447,1557456-1557457,1557462,1557593,1558145,1558241,1558373,1558774,1559814,1560048,1560056,1560176,1560203,1560205,1560699,1561286,1561290,1561305,1561311,1561327,
 
1561467,1562767,1563238,1563683,1563958,1564111,1564113,1564463,1564473,1564493,1566096,1566273,1569078,1569743,1570611,1570622,1570639-1570640,1571207,1571219,1571247,1573161,1573498,1573639,1574019,1574201,1574404,1575508,1576259,1576331,1576378,1576506,1576511,1576757,1576839,1577268,1577744,1579155,1579161,1579277,1579309,1580455,1580850,1581386,1581972,1581997,1582762,1583040,1583427-1583428,1583551,1583674,1583681,1583689,1583696,1584873,1585033,1585574,1585958-1585959,1586987,1587841,1587843,1588733,1589589,1589602,1589606,1589612,1589625,1589669,1592530,1592588,1592745,1592977,1593902,1593908,1593952,1597239,1597464,1598113,1598327,1598475,1598544,1598913,1603439,1603732,1603739,1604357,1604363,1604522,1604554,1605029,1605269,1605348,1605354,1605707,1607457,1608355,1608495,1608526,1608698,1609047,1609065,1609076,1609087,1609149,1609155,1609167,1609184,1609389,1609394,1609398,1609406,1609418,1609885,1609889,1610355,1610420,1610425,1610685,1610918,1610925,1611321,1612190,16122
 
02,1613121,1614019,1614025,1614280,1614355,1614366,1614556,1615296-1615297,1616272,1616684,1616940,1617229,1617473,1617480,1617936,1617938,1618395,1618397,1618570,1618831,1619087,1619098,1619300,1621335,1621363,1621413,1621436,1621438-1621439,1621442,1621599,1621683,1622050,1622170,1622672,1623370,1624538,1624767,1624809,1624817,1626797,1627230,1627763,1628096,1628130,1628937,1628940,1629382,1629391,1629426,1631203,1631299-1631300,1632764,1632793,1633100,1633188,1633550,1636864,1637883,1639606,1639835,1639840-1639841,1639846,1639863,1639887,1640288,1640299,1640515,1640717,1641066,1641131,1641165,1641548,1641804,1642423,1643341,1644904,1645950,1646204,1646935,1646977,1646984,1647266,1647338,1647559,1647606,1648668,1649239,1649393,1650240,1650583,1650642,1650678,1650882,1650938,1651593,1652361,1652725,1652731,1652739,1653456,1654175,1654273,1655046,1655795,1656983,1660389,1660444,1661358,1661612,1661778,1661853,1661873,1661940,1661951,1661977,1662361,1662500,1663912,1663979,1664602,16
 
64604,1666633,1666836,1666939,1666949,1667055,1667253,1668214,1668246,1668258,1668265,1668314,1669317,1670952,1672862,1672873,1673764,1674447,1674491,1674496,1674908,1676674,1677123,1677597,1677769-1677770,1678294,1678882,1678911,1679689,1679697,1680155,1680304,1680672,1680733,1680840,1680873,1680881,1682272,1682415,1683998,1686566,1686569,1686574,1686583,1686635

Modified: ofbiz/branches/release13.07/applications/order/servicedef/secas.xml
URL: 
http://svn.apache.org/viewvc/ofbiz/branches/release13.07/applications/order/servicedef/secas.xml?rev=1686645&r1=1686644&r2=1686645&view=diff
==============================================================================
--- ofbiz/branches/release13.07/applications/order/servicedef/secas.xml 
(original)
+++ ofbiz/branches/release13.07/applications/order/servicedef/secas.xml Sat Jun 
20 16:01:54 2015
@@ -458,4 +458,7 @@ under the License.
         <condition field-name="orderTypeId" operator="equals" 
value="SALES_ORDER"/>
         <action service="checkOrderItemForProductGroupOrder" mode="sync"/>
     </eca>
+    <eca service="updateShipGroupShipInfo" event="commit">
+        <action service="resetGrandTotal" mode="sync"/>
+    </eca>
 </service-eca>

Modified: ofbiz/branches/release13.07/applications/order/servicedef/services.xml
URL: 
http://svn.apache.org/viewvc/ofbiz/branches/release13.07/applications/order/servicedef/services.xml?rev=1686645&r1=1686644&r2=1686645&view=diff
==============================================================================
--- ofbiz/branches/release13.07/applications/order/servicedef/services.xml 
(original)
+++ ofbiz/branches/release13.07/applications/order/servicedef/services.xml Sat 
Jun 20 16:01:54 2015
@@ -1148,4 +1148,12 @@ under the License.
         <auto-attributes mode="IN" entity-name="OrderItemGroupOrder" 
include="pk" optional="false"/>
     </service>
     
+    <service name="updateShipGroupShipInfo" engine="java" 
location="org.ofbiz.order.order.OrderServices" invoke="updateShipGroupShipInfo" 
auth="true">
+            <description>Update Shipping Information on Order 
View</description>
+         <attribute name="contactMechId" type="String" mode="IN" 
optional="false"/>
+         <attribute name="orderId" type="String" mode="IN" optional="true"/>
+         <attribute name="oldContactMechId" type="String" mode="IN" 
optional="true"/>
+         <attribute name="shipGroupSeqId" type="String" mode="IN" 
optional="true"/>
+         <attribute name="shipmentMethod" type="String" mode="IN" 
optional="true"/>
+    </service>
 </services>

Modified: 
ofbiz/branches/release13.07/applications/order/src/org/ofbiz/order/order/OrderReadHelper.java
URL: 
http://svn.apache.org/viewvc/ofbiz/branches/release13.07/applications/order/src/org/ofbiz/order/order/OrderReadHelper.java?rev=1686645&r1=1686644&r2=1686645&view=diff
==============================================================================
--- 
ofbiz/branches/release13.07/applications/order/src/org/ofbiz/order/order/OrderReadHelper.java
 (original)
+++ 
ofbiz/branches/release13.07/applications/order/src/org/ofbiz/order/order/OrderReadHelper.java
 Sat Jun 20 16:01:54 2015
@@ -2916,4 +2916,17 @@ public class OrderReadHelper {
        result.put("taxGrandTotal", taxGrandTotal);
        return result;
    }
+   
+    public List<BigDecimal> getShippableSizes(String shipGrouSeqId) {
+        List<BigDecimal> shippableSizes = FastList.newInstance();
+        List<GenericValue> validItems = getValidOrderItems(shipGrouSeqId);
+        if (validItems != null) {
+            Iterator<GenericValue> i = validItems.iterator();
+            while (i.hasNext()) {
+                GenericValue item = i.next();
+                shippableSizes.add(this.getItemSize(item));
+            }
+        }
+        return shippableSizes;
+    }
 }

Modified: 
ofbiz/branches/release13.07/applications/order/src/org/ofbiz/order/order/OrderServices.java
URL: 
http://svn.apache.org/viewvc/ofbiz/branches/release13.07/applications/order/src/org/ofbiz/order/order/OrderServices.java?rev=1686645&r1=1686644&r2=1686645&view=diff
==============================================================================
--- 
ofbiz/branches/release13.07/applications/order/src/org/ofbiz/order/order/OrderServices.java
 (original)
+++ 
ofbiz/branches/release13.07/applications/order/src/org/ofbiz/order/order/OrderServices.java
 Sat Jun 20 16:01:54 2015
@@ -5863,4 +5863,110 @@ public class OrderServices {
 
         return ServiceUtil.returnSuccess();
     }
+
+    /**
+     * This service runs when you update shipping method of Order from order 
view page.
+     */
+    public static Map<String, Object> updateShipGroupShipInfo(DispatchContext 
dctx, Map<String, ? extends Object> context) {
+        LocalDispatcher dispatcher = dctx.getDispatcher();
+        Delegator delegator = dctx.getDelegator();
+        Locale locale = (Locale) context.get("locale");
+        GenericValue userLogin  = (GenericValue)context.get("userLogin");
+        String orderId = (String)context.get("orderId");
+        String shipGroupSeqId = (String)context.get("shipGroupSeqId");
+        String contactMechId = (String)context.get("contactMechId");
+        String oldContactMechId = (String)context.get("oldContactMechId");
+        String shipmentMethod = (String)context.get("shipmentMethod");
+
+        //load cart from order to update new shipping method or address
+        ShoppingCart shoppingCart = null;
+        try {
+            shoppingCart = loadCartForUpdate(dispatcher, delegator, userLogin, 
orderId);
+        } catch(GeneralException e) {
+            Debug.logError(e, module);
+        }
+
+        String message = null;
+        if (UtilValidate.isNotEmpty(shipGroupSeqId)) {
+            OrderReadHelper orh = new OrderReadHelper(delegator, orderId);
+            List<GenericValue> shippingMethods = null;
+            String shipmentMethodTypeId = null;
+            String carrierPartyId = null;
+
+            // get shipment method from OrderItemShipGroup, if not available 
in parameters
+            if (UtilValidate.isNotEmpty(shipmentMethod)) {
+                String[] arr = shipmentMethod.split( "@" );
+                shipmentMethodTypeId = arr[0];
+                carrierPartyId = arr[1];
+            } else {
+                GenericValue orderItemshipGroup = 
orh.getOrderItemShipGroup(shipGroupSeqId);
+                shipmentMethodTypeId = 
orderItemshipGroup.getString("shipmentMethodTypeId");
+                carrierPartyId = 
orderItemshipGroup.getString("carrierPartyId");
+            }
+            int groupIdx =Integer.parseInt(shipGroupSeqId);
+
+            /* check whether new selected contact address is same as old 
contact.
+               If contact address is different, get applicable ship methods 
for changed contact */
+            if (UtilValidate.isNotEmpty(oldContactMechId) && 
oldContactMechId.equals(contactMechId)) {
+                shoppingCart.setShipmentMethodTypeId(groupIdx - 1, 
shipmentMethodTypeId);
+                shoppingCart.setCarrierPartyId(groupIdx - 1, carrierPartyId);
+            } else {
+                Map<String, BigDecimal> shippableItemFeatures = 
orh.getFeatureIdQtyMap(shipGroupSeqId);
+                BigDecimal shippableTotal = 
orh.getShippableTotal(shipGroupSeqId);
+                BigDecimal shippableWeight = 
orh.getShippableWeight(shipGroupSeqId);
+                List<BigDecimal> shippableItemSizes = 
orh.getShippableSizes(shipGroupSeqId);
+
+                GenericValue shippingAddress = null;
+                if(UtilValidate.isEmpty(shippingAddress)) {
+                    shippingAddress = orh.getShippingAddress(shipGroupSeqId);
+                }
+
+                shippingMethods = 
ProductStoreWorker.getAvailableStoreShippingMethods(delegator, 
orh.getProductStoreId(),
+                        shippingAddress, shippableItemSizes, 
shippableItemFeatures, shippableWeight, shippableTotal);
+
+                boolean isShippingMethodAvailable = false;
+                // search shipping method for ship group is applicable to new 
address or not.
+                for (GenericValue shippingMethod : shippingMethods) {
+                    isShippingMethodAvailable = 
shippingMethod.getString("partyId").equals(carrierPartyId) && 
shippingMethod.getString("shipmentMethodTypeId").equals(shipmentMethodTypeId);
+                    if (isShippingMethodAvailable) {
+                        shoppingCart.setShipmentMethodTypeId(groupIdx - 1, 
shipmentMethodTypeId);
+                        shoppingCart.setCarrierPartyId(groupIdx - 1, 
carrierPartyId);
+                        break;
+                    }
+                }
+
+                // set first shipping method from list, if shipping method for 
ship group is not applicable to new ship address.
+                if(!isShippingMethodAvailable) {
+                    shoppingCart.setShipmentMethodTypeId(groupIdx - 1, 
shippingMethods.get(0).getString("shipmentMethodTypeId"));
+                    shoppingCart.setCarrierPartyId(groupIdx - 1, 
shippingMethods.get(0).getString("carrierPartyId"));
+
+                    String newShipMethTypeDesc =null;
+                    String shipMethTypeDesc=null;
+                    try {
+                        shipMethTypeDesc = 
delegator.findOne("ShipmentMethodType", UtilMisc.toMap("shipmentMethodTypeId", 
shipmentMethodTypeId), false).getString("description");
+                        newShipMethTypeDesc = 
delegator.findOne("ShipmentMethodType", UtilMisc.toMap("shipmentMethodTypeId", 
shippingMethods.get(0).getString("shipmentMethodTypeId")), 
false).getString("description");
+                    } catch(GenericEntityException e) {
+                        Debug.logError(e, module);
+                    }
+                    // message to notify user for not applicability of 
shipping method
+                    message= "Shipping Method "+carrierPartyId+" 
"+shipMethTypeDesc+" is not applicable to shipping address. 
"+shippingMethods.get(0).getString("carrierPartyId")+" "+newShipMethTypeDesc+" 
has been set for shipping address.";
+                }
+                shoppingCart.setShippingContactMechId(groupIdx-1, 
contactMechId);
+            }
+        }
+
+        // save cart after updating shipping method and shipping address.
+        Map<String, Object> changeMap = new HashMap<String, Object>();
+        try {
+            saveUpdatedCartToOrder(dispatcher, delegator, shoppingCart, 
locale, userLogin, orderId, changeMap, true, false);
+        } catch(GeneralException e) {
+            Debug.logError(e, module);
+        }
+
+        if (UtilValidate.isNotEmpty(message)) {
+            return ServiceUtil.returnSuccess(message);
+        } else {
+            return ServiceUtil.returnSuccess();
+        }
+    }
 }

Modified: 
ofbiz/branches/release13.07/applications/order/webapp/ordermgr/WEB-INF/actions/order/OrderView.groovy
URL: 
http://svn.apache.org/viewvc/ofbiz/branches/release13.07/applications/order/webapp/ordermgr/WEB-INF/actions/order/OrderView.groovy?rev=1686645&r1=1686644&r2=1686645&view=diff
==============================================================================
--- 
ofbiz/branches/release13.07/applications/order/webapp/ordermgr/WEB-INF/actions/order/OrderView.groovy
 (original)
+++ 
ofbiz/branches/release13.07/applications/order/webapp/ordermgr/WEB-INF/actions/order/OrderView.groovy
 Sat Jun 20 16:01:54 2015
@@ -29,6 +29,7 @@ import org.ofbiz.order.order.*;
 import org.ofbiz.party.contact.*;
 import org.ofbiz.product.inventory.InventoryWorker;
 import org.ofbiz.product.catalog.CatalogWorker;
+import org.ofbiz.product.store.ProductStoreWorker;
 import org.ofbiz.accounting.payment.*;
 
 import javolution.util.FastMap;
@@ -404,8 +405,19 @@ if (orderHeader) {
     context.shippingContactMechList = shippingContactMechList;
 
     // list to find all the shipmentMethods from the view named 
"ProductStoreShipmentMethView".
-    if (productStore) {
-        context.productStoreShipmentMethList = 
delegator.findByAnd('ProductStoreShipmentMethView', [productStoreId: 
productStore.productStoreId], ['sequenceNumber'], true);
+    shipGroupShippingMethods = [:];
+    shipGroups.each { shipGroup ->
+        shipGroupSeqId = shipGroup.shipGroupSeqId;
+        shippableItemFeatures = 
orderReadHelper.getFeatureIdQtyMap(shipGroupSeqId);
+        shippableTotal = orderReadHelper.getShippableTotal(shipGroupSeqId);
+        shippableWeight = orderReadHelper.getShippableWeight(shipGroupSeqId);
+        shippableItemSizes = orderReadHelper.getShippableSizes(shipGroupSeqId);
+        shippingAddress = orderReadHelper.getShippingAddress(shipGroupSeqId);
+
+        List<GenericValue> productStoreShipmentMethList = 
ProductStoreWorker.getAvailableStoreShippingMethods(delegator, 
orderReadHelper.getProductStoreId(),
+                shippingAddress, shippableItemSizes, shippableItemFeatures, 
shippableWeight, shippableTotal);
+        shipGroupShippingMethods.put(shipGroupSeqId, 
productStoreShipmentMethList);
+        context.shipGroupShippingMethods = shipGroupShippingMethods;
     }
 
     // Get a map of returnable items

Modified: 
ofbiz/branches/release13.07/applications/order/webapp/ordermgr/WEB-INF/controller.xml
URL: 
http://svn.apache.org/viewvc/ofbiz/branches/release13.07/applications/order/webapp/ordermgr/WEB-INF/controller.xml?rev=1686645&r1=1686644&r2=1686645&view=diff
==============================================================================
--- 
ofbiz/branches/release13.07/applications/order/webapp/ordermgr/WEB-INF/controller.xml
 (original)
+++ 
ofbiz/branches/release13.07/applications/order/webapp/ordermgr/WEB-INF/controller.xml
 Sat Jun 20 16:01:54 2015
@@ -1695,7 +1695,7 @@ under the License.
     <request-map uri="addShippingAddress">
         <security https="true" auth="true"/>
         <event type="service" invoke="createUpdateShippingAddress"/>
-        <response name="success" type="request" 
value="updateOrderItemShipGroup"/>
+        <response name="success" type="request" 
value="updateShipGroupShipInfo"/>
         <response name="error" type="view" value="orderview"/>
     </request-map>
     <request-map uri="getAssociatedStateList">
@@ -1850,6 +1850,13 @@ under the License.
         <response name="success" type="view" value="splitship"/>
         <response name="error" type="request" value="splitship"/>
     </request-map>
+    
+    <request-map uri="updateShipGroupShipInfo">
+        <security https="true" auth="true"/>
+        <event type="service" invoke="updateShipGroupShipInfo"/>
+        <response name="success" type="view" value="orderview"/>
+        <response name="error" type="view" value="orderview"/>
+    </request-map>
 
     <!--
         These are just examples of reports developed using JasperReport and 
not really

Modified: 
ofbiz/branches/release13.07/applications/order/webapp/ordermgr/order/ordershippinginfo.ftl
URL: 
http://svn.apache.org/viewvc/ofbiz/branches/release13.07/applications/order/webapp/ordermgr/order/ordershippinginfo.ftl?rev=1686645&r1=1686644&r2=1686645&view=diff
==============================================================================
--- 
ofbiz/branches/release13.07/applications/order/webapp/ordermgr/order/ordershippinginfo.ftl
 (original)
+++ 
ofbiz/branches/release13.07/applications/order/webapp/ordermgr/order/ordershippinginfo.ftl
 Sat Jun 20 16:01:54 2015
@@ -203,7 +203,7 @@ under the License.
        <br class="clear"/>
     </div>
     <div class="screenlet-body" 
id="ShipGroupScreenletBody_${shipGroup.shipGroupSeqId}">
-        <form name="updateOrderItemShipGroup" method="post" 
action="<@ofbizUrl>updateOrderItemShipGroup</@ofbizUrl>">
+         <form name="updateOrderItemShipGroup" method="post" 
action="<@ofbizUrl>updateShipGroupShipInfo</@ofbizUrl>">
         <input type="hidden" name="orderId" value="${orderId?if_exists}"/>
         <input type="hidden" name="shipGroupSeqId" 
value="${shipGroup.shipGroupSeqId?if_exists}"/>
         <input type="hidden" name="contactMechPurposeTypeId" 
value="SHIPPING_LOCATION"/>
@@ -256,7 +256,7 @@ under the License.
                                 <#else>
                                 <option value=""/>
                                 </#if>
-                                <#list productStoreShipmentMethList as 
productStoreShipmentMethod>
+                                <#list 
shipGroupShippingMethods[shipGroup.shipGroupSeqId] as 
productStoreShipmentMethod>
                                 <#assign shipmentMethodTypeAndParty = 
productStoreShipmentMethod.shipmentMethodTypeId + "@" + 
productStoreShipmentMethod.partyId + "@" + 
productStoreShipmentMethod.roleTypeId>
                                 <#if 
productStoreShipmentMethod.partyId?has_content || 
productStoreShipmentMethod?has_content>
                                 <option 
value="${shipmentMethodTypeAndParty?if_exists}"><#if 
productStoreShipmentMethod.partyId != 
"_NA_">${productStoreShipmentMethod.partyId?if_exists}</#if>&nbsp;${productStoreShipmentMethod.get("description",locale)?default("")}</option>


Reply via email to