[jboss-cvs] JBossAS SVN: r108293 - projects/metadata/common/trunk/src/main/java/org/jboss/metadata/annotation/creator/ws.

[jboss-cvs-commits at lists.jboss.org]

Author: richard.opalka at jboss.com
Date: 2010-09-23 09:48:21 -0400 (Thu, 23 Sep 2010)
New Revision: 108293

Modified:
   projects/metadata/common/trunk/src/main/java/org/jboss/metadata/annotation/creator/ws/WebServiceRefClassProcessor.java
   projects/metadata/common/trunk/src/main/java/org/jboss/metadata/annotation/creator/ws/WebServiceRefsClassProcessor.java
Log:
[JBMETA-298] @WebServiceRef type attribute have to be specified if this annotation is used on class type

Modified: projects/metadata/common/trunk/src/main/java/org/jboss/metadata/annotation/creator/ws/WebServiceRefClassProcessor.java
===================================================================
--- projects/metadata/common/trunk/src/main/java/org/jboss/metadata/annotation/creator/ws/WebServiceRefClassProcessor.java	2010-09-23 12:26:52 UTC (rev 108292)
+++ projects/metadata/common/trunk/src/main/java/org/jboss/metadata/annotation/creator/ws/WebServiceRefClassProcessor.java	2010-09-23 13:48:21 UTC (rev 108293)
@@ -29,6 +29,7 @@
 
 /**
  * @author <a href="mailto:emuckenh at redhat.com">Emanuel Muckenhuber</a>
+ * @author <a href="mailto:ropalka at redhat.com">Richard Opalka</a>
  * @version $Revision$
  */
 public class WebServiceRefClassProcessor
@@ -56,7 +57,7 @@
    @Override
    protected String getType(Class<?> element)
    {
-      return element.getName();
+      throw new IllegalStateException("@WebServiceRef annotation on type '"+ element.getName() +"' must define a type.");
    }
 
    @Override

Modified: projects/metadata/common/trunk/src/main/java/org/jboss/metadata/annotation/creator/ws/WebServiceRefsClassProcessor.java
===================================================================
--- projects/metadata/common/trunk/src/main/java/org/jboss/metadata/annotation/creator/ws/WebServiceRefsClassProcessor.java	2010-09-23 12:26:52 UTC (rev 108292)
+++ projects/metadata/common/trunk/src/main/java/org/jboss/metadata/annotation/creator/ws/WebServiceRefsClassProcessor.java	2010-09-23 13:48:21 UTC (rev 108293)
@@ -35,11 +35,12 @@
 
 /**
  * @WebServiceRefs processor
- * 
+ *
  * @author <a href="mailto:emuckenh at redhat.com">Emanuel Muckenhuber</a>
+ * @author <a href="mailto:ropalka at redhat.com">Richard Opalka</a>
  * @version $Revision$
  */
-public class WebServiceRefsClassProcessor 
+public class WebServiceRefsClassProcessor
    extends WebServiceRefClassProcessor
    implements Processor<ServiceReferencesMetaData, Class<?>>
 {
@@ -48,14 +49,14 @@
    {
       super(finder);
    }
-   
+
    @Override
    public void process(ServiceReferencesMetaData refsMetaData, Class<?> type)
    {
       WebServiceRefs annotation = finder.getAnnotation(type, WebServiceRefs.class);
       if(annotation == null)
          return;
-      
+
       WebServiceRef[] references = annotation.value();
       if(references != null)
       {
@@ -65,27 +66,9 @@
    }
 
    @Override
-   protected String getName(Class<?> element)
-   {
-      String name = element.getSimpleName();
-      return name;
-   }
-
-   @Override
-   protected String getType(Class<?> element)
-   {
-      return element.getName();
-   }
-
-   @Override
-   protected String getDeclaringClass(Class<?> element)
-   {
-      return element.getName();
-   }
-
-   @Override
    public Collection<Class<? extends Annotation>> getAnnotationTypes()
    {
       return ProcessorUtils.createAnnotationSet(WebServiceRefs.class);
    }
+
 }