Add support for package binding in the JibxMashaller [SPR-7805]

[spring-projects-issues]

Tiago Stürmer Daitx opened SPR-7805 and commented

As per https://jira.springframework.org/browse/SWS-229 JibxMashaller already supports all the mapped classes in the IBindingFactory (wonder why the docs don't say anything about that) but I needed a way to set it per package so I have created my own JibxMarshaller and have been using it successfully so far.

By default it will ignore targetPackage property if targetClass property is set to keep the usual behavior.

I have also updated it to fix https://jira.springsource.org/browse/SWS-659 as stated on the forum because I had a similar scenario where I needed multiple marshallers (one per root element). Since changing this method will break anyone who has been using the "set class and get binding for all mapped classes" feature/bug I'm adding two patches: one with all the changes and another with everything but the supports(Class) method change.

BTW, I generated the patch against JibxMarshaller from Spring 3.0.5. Let me know if I really need to do it against SWS 1.5.9.

---

Affects: 3.0.5

Attachments:

• jibxmarshaller-targetpackage-only.patch (2.66 kB)
• jibxmarshaller-targetpackage-with-supports.patch (2.99 kB)

Issue Links:

• JibxMarshaller supports method returns true for classes not in the configured binding [SPR-7804] #12460 JibxMarshaller supports method returns true for classes not in the configured binding

Referenced from: commits fa4f90e

[spring-projects-issues]

Arjen Poutsma commented

No need to create a SWS version of this patch, as SWS's OXM has been removed completely in favor of SPR's OXM.

[spring-projects-issues]

Arjen Poutsma commented

I've applied the -with-supports patch. This can result in some issues with regards to backward compatibility, but as the patch provides a clear improvement over the current code, I'm OK with doing this for Spring 3.1.

Thanks!