Actually, this is turning out to be a little tougher than it seemed at first glance - because of SOAP.
It's easy to check for XML serialization to/from the entity engine because the root element is always <ofbiz-ser>. It's easy to check for SOAP response serialization because the root element is always <Response>. The problem is with a SOAP request - the root element could be anything. From my perspective (and I know very little about SOAP, so take this for what it is worth) OFBiz should have SoapRequest and SoapResponse classes, and those objects should be serialized/deserialized in XML - instead of the current implementation. -Adrian --- On Sat, 2/13/10, Adrian Crum <[email protected]> wrote: > From: Adrian Crum <[email protected]> > Subject: Re: XmlSerializer.java > To: [email protected] > Date: Saturday, February 13, 2010, 8:01 PM > I agree. That's exactly what I did in > my local copy. :-) > > -Adrian > > --- On Sat, 2/13/10, David E Jones <[email protected]> > wrote: > > > From: David E Jones <[email protected]> > > Subject: Re: XmlSerializer.java > > To: [email protected] > > Date: Saturday, February 13, 2010, 7:58 PM > > > > We should at least keep the deserialization code > around > > forever for backward compatibility. You can select > which > > deserializer to use based on the root element or > something, > > that shouldn't be tough. > > > > -David > > > > > > On Feb 13, 2010, at 8:53 PM, Adrian Crum wrote: > > > > > I recently added the XStream library to the > project - > > which enables serializing/deserializing Java objects > to XML > > or JSON. > > > > > > With that addition, the XmlSerializer class is > no > > longer needed. The XStream library is more > sophisticated and > > extensible than the home-grown code. > > > > > > Switching XML serializers introduces a potential > > compatibility problem with existing installations - > since > > Java objects are stored in the database in XML format. > I > > believe I have come up with a strategy that provides > a > > migration path, and I'm testing some ideas now. > > > > > > So, I just wanted to let the community know what > I > > have in mind and see if there are any comments. > > > > > > -Adrian > > > > > > > > > > > > > > > > > > > >
