Decouple domain and XML parsing concerns

[agseco]

Currently, domain classes like PaymentMethod are tied to SimpleXML's annotations. I think this is not good and perhaps they should be decoupled from this concerns somehow...

What do you think?

[mfornos]

👍 Should be decoupled, but first should be decoupled the XML provider, in fact SimpleXML uses StAX under the hood. I think that is not a big issue, since SimpleXML is a lightweight dependency and allows to plug your preferred StAX implementation.

[agseco]

👌 let's do that.

[mfornos]

The model also has dependencies on SimpleXML annotations.