Author: erwan
Date: Mon Feb 28 14:45:57 2011
New Revision: 1075356
URL:
http://svn.apache.org/viewvc?rev=1075356&view=revLog:
String test should be using .equals instead of ==
Modified:
ofbiz/trunk/applications/product/src/org/ofbiz/product/spreadsheetimport/ImportProductHelper.java
Modified: ofbiz/trunk/applications/product/src/org/ofbiz/product/spreadsheetimport/ImportProductHelper.java
URL:
http://svn.apache.org/viewvc/ofbiz/trunk/applications/product/src/org/ofbiz/product/spreadsheetimport/ImportProductHelper.java?rev=1075356&r1=1075355&r2=1075356&view=diff==============================================================================
--- ofbiz/trunk/applications/product/src/org/ofbiz/product/spreadsheetimport/ImportProductHelper.java (original)
+++ ofbiz/trunk/applications/product/src/org/ofbiz/product/spreadsheetimport/ImportProductHelper.java Mon Feb 28 14:45:57 2011
@@ -68,7 +68,7 @@ public class ImportProductHelper {
tmpProductGV = delegator.findByPrimaryKey("Product", UtilMisc
.toMap("productId", productId));
if (tmpProductGV != null
- && tmpProductGV.getString("productId") == productId)
+ && productId.equals(tmpProductGV.getString("productId")))
productExists = true;
} catch (GenericEntityException e) {
Debug.logError("Problem in reading data of product", module);