Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

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

Jump to bottom

Update documentation for <context:annotation-config/> #26782

Merged
merged 1 commit into from Apr 9, 2021
Merged

Update documentation for <context:annotation-config/> #26782

merged 1 commit into from Apr 9, 2021

Conversation

hrybs
Copy link
Contributor

@hrybs hrybs commented Apr 9, 2021 •
edited

The official documentation contains incorrect description on <context:annotation-config/>:

  1. No RequiredAnnotationBeanPostProcessor is automatically registered.
    According to source code of AnnotationConfigBeanDefinitionParser class we see that there is a using of AnnotationConfigUtils class. Investigating the code of letter one we see that there is registration of 5 BeanPostProcessor and BeanFactoryPostProcessor classes.
  2. There are 2 additional processor that are registered when using aforementioned annotation
  3. AutowiredAnnotationBeanPostProcessor also processes @lookup annotation.
  4. CommonAnnotationBeanPostProcessor also processes JAX-WS @WebServiceRef and EJB3 @ejb annotations

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged or decided on label Apr 9, 2021
@sbrannen sbrannen self-assigned this Apr 9, 2021
@sbrannen sbrannen added type: documentation A documentation task and removed status: waiting-for-triage An issue we've not yet triaged or decided on labels Apr 9, 2021
@sbrannen sbrannen added this to the 5.3.6 milestone Apr 9, 2021
@sbrannen sbrannen changed the title fix annotation-config description Update documentation for <context:annotation-config/> Apr 9, 2021
@sbrannen sbrannen merged commit 35ea33e into spring-projects:master Apr 9, 2021
sbrannen added a commit that referenced this pull request Apr 9, 2021
@sbrannen
Polish contribution
0f54585 
See gh-26782
@sbrannen
Copy link
Member

sbrannen commented Apr 9, 2021

This has been merged into master in 35ea33e and revised in 0f54585.

Thanks

@sync-by-unito sync-by-unito bot mentioned this pull request Apr 14, 2021
Merged
Zoran0104 pushed a commit to Zoran0104/spring-framework that referenced this pull request Aug 20, 2021
@hrybs @Zoran0104
Update documentation for <context:annotation-config/>
1c339d0 
Closes spring-projectsgh-26782
Zoran0104 pushed a commit to Zoran0104/spring-framework that referenced this pull request Aug 20, 2021
@sbrannen @Zoran0104
Polish contribution
d335b88 
See spring-projectsgh-26782
lxbzmy pushed a commit to lxbzmy/spring-framework that referenced this pull request Mar 26, 2022
@hrybs @lxbzmy
Update documentation for <context:annotation-config/>
a9471ff 
Closes spring-projectsgh-26782
lxbzmy pushed a commit to lxbzmy/spring-framework that referenced this pull request Mar 26, 2022
@sbrannen @lxbzmy
Polish contribution
b2d0fea 
See spring-projectsgh-26782
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@sbrannen sbrannen

Labels
type: documentation A documentation task
Projects
None yet
Milestone
5.3.6
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@hrybs @sbrannen @spring-projects-issues