[jbossws-commits] JBossWS SVN: r5795 - in stack/native/trunk/src: test/java/org/jboss/test/ws/jaxws/xop and 1 other directory.

Author: heiko.braun(a)jboss.com Date: 2008-02-25 07:55:56 -0500 (Mon, 25 Feb 2008) New Revision: 5795 Modified: stack/native/trunk/src/main/java/org/jboss/ws/extensions/xop/jaxws/ReflectiveAttachmentRefScanner.java stack/native/trunk/src/test/java/org/jboss/test/ws/jaxws/xop/MimeDeclarationTestCase.java Log: Fix JBWS-2012: Attachment support for JAX-WS collections Modified: stack/native/trunk/src/main/java/org/jboss/ws/extensions/xop/jaxws/ReflectiveAttachmentRefScanner.java =================================================================== --- stack/native/trunk/src/main/java/org/jboss/ws/extensions/xop/jaxws/ReflectiveAttachmentRefScanner.java 2008-02-25 12:08:24 UTC (rev 5794) +++ stack/native/trunk/src/main/java/org/jboss/ws/extensions/xop/jaxws/ReflectiveAttachmentRefScanner.java 2008-02-25 12:55:56 UTC (rev 5795) @@ -30,6 +30,8 @@ import java.awt.*; import java.lang.reflect.Field; import java.lang.reflect.Method; +import java.lang.reflect.ParameterizedType; +import java.lang.reflect.Type; import java.lang.annotation.Annotation; import java.util.ArrayList; import java.util.List; @@ -82,7 +84,9 @@ Class<?> type = field.getType(); boolean exceptionToTheRule = isAttachmentDataType(type); - + if (! exceptionToTheRule) { + type = getFieldComponentType(field); + } // only non JDK types are inspected except for byte[] and java.lang.String if( !alreadyScanned(field) && (exceptionToTheRule || !isJDKType(type)) ) { @@ -218,6 +222,31 @@ scannedFields.clear(); } + /** + * In the case of an array T[] or a List<T> returns T, else returns the field type + * + * @param clazz + * @return the type of the field, if the field is an array returns the component type, + * if the field is declared as List<T> returns T + */ + private Class<?> getFieldComponentType(Field field) { + Class<?> fieldType = field.getType(); + if (fieldType.isArray()) { + return fieldType.getComponentType(); + } else if (List.class.isAssignableFrom(fieldType)) { + if (field.getGenericType() instanceof ParameterizedType) { + ParameterizedType paramType = (ParameterizedType) field.getGenericType(); + if ((paramType.getRawType() instanceof Class) && List.class.isAssignableFrom((Class<?>) paramType.getRawType())) { + Type[] actualTypes = paramType.getActualTypeArguments(); + if (actualTypes.length == 1 && (actualTypes[0] instanceof Class)) { + return (Class<?>) actualTypes[0]; + } + } + } + } + return fieldType; + } + private static boolean isAttachmentDataType(Class clazz) { for(Class cl : SUPPORTED_TYPES) { Modified: stack/native/trunk/src/test/java/org/jboss/test/ws/jaxws/xop/MimeDeclarationTestCase.java =================================================================== --- stack/native/trunk/src/test/java/org/jboss/test/ws/jaxws/xop/MimeDeclarationTestCase.java 2008-02-25 12:08:24 UTC (rev 5794) +++ stack/native/trunk/src/test/java/org/jboss/test/ws/jaxws/xop/MimeDeclarationTestCase.java 2008-02-25 12:55:56 UTC (rev 5795) @@ -108,6 +108,20 @@ assertFalse("MTOM should be disabled", XOPContext.isMTOMEnabled()); } + public void testNestedArray() throws Exception + { + AttachmentScanResult mimeType = SCANNER.scanBean(NestedArray.class); + assertNotNull("Unable to find xop declaration", mimeType); + assertEquals("text/plain", mimeType.getMimeType()); + } + + public void testNestedList() throws Exception + { + AttachmentScanResult mimeType = SCANNER.scanBean(NestedList.class); + assertNotNull("Unable to find xop declaration", mimeType); + assertEquals("text/plain", mimeType.getMimeType()); + } + class FieldAnnotation { @XmlMimeType("text/xml") @@ -158,4 +172,12 @@ @XmlMimeType("text/plain") String data; } + + class NestedArray { + Nested[] nested; + } + + class NestedList { + List<Nested> nested; + } }