You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ofbiz.apache.org by jo...@apache.org on 2007/07/18 03:15:15 UTC

svn commit: r557109 - /ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/callops/SetServiceFields.java

Author: jonesde
Date: Tue Jul 17 18:15:13 2007
New Revision: 557109

URL: http://svn.apache.org/viewvc?view=rev&rev=557109
Log:
Small change to correct problem with the last one; when the type conversion fails we don't want to put the original value in, it will cause a service call exception; set to null to just leave out

Modified:
    ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/callops/SetServiceFields.java

Modified: ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/callops/SetServiceFields.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/callops/SetServiceFields.java?view=diff&rev=557109&r1=557108&r2=557109
==============================================================================
--- ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/callops/SetServiceFields.java (original)
+++ ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/callops/SetServiceFields.java Tue Jul 17 18:15:13 2007
@@ -102,7 +102,9 @@
                     } catch (GeneralException e) {
                         String errMsg = "Could not convert field value for the parameter/attribute: [" + modelParam.name + "] on the [" + serviceName + "] service to the [" + modelParam.type + "] type for the value [" + value + "]: " + e.toString();
                         Debug.logError(e, errMsg, module);
+                        // add the message to the list and set the value to null - tried and failed, just leave it out
                         messages.add(errMsg);
+                        value = null;
                     }
                 }