You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ofbiz.apache.org by ja...@apache.org on 2006/08/13 07:32:32 UTC

svn commit: r431145 - /incubator/ofbiz/trunk/applications/product/src/org/ofbiz/product/product/ProductEvents.java

Author: jacopoc
Date: Sat Aug 12 22:32:31 2006
New Revision: 431145

URL: http://svn.apache.org/viewvc?rev=431145&view=rev
Log:
Applied patch from Scott Gray: the code is checking to see if productId is empty twice, instead of checking productId and AttrName. 

Modified:
    incubator/ofbiz/trunk/applications/product/src/org/ofbiz/product/product/ProductEvents.java

Modified: incubator/ofbiz/trunk/applications/product/src/org/ofbiz/product/product/ProductEvents.java
URL: http://svn.apache.org/viewvc/incubator/ofbiz/trunk/applications/product/src/org/ofbiz/product/product/ProductEvents.java?rev=431145&r1=431144&r2=431145&view=diff
==============================================================================
--- incubator/ofbiz/trunk/applications/product/src/org/ofbiz/product/product/ProductEvents.java (original)
+++ incubator/ofbiz/trunk/applications/product/src/org/ofbiz/product/product/ProductEvents.java Sat Aug 12 22:32:31 2006
@@ -600,7 +600,7 @@
 
         if (!UtilValidate.isNotEmpty(productId))
             errMsg += ("<li>" + UtilProperties.getMessage(resource,"productevents.product_ID_missing", UtilHttp.getLocale(request)));
-        if (!UtilValidate.isNotEmpty(productId))
+        if (!UtilValidate.isNotEmpty(attrName))
             errMsg += ("<li>" + UtilProperties.getMessage(resource,"productevents.attribute_name_missing", UtilHttp.getLocale(request)));
         if (errMsg.length() > 0) {
             errMsg += ("<b>" + UtilProperties.getMessage(resource,"productevents.following_errors_occurred", UtilHttp.getLocale(request)));