You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ofbiz.apache.org by er...@apache.org on 2011/02/28 15:45:58 UTC

svn commit: r1075356 - /ofbiz/trunk/applications/product/src/org/ofbiz/product/spreadsheetimport/ImportProductHelper.java

Author: erwan
Date: Mon Feb 28 14:45:57 2011
New Revision: 1075356

URL: http://svn.apache.org/viewvc?rev=1075356&view=rev
Log:
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);