Remove mod-menu for 1.17

build.gradle

@@ -28,7 +28,7 @@ dependencies {
 
     modImplementation "net.fabricmc:fabric-loader:${project.loader_version}"
 
-    modImplementation "com.terraformersmc:modmenu:${project.modmenu_version}"
+    // modImplementation "com.terraformersmc:modmenu:${project.modmenu_version}"
     modImplementation "net.fabricmc.fabric-api:fabric-api:${project.fabric_version}"
     modImplementation include("io.github.cottonmc:LibGui:${project.libgui_version}")
 }

src/main/java/me/bionicbeanie/mods/savecoords/gui/impl/DIContainer.java

@@ -1,14 +1,11 @@
 package me.bionicbeanie.mods.savecoords.gui.impl;
 
-import com.terraformersmc.modmenu.api.ConfigScreenFactory;
-
 import me.bionicbeanie.mods.savecoords.IFileStore;
 import me.bionicbeanie.mods.savecoords.IKeyBinds;
 import me.bionicbeanie.mods.savecoords.IModGui;
 import me.bionicbeanie.mods.savecoords.IPlayerLocator;
 import me.bionicbeanie.mods.savecoords.impl.Factory;
 import net.minecraft.client.MinecraftClient;
-import net.minecraft.client.gui.screen.Screen;
 
 //All of this works because single-threaded initialization!! Sed lyf :(
 public class DIContainer {
@@ -19,7 +16,7 @@ public class DIContainer {
     private static IPlayerLocator playerLocator;
     private static IModGui modGui;
     private static IKeyBinds keyBinds;
-    private static ConfigScreenFactory<Screen> modMenuScreenFactory;
+    //private static ConfigScreenFactory<Screen> modMenuScreenFactory;
     private static PingPositionOperation pingPositionOperation;
 
     public static IModGui getModGui() {
@@ -32,25 +29,25 @@ public class DIContainer {
         return keyBinds;
     }
 
-    public static ConfigScreenFactory<Screen> getModMenuScreenFactory() {
+//    public static ConfigScreenFactory<Screen> getModMenuScreenFactory() {
-        initialize();
+//        initialize();
-
+//
-        if (modMenuScreenFactory == null) {
+//        if (modMenuScreenFactory == null) {
-            modMenuScreenFactory = (parent) -> {
+//            modMenuScreenFactory = (parent) -> {
-                ConfigViewHandler handler = new ConfigViewHandler();
+//                ConfigViewHandler handler = new ConfigViewHandler();
-
+//
-                handler.onSave(() -> {
+//                handler.onSave(() -> {
-                    new SaveConfigsOperation(keyBinds, fileStore, handler::getState).run();
+//                    new SaveConfigsOperation(keyBinds, fileStore, handler::getState).run();
-                    guiController.closeScreen();
+//                    guiController.closeScreen();
-                });
+//                });
-
+//
-                handler.onBack(() -> guiController.closeScreen());
+//                handler.onBack(() -> guiController.closeScreen());
-
+//
-                return handler.createView(keyBinds.getAllBinds());
+//                return handler.createView(keyBinds.getAllBinds());
-            };
+//            };
-        }
+//        }
-        return modMenuScreenFactory;
+//        return modMenuScreenFactory;
-    }
+//    }
     
     public static Runnable getPingPositionOperation() {
         initialize();
@@ -59,7 +56,6 @@ public class DIContainer {
 
     private static void initialize() {
         if (!initialized) {
-
             initialized = true;
 
             MinecraftClient minecraftClient = MinecraftClient.getInstance();

src/main/java/me/bionicbeanie/mods/savecoords/gui/impl/ListViewHandler.java

@@ -82,7 +82,8 @@ class ListViewHandler extends ViewHandlerBase<Void> {
             List<PlayerPosition> positions = fileStore.listPositions();
             Collections.sort(positions, (p1, p2) -> {
                 if (p1.getPositionMetadata() != null && p2.getPositionMetadata() != null) {
-                    return p2.getPositionMetadata().getLastModified().compareTo(p1.getPositionMetadata().getLastModified());
+                    return p2.getPositionMetadata().getLastModifiedMillis() > p1.getPositionMetadata().getLastModifiedMillis() ? 1
+                            : 0;
                 }
 
                 return -1;

src/main/java/me/bionicbeanie/mods/savecoords/gui/impl/RootGridPanel.java

@@ -6,6 +6,7 @@ import java.util.List;
 import io.github.cottonmc.cotton.gui.GuiDescription;
 import io.github.cottonmc.cotton.gui.widget.WGridPanel;
 import io.github.cottonmc.cotton.gui.widget.WWidget;
+import io.github.cottonmc.cotton.gui.widget.data.Insets;
 import me.bionicbeanie.mods.savecoords.gui.IRootPanel;
 
 class RootGridPanel extends WGridPanel implements IRootPanel{
@@ -15,6 +16,7 @@ class RootGridPanel extends WGridPanel implements IRootPanel{
     
     public RootGridPanel(GuiDescription guiDescription) {
         this(18);
+        setInsets(Insets.ROOT_PANEL);
         
         this.guiDescription = guiDescription;
     }

src/main/java/me/bionicbeanie/mods/savecoords/gui/impl/SaveConfigsOperation.java

@@ -30,7 +30,7 @@ public class SaveConfigsOperation extends ViewOperationBase<List<IKeyBinding>>{
         for (IKeyBinding binding : state) {
             Config config = new Config();
             config.setName(binding.getName());
-            config.setType(parseTypeInt(binding.getType()));
+            config.setType(parseTypeString(binding.getType()));
             config.setCode(binding.getCode());
             
             keyBinds.updateKeyBind(binding.getName(), binding.getType(), binding.getCode());
@@ -43,11 +43,11 @@ public class SaveConfigsOperation extends ViewOperationBase<List<IKeyBinding>>{
         fileStore.writeConfigs(configData);
     }
 
-    private int parseTypeInt(Type type) {
+    private String parseTypeString(Type type) {
         if(type == Type.KEYSYM) {
-            return 0;
+            return "KEYSYM";
         }
         
-        return 1;
+        return "MOUSE";
     }
 }

src/main/java/me/bionicbeanie/mods/savecoords/impl/FileStore.java

@@ -141,15 +141,7 @@ class FileStore implements IFileStore {
             return data;
 
         } catch (Exception e) {
-            // Fallback for old versions
+            throw e;
-            ModData data = new ModData();
-            data.setDefaultWorldName("");
-            data.setPositions(gson.fromJson(String.join("", lines), PlayerPosition[].class));
-            data.setConfigData(new ConfigData());
-
-            dump(data);
-
-            return data;
         }
     }
 

src/main/java/me/bionicbeanie/mods/savecoords/impl/KeyBinds.java

@@ -154,8 +154,8 @@ class KeyBinds implements IKeyBinds {
         
     }
 
-    private Type parseType(int type) {
+    private Type parseType(String type) {
-        if(type == 0) {
+        if("KEYSYM".equalsIgnoreCase(type)) {
             return Type.KEYSYM;
         }
         

src/main/java/me/bionicbeanie/mods/savecoords/model/ConfigData.java

@@ -33,7 +33,7 @@ public class ConfigData {
 
     public static class Config{
         private String name;
-        private int type;
+        private String type;
         private int code;
         
         public int getCode() {
@@ -42,10 +42,10 @@ public class ConfigData {
         public void setCode(int code) {
             this.code = code;
         }
-        public int getType() {
+        public String getType() {
             return type;
         }
-        public void setType(int type) {
+        public void setType(String type) {
             this.type = type;
         }
         public String getName() {

src/main/java/me/bionicbeanie/mods/savecoords/model/PositionMetadata.java

@@ -1,17 +1,15 @@
 package me.bionicbeanie.mods.savecoords.model;
 
-import java.time.LocalDateTime;
-
 public class PositionMetadata {
 
     private String worldName, notes;
-    private LocalDateTime created, lastModified;
+    private long createdMillis, lastModifiedMillis;
 
     public PositionMetadata(String worldName, String notes) {
         this.worldName = worldName;
         this.notes = notes;
-        this.created = LocalDateTime.now();
+        this.createdMillis = System.currentTimeMillis();
-        this.lastModified = this.created;
+        this.lastModifiedMillis = this.createdMillis;
     }
 
     public String getWorldName() {
@@ -30,15 +28,15 @@ public class PositionMetadata {
         this.notes = notes;
     }
 
-    public LocalDateTime getCreated() {
+    public long getCreatedMillis() {
-        return created;
+        return createdMillis;
     }
 
-    public LocalDateTime getLastModified() {
+    public long getLastModifiedMillis() {
-        return lastModified;
+        return lastModifiedMillis;
     }
 
     public void updateLastModified() {
-        this.lastModified = LocalDateTime.now();
+        this.lastModifiedMillis = System.currentTimeMillis();
     }
 }

src/main/java/me/bionicbeanie/mods/savecoords/modmenu/SaveCoordinatesModMenu.java

@@ -1,14 +1,14 @@
-package me.bionicbeanie.mods.savecoords.modmenu;
+//package me.bionicbeanie.mods.savecoords.modmenu;
-
+//
-import com.terraformersmc.modmenu.api.ConfigScreenFactory;
+//import com.terraformersmc.modmenu.api.ConfigScreenFactory;
-import com.terraformersmc.modmenu.api.ModMenuApi;
+//import com.terraformersmc.modmenu.api.ModMenuApi;
-
+//
-import me.bionicbeanie.mods.savecoords.gui.impl.DIContainer;
+//import me.bionicbeanie.mods.savecoords.gui.impl.DIContainer;
-
+//
-public class SaveCoordinatesModMenu implements ModMenuApi {
+//public class SaveCoordinatesModMenu implements ModMenuApi {
-
+//
-    @Override
+//    @Override
-    public ConfigScreenFactory<?> getModConfigScreenFactory() {
+//    public ConfigScreenFactory<?> getModConfigScreenFactory() {
-        return DIContainer.getModMenuScreenFactory();
+//        return DIContainer.getModMenuScreenFactory();
-    }
+//    }
-}
+//}