BugFix: `/kb list` or `/kb` shows `null` for all incremental backups.1.4.5

2 files changed, 22 insertions, 2 deletions

diff --git a/src/main/java/com/keuin/kbackupfabric/KBCommands.java b/src/main/java/com/keuin/kbackupfabric/KBCommands.java
index 90a4130..9f00790 100644
--- a/src/main/java/com/keuin/kbackupfabric/KBCommands.java
+++ b/src/main/java/com/keuin/kbackupfabric/KBCommands.java
@@ -25,6 +25,7 @@ import java.util.ArrayList;
 import java.util.Arrays;
 import java.util.List;
 import java.util.Objects;
+import java.util.function.Function;
 
 import static com.keuin.kbackupfabric.backup.BackupFilesystemUtil.*;
 import static com.keuin.kbackupfabric.util.PrintUtil.*;
@@ -90,6 +91,18 @@ public final class KBCommands {
                                 || name.toLowerCase().endsWith(".kbi"))
         );
 
+        Function<File, String> backupInformationProvider = new Function<File, String>() {
+            @Override
+            public String apply(File file) {
+                Objects.requireNonNull(file);
+                if (file.getName().toLowerCase().endsWith(".zip"))
+                    return getPrimitiveBackupInformationString(file.getName(), file.length());
+                else if (file.getName().toLowerCase().endsWith(".kbi"))
+                    return getIncrementalBackupInformationString(file.getName());
+                return file.getName();
+            }
+        };
+
         synchronized (backupFileNameList) {
             backupFileNameList.clear();
             if (files != null) {
@@ -102,7 +115,7 @@ public final class KBCommands {
                 for (File file : files) {
                     ++i;
                     String backupFileName = file.getName();
-                    msgInfo(context, String.format("[%d] %s", i, getPrimitiveBackupInformationString(backupFileName, file.length())));
+                    msgInfo(context, String.format("[%d] %s", i, backupInformationProvider.apply(file)));
                     backupFileNameList.add(backupFileName);
                 }
             } else {
@@ -383,6 +396,13 @@ public final class KBCommands {
         );
     }
 
+    private static String getIncrementalBackupInformationString(String backupFileName) {
+        return String.format(
+                "(Incremental) %s",
+                new IncrementalBackupFileNameEncoder().decode(backupFileName)
+        );
+    }
+
 //    /**
 //     * Select the backup method we use.
 //     * @param context the context.
diff --git a/src/main/resources/fabric.mod.json b/src/main/resources/fabric.mod.json
index ce26dd2..e2d9d39 100644
--- a/src/main/resources/fabric.mod.json
+++ b/src/main/resources/fabric.mod.json
@@ -13,7 +13,7 @@
   },
   "license": "GPL V3",
   "icon": "assets/kbackupfabric/icon.png",
-  "environment": "*",
+  "environment": "server",
   "entrypoints": {
     "main": [
       "com.keuin.kbackupfabric.KBPluginEvents"