Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Michael Vitz opened SPR-14552 and commented
The reference documentation at http://docs.spring.io/spring/docs/current/spring-framework-reference/htmlsingle/#beans-java-composing-configuration-classes states that "Constructor injection in @Configuration classes is only supported as of Spring Framework 4.3.". However if looking in the javadoc API for @Configuration at http://docs.spring.io/spring-framework/docs/current/javadoc-api/org/springframework/context/annotation/Configuration.html the following is said: "@Configuration classes must have a default/no-arg constructor and may not use @Autowired constructor parameters. Any nested configuration classes must be static." I think the javadoc needs to be fixed here.
@Configuration
@Autowired
Affects: 4.3.2
Referenced from: commits 8aadb8d, 4854989
The text was updated successfully, but these errors were encountered:
Stéphane Nicoll commented
Good catch, thanks!
snicoll
Successfully merging a pull request may close this issue.
Michael Vitz opened SPR-14552 and commented
The reference documentation at http://docs.spring.io/spring/docs/current/spring-framework-reference/htmlsingle/#beans-java-composing-configuration-classes states that "Constructor injection in
@Configuration
classes is only supported as of Spring Framework 4.3.". However if looking in the javadoc API for@Configuration
at http://docs.spring.io/spring-framework/docs/current/javadoc-api/org/springframework/context/annotation/Configuration.html the following is said: "@Configuration
classes must have a default/no-arg constructor and may not use@Autowired
constructor parameters. Any nested configuration classes must be static." I think the javadoc needs to be fixed here.Affects: 4.3.2
Referenced from: commits 8aadb8d, 4854989
The text was updated successfully, but these errors were encountered: