Skip to content

Commit

Permalink
Revert "GH-3697: DelegatingInvocableHandler.invoke() returning null"
Browse files Browse the repository at this point in the history
This reverts commit 9e34397.
  • Loading branch information
sobychacko committed Jan 16, 2025
1 parent 9e34397 commit d28d612
Showing 1 changed file with 3 additions and 10 deletions.
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
/*
* Copyright 2016-2025 the original author or authors.
* Copyright 2016-2024 the original author or authors.
*
* Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License.
Expand Down Expand Up @@ -55,7 +55,6 @@
*
* @author Gary Russell
* @author Wang Zhiyang
* @author Ivan Malutin
*
*/
public class DelegatingInvocableHandler {
Expand Down Expand Up @@ -168,7 +167,6 @@ public boolean isAsyncReplies() {
* @throws Exception raised if no suitable argument resolver can be found,
* or the method raised an exception.
*/
@Nullable
public Object invoke(Message<?> message, Object... providedArgs) throws Exception { //NOSONAR
Class<?> payloadClass = message.getPayload().getClass();
InvocableHandlerMethod handler = getHandlerForPayload(payloadClass);
Expand All @@ -188,13 +186,8 @@ public Object invoke(Message<?> message, Object... providedArgs) throws Exceptio
else {
result = handler.invoke(message, providedArgs);
}
if (result != null) {
Expression replyTo = this.handlerSendTo.get(handler);
return new InvocationResult(result, replyTo, this.handlerReturnsMessage.get(handler));
}
else {
return null;
}
Expression replyTo = this.handlerSendTo.get(handler);
return new InvocationResult(result, replyTo, this.handlerReturnsMessage.get(handler));
}

/**
Expand Down

0 comments on commit d28d612

Please sign in to comment.