Validate SCA annotations in Java impls are not on static members (JCA90002)

git-svn-id: http://svn.us.apache.org/repos/asf/tuscany@825827 13f79535-47bb-0310-9956-ffa450edef68

1 files changed, 27 insertions, 0 deletions

diff --git a/java/sca/modules/implementation-java/src/main/java/org/apache/tuscany/sca/implementation/java/xml/JavaImplementationProcessor.java b/java/sca/modules/implementation-java/src/main/java/org/apache/tuscany/sca/implementation/java/xml/JavaImplementationProcessor.java
index 1653e935dc..df67814bd4 100644
--- a/java/sca/modules/implementation-java/src/main/java/org/apache/tuscany/sca/implementation/java/xml/JavaImplementationProcessor.java
+++ b/java/sca/modules/implementation-java/src/main/java/org/apache/tuscany/sca/implementation/java/xml/JavaImplementationProcessor.java
@@ -25,6 +25,7 @@ import static org.apache.tuscany.sca.implementation.java.xml.JavaImplementationC
 import static org.apache.tuscany.sca.implementation.java.xml.JavaImplementationConstants.IMPLEMENTATION_JAVA_QNAME;
 import static org.apache.tuscany.sca.implementation.java.xml.JavaImplementationConstants.SCA11_NS;
 
+import java.lang.annotation.Annotation;
 import java.lang.reflect.Field;
 import java.lang.reflect.Method;
 import java.lang.reflect.Modifier;
@@ -187,6 +188,8 @@ public class JavaImplementationProcessor implements StAXArtifactProcessor<JavaIm
 	            return;
 	        }
 
+	        checkNoStaticAnnotations(monitor, javaImplementation);
+	        
 	        javaImplementation.setUnresolved(false);
 	        mergeComponentType(resolver, javaImplementation, context);
 
@@ -199,6 +202,30 @@ public class JavaImplementationProcessor implements StAXArtifactProcessor<JavaIm
     	} // end try
     } // end method
 
+    private void checkNoStaticAnnotations(Monitor monitor, JavaImplementation javaImplementation) {
+        if (javaImplementation.getJavaClass() != null) {
+            Class<?> clazz = javaImplementation.getJavaClass();
+            for (Method m : clazz.getMethods()) {
+                if (Modifier.isStatic(m.getModifiers())) {
+                    for (Annotation a : m.getAnnotations()) {
+                        if (a.annotationType().getName().startsWith("org.oasisopen.sca.annotation")) {
+                            error(monitor, "IllegalSCAAnnotation", javaFactory, javaImplementation.getName(), m.getName());
+                        }
+                    }
+                }
+            }
+            for (Field f : clazz.getFields()) {
+                if (Modifier.isStatic(f.getModifiers())) {
+                    for (Annotation a : f.getAnnotations()) {
+                        if (a.annotationType().getName().startsWith("org.oasisopen.sca.annotation")) {
+                            error(monitor, "IllegalSCAAnnotation", javaFactory, javaImplementation.getName(), f.getName());
+                        }
+                    }
+                }
+            }
+        }
+    }
+
     private JavaElementImpl getMemeber(JavaImplementation impl, String name, Class<?> type) {
         String setter = JavaIntrospectionHelper.toSetter(name);
         try {