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.

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

'default-lazy-init' attribute is not processed when XSD validation is disabled [SPR-8335] #12983

Closed
spring-projects-issues opened this issue May 13, 2011 · 4 comments

Comments

@spring-projects-issues
Copy link
Collaborator

@spring-projects-issues spring-projects-issues commented May 13, 2011

Andruschuk Borislav opened SPR-8335 and commented

I've noticed that default-lazy-init attribute inside beans element is not processed when XSD validation is disabled. Thus all singletons that should be lazy become initialized. Tests are attached to the issue.


Affects: 3.0.5

Attachments:

Referenced from: pull request #2071, and commits 21d3bf8, 4938dab, cba355a

Backported to: 5.0.12, 4.3.22

3 votes, 5 watchers

@spring-projects-issues
Copy link
Collaborator Author

@spring-projects-issues spring-projects-issues commented Dec 26, 2018

Sébastien Deleuze commented

Surprising but I can confirm this issue.

@spring-projects-issues
Copy link
Collaborator Author

@spring-projects-issues spring-projects-issues commented Jan 6, 2019

Michel Schudel commented

Sébastien Deleuze

The root cause is that the xml parser does not fill in default values from the xsd when validation is disabled. The beans parser should take this into account.

I created a pull request that should solve the issue:

#2071

 

@spring-projects-issues
Copy link
Collaborator Author

@spring-projects-issues spring-projects-issues commented Jan 7, 2019

Juergen Hoeller commented

This looks valid indeed and is worth backporting to 5.0.12 and 4.3.22 from my perspective.

@spring-projects-issues
Copy link
Collaborator Author

@spring-projects-issues spring-projects-issues commented Jan 8, 2019

Sébastien Deleuze commented

Merged, thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants