You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ofbiz.apache.org by de...@apache.org on 2016/03/19 14:31:02 UTC

svn commit: r1735773 - /ofbiz/trunk/applications/product/src/org/ofbiz/product/category/CatalogUrlServlet.java

Author: deepak
Date: Sat Mar 19 13:31:02 2016
New Revision: 1735773

URL: http://svn.apache.org/viewvc?rev=1735773&view=rev
Log:
(OFBIZ-5918) Fixed NPE for ecommerce/products url, If user enter invalid url ecommerce/products then redirect it to main.

Modified:
    ofbiz/trunk/applications/product/src/org/ofbiz/product/category/CatalogUrlServlet.java

Modified: ofbiz/trunk/applications/product/src/org/ofbiz/product/category/CatalogUrlServlet.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/product/src/org/ofbiz/product/category/CatalogUrlServlet.java?rev=1735773&r1=1735772&r2=1735773&view=diff
==============================================================================
--- ofbiz/trunk/applications/product/src/org/ofbiz/product/category/CatalogUrlServlet.java (original)
+++ ofbiz/trunk/applications/product/src/org/ofbiz/product/category/CatalogUrlServlet.java Sat Mar 19 13:31:02 2016
@@ -82,6 +82,11 @@ public class CatalogUrlServlet extends H
 
         String productId = null;
         String categoryId = null;
+
+        if (pathElements == null) {
+            RequestDispatcher rd = request.getRequestDispatcher("/" + CONTROL_MOUNT_POINT + "/main");
+            rd.forward(request, response);
+        } else {
         try {
             String lastPathElement = pathElements.get(pathElements.size() - 1);
             if (lastPathElement.startsWith("p_")) {
@@ -146,6 +151,7 @@ public class CatalogUrlServlet extends H
 
         RequestDispatcher rd = request.getRequestDispatcher("/" + CONTROL_MOUNT_POINT + "/" + (productId != null ? PRODUCT_REQUEST : CATEGORY_REQUEST));
         rd.forward(request, response);
+        }
     }
 
     /**