Upgrade lastnpe EEA to 2.4.0 (#2813)

For release notes, see: https://github.com/lastnpe/eclipse-null-eea-augments/releases/tag/v2.4.0 Signed-off-by: Wouter Born <github@maindrain.net>
openhab · Oct 20, 2024 · f5f3a3e · f5f3a3e
1 parent 5bc928d
commit f5f3a3e
Show file tree

Hide file tree

Showing 4 changed files with 13 additions and 6 deletions.
diff --git a/...cometvisu/src/main/java/org/openhab/ui/cometvisu/internal/backend/sitemap/VisuConfig.java b/...cometvisu/src/main/java/org/openhab/ui/cometvisu/internal/backend/sitemap/VisuConfig.java
@@ -16,6 +16,7 @@
 import java.io.StringWriter;
 import java.math.BigDecimal;
 import java.math.BigInteger;
+import java.util.Objects;
 
 import javax.servlet.http.HttpServletRequest;
 import javax.xml.XMLConstants;
@@ -117,7 +118,7 @@ public String getConfigXml(HttpServletRequest req) {
 
         // set relative path to XSD file
         File rootFolder = new File(Config.cometvisuWebfolder);
-        File file = new File(rootFolder, req.getPathInfo());
+        File file = new File(rootFolder, Objects.requireNonNull(req.getPathInfo(), "Path info should not be null"));
         String relXsd = "";
         File parent = file.getParentFile();
         File schema = new File(parent, "visu_config.xsd");

diff --git a/...ab.ui.cometvisu/src/main/java/org/openhab/ui/cometvisu/internal/util/ClientInstaller.java b/...ab.ui.cometvisu/src/main/java/org/openhab/ui/cometvisu/internal/util/ClientInstaller.java
@@ -309,7 +309,10 @@ private void extractFolder(String folderName, ZipFile zipFile, String destDir) {
                         // never ever overwrite existing config files
                         continue;
                     }
-                    new File(file.getParent()).mkdirs();
+                    String parent = file.getParent();
+                    if (parent != null) {
+                        new File(parent).mkdirs();
+                    }
 
                     try (InputStream is = zipFile.getInputStream(entry); OutputStream os = new FileOutputStream(file)) {
                         for (int len; (len = is.read(BUFFER)) != -1;) {

diff --git a/...org.openhab.ui.cometvisu/src/main/java/org/openhab/ui/cometvisu/internal/util/FsUtil.java b/...org.openhab.ui.cometvisu/src/main/java/org/openhab/ui/cometvisu/internal/util/FsUtil.java
@@ -113,9 +113,12 @@ public boolean isInTrash(File file) {
     }
 
     public void saveFile(File file, InputStream fileInputStream, String hash) throws FileOperationException {
-        File parentDir = new File(file.getParent());
-        if (!parentDir.exists()) {
-            parentDir.mkdirs();
+        String parent = file.getParent();
+        if (parent != null) {
+            File parentDir = new File(parent);
+            if (!parentDir.exists()) {
+                parentDir.mkdirs();
+            }
         }
         // check hash
         try {

diff --git a/pom.xml b/pom.xml
@@ -66,7 +66,7 @@
     <maven.compiler.release>${oh.java.version}</maven.compiler.release>
 
     <bnd.version>7.0.0</bnd.version>
-    <eea.version>2.3.0</eea.version>
+    <eea.version>2.4.0</eea.version>
     <jackson.version>2.16.0</jackson.version>
     <karaf.version>4.4.6</karaf.version>
     <ohc.version>4.3.0-SNAPSHOT</ohc.version>