Skip to content

Commit a6b99db

Browse files
committed
Fix new Sonar smells
1 parent 5ac262f commit a6b99db

File tree

2 files changed

+11
-9
lines changed

2 files changed

+11
-9
lines changed

spring-integration-core/src/main/java/org/springframework/integration/expression/ReloadableResourceBundleExpressionSource.java

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -294,8 +294,9 @@ private PropertiesHolder getMergedProperties(Locale locale) {
294294
for (int j = filenames.size() - 1; j >= 0; j--) {
295295
String filename = filenames.get(j);
296296
PropertiesHolder propHolder = getProperties(filename);
297-
if (propHolder.getProperties() != null) {
298-
mergedProps.putAll(propHolder.getProperties());
297+
Properties propHolderProperties = propHolder.getProperties();
298+
if (propHolderProperties != null) {
299+
mergedProps.putAll(propHolderProperties);
299300
}
300301
}
301302
}

spring-integration-core/src/main/java/org/springframework/integration/handler/AbstractMessageHandler.java

Lines changed: 8 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -38,11 +38,12 @@
3838
public abstract class AbstractMessageHandler extends MessageHandlerSupport
3939
implements MessageHandler, CoreSubscriber<Message<?>> {
4040

41-
@SuppressWarnings("deprecation")
41+
@SuppressWarnings("deprecation") // NOSONAR
4242
public void handleMessage(Message<?> message) {
43-
Assert.notNull(message, "Message must not be null");
43+
Message<?> messageToUse = message;
44+
Assert.notNull(messageToUse, "Message must not be null");
4445
if (isLoggingEnabled() && this.logger.isDebugEnabled()) {
45-
this.logger.debug(this + " received message: " + message);
46+
this.logger.debug(this + " received message: " + messageToUse);
4647
}
4748
org.springframework.integration.support.management.MetricsContext start = null;
4849
SampleFacade sample = null;
@@ -52,19 +53,19 @@ public void handleMessage(Message<?> message) {
5253
}
5354
try {
5455
if (shouldTrack()) {
55-
message = MessageHistory.write(message, this, getMessageBuilderFactory());
56+
messageToUse = MessageHistory.write(messageToUse, this, getMessageBuilderFactory());
5657
}
5758
AbstractMessageHandlerMetrics handlerMetrics = getHandlerMetrics();
5859
if (isCountsEnabled()) {
5960
start = handlerMetrics.beforeHandle();
60-
handleMessageInternal(message);
61+
handleMessageInternal(messageToUse);
6162
if (sample != null) {
6263
sample.stop(sendTimer());
6364
}
6465
handlerMetrics.afterHandle(start, true);
6566
}
6667
else {
67-
handleMessageInternal(message);
68+
handleMessageInternal(messageToUse);
6869
}
6970
}
7071
catch (Exception e) {
@@ -74,7 +75,7 @@ public void handleMessage(Message<?> message) {
7475
if (isCountsEnabled()) {
7576
getHandlerMetrics().afterHandle(start, false);
7677
}
77-
throw IntegrationUtils.wrapInHandlingExceptionIfNecessary(message,
78+
throw IntegrationUtils.wrapInHandlingExceptionIfNecessary(messageToUse,
7879
() -> "error occurred in message handler [" + this + "]", e);
7980
}
8081
}

0 commit comments

Comments
 (0)