Great, thanks!

Looking forward :)

Jacques

Le 31/03/2016 12:43, Divesh Dutta a écrit :
Hi Jacques,

Will look into it soon.

Thanks
--
Divesh Dutta.

On Tue, Mar 22, 2016 at 2:49 PM, Jacques Le Roux <
jacques.le.r...@les7arts.com> wrote:

Reverted at r1736164

Jacques


Le 19/03/2016 16:26, Jacques Le Roux a écrit :

Hi Divesh,

Sorry but after this change few tests fail:
https://ci.apache.org/builders/ofbiz-trunk
https://ci.apache.org/builders/ofbiz-trunk/builds/477
https://ci.apache.org/projects/ofbiz/logs/trunk/html/

Jacques

Le 19/03/2016 15:38, diveshdu...@apache.org a écrit :

Author: diveshdutta
Date: Sat Mar 19 14:38:48 2016
New Revision: 1735777

URL: http://svn.apache.org/viewvc?rev=1735777&view=rev
Log:
[OFBIZ-1585]

Modified:
ofbiz/trunk/applications/order/config/OrderErrorUiLabels.xml

ofbiz/trunk/applications/order/src/org/ofbiz/order/shoppingcart/CheckOutEvents.java

ofbiz/trunk/specialpurpose/ecommerce/webapp/ecommerce/WEB-INF/controller.xml

Modified: ofbiz/trunk/applications/order/config/OrderErrorUiLabels.xml
URL:
http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/config/OrderErrorUiLabels.xml?rev=1735777&r1=1735776&r2=1735777&view=diff

==============================================================================
--- ofbiz/trunk/applications/order/config/OrderErrorUiLabels.xml
(original)
+++ ofbiz/trunk/applications/order/config/OrderErrorUiLabels.xml Sat Mar
19 14:38:48 2016
@@ -6699,4 +6699,10 @@
           <value xml:lang="vi">Bạn không thể thêm sản phẩm
ảo ('Virtual' product) tại đây, bạn hãy chỠn trong các hà ng
hóa liên quan (related variants).</value>
           <value xml:lang="zh">åœ¨è¿™é‡Œä½ ä¸ èƒ½æ·»åŠ ä¸€ä¸ªè™šæ‹Ÿäº§å“
ï¼Œè¯·é€‰æ‹©ä¸€ä¸ªç›¸å…³çš„å ˜åž‹ã€‚</value>
       </property>
+    <property key="OrderNoPaymentMethodSelected">
+        <value xml:lang="en">No Payment Method selected </value>
+    </property>
+    <property key="OrderNoShipmentMethodSelected">
+        <value xml:lang="en">No Shipment Method selected</value>
+    </property>
   </resource>

Modified:
ofbiz/trunk/applications/order/src/org/ofbiz/order/shoppingcart/CheckOutEvents.java
URL:
http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/src/org/ofbiz/order/shoppingcart/CheckOutEvents.java?rev=1735777&r1=1735776&r2=1735777&view=diff

==============================================================================
---
ofbiz/trunk/applications/order/src/org/ofbiz/order/shoppingcart/CheckOutEvents.java
(original)
+++
ofbiz/trunk/applications/order/src/org/ofbiz/order/shoppingcart/CheckOutEvents.java
Sat Mar 19 14:38:48 2016
@@ -420,6 +420,24 @@ public class CheckOutEvents {
           return "success";
       }
   +    // Check for payment method and shipping method exist for
checkout process of anonymous user
+    public static String checkoutValidation(HttpServletRequest request,
HttpServletResponse response) {
+        ShoppingCart cart = (ShoppingCart)
request.getSession().getAttribute("shoppingCart");
+        List<GenericValue> paymentMethods = cart.getPaymentMethods();
+        if (UtilValidate.isEmpty(paymentMethods)) {
+            String errMsg = UtilProperties.getMessage(resource_error,
"OrderNoPaymentMethodSelected", (cart != null ? cart.getLocale() :
UtilHttp.getLocale(request)));
+            request.setAttribute("_ERROR_MESSAGE_",errMsg);
+            return "error";
+        }
+        String shipmentMethod = cart.getShipmentMethodTypeId();
+        if (UtilValidate.isEmpty(shipmentMethod)) {
+            String errMsg = UtilProperties.getMessage(resource_error,
"OrderNoShipmentMethodSelected", (cart != null ? cart.getLocale() :
UtilHttp.getLocale(request)));
+            request.setAttribute("_ERROR_MESSAGE_",errMsg);
+            return "error";
+        }
+        return "success";
+    }
+
       // Create order event - uses createOrder service for processing
       public static String createOrder(HttpServletRequest request,
HttpServletResponse response) {
           HttpSession session = request.getSession();
@@ -429,6 +447,8 @@ public class CheckOutEvents {
           GenericValue userLogin = (GenericValue)
session.getAttribute("userLogin");
           CheckOutHelper checkOutHelper = new CheckOutHelper(dispatcher,
delegator, cart);
           Map<String, Object> callResult;
+        String result = checkoutValidation(request,response);
+        if ("error".equals(result)) return "error";
             if (UtilValidate.isEmpty(userLogin)) {
               userLogin = cart.getUserLogin();

Modified:
ofbiz/trunk/specialpurpose/ecommerce/webapp/ecommerce/WEB-INF/controller.xml
URL:
http://svn.apache.org/viewvc/ofbiz/trunk/specialpurpose/ecommerce/webapp/ecommerce/WEB-INF/controller.xml?rev=1735777&r1=1735776&r2=1735777&view=diff

==============================================================================
---
ofbiz/trunk/specialpurpose/ecommerce/webapp/ecommerce/WEB-INF/controller.xml
(original)
+++
ofbiz/trunk/specialpurpose/ecommerce/webapp/ecommerce/WEB-INF/controller.xml
Sat Mar 19 14:38:48 2016
@@ -733,7 +733,7 @@ under the License.
           <event type="java"
path="org.ofbiz.order.shoppingcart.CheckOutEvents" invoke="createOrder"/>
           <response name="sales_order" type="request"
value="checkBlacklist"/>
           <response name="work_order" type="request"
value="checkBlacklist"/>
-        <response name="error" type="view" value="checkoutreview"/>
+        <response name="error" type="view"
value="quickAnonOrderReview"/>
       </request-map>
       <request-map uri="processpayment">
           <security https="true" direct-request="false"/>




Reply via email to