Skip to content

Commit 33ef910

Browse files
divconrstoyanchev
authored andcommitted
Minor refactoring in PayloadMethodArgumentResolver
Closes gh-30168
1 parent 8463ead commit 33ef910

File tree

1 file changed

+3
-7
lines changed

1 file changed

+3
-7
lines changed

spring-messaging/src/main/java/org/springframework/messaging/handler/annotation/support/PayloadMethodArgumentResolver.java

Lines changed: 3 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -136,11 +136,7 @@ public Object resolveArgument(MethodParameter parameter, Message<?> message) thr
136136

137137
Class<?> targetClass = resolveTargetClass(parameter, message);
138138
Class<?> payloadClass = payload.getClass();
139-
if (ClassUtils.isAssignable(targetClass, payloadClass)) {
140-
validate(message, parameter, payload);
141-
return (isOptionalTargetClass ? Optional.of(payload) : payload);
142-
}
143-
else {
139+
if (!ClassUtils.isAssignable(targetClass, payloadClass)) {
144140
if (this.converter instanceof SmartMessageConverter smartConverter) {
145141
payload = smartConverter.fromMessage(message, targetClass, parameter);
146142
}
@@ -151,9 +147,9 @@ public Object resolveArgument(MethodParameter parameter, Message<?> message) thr
151147
throw new MessageConversionException(message, "Cannot convert from [" +
152148
payloadClass.getName() + "] to [" + targetClass.getName() + "] for " + message);
153149
}
154-
validate(message, parameter, payload);
155-
return (isOptionalTargetClass ? Optional.of(payload) : payload);
156150
}
151+
validate(message, parameter, payload);
152+
return (isOptionalTargetClass ? Optional.of(payload) : payload);
157153
}
158154

159155
private String getParameterName(MethodParameter param) {

0 commit comments

Comments
 (0)