-
Notifications
You must be signed in to change notification settings - Fork 38.8k
Closed
Labels
in: coreIssues in core modules (aop, beans, core, context, expression)Issues in core modules (aop, beans, core, context, expression)type: enhancementA general enhancementA general enhancement
Milestone
Description
The TypeDescriptor for the varargs component type can currently be null in SpEL's ReflectionHelper.convertAllMethodHandleArguments(...) method; however, that should not be possible.
Whereas, in compareArgumentsVarargs(...) and convertArguments(...) the varargs component type can never be null.
My assumption is that we are using an incorrect combination of Class, ResolvableType, and TypeDescriptor in
convertAllMethodHandleArguments(...).
I discovered this while working on related SpEL issues and added the following TODO which this issue aims to address.
Lines 386 to 387 in 4de2aad
| // TODO Determine why componentTypeDesc can be null. | |
| // Assert.state(componentTypeDesc != null, "Component type must not be null for a varargs array"); |
Metadata
Metadata
Assignees
Labels
in: coreIssues in core modules (aop, beans, core, context, expression)Issues in core modules (aop, beans, core, context, expression)type: enhancementA general enhancementA general enhancement