diff --git a/spring-beans/src/main/java/org/springframework/beans/BeanUtils.java b/spring-beans/src/main/java/org/springframework/beans/BeanUtils.java index e0e4b09ef94..a8770c46da3 100644 --- a/spring-beans/src/main/java/org/springframework/beans/BeanUtils.java +++ b/spring-beans/src/main/java/org/springframework/beans/BeanUtils.java @@ -378,13 +378,28 @@ public abstract class BeanUtils { * Find a JavaBeans {@code PropertyDescriptor} for the given method, * with the method either being the read method or the write method for * that bean property. - * @param method the method to find a corresponding PropertyDescriptor for + * @param method the method to find a corresponding PropertyDescriptor for, + * introspecting its declaring class * @return the corresponding PropertyDescriptor, or {@code null} if none * @throws BeansException if PropertyDescriptor lookup fails */ public static PropertyDescriptor findPropertyForMethod(Method method) throws BeansException { + return findPropertyForMethod(method, method.getDeclaringClass()); + } + + /** + * Find a JavaBeans {@code PropertyDescriptor} for the given method, + * with the method either being the read method or the write method for + * that bean property. + * @param method the method to find a corresponding PropertyDescriptor for + * @param clazz the (most specific) class to introspect for descriptors + * @return the corresponding PropertyDescriptor, or {@code null} if none + * @throws BeansException if PropertyDescriptor lookup fails + * @since 4.0.9 + */ + public static PropertyDescriptor findPropertyForMethod(Method method, Class clazz) throws BeansException { Assert.notNull(method, "Method must not be null"); - PropertyDescriptor[] pds = getPropertyDescriptors(method.getDeclaringClass()); + PropertyDescriptor[] pds = getPropertyDescriptors(clazz); for (PropertyDescriptor pd : pds) { if (method.equals(pd.getReadMethod()) || method.equals(pd.getWriteMethod())) { return pd; @@ -590,11 +605,11 @@ public abstract class BeanUtils { actualEditable = editable; } PropertyDescriptor[] targetPds = getPropertyDescriptors(actualEditable); - List ignoreList = (ignoreProperties != null) ? Arrays.asList(ignoreProperties) : null; + List ignoreList = (ignoreProperties != null ? Arrays.asList(ignoreProperties) : null); for (PropertyDescriptor targetPd : targetPds) { Method writeMethod = targetPd.getWriteMethod(); - if (writeMethod != null && (ignoreProperties == null || (!ignoreList.contains(targetPd.getName())))) { + if (writeMethod != null && (ignoreList == null || (!ignoreList.contains(targetPd.getName())))) { PropertyDescriptor sourcePd = getPropertyDescriptor(source.getClass(), targetPd.getName()); if (sourcePd != null) { Method readMethod = sourcePd.getReadMethod(); diff --git a/spring-beans/src/main/java/org/springframework/beans/factory/annotation/AutowiredAnnotationBeanPostProcessor.java b/spring-beans/src/main/java/org/springframework/beans/factory/annotation/AutowiredAnnotationBeanPostProcessor.java index bd0e9e37fa9..4f8013597cf 100644 --- a/spring-beans/src/main/java/org/springframework/beans/factory/annotation/AutowiredAnnotationBeanPostProcessor.java +++ b/spring-beans/src/main/java/org/springframework/beans/factory/annotation/AutowiredAnnotationBeanPostProcessor.java @@ -397,14 +397,11 @@ public class AutowiredAnnotationBeanPostProcessor extends InstantiationAwareBean } } for (Method method : targetClass.getDeclaredMethods()) { - AnnotationAttributes ann = null; Method bridgedMethod = BridgeMethodResolver.findBridgedMethod(method); - if (BridgeMethodResolver.isVisibilityBridgeMethodPair(method, bridgedMethod)) { - ann = findAutowiredAnnotation(bridgedMethod); - } - else if (!method.isBridge()) { - ann = findAutowiredAnnotation(method); + if (!BridgeMethodResolver.isVisibilityBridgeMethodPair(method, bridgedMethod)) { + continue; } + AnnotationAttributes ann = findAutowiredAnnotation(bridgedMethod); if (ann != null && method.equals(ClassUtils.getMostSpecificMethod(method, clazz))) { if (Modifier.isStatic(method.getModifiers())) { if (logger.isWarnEnabled()) { @@ -418,7 +415,7 @@ public class AutowiredAnnotationBeanPostProcessor extends InstantiationAwareBean } } boolean required = determineRequiredStatus(ann); - PropertyDescriptor pd = BeanUtils.findPropertyForMethod(method); + PropertyDescriptor pd = BeanUtils.findPropertyForMethod(bridgedMethod, clazz); currElements.add(new AutowiredMethodElement(method, required, pd)); } } diff --git a/spring-beans/src/test/java/org/springframework/beans/factory/annotation/AutowiredAnnotationBeanPostProcessorTests.java b/spring-beans/src/test/java/org/springframework/beans/factory/annotation/AutowiredAnnotationBeanPostProcessorTests.java index 4ca7eae9cca..8c0ff7db989 100644 --- a/spring-beans/src/test/java/org/springframework/beans/factory/annotation/AutowiredAnnotationBeanPostProcessorTests.java +++ b/spring-beans/src/test/java/org/springframework/beans/factory/annotation/AutowiredAnnotationBeanPostProcessorTests.java @@ -1795,7 +1795,7 @@ public class AutowiredAnnotationBeanPostProcessorTests { } - public static class ExtendedResourceInjectionBean extends ResourceInjectionBean { + static class NonPublicResourceInjectionBean extends ResourceInjectionBean { @Autowired public final ITestBean testBean3 = null; @@ -1808,7 +1808,7 @@ public class AutowiredAnnotationBeanPostProcessorTests { public boolean baseInjected = false; - public ExtendedResourceInjectionBean() { + public NonPublicResourceInjectionBean() { } @Override @@ -1851,12 +1851,11 @@ public class AutowiredAnnotationBeanPostProcessorTests { } - public static class TypedExtendedResourceInjectionBean extends ExtendedResourceInjectionBean { - + public static class TypedExtendedResourceInjectionBean extends NonPublicResourceInjectionBean { } - public static class OverriddenExtendedResourceInjectionBean extends ExtendedResourceInjectionBean { + public static class OverriddenExtendedResourceInjectionBean extends NonPublicResourceInjectionBean { public boolean subInjected = false; @@ -2251,7 +2250,6 @@ public class AutowiredAnnotationBeanPostProcessorTests { private TestBean testBean3; - @MyAutowired(optional = true) protected void setTestBean3(TestBean testBean3) { this.testBean3 = testBean3; diff --git a/spring-context/src/main/java/org/springframework/context/annotation/CommonAnnotationBeanPostProcessor.java b/spring-context/src/main/java/org/springframework/context/annotation/CommonAnnotationBeanPostProcessor.java index 2b911799796..7c971853294 100644 --- a/spring-context/src/main/java/org/springframework/context/annotation/CommonAnnotationBeanPostProcessor.java +++ b/spring-context/src/main/java/org/springframework/context/annotation/CommonAnnotationBeanPostProcessor.java @@ -58,6 +58,7 @@ import org.springframework.beans.factory.config.ConfigurableBeanFactory; import org.springframework.beans.factory.config.DependencyDescriptor; import org.springframework.beans.factory.config.InstantiationAwareBeanPostProcessor; import org.springframework.beans.factory.support.RootBeanDefinition; +import org.springframework.core.BridgeMethodResolver; import org.springframework.core.MethodParameter; import org.springframework.core.Ordered; import org.springframework.jndi.support.SimpleJndiBeanFactory; @@ -344,46 +345,50 @@ public class CommonAnnotationBeanPostProcessor extends InitDestroyAnnotationBean if (Modifier.isStatic(field.getModifiers())) { throw new IllegalStateException("@WebServiceRef annotation is not supported on static fields"); } - currElements.add(new WebServiceRefElement(field, null)); + currElements.add(new WebServiceRefElement(field, field, null)); } else if (ejbRefClass != null && field.isAnnotationPresent(ejbRefClass)) { if (Modifier.isStatic(field.getModifiers())) { throw new IllegalStateException("@EJB annotation is not supported on static fields"); } - currElements.add(new EjbRefElement(field, null)); + currElements.add(new EjbRefElement(field, field, null)); } else if (field.isAnnotationPresent(Resource.class)) { if (Modifier.isStatic(field.getModifiers())) { throw new IllegalStateException("@Resource annotation is not supported on static fields"); } if (!ignoredResourceTypes.contains(field.getType().getName())) { - currElements.add(new ResourceElement(field, null)); + currElements.add(new ResourceElement(field, field, null)); } } } for (Method method : targetClass.getDeclaredMethods()) { - if (!method.isBridge() && method.equals(ClassUtils.getMostSpecificMethod(method, clazz))) { - if (webServiceRefClass != null && method.isAnnotationPresent(webServiceRefClass)) { + Method bridgedMethod = BridgeMethodResolver.findBridgedMethod(method); + if (!BridgeMethodResolver.isVisibilityBridgeMethodPair(method, bridgedMethod)) { + continue; + } + if (method.equals(ClassUtils.getMostSpecificMethod(method, clazz))) { + if (webServiceRefClass != null && bridgedMethod.isAnnotationPresent(webServiceRefClass)) { if (Modifier.isStatic(method.getModifiers())) { throw new IllegalStateException("@WebServiceRef annotation is not supported on static methods"); } if (method.getParameterTypes().length != 1) { throw new IllegalStateException("@WebServiceRef annotation requires a single-arg method: " + method); } - PropertyDescriptor pd = BeanUtils.findPropertyForMethod(method); - currElements.add(new WebServiceRefElement(method, pd)); + PropertyDescriptor pd = BeanUtils.findPropertyForMethod(bridgedMethod, clazz); + currElements.add(new WebServiceRefElement(method, bridgedMethod, pd)); } - else if (ejbRefClass != null && method.isAnnotationPresent(ejbRefClass)) { + else if (ejbRefClass != null && bridgedMethod.isAnnotationPresent(ejbRefClass)) { if (Modifier.isStatic(method.getModifiers())) { throw new IllegalStateException("@EJB annotation is not supported on static methods"); } if (method.getParameterTypes().length != 1) { throw new IllegalStateException("@EJB annotation requires a single-arg method: " + method); } - PropertyDescriptor pd = BeanUtils.findPropertyForMethod(method); - currElements.add(new EjbRefElement(method, pd)); + PropertyDescriptor pd = BeanUtils.findPropertyForMethod(bridgedMethod, clazz); + currElements.add(new EjbRefElement(method, bridgedMethod, pd)); } - else if (method.isAnnotationPresent(Resource.class)) { + else if (bridgedMethod.isAnnotationPresent(Resource.class)) { if (Modifier.isStatic(method.getModifiers())) { throw new IllegalStateException("@Resource annotation is not supported on static methods"); } @@ -392,8 +397,8 @@ public class CommonAnnotationBeanPostProcessor extends InitDestroyAnnotationBean throw new IllegalStateException("@Resource annotation requires a single-arg method: " + method); } if (!ignoredResourceTypes.contains(paramTypes[0].getName())) { - PropertyDescriptor pd = BeanUtils.findPropertyForMethod(method); - currElements.add(new ResourceElement(method, pd)); + PropertyDescriptor pd = BeanUtils.findPropertyForMethod(bridgedMethod, clazz); + currElements.add(new ResourceElement(method, bridgedMethod, pd)); } } } @@ -519,9 +524,8 @@ public class CommonAnnotationBeanPostProcessor extends InitDestroyAnnotationBean */ private class ResourceElement extends LookupElement { - public ResourceElement(Member member, PropertyDescriptor pd) { + public ResourceElement(Member member, AnnotatedElement ae, PropertyDescriptor pd) { super(member, pd); - AnnotatedElement ae = (AnnotatedElement) member; Resource resource = ae.getAnnotation(Resource.class); String resourceName = resource.name(); Class resourceType = resource.type(); @@ -564,9 +568,8 @@ public class CommonAnnotationBeanPostProcessor extends InitDestroyAnnotationBean private final String wsdlLocation; - public WebServiceRefElement(Member member, PropertyDescriptor pd) { + public WebServiceRefElement(Member member, AnnotatedElement ae, PropertyDescriptor pd) { super(member, pd); - AnnotatedElement ae = (AnnotatedElement) member; WebServiceRef resource = ae.getAnnotation(WebServiceRef.class); String resourceName = resource.name(); Class resourceType = resource.type(); @@ -612,7 +615,7 @@ public class CommonAnnotationBeanPostProcessor extends InitDestroyAnnotationBean } if (StringUtils.hasLength(this.wsdlLocation)) { try { - Constructor ctor = this.lookupType.getConstructor(new Class[] {URL.class, QName.class}); + Constructor ctor = this.lookupType.getConstructor(URL.class, QName.class); WebServiceClient clientAnn = this.lookupType.getAnnotation(WebServiceClient.class); if (clientAnn == null) { throw new IllegalStateException("JAX-WS Service class [" + this.lookupType.getName() + @@ -648,9 +651,8 @@ public class CommonAnnotationBeanPostProcessor extends InitDestroyAnnotationBean private final String beanName; - public EjbRefElement(Member member, PropertyDescriptor pd) { + public EjbRefElement(Member member, AnnotatedElement ae, PropertyDescriptor pd) { super(member, pd); - AnnotatedElement ae = (AnnotatedElement) member; EJB resource = ae.getAnnotation(EJB.class); String resourceBeanName = resource.beanName(); String resourceName = resource.name(); diff --git a/spring-context/src/test/java/org/springframework/beans/factory/annotation/BridgeMethodAutowiringTests.java b/spring-context/src/test/java/org/springframework/beans/factory/annotation/BridgeMethodAutowiringTests.java index 2358ae163a2..381c76d9706 100644 --- a/spring-context/src/test/java/org/springframework/beans/factory/annotation/BridgeMethodAutowiringTests.java +++ b/spring-context/src/test/java/org/springframework/beans/factory/annotation/BridgeMethodAutowiringTests.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2012 the original author or authors. + * Copyright 2002-2014 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -29,35 +29,38 @@ import static org.junit.Assert.*; public class BridgeMethodAutowiringTests { @Test - public void SPR_8434() { + public void SPR8434() { AnnotationConfigApplicationContext ctx = new AnnotationConfigApplicationContext(); ctx.register(UserServiceImpl.class, Foo.class); ctx.refresh(); assertNotNull(ctx.getBean(UserServiceImpl.class).object); } -} + static abstract class GenericServiceImpl { -abstract class GenericServiceImpl { + public abstract void setObject(D object); + } - public abstract void setObject(D object); -} + public static class UserServiceImpl extends GenericServiceImpl { + protected Foo object; -class UserServiceImpl extends GenericServiceImpl { + @Override + @Inject + @Named("userObject") + public void setObject(Foo object) { + if (this.object != null) { + throw new IllegalStateException("Already called"); + } + this.object = object; + } + } - protected Foo object; - @Override - @Inject - @Named("userObject") - public void setObject(Foo object) { - this.object = object; + @Component("userObject") + public static class Foo { } -} - -@Component("userObject") -class Foo { } +} diff --git a/spring-context/src/test/java/org/springframework/context/annotation/CommonAnnotationBeanPostProcessorTests.java b/spring-context/src/test/java/org/springframework/context/annotation/CommonAnnotationBeanPostProcessorTests.java index 04a79a11594..c44662b418a 100644 --- a/spring-context/src/test/java/org/springframework/context/annotation/CommonAnnotationBeanPostProcessorTests.java +++ b/spring-context/src/test/java/org/springframework/context/annotation/CommonAnnotationBeanPostProcessorTests.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2013 the original author or authors. + * Copyright 2002-2014 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -566,20 +566,20 @@ public class CommonAnnotationBeanPostProcessorTests { } - public static class ExtendedResourceInjectionBean extends ResourceInjectionBean { + static class NonPublicResourceInjectionBean extends ResourceInjectionBean { @Resource(name="testBean4", type=TestBean.class) protected ITestBean testBean3; - private ITestBean testBean4; + private B testBean4; @Resource - private INestedTestBean testBean5; + INestedTestBean testBean5; - private INestedTestBean testBean6; + INestedTestBean testBean6; @Resource - private BeanFactory beanFactory; + BeanFactory beanFactory; @Override @Resource @@ -588,12 +588,18 @@ public class CommonAnnotationBeanPostProcessorTests { } @Resource(name="${tb}", type=ITestBean.class) - private void setTestBean4(ITestBean testBean4) { + private void setTestBean4(B testBean4) { + if (this.testBean4 != null) { + throw new IllegalStateException("Already called"); + } this.testBean4 = testBean4; } @Resource public void setTestBean6(INestedTestBean testBean6) { + if (this.testBean6 != null) { + throw new IllegalStateException("Already called"); + } this.testBean6 = testBean6; } @@ -601,7 +607,7 @@ public class CommonAnnotationBeanPostProcessorTests { return testBean3; } - public ITestBean getTestBean4() { + public B getTestBean4() { return testBean4; } @@ -630,6 +636,10 @@ public class CommonAnnotationBeanPostProcessorTests { } + public static class ExtendedResourceInjectionBean extends NonPublicResourceInjectionBean { + } + + public static class ExtendedEjbInjectionBean extends ResourceInjectionBean { @EJB(name="testBean4", beanInterface=TestBean.class) @@ -653,11 +663,17 @@ public class CommonAnnotationBeanPostProcessorTests { @EJB(beanName="testBean3", beanInterface=ITestBean.class) private void setTestBean4(ITestBean testBean4) { + if (this.testBean4 != null) { + throw new IllegalStateException("Already called"); + } this.testBean4 = testBean4; } @EJB public void setTestBean6(INestedTestBean testBean6) { + if (this.testBean6 != null) { + throw new IllegalStateException("Already called"); + } this.testBean6 = testBean6; } diff --git a/spring-orm/src/main/java/org/springframework/orm/jpa/support/PersistenceAnnotationBeanPostProcessor.java b/spring-orm/src/main/java/org/springframework/orm/jpa/support/PersistenceAnnotationBeanPostProcessor.java index a35d7cfdd60..eff30fe3a5d 100644 --- a/spring-orm/src/main/java/org/springframework/orm/jpa/support/PersistenceAnnotationBeanPostProcessor.java +++ b/spring-orm/src/main/java/org/springframework/orm/jpa/support/PersistenceAnnotationBeanPostProcessor.java @@ -50,6 +50,7 @@ import org.springframework.beans.factory.config.DestructionAwareBeanPostProcesso import org.springframework.beans.factory.config.InstantiationAwareBeanPostProcessor; import org.springframework.beans.factory.support.MergedBeanDefinitionPostProcessor; import org.springframework.beans.factory.support.RootBeanDefinition; +import org.springframework.core.BridgeMethodResolver; import org.springframework.core.Ordered; import org.springframework.core.PriorityOrdered; import org.springframework.jndi.JndiLocatorDelegate; @@ -405,19 +406,21 @@ public class PersistenceAnnotationBeanPostProcessor do { LinkedList currElements = new LinkedList(); for (Field field : targetClass.getDeclaredFields()) { - PersistenceContext pc = field.getAnnotation(PersistenceContext.class); - PersistenceUnit pu = field.getAnnotation(PersistenceUnit.class); - if (pc != null || pu != null) { + if (field.isAnnotationPresent(PersistenceContext.class) || + field.isAnnotationPresent(PersistenceUnit.class)) { if (Modifier.isStatic(field.getModifiers())) { throw new IllegalStateException("Persistence annotations are not supported on static fields"); } - currElements.add(new PersistenceElement(field, null)); + currElements.add(new PersistenceElement(field, field, null)); } } for (Method method : targetClass.getDeclaredMethods()) { - PersistenceContext pc = method.getAnnotation(PersistenceContext.class); - PersistenceUnit pu = method.getAnnotation(PersistenceUnit.class); - if ((pc != null || pu != null) && !method.isBridge() && + Method bridgedMethod = BridgeMethodResolver.findBridgedMethod(method); + if (!BridgeMethodResolver.isVisibilityBridgeMethodPair(method, bridgedMethod)) { + continue; + } + if ((bridgedMethod.isAnnotationPresent(PersistenceContext.class) || + bridgedMethod.isAnnotationPresent(PersistenceUnit.class)) && method.equals(ClassUtils.getMostSpecificMethod(method, clazz))) { if (Modifier.isStatic(method.getModifiers())) { throw new IllegalStateException("Persistence annotations are not supported on static methods"); @@ -425,8 +428,8 @@ public class PersistenceAnnotationBeanPostProcessor if (method.getParameterTypes().length != 1) { throw new IllegalStateException("Persistence annotation requires a single-arg method: " + method); } - PropertyDescriptor pd = BeanUtils.findPropertyForMethod(method); - currElements.add(new PersistenceElement(method, pd)); + PropertyDescriptor pd = BeanUtils.findPropertyForMethod(bridgedMethod, clazz); + currElements.add(new PersistenceElement(method, bridgedMethod, pd)); } } elements.addAll(0, currElements); @@ -621,9 +624,8 @@ public class PersistenceAnnotationBeanPostProcessor private Properties properties; - public PersistenceElement(Member member, PropertyDescriptor pd) { + public PersistenceElement(Member member, AnnotatedElement ae, PropertyDescriptor pd) { super(member, pd); - AnnotatedElement ae = (AnnotatedElement) member; PersistenceContext pc = ae.getAnnotation(PersistenceContext.class); PersistenceUnit pu = ae.getAnnotation(PersistenceUnit.class); Class resourceType = EntityManager.class; diff --git a/spring-orm/src/test/java/org/springframework/orm/jpa/support/PersistenceInjectionTests.java b/spring-orm/src/test/java/org/springframework/orm/jpa/support/PersistenceInjectionTests.java index b907a2d5dfc..faa2ab62717 100644 --- a/spring-orm/src/test/java/org/springframework/orm/jpa/support/PersistenceInjectionTests.java +++ b/spring-orm/src/test/java/org/springframework/orm/jpa/support/PersistenceInjectionTests.java @@ -774,7 +774,7 @@ public class PersistenceInjectionTests extends AbstractEntityManagerFactoryBeanT @SuppressWarnings("serial") - public static class SpecificPublicPersistenceContextSetter extends DefaultPublicPersistenceContextSetter { + static class PublicPersistenceContextSetterOnNonPublicClass extends DefaultPublicPersistenceContextSetter { @Override @PersistenceContext(unitName="unit2", type = PersistenceContextType.EXTENDED) @@ -784,6 +784,11 @@ public class PersistenceInjectionTests extends AbstractEntityManagerFactoryBeanT } + @SuppressWarnings("serial") + public static class SpecificPublicPersistenceContextSetter extends PublicPersistenceContextSetterOnNonPublicClass { + } + + public static class DefaultPrivatePersistenceUnitField { @PersistenceUnit