svn commit: r1033928 - /ofbiz/trunk/applications/product/src/org/ofbiz/product/product/KeywordIndex.java

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

svn commit: r1033928 - /ofbiz/trunk/applications/product/src/org/ofbiz/product/product/KeywordIndex.java

ashish-18
Author: ashish
Date: Thu Nov 11 13:08:38 2010
New Revision: 1033928

URL: http://svn.apache.org/viewvc?rev=1033928&view=rev
Log:
Fixing NPE. Thanks Pranay!

Modified:
    ofbiz/trunk/applications/product/src/org/ofbiz/product/product/KeywordIndex.java

Modified: ofbiz/trunk/applications/product/src/org/ofbiz/product/product/KeywordIndex.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/product/src/org/ofbiz/product/product/KeywordIndex.java?rev=1033928&r1=1033927&r2=1033928&view=diff
==============================================================================
--- ofbiz/trunk/applications/product/src/org/ofbiz/product/product/KeywordIndex.java (original)
+++ ofbiz/trunk/applications/product/src/org/ofbiz/product/product/KeywordIndex.java Thu Nov 11 13:08:38 2010
@@ -32,6 +32,7 @@ import org.ofbiz.base.util.GeneralExcept
 import org.ofbiz.base.util.UtilDateTime;
 import org.ofbiz.base.util.UtilMisc;
 import org.ofbiz.base.util.UtilProperties;
+import org.ofbiz.base.util.UtilValidate;
 import org.ofbiz.common.KeywordSearchUtil;
 import org.ofbiz.content.data.DataResourceWorker;
 import org.ofbiz.entity.Delegator;
@@ -182,10 +183,11 @@ public class KeywordIndex {
                 }
             }
         }
-
-        for (String str: strings) {
-            // call process keywords method here
-            KeywordSearchUtil.processKeywordsForIndex(str, keywords, separators, stopWordBagAnd, stopWordBagOr, removeStems, stemSet);
+        if (UtilValidate.isNotEmpty(strings)) {
+            for (String str: strings) {
+                // call process keywords method here
+                KeywordSearchUtil.processKeywordsForIndex(str, keywords, separators, stopWordBagAnd, stopWordBagOr, removeStems, stemSet);
+            }
         }
 
         List<GenericValue> toBeStored = FastList.newInstance();