FileUtils isFolderEmpty should throw IOException

- And this also eliminates a warning in Eclipse 4.31 about a potential resource leak
archimatetool · Feb 27, 2024 · 3ef1be6 · 3ef1be6
1 parent 3244726
commit 3ef1be6
Showing 1 changed file with 1 addition and 4 deletions.
diff --git a/com.archimatetool.editor/src/com/archimatetool/editor/utils/FileUtils.java b/com.archimatetool.editor/src/com/archimatetool/editor/utils/FileUtils.java
@@ -416,7 +416,7 @@ public static String getValidFileName(String name) {
      * @param folder
      * @return true if the folder is empty
      */
-    public static boolean isFolderEmpty(File folder) {
+    public static boolean isFolderEmpty(File folder) throws IOException {
         if(!(folder != null && folder.exists() && folder.isDirectory())) {
             return true;
         }
@@ -426,9 +426,6 @@ public static boolean isFolderEmpty(File folder) {
                                         .filter(path -> !path.endsWith(".DS_Store"))) { // Ignore Mac file
             return entries.findFirst().isEmpty();
         }
-        catch(IOException ex) {
-            return true;
-        }
     }
 }