Revert "Fix #22156: Bug in annotation handling in case of plural annotation handlers" (#22175)

yaminikb · web-flow · commit 61d5fbfe00a5 · 2017-08-04T18:00:46.000+05:30
diff --git a/appserver/connectors/connectors-runtime/src/main/java/com/sun/enterprise/connectors/deployment/annotation/handlers/AdministeredObjectDefinitionHandler.java b/appserver/connectors/connectors-runtime/src/main/java/com/sun/enterprise/connectors/deployment/annotation/handlers/AdministeredObjectDefinitionHandler.java
@@ -49,7 +49,6 @@
 import org.glassfish.apf.HandlerProcessingResult;
 import org.glassfish.deployment.common.JavaEEResourceType;
 import org.glassfish.deployment.common.RootDeploymentDescriptor;
-import org.glassfish.internal.deployment.AnnotationTypesProvider;
 import org.jvnet.hk2.annotations.Service;
 
 import javax.interceptor.AroundInvoke;
@@ -70,10 +69,7 @@
 @AnnotationHandlerFor(AdministeredObjectDefinition.class)
 public class AdministeredObjectDefinitionHandler extends AbstractResourceHandler {
     
-    public AdministeredObjectDefinitionHandler() {}
-
-    public AdministeredObjectDefinitionHandler(AnnotationTypesProvider ejbProvider) {
-        super(ejbProvider);
+    public AdministeredObjectDefinitionHandler() {
     }
 
     @Override
diff --git a/appserver/connectors/connectors-runtime/src/main/java/com/sun/enterprise/connectors/deployment/annotation/handlers/AdministeredObjectDefinitionsHandler.java b/appserver/connectors/connectors-runtime/src/main/java/com/sun/enterprise/connectors/deployment/annotation/handlers/AdministeredObjectDefinitionsHandler.java
@@ -69,7 +69,8 @@ public class AdministeredObjectDefinitionsHandler extends AbstractResourceHandle
     protected final static LocalStringManagerImpl localStrings =
             new LocalStringManagerImpl(AdministeredObjectDefinitionsHandler.class);
 
-    public AdministeredObjectDefinitionsHandler() {}
+    public AdministeredObjectDefinitionsHandler() {
+    }
 
     @Override
     protected HandlerProcessingResult processAnnotation(AnnotationInfo ainfo,  ResourceContainerContext[] rcContexts)
@@ -91,7 +92,7 @@ protected HandlerProcessingResult processAnnotation(AnnotationInfo ainfo,  Resou
                 }else{
                     duplicates.add(defnName);
                 }
-                AdministeredObjectDefinitionHandler handler = new AdministeredObjectDefinitionHandler(ejbProvider);
+                AdministeredObjectDefinitionHandler handler = new AdministeredObjectDefinitionHandler();
                 handler.processAnnotation(defn, ainfo, rcContexts);
             }
             duplicates.clear();
diff --git a/appserver/connectors/connectors-runtime/src/main/java/com/sun/enterprise/connectors/deployment/annotation/handlers/ConnectionFactoryDefinitionHandler.java b/appserver/connectors/connectors-runtime/src/main/java/com/sun/enterprise/connectors/deployment/annotation/handlers/ConnectionFactoryDefinitionHandler.java
@@ -49,7 +49,6 @@
 import org.glassfish.apf.HandlerProcessingResult;
 import org.glassfish.deployment.common.JavaEEResourceType;
 import org.glassfish.deployment.common.RootDeploymentDescriptor;
-import org.glassfish.internal.deployment.AnnotationTypesProvider;
 import org.jvnet.hk2.annotations.Service;
 
 import javax.interceptor.AroundInvoke;
@@ -70,12 +69,11 @@
 @AnnotationHandlerFor(ConnectionFactoryDefinition.class)
 public class ConnectionFactoryDefinitionHandler extends AbstractResourceHandler {
 
-    public ConnectionFactoryDefinitionHandler() {}
-
-    public ConnectionFactoryDefinitionHandler(AnnotationTypesProvider ejbProvider) {
-        super(ejbProvider);
+    
+    public ConnectionFactoryDefinitionHandler() {
     }
 
+
     @Override
     protected HandlerProcessingResult processAnnotation(AnnotationInfo ainfo, ResourceContainerContext[] rcContexts)
             throws AnnotationProcessorException {
diff --git a/appserver/connectors/connectors-runtime/src/main/java/com/sun/enterprise/connectors/deployment/annotation/handlers/ConnectionFactoryDefinitionsHandler.java b/appserver/connectors/connectors-runtime/src/main/java/com/sun/enterprise/connectors/deployment/annotation/handlers/ConnectionFactoryDefinitionsHandler.java
@@ -68,7 +68,10 @@ public class ConnectionFactoryDefinitionsHandler extends AbstractResourceHandler
     protected final static LocalStringManagerImpl localStrings =
             new LocalStringManagerImpl(ConnectionFactoryDefinitionsHandler.class);
 
-    public ConnectionFactoryDefinitionsHandler() {}
+    
+    public ConnectionFactoryDefinitionsHandler() {
+    }
+
 
     @Override
     protected HandlerProcessingResult processAnnotation(AnnotationInfo ainfo,  ResourceContainerContext[] rcContexts)
@@ -95,7 +98,7 @@ protected HandlerProcessingResult processAnnotation(AnnotationInfo ainfo,  Resou
                 }else{
                     duplicates.add(defnName);
                 }
-                ConnectionFactoryDefinitionHandler handler = new ConnectionFactoryDefinitionHandler(ejbProvider);
+                ConnectionFactoryDefinitionHandler handler = new ConnectionFactoryDefinitionHandler();
                 handler.processAnnotation(defn, ainfo, rcContexts);
             }
             duplicates.clear();
diff --git a/appserver/deployment/dol/src/main/java/com/sun/enterprise/deployment/annotation/handlers/AbstractResourceHandler.java b/appserver/deployment/dol/src/main/java/com/sun/enterprise/deployment/annotation/handlers/AbstractResourceHandler.java
@@ -51,7 +51,6 @@
 import org.glassfish.apf.AnnotationInfo;
 import org.glassfish.apf.AnnotationProcessorException;
 import org.glassfish.apf.HandlerProcessingResult;
-import org.glassfish.internal.deployment.AnnotationTypesProvider;
 
 import java.lang.annotation.Annotation;
 import java.lang.reflect.Method;
@@ -72,14 +71,6 @@
  * @author Shing Wai Chan
  */
 public abstract class AbstractResourceHandler extends AbstractHandler {
-
-    protected AbstractResourceHandler() {
-    }
-
-    protected AbstractResourceHandler(AnnotationTypesProvider ejbProvider) {
-        this.ejbProvider = ejbProvider;
-    }
-
     /**
      * Process Annotation with given ResourceContainerContexts.
      * @param ainfo
diff --git a/appserver/jdbc/jdbc-runtime/src/main/java/org/glassfish/jdbcruntime/deployment/annotation/handlers/DataSourceDefinitionHandler.java b/appserver/jdbc/jdbc-runtime/src/main/java/org/glassfish/jdbcruntime/deployment/annotation/handlers/DataSourceDefinitionHandler.java
@@ -49,7 +49,6 @@
 import org.glassfish.apf.HandlerProcessingResult;
 import org.glassfish.deployment.common.JavaEEResourceType;
 import org.glassfish.deployment.common.RootDeploymentDescriptor;
-import org.glassfish.internal.deployment.AnnotationTypesProvider;
 import org.jvnet.hk2.annotations.Service;
 
 import javax.annotation.sql.DataSourceDefinition;
@@ -69,10 +68,7 @@
 @AnnotationHandlerFor(DataSourceDefinition.class)
 public class DataSourceDefinitionHandler extends AbstractResourceHandler {
 
-    public DataSourceDefinitionHandler() {}
-
-    public DataSourceDefinitionHandler(AnnotationTypesProvider ejbProvider) {
-        super(ejbProvider);
+    public DataSourceDefinitionHandler() {
     }
 
     protected HandlerProcessingResult processAnnotation(AnnotationInfo ainfo, ResourceContainerContext[] rcContexts)
diff --git a/appserver/jdbc/jdbc-runtime/src/main/java/org/glassfish/jdbcruntime/deployment/annotation/handlers/DataSourceDefinitionsHandler.java b/appserver/jdbc/jdbc-runtime/src/main/java/org/glassfish/jdbcruntime/deployment/annotation/handlers/DataSourceDefinitionsHandler.java
@@ -94,7 +94,7 @@ protected HandlerProcessingResult processAnnotation(AnnotationInfo ainfo, Resour
                 }else{
                     duplicates.add(defnName);
                 }
-                DataSourceDefinitionHandler handler = new DataSourceDefinitionHandler(ejbProvider);
+                DataSourceDefinitionHandler handler = new DataSourceDefinitionHandler();
                 handler.processAnnotation(defn, ainfo, rcContexts);
             }
             duplicates.clear();
diff --git a/appserver/jms/jms-core/src/main/java/com/sun/enterprise/connectors/jms/deployment/annotation/handlers/JMSConnectionFactoryDefinitionHandler.java b/appserver/jms/jms-core/src/main/java/com/sun/enterprise/connectors/jms/deployment/annotation/handlers/JMSConnectionFactoryDefinitionHandler.java
@@ -49,7 +49,6 @@
 import org.glassfish.apf.HandlerProcessingResult;
 import org.glassfish.deployment.common.JavaEEResourceType;
 import org.glassfish.deployment.common.RootDeploymentDescriptor;
-import org.glassfish.internal.deployment.AnnotationTypesProvider;
 import org.jvnet.hk2.annotations.Service;
 
 import javax.interceptor.AroundInvoke;
@@ -67,10 +66,7 @@
 @AnnotationHandlerFor(JMSConnectionFactoryDefinition.class)
 public class JMSConnectionFactoryDefinitionHandler extends AbstractResourceHandler {
 
-    public JMSConnectionFactoryDefinitionHandler() {}
-
-    public JMSConnectionFactoryDefinitionHandler(AnnotationTypesProvider ejbProvider) {
-        super(ejbProvider);
+    public JMSConnectionFactoryDefinitionHandler() {
     }
 
     protected HandlerProcessingResult processAnnotation(AnnotationInfo ainfo, ResourceContainerContext[] rcContexts)
diff --git a/appserver/jms/jms-core/src/main/java/com/sun/enterprise/connectors/jms/deployment/annotation/handlers/JMSConnectionFactoryDefinitionsHandler.java b/appserver/jms/jms-core/src/main/java/com/sun/enterprise/connectors/jms/deployment/annotation/handlers/JMSConnectionFactoryDefinitionsHandler.java
@@ -56,7 +56,6 @@
 import org.glassfish.apf.AnnotationInfo;
 import org.glassfish.apf.AnnotationProcessorException;
 import org.glassfish.apf.HandlerProcessingResult;
-import org.glassfish.internal.deployment.AnnotationTypesProvider;
 import org.jvnet.hk2.annotations.Service;
 
 @Service
@@ -66,7 +65,9 @@ public class JMSConnectionFactoryDefinitionsHandler extends AbstractResourceHand
     protected final static LocalStringManagerImpl localStrings =
             new LocalStringManagerImpl(JMSConnectionFactoryDefinitionsHandler.class);
 
-    public JMSConnectionFactoryDefinitionsHandler() {}
+
+    public JMSConnectionFactoryDefinitionsHandler() {
+    }
 
     @Override
     protected HandlerProcessingResult processAnnotation(AnnotationInfo ainfo,  ResourceContainerContext[] rcContexts)
@@ -87,7 +88,7 @@ protected HandlerProcessingResult processAnnotation(AnnotationInfo ainfo,  Resou
                 } else {
                     duplicates.add(defnName);
                 }
-                JMSConnectionFactoryDefinitionHandler handler = new JMSConnectionFactoryDefinitionHandler(ejbProvider);
+                JMSConnectionFactoryDefinitionHandler handler = new JMSConnectionFactoryDefinitionHandler();
                 handler.processAnnotation(defn, ainfo, rcContexts);
             }
             duplicates.clear();
diff --git a/appserver/jms/jms-core/src/main/java/com/sun/enterprise/connectors/jms/deployment/annotation/handlers/JMSDestinationDefinitionHandler.java b/appserver/jms/jms-core/src/main/java/com/sun/enterprise/connectors/jms/deployment/annotation/handlers/JMSDestinationDefinitionHandler.java
@@ -49,7 +49,6 @@
 import org.glassfish.apf.HandlerProcessingResult;
 import org.glassfish.deployment.common.JavaEEResourceType;
 import org.glassfish.deployment.common.RootDeploymentDescriptor;
-import org.glassfish.internal.deployment.AnnotationTypesProvider;
 import org.jvnet.hk2.annotations.Service;
 
 import javax.interceptor.AroundInvoke;
@@ -67,10 +66,7 @@
 @AnnotationHandlerFor(JMSDestinationDefinition.class)
 public class JMSDestinationDefinitionHandler extends AbstractResourceHandler {
 
-    public JMSDestinationDefinitionHandler() {}
-
-    public JMSDestinationDefinitionHandler(AnnotationTypesProvider ejbProvider) {
-        super(ejbProvider);
+    public JMSDestinationDefinitionHandler() {
     }
 
     protected HandlerProcessingResult processAnnotation(AnnotationInfo ainfo, ResourceContainerContext[] rcContexts)
diff --git a/appserver/jms/jms-core/src/main/java/com/sun/enterprise/connectors/jms/deployment/annotation/handlers/JMSDestinationDefinitionsHandler.java b/appserver/jms/jms-core/src/main/java/com/sun/enterprise/connectors/jms/deployment/annotation/handlers/JMSDestinationDefinitionsHandler.java
@@ -65,7 +65,9 @@ public class JMSDestinationDefinitionsHandler extends AbstractResourceHandler {
     protected final static LocalStringManagerImpl localStrings =
             new LocalStringManagerImpl(JMSDestinationDefinitionsHandler.class);
 
-    public JMSDestinationDefinitionsHandler() {}
+
+    public JMSDestinationDefinitionsHandler() {
+    }
 
     @Override
     protected HandlerProcessingResult processAnnotation(AnnotationInfo ainfo,  ResourceContainerContext[] rcContexts)
@@ -86,7 +88,7 @@ protected HandlerProcessingResult processAnnotation(AnnotationInfo ainfo,  Resou
                 } else {
                     duplicates.add(defnName);
                 }
-                JMSDestinationDefinitionHandler handler = new JMSDestinationDefinitionHandler(ejbProvider);
+                JMSDestinationDefinitionHandler handler = new JMSDestinationDefinitionHandler();
                 handler.processAnnotation(defn, ainfo, rcContexts);
             }
             duplicates.clear();
diff --git a/appserver/resources/javamail/javamail-runtime/src/main/java/org/glassfish/resources/javamail/annotation/handler/MailSessionDefinitionHandler.java b/appserver/resources/javamail/javamail-runtime/src/main/java/org/glassfish/resources/javamail/annotation/handler/MailSessionDefinitionHandler.java
@@ -50,7 +50,6 @@
 import org.glassfish.apf.HandlerProcessingResult;
 import org.glassfish.deployment.common.JavaEEResourceType;
 import org.glassfish.deployment.common.RootDeploymentDescriptor;
-import org.glassfish.internal.deployment.AnnotationTypesProvider;
 import org.jvnet.hk2.annotations.Service;
 
 import javax.interceptor.AroundInvoke;
@@ -79,10 +78,8 @@ public class MailSessionDefinitionHandler extends AbstractResourceHandler {
     protected final static LocalStringManagerImpl localStrings =
             new LocalStringManagerImpl(MailSessionDefinitionHandler.class);
 
-    public MailSessionDefinitionHandler() {}
+    public MailSessionDefinitionHandler() {
 
-    public MailSessionDefinitionHandler(AnnotationTypesProvider ejbProvider) {
-        super(ejbProvider);
     }
 
     @Override
diff --git a/appserver/resources/javamail/javamail-runtime/src/main/java/org/glassfish/resources/javamail/annotation/handler/MailSessionDefinitionsHandler.java b/appserver/resources/javamail/javamail-runtime/src/main/java/org/glassfish/resources/javamail/annotation/handler/MailSessionDefinitionsHandler.java
@@ -71,7 +71,9 @@ public class MailSessionDefinitionsHandler extends AbstractResourceHandler {
     protected final static LocalStringManagerImpl localStrings =
             new LocalStringManagerImpl(MailSessionDefinitionsHandler.class);
 
-    public MailSessionDefinitionsHandler() {}
+    public MailSessionDefinitionsHandler() {
+
+    }
 
     @Override
     protected HandlerProcessingResult processAnnotation(AnnotationInfo ainfo, ResourceContainerContext[] rcContexts) throws AnnotationProcessorException {
@@ -92,7 +94,7 @@ protected HandlerProcessingResult processAnnotation(AnnotationInfo ainfo, Resour
                 } else {
                     duplicates.add(defnName);
                 }
-                MailSessionDefinitionHandler handler = new MailSessionDefinitionHandler(ejbProvider);
+                MailSessionDefinitionHandler handler = new MailSessionDefinitionHandler();
                 handler.processAnnotation(defn, ainfo, rcContexts);
             }
             duplicates.clear();

Original file line number	Diff line number	Diff line change
`@@ -94,7 +94,7 @@ protected HandlerProcessingResult processAnnotation(AnnotationInfo ainfo, Resour`
`94`	`94`	`}else{`
`95`	`95`	`duplicates.add(defnName);`
`96`	`96`	`}`
`97`		`- DataSourceDefinitionHandler handler = new DataSourceDefinitionHandler(ejbProvider);`
	`97`	`+ DataSourceDefinitionHandler handler = new DataSourceDefinitionHandler();`
`98`	`98`	`handler.processAnnotation(defn, ainfo, rcContexts);`
`99`	`99`	`}`
`100`	`100`	`duplicates.clear();`