Spring/spring-integration
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Fix new Sonar smells

Browse Source
This commit is contained in:
Artem Bilan
2019-06-22 13:43:23 -04:00
parent bda5221720
commit e833a44a38
2 changed files with 6 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
spring-integration-core/src/main/java/org/springframework/integration/history/MessageHistoryConfigurer.java
View File

@@ -52,7 +52,7 @@ import org.springframework.util.StringUtils;
@IntegrationManagedResource
public class MessageHistoryConfigurer implements SmartLifecycle, BeanFactoryAware, DestructionAwareBeanPostProcessor {
private static final Log logger = LogFactory.getLog(MessageHistoryConfigurer.class);
private static final Log LOGGER = LogFactory.getLog(MessageHistoryConfigurer.class);
private final Set<TrackableComponent> currentlyTrackedComponents = ConcurrentHashMap.newKeySet();
@@ -154,8 +154,8 @@ public class MessageHistoryConfigurer implements SmartLifecycle, BeanFactoryAwar
component.setShouldTrack(shouldTrack);
if (shouldTrack) {
this.currentlyTrackedComponents.add(component);
if (this.logger.isInfoEnabled()) {
this.logger.info("Enabling MessageHistory tracking for component '" + componentName + "'");
if (this.LOGGER.isInfoEnabled()) {
this.LOGGER.info("Enabling MessageHistory tracking for component '" + componentName + "'");
}
}
}
@@ -217,8 +217,8 @@ public class MessageHistoryConfigurer implements SmartLifecycle, BeanFactoryAwar
if (this.running) {
this.currentlyTrackedComponents.forEach(component -> {
component.setShouldTrack(false);
if (this.logger.isInfoEnabled()) {
this.logger.info("Disabling MessageHistory tracking for component '"
if (this.LOGGER.isInfoEnabled()) {
this.LOGGER.info("Disabling MessageHistory tracking for component '"
+ component.getComponentName() + "'");
}
});

2
spring-integration-jms/src/main/java/org/springframework/integration/jms/config/JmsChannelFactoryBean.java
View File

@@ -520,7 +520,7 @@ public class JmsChannelFactoryBean extends AbstractFactoryBean<AbstractJmsChanne
@Override
protected void destroyInstance(AbstractJmsChannel instance) {
instance.destroy();
this.channel.destroy();
}
}