[KULRICE-5188] Need to remove ImmutableListAdapter and ImmutableCollectionAdapter from our service definitions and come up with a better solution because it is producing undesirable XML Created: 06/Jun/11  Updated: 23/Feb/12  Resolved: 13/Jun/11

Status: Closed
Project: Kuali Rice Development
Component/s: Development, Version Compatibility
Affects Version/s: None
Fix Version/s: 2.0.0-m5, 2.0
Security Level: Public (Public: Anyone can view)

Type: Bug Fix Priority: Blocker
Reporter: Eric Westfall Assignee: Jason Whaley (Inactive)
Resolution: Fixed Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

KAI Review Status: Not Required
KTI Review Status: Not Required


Currently, the ImmutableListAdapter is producing XML schema like the following (this example is from the GroupService):

<xs:complexType name="getDirectParentGroupIdsResponse">
     <xs:element minOccurs="0" name="directParentGroupIds" type="ns1:anyTypeArray"/>

In this case, anyTypeArray comes from the http://jaxb.dev.java.net/array schema and does not properly use the schema types from the elements of the "array". This is because this adapater is currently declared as follows:

public class ImmutableListAdapter extends XmlAdapter<Object[], List<?>> {

This use of Object[] triggers the issue. Unfortunately, because of the nature of type erasure and the way that the JAXB adapter system works, there is no way to create a single, general-purpose adapter that targets the appropriate parameterized list type.

Possible options here include:

  1. Look at the possibility introducing a custom CXF interceptor which will translate appropriately, would need to define a custom marker annotation to identify which lists/collections need to be unmodifiable
  2. Add some functionality to the KSB so that it creates it's own client-side proxies that consider immutability on list return types, would still need a custom annotation as above
  3. Create an adapter class per immutable parameterized list type and annotate using that

Either way, the current usage of ImmutableListAdapter and ImmutableCollectionAdapter needs to be removed.

Comment by Jason Whaley (Inactive) [ 07/Jun/11 ]

Polled the cxf-user mailing list for feedback on how best to handle this.

Thread is here: http://cxf.547215.n5.nabble.com/XmlAdapter-or-CXF-Interceptor-to-enforce-immutability-on-returned-Collections-Lists-td4462231.html

Comment by Jason Whaley (Inactive) [ 13/Jun/11 ]

Marking resolved as of r20889 in trunk.

Comment by Rice-CI User (Inactive) [ 14/Jun/11 ]

Integrated in rice-trunk-nightly #86 (See http://ci.rice.kuali.org/job/rice-trunk-nightly/86/)
KULRICE-5188: committing the actual interceptor used to replayce Immutable*Adapters
KULRICE-5188: Removed Immutable*Adapters and replaced with a CXF interceptor

Comment by Jessica Coltrin (Inactive) [ 23/Feb/12 ]

Closing since these items are now in the release notes.

Generated at Thu Oct 22 02:07:52 CDT 2020 using JIRA 7.0.11#70121-sha1:19d24976997c1d95f06f3e327e087be0b71f28d4.