Author: arne
Date: Fri Jan 4 18:05:03 2013
New Revision: 1429007
URL: http://svn.apache.org/viewvc?rev=1429007&view=rev
Log:
OWB-745: Removed DefinitionUtil.defineInjectedMethods
Modified:
openwebbeans/trunk/webbeans-impl/src/main/java/org/apache/webbeans/component/creation/AbstractInjectedTargetBeanCreator.java
openwebbeans/trunk/webbeans-impl/src/main/java/org/apache/webbeans/config/DefinitionUtil.java
openwebbeans/trunk/webbeans-impl/src/main/java/org/apache/webbeans/config/ManagedBeanConfigurator.java
openwebbeans/trunk/webbeans-impl/src/main/java/org/apache/webbeans/util/WebBeansAnnotatedTypeUtil.java
openwebbeans/trunk/webbeans-impl/src/main/java/org/apache/webbeans/util/WebBeansUtil.java
Modified:
openwebbeans/trunk/webbeans-impl/src/main/java/org/apache/webbeans/component/creation/AbstractInjectedTargetBeanCreator.java
URL:
http://svn.apache.org/viewvc/openwebbeans/trunk/webbeans-impl/src/main/java/org/apache/webbeans/component/creation/AbstractInjectedTargetBeanCreator.java?rev=1429007&r1=1429006&r2=1429007&view=diff
==============================================================================
---
openwebbeans/trunk/webbeans-impl/src/main/java/org/apache/webbeans/component/creation/AbstractInjectedTargetBeanCreator.java
(original)
+++
openwebbeans/trunk/webbeans-impl/src/main/java/org/apache/webbeans/component/creation/AbstractInjectedTargetBeanCreator.java
Fri Jan 4 18:05:03 2013
@@ -72,16 +72,7 @@ public abstract class AbstractInjectedTa
*/
public void defineInjectedMethods()
{
- AbstractInjectionTargetBean bean = getBean();
- if(isDefaultMetaDataProvider())
- {
-
bean.getWebBeansContext().getDefinitionUtil().defineInjectedMethods(bean);
- }
- else
- {
-
bean.getWebBeansContext().getAnnotatedTypeUtil().defineInjectedMethods(bean,
getAnnotatedType());
- }
-
+
webBeansContext.getAnnotatedTypeUtil().defineInjectedMethods(getBean(),
getAnnotatedType());
}
/**
Modified:
openwebbeans/trunk/webbeans-impl/src/main/java/org/apache/webbeans/config/DefinitionUtil.java
URL:
http://svn.apache.org/viewvc/openwebbeans/trunk/webbeans-impl/src/main/java/org/apache/webbeans/config/DefinitionUtil.java?rev=1429007&r1=1429006&r2=1429007&view=diff
==============================================================================
---
openwebbeans/trunk/webbeans-impl/src/main/java/org/apache/webbeans/config/DefinitionUtil.java
(original)
+++
openwebbeans/trunk/webbeans-impl/src/main/java/org/apache/webbeans/config/DefinitionUtil.java
Fri Jan 4 18:05:03 2013
@@ -26,7 +26,6 @@ import java.lang.reflect.Method;
import java.lang.reflect.Modifier;
import java.lang.reflect.ParameterizedType;
import java.lang.reflect.Type;
-import java.lang.reflect.TypeVariable;
import java.util.ArrayList;
import java.util.HashSet;
import java.util.Iterator;
@@ -829,122 +828,6 @@ public final class DefinitionUtil
return component;
}
- public <T> void defineInjectedMethods(AbstractInjectionTargetBean<T> bean)
- {
- Asserts.assertNotNull(bean, "bean parameter can not be null");
-
- Class<T> clazz = bean.getReturnType();
-
- // From bean class definition
- defineInternalInjectedMethods(bean, clazz, false);
-
- // From inheritance hierarchy
- defineInternalInjectedMethodsRecursively(bean, clazz);
- }
-
- public <T> void
defineInternalInjectedMethodsRecursively(AbstractInjectionTargetBean<T>
component, Class<T> clazz)
- {
- // From inheritance
- Class<?> superClazz = clazz.getSuperclass();
- if (!superClazz.equals(Object.class))
- {
- // From super class
- defineInternalInjectedMethods(component, (Class<T>) superClazz,
true);
-
- // From super class type hierarchy
- defineInternalInjectedMethodsRecursively(component, (Class<T>)
superClazz);
- }
-
- }
-
- private <T> void
defineInternalInjectedMethods(AbstractInjectionTargetBean<T> component,
Class<T> clazz, boolean fromInherited)
- {
-
- Method[] methods =
webBeansContext.getSecurityService().doPrivilegedGetDeclaredMethods(clazz);
-
- for (Method method : methods)
- {
- boolean isInitializer = AnnotationUtil.hasMethodAnnotation(method,
Inject.class);
-
- if (isInitializer)
- {
- //Do not support static
- if(Modifier.isStatic(method.getModifiers()))
- {
- continue;
- }
-
- checkForInjectedInitializerMethod(clazz, method,
webBeansContext);
- }
- else
- {
- continue;
- }
-
- if (!Modifier.isStatic(method.getModifiers()))
- {
- if (!fromInherited)
- {
- component.addInjectedMethod(method);
- addMethodInjectionPointMetaData(component, method);
- }
- else
- {
- Method[] beanMethods =
webBeansContext.getSecurityService().doPrivilegedGetDeclaredMethods(component.getReturnType());
- boolean defined = false;
- for (Method beanMethod : beanMethods)
- {
- if(ClassUtil.isOverridden(beanMethod, method))
- {
- defined = true;
- break;
- }
- }
-
- if(!defined)
- {
- component.addInjectedMethodToSuper(method);
- addMethodInjectionPointMetaData(component, method);
- }
- }
- }
- }
-
- }
-
- /**
- * add the definitions for a @Initializer method.
- */
- private static <T> void checkForInjectedInitializerMethod(Class<T> clazz,
Method method,
- WebBeansContext
webBeansContext)
- {
- TypeVariable<?>[] args = method.getTypeParameters();
- if(args.length > 0)
- {
- throw new WebBeansConfigurationException("Initializer methods must
not be generic but method : " + method.getName() + " in bean class : "
- + clazz + " is defined as
generic");
- }
-
- Annotation[][] anns = method.getParameterAnnotations();
- Type[] type = method.getGenericParameterTypes();
- for (int i = 0; i < anns.length; i++)
- {
- Annotation[] a = anns[i];
- Type t = type[i];
-
webBeansContext.getAnnotationManager().checkForNewQualifierForDeployment(t,
clazz, method.getName(), a);
- }
-
- if (method.getAnnotation(Produces.class) == null)
- {
- WebBeansUtil.checkInjectedMethodParameterConditions(method, clazz);
- }
- else
- {
- throw new WebBeansConfigurationException("Initializer method : " +
method.getName() + " in class : " + clazz.getName()
- + " can not be annotated
with @Produces");
- }
- }
-
/**
* Configure bean instance interceptor stack.
* @param bean bean instance
Modified:
openwebbeans/trunk/webbeans-impl/src/main/java/org/apache/webbeans/config/ManagedBeanConfigurator.java
URL:
http://svn.apache.org/viewvc/openwebbeans/trunk/webbeans-impl/src/main/java/org/apache/webbeans/config/ManagedBeanConfigurator.java?rev=1429007&r1=1429006&r2=1429007&view=diff
==============================================================================
---
openwebbeans/trunk/webbeans-impl/src/main/java/org/apache/webbeans/config/ManagedBeanConfigurator.java
(original)
+++
openwebbeans/trunk/webbeans-impl/src/main/java/org/apache/webbeans/config/ManagedBeanConfigurator.java
Fri Jan 4 18:05:03 2013
@@ -178,7 +178,7 @@ public final class ManagedBeanConfigurat
webBeansContext.getAnnotatedTypeUtil().defineDisposalMethods(component,
component.getAnnotatedType());
webBeansContext.getAnnotatedTypeUtil().defineInjectedFields(component,
component.getAnnotatedType());
- definitionUtil.defineInjectedMethods(component);
+
webBeansContext.getAnnotatedTypeUtil().defineInjectedMethods(component,
component.getAnnotatedType());
definitionUtil.defineObserverMethods(component, clazz);
return component;
Modified:
openwebbeans/trunk/webbeans-impl/src/main/java/org/apache/webbeans/util/WebBeansAnnotatedTypeUtil.java
URL:
http://svn.apache.org/viewvc/openwebbeans/trunk/webbeans-impl/src/main/java/org/apache/webbeans/util/WebBeansAnnotatedTypeUtil.java?rev=1429007&r1=1429006&r2=1429007&view=diff
==============================================================================
---
openwebbeans/trunk/webbeans-impl/src/main/java/org/apache/webbeans/util/WebBeansAnnotatedTypeUtil.java
(original)
+++
openwebbeans/trunk/webbeans-impl/src/main/java/org/apache/webbeans/util/WebBeansAnnotatedTypeUtil.java
Fri Jan 4 18:05:03 2013
@@ -335,8 +335,6 @@ public final class WebBeansAnnotatedType
addMethodInjectionPointMetaData(bean, annotatedMethod);
}
}
-
-
webBeansContext.getDefinitionUtil().defineInternalInjectedMethodsRecursively(bean,
annotatedType.getJavaClass());
}
public <X> void defineInjectedFields(AbstractInjectionTargetBean<X>
bean,AnnotatedType<X> annotatedType)
Modified:
openwebbeans/trunk/webbeans-impl/src/main/java/org/apache/webbeans/util/WebBeansUtil.java
URL:
http://svn.apache.org/viewvc/openwebbeans/trunk/webbeans-impl/src/main/java/org/apache/webbeans/util/WebBeansUtil.java?rev=1429007&r1=1429006&r2=1429007&view=diff
==============================================================================
---
openwebbeans/trunk/webbeans-impl/src/main/java/org/apache/webbeans/util/WebBeansUtil.java
(original)
+++
openwebbeans/trunk/webbeans-impl/src/main/java/org/apache/webbeans/util/WebBeansUtil.java
Fri Jan 4 18:05:03 2013
@@ -586,7 +586,7 @@ public final class WebBeansUtil
definitionUtil.addConstructorInjectionPointMetaData(comp,
comp.getConstructor());
webBeansContext.getAnnotatedTypeUtil().defineInjectedFields(comp,
comp.getAnnotatedType());
- definitionUtil.defineInjectedMethods(comp);
+ webBeansContext.getAnnotatedTypeUtil().defineInjectedMethods(comp,
comp.getAnnotatedType());
}
else if (EJBWebBeansConfigurator.isSessionBean(clazz, webBeansContext))
{