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

Parameter missing in message format #2956

Closed
mawen12 opened this issue Oct 16, 2023 · 0 comments · Fixed by #2958
Closed

Parameter missing in message format #2956

mawen12 opened this issue Oct 16, 2023 · 0 comments · Fixed by #2958
Assignees
Labels
type: bug A general bug

Comments

@mawen12
Copy link
Contributor

mawen12 commented Oct 16, 2023

The Method getRequiredBeanClassName of ConfigurationUtils has lost the placeholder for beanDefinition.

if result is null, The error message lost beanDefinition information.

public static String getRequiredBeanClassName(BeanDefinition beanDefinition) {

	Assert.notNull(beanDefinition, "BeanDefinition must not be null");

	String result = beanDefinition.getBeanClassName();

	if (result == null) {
		throw new IllegalArgumentException(
                                 // lose the placeholder for beanDefinition
				String.format("Could not obtain required bean class name from BeanDefinition", beanDefinition));
	}

	return result;
}
@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Oct 16, 2023
@mp911de mp911de added type: bug A general bug and removed status: waiting-for-triage An issue we've not yet triaged labels Oct 16, 2023
@mp911de mp911de self-assigned this Oct 16, 2023
@mp911de mp911de linked a pull request Oct 17, 2023 that will close this issue
@mp911de mp911de changed the title Complete Error Message Parameter missing in message format Oct 17, 2023
mp911de pushed a commit that referenced this issue Oct 17, 2023
mp911de pushed a commit that referenced this issue Oct 17, 2023
mp911de pushed a commit that referenced this issue Oct 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug A general bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants