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

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

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

jacopoc
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)));