Author: jleroux Date: Sat Dec 31 10:16:20 2011 New Revision: 1226059 URL: http://svn.apache.org/viewvc?rev=1226059&view=rev Log: "Applied fix from trunk for revision: 1226055 " ------------------------------------------------------------------------ r1226055 | jleroux | 2011-12-31 11:13:34 +0100 (sam., 31 déc. 2011) | 9 lines
Should close "Null values are not synchronized in http mode" https://issues.apache.org/jira/browse/OFBIZ-4602 I followed Adrian's advice. I tried to create an order with a dropship1 item (like in Jira 4637) and it worked perfectly. I guess, as we were suspecting, using set(name, ""); instead of element.setAttribute(name, "null"); had side effects. Anyway, as Adrian pointed out, the right returned string should be "[null-field]" and it's now correct/consistent. On the other side (entity sync clients), of course, now "[null-field]" should be handled instead of "" or "null" ------------------------------------------------------------------------ Modified: ofbiz/branches/release4.0/ (props changed) ofbiz/branches/release4.0/framework/entity/src/org/ofbiz/entity/GenericEntity.java Propchange: ofbiz/branches/release4.0/ ------------------------------------------------------------------------------ --- svn:mergeinfo (original) +++ svn:mergeinfo Sat Dec 31 10:16:20 2011 @@ -1 +1 @@ -/ofbiz/trunk:536399,539836-539837,618970,627900,629279,674173,676162,676227,676246,679704,690644,705862,706035,706055,706067,706692,721839,721887,728935,737443,738870,741491,808792,814731,827730,890245,942884,943168,954956,958343,958514,965916,967098,980935,981123 +/ofbiz/trunk:536399,539836-539837,618970,627900,629279,674173,676162,676227,676246,679704,690644,705862,706035,706055,706067,706692,721839,721887,728935,737443,738870,741491,808792,814731,827730,890245,942884,943168,954956,958343,958514,965916,967098,980935,981123,1226055 Modified: ofbiz/branches/release4.0/framework/entity/src/org/ofbiz/entity/GenericEntity.java URL: http://svn.apache.org/viewvc/ofbiz/branches/release4.0/framework/entity/src/org/ofbiz/entity/GenericEntity.java?rev=1226059&r1=1226058&r2=1226059&view=diff ============================================================================== --- ofbiz/branches/release4.0/framework/entity/src/org/ofbiz/entity/GenericEntity.java (original) +++ ofbiz/branches/release4.0/framework/entity/src/org/ofbiz/entity/GenericEntity.java Sat Dec 31 10:16:20 2011 @@ -944,6 +944,8 @@ public class GenericEntity extends Obser } else { element.setAttribute(name, value); } + } else { + element.setAttribute(name, GenericEntity.NULL_FIELD.toString()); } }
