qa: fix some sonarcloud reported code smells

geoserver · Mar 15, 2024 · 9e660ec · 9e660ec
1 parent b22b8b9
commit 9e660ec
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 13 deletions.
diff --git a/...main/java/org/geoserver/cloud/event/remote/datadir/RemoteEventDataDirectoryProcessor.java b/...main/java/org/geoserver/cloud/event/remote/datadir/RemoteEventDataDirectoryProcessor.java
@@ -135,7 +135,7 @@ public void onRemoteAddEvent(InfoAdded<? extends Info> event) {
             case CatalogInfo info -> facade.add(info);
             case ServiceInfo config -> configFacade.add(config);
             case SettingsInfo config -> configFacade.add(config);
-            case LoggingInfo config -> log.debug("ignoring unused LoggingInfo");
+            case LoggingInfo config -> log.debug("ignoring unused LoggingInfo {}", config);
             default -> log.warn("Don't know how to handle remote envent {})", event);
         }
     }
@@ -189,18 +189,16 @@ private Info loadInfo(InfoModified event) {
             Class<? extends CatalogInfo> ctype = (Class<? extends CatalogInfo>) type.getType();
             return catalogFacade().get(objectId, ctype).orElse(null);
         }
-        Info configInfo =
-                switch (type) {
-                    case GEOSERVER -> configFacade.getGlobal();
-                    case SERVICE -> configFacade.getService(objectId, ServiceInfo.class);
-                    case SETTINGS -> configFacade.getSettings(objectId);
-                    case LOGGING -> configFacade.getLogging();
-                    default -> {
-                        log.warn("Don't know how to handle remote modify envent {}", event);
-                        yield null;
-                    }
-                };
-        return configInfo;
+        return switch (type) {
+            case GEOSERVER -> configFacade.getGlobal();
+            case SERVICE -> configFacade.getService(objectId, ServiceInfo.class);
+            case SETTINGS -> configFacade.getSettings(objectId);
+            case LOGGING -> configFacade.getLogging();
+            default -> {
+                log.warn("Don't know how to handle remote modify envent {}", event);
+                yield null;
+            }
+        };
     }
 
     @EventListener(DefaultWorkspaceSet.class)

diff --git a/...og/plugin/src/main/java/org/geoserver/catalog/plugin/CatalogInfoRepositoryHolderImpl.java b/...og/plugin/src/main/java/org/geoserver/catalog/plugin/CatalogInfoRepositoryHolderImpl.java
@@ -50,6 +50,7 @@ public <T extends CatalogInfo, R extends CatalogInfoRepository<T>> R repositoryF
         return (R) repos.forObject(info);
     }
 
+    @SuppressWarnings("java:S1452")
     public List<CatalogInfoRepository<?>> all() {
         return repos.getAll();
     }