svn commit: r1226059 - in /ofbiz/branches/release4.0: ./ framework/entity/src/org/ofbiz/entity/GenericEntity.java

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());
             }
         }