svn commit: r1143520 - in /openjpa/trunk/openjpa-persistence/src/main/java/org/apache/openjpa/persistence/meta: AnnotationProcessor6.java SourceAnnotationHandler.java

[mikedd]

Author: mikedd
Date: Wed Jul  6 18:18:43 2011
New Revision: 1143520

URL: http://svn.apache.org/viewvc?rev=1143520&view=rev
Log:
OPENJPA-2025: Generate SingularAttribute for unannotated arrays, ListAttribute 
will be used for @PersistentCollections.

Modified:
    
openjpa/trunk/openjpa-persistence/src/main/java/org/apache/openjpa/persistence/meta/AnnotationProcessor6.java
    
openjpa/trunk/openjpa-persistence/src/main/java/org/apache/openjpa/persistence/meta/SourceAnnotationHandler.java

Modified: 
openjpa/trunk/openjpa-persistence/src/main/java/org/apache/openjpa/persistence/meta/AnnotationProcessor6.java
URL: 
http://svn.apache.org/viewvc/openjpa/trunk/openjpa-persistence/src/main/java/org/apache/openjpa/persistence/meta/AnnotationProcessor6.java?rev=1143520&r1=1143519&r2=1143520&view=diff
==============================================================================
--- 
openjpa/trunk/openjpa-persistence/src/main/java/org/apache/openjpa/persistence/meta/AnnotationProcessor6.java
 (original)
+++ 
openjpa/trunk/openjpa-persistence/src/main/java/org/apache/openjpa/persistence/meta/AnnotationProcessor6.java
 Wed Jul  6 18:18:43 2011
@@ -48,6 +48,7 @@ import javax.tools.JavaFileObject;
 import org.apache.openjpa.lib.util.Localizer;
 import org.apache.openjpa.meta.MetaDataFactory;
 import org.apache.openjpa.persistence.PersistenceMetaDataFactory;
+import org.apache.openjpa.persistence.PersistentCollection;
 import org.apache.openjpa.persistence.util.SourceCode;
 
 
@@ -165,9 +166,10 @@ public class AnnotationProcessor6 extend
      *  
      */
     private TypeCategory toMetaModelTypeCategory(TypeMirror mirror, 
-        String name) {
-        if (mirror.getKind() == TypeKind.ARRAY)
+        String name, boolean persistentCollection) {   
+        if (mirror.getKind() == TypeKind.ARRAY && persistentCollection ) {
             return TypeCategory.LIST;
+        }
         if (CLASSNAMES_COLLECTION.contains(name))
             return TypeCategory.COLLECTION;
         if (CLASSNAMES_LIST.contains(name))
@@ -241,10 +243,13 @@ public class AnnotationProcessor6 extend
             Set<? extends Element> members = handler.getPersistentMembers(e);
             
             for (Element m : members) {
+                boolean isPersistentCollection = 
m.getAnnotation(PersistentCollection.class) != null; 
+                
                 TypeMirror decl  = handler.getDeclaredType(m);
                 String fieldName = handler.getPersistentMemberName(m);
-                String fieldType = handler.getDeclaredTypeName(decl);
-                TypeCategory typeCategory = toMetaModelTypeCategory(decl, 
fieldType);
+                String fieldType = handler.getDeclaredTypeName(decl, true, 
isPersistentCollection);  
+                TypeCategory typeCategory =
+                    toMetaModelTypeCategory(decl, fieldType, 
isPersistentCollection);
                 String metaModelType = typeCategory.getMetaModelType();
                 SourceCode.Field modelField = null;
                 switch (typeCategory) {

Modified: 
openjpa/trunk/openjpa-persistence/src/main/java/org/apache/openjpa/persistence/meta/SourceAnnotationHandler.java
URL: 
http://svn.apache.org/viewvc/openjpa/trunk/openjpa-persistence/src/main/java/org/apache/openjpa/persistence/meta/SourceAnnotationHandler.java?rev=1143520&r1=1143519&r2=1143520&view=diff
==============================================================================
--- 
openjpa/trunk/openjpa-persistence/src/main/java/org/apache/openjpa/persistence/meta/SourceAnnotationHandler.java
 (original)
+++ 
openjpa/trunk/openjpa-persistence/src/main/java/org/apache/openjpa/persistence/meta/SourceAnnotationHandler.java
 Wed Jul  6 18:18:43 2011
@@ -548,6 +548,10 @@ public class SourceAnnotationHandler 
        return getDeclaredTypeName(mirror, true);
     }
     
+    String getDeclaredTypeName(TypeMirror mirror, boolean box) {
+        return getDeclaredTypeName(mirror, box, false);
+    }
+    
      /**
      * Get the element name of the class the given mirror represents. If the
      * mirror is primitive then returns the corresponding boxed class name.
@@ -556,12 +560,17 @@ public class SourceAnnotationHandler 
      * <code>java.util.Set&lt;java.lang.String&gt;</code> this method will 
      * return <code>java.util.Set</code>.
      */
-    String getDeclaredTypeName(TypeMirror mirror, boolean box) {
+    String getDeclaredTypeName(TypeMirror mirror, boolean box, boolean 
persistentCollection) {
         if (mirror == null || mirror.getKind() == TypeKind.NULL || 
mirror.getKind() == TypeKind.WILDCARD)
             return "java.lang.Object";
        if (mirror.getKind() == TypeKind.ARRAY) {
-               TypeMirror comp = ((ArrayType)mirror).getComponentType();
-               return getDeclaredTypeName(comp, false);
+           if(persistentCollection) { 
+               TypeMirror comp = ((ArrayType)mirror).getComponentType();
+               return getDeclaredTypeName(comp, false);
+           }
+           else { 
+               return mirror.toString();
+           }
        }
        mirror = box ? box(mirror) : mirror;
        if (isPrimitive(mirror))