Spring/spring-framework
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

AutowiredAnnotationBeanPostProcessor tolerates annotated no-arg constructors

Browse Source 
Issue: SPR-15005
(cherry picked from commit 8b5ee4e)
This commit is contained in:
Juergen Hoeller
2016-12-12 22:26:24 +01:00
parent 8190e7838f
commit 934fffec68
2 changed files with 18 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
spring-beans/src/main/java/org/springframework/beans/factory/annotation/AutowiredAnnotationBeanPostProcessor.java
View File

@@ -313,10 +313,6 @@ public class AutowiredAnnotationBeanPostProcessor extends InstantiationAwareBean
". Found constructor with 'required' Autowired annotation already: " +
requiredConstructor);
}
if (candidate.getParameterTypes().length == 0) {
throw new IllegalStateException(
"Autowired annotation requires at least one argument: " + candidate);
}
boolean required = determineRequiredStatus(ann);
if (required) {
if (!candidates.isEmpty()) {

22
spring-beans/src/test/java/org/springframework/beans/factory/annotation/InjectAnnotationBeanPostProcessorTests.java
View File

@@ -1,5 +1,5 @@
/*
* Copyright 2002-2014 the original author or authors.
* Copyright 2002-2016 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.
@@ -742,6 +742,15 @@ public class InjectAnnotationBeanPostProcessorTests {
bf.destroySingletons();
}
@Test
public void testAnnotatedDefaultConstructor() {
DefaultListableBeanFactory bf = new DefaultListableBeanFactory();
bf.addBeanPostProcessor(new AutowiredAnnotationBeanPostProcessor());
bf.registerBeanDefinition("annotatedBean", new RootBeanDefinition(AnnotatedDefaultConstructorBean.class));
assertNotNull(bf.getBean("annotatedBean"));
}
public static class ResourceInjectionBean {
@@ -750,7 +759,6 @@ public class InjectAnnotationBeanPostProcessorTests {
private TestBean testBean2;
@Inject
public void setTestBean2(TestBean testBean2) {
if (this.testBean2 != null) {
@@ -819,7 +827,6 @@ public class InjectAnnotationBeanPostProcessorTests {
public static class TypedExtendedResourceInjectionBean extends ExtendedResourceInjectionBean<NestedTestBean> {
}
@@ -1087,7 +1094,6 @@ public class InjectAnnotationBeanPostProcessorTests {
@Inject
private Map<String, TestBean> testBeanMap;
public Map<String, TestBean> getTestBeanMap() {
return this.testBeanMap;
}
@@ -1346,4 +1352,12 @@ public class InjectAnnotationBeanPostProcessorTests {
}
}
public static class AnnotatedDefaultConstructorBean {
@Inject
public AnnotatedDefaultConstructorBean() {
}
}
}