svn commit: r820249 - /ofbiz/trunk/applications/product/src/org/ofbiz/product/product/ProductSearchSession.java

[ashish-18]

Author: ashish
Date: Wed Sep 30 11:18:03 2009
New Revision: 820249

URL: http://svn.apache.org/viewvc?rev=820249&view=rev
Log:
Reverted changes of previous commit.
This can create problem for the projects in which "prodSearchExcludeVariants" field is not been set in past so the not-empty check will allow variant product to be included on the search.

Thanks Scott for your comment.

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

Modified: ofbiz/trunk/applications/product/src/org/ofbiz/product/product/ProductSearchSession.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/product/src/org/ofbiz/product/product/ProductSearchSession.java?rev=820249&r1=820248&r2=820249&view=diff
==============================================================================
--- ofbiz/trunk/applications/product/src/org/ofbiz/product/product/ProductSearchSession.java (original)
+++ ofbiz/trunk/applications/product/src/org/ofbiz/product/product/ProductSearchSession.java Wed Sep 30 11:18:03 2009
@@ -767,7 +767,7 @@
         }
 
         // check the ProductStore to see if we should add the ExcludeVariantsConstraint
-        if (productStore != null && UtilValidate.isNotEmpty(productStore.getString("prodSearchExcludeVariants")) && !"N".equals(productStore.getString("prodSearchExcludeVariants"))) {
+        if (productStore != null && !"N".equals(productStore.getString("prodSearchExcludeVariants"))) {
             searchAddConstraint(new ProductSearch.ExcludeVariantsConstraint(), session);
             // not consider this a change for now, shouldn't change often: constraintsChanged = true;
         }