search

caravelo / spreedly-java-client

Spreedly Client for Java
5 stars 0 forks source link

Decouple domain and XML parsing concerns #4

Open agseco opened 9 years ago

agseco commented 9 years ago

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 commented 9 years ago

:+1: 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 commented 9 years ago

:ok_hand: let's do that.

mfornos commented 9 years ago

The model also has dependencies on SimpleXML annotations.