From 7c36617431a7b76d77c5eb9d04ec01fc9d2cb924 Mon Sep 17 00:00:00 2001 From: Aaron <51387595+AzureAaron@users.noreply.github.com> Date: Fri, 10 May 2024 18:37:11 -0400 Subject: More null safety for newer fields --- src/main/java/de/hysky/skyblocker/config/ConfigDatafixer.java | 11 +++++------ 1 file changed, 5 insertions(+), 6 deletions(-) (limited to 'src/main/java') diff --git a/src/main/java/de/hysky/skyblocker/config/ConfigDatafixer.java b/src/main/java/de/hysky/skyblocker/config/ConfigDatafixer.java index 90f349fc..7d0bb2d2 100644 --- a/src/main/java/de/hysky/skyblocker/config/ConfigDatafixer.java +++ b/src/main/java/de/hysky/skyblocker/config/ConfigDatafixer.java @@ -257,7 +257,7 @@ public class ConfigDatafixer { dungeonsNew.addProperty("playerSecretsTracker", JsonHelper.getBoolean(dungeonsOld, "playerSecretsTracker").orElse(false)); dungeonsNew.add("starredMobGlow", dungeonsOld.get("starredMobGlow")); dungeonsNew.addProperty("starredMobBoundingBoxes", JsonHelper.getBoolean(dungeonsOld, "starredMobBoundingBoxes").orElse(true)); - dungeonsNew.add("allowDroppingProtectedItems", dungeonsOld.get("allowDroppingProtectedItems")); + dungeonsNew.addProperty("allowDroppingProtectedItems", JsonHelper.getBoolean(dungeonsOld, "allowDroppingProtectedItems").orElse(false)); }; DataFixer map = (oldFmt, newFmt) -> { @@ -281,10 +281,10 @@ public class ConfigDatafixer { solverConfig.add("creeperSolver", dungeonsOld.get("creeperSolver")); solverConfig.add("solveTrivia", dungeonsOld.get("solveTrivia")); solverConfig.add("solveTicTacToe", dungeonsOld.get("solveTicTacToe")); - solverConfig.add("solveWaterboard", dungeonsOld.get("solveWaterboard")); - solverConfig.add("solveBoulder", dungeonsOld.get("solveBoulder")); - solverConfig.add("solveIceFill", dungeonsOld.get("solveIceFill")); - solverConfig.add("solveSilverfish", dungeonsOld.get("solveSilverfish")); + solverConfig.addProperty("solveWaterboard", JsonHelper.getBoolean(dungeonsOld, "solveWaterboard").orElse(true)); + solverConfig.addProperty("solveBoulder", JsonHelper.getBoolean(dungeonsOld, "solveBoulder").orElse(true)); + solverConfig.addProperty("solveIceFill", JsonHelper.getBoolean(dungeonsOld, "solveIceFill").orElse(true)); + solverConfig.addProperty("solveSilverfish", JsonHelper.getBoolean(dungeonsOld, "solveSilverfish").orElse(true)); newFmt.getAsJsonObject("dungeons").add("puzzleSolvers", solverConfig); }; @@ -377,7 +377,6 @@ public class ConfigDatafixer { return new DataFixer[] { mainFixer }; } - private static DataFixer[] getQuickNavDataFixerRules() { DataFixer toggle = (oldFmt, newFmt) -> newFmt.getAsJsonObject("quickNav").add("enableQuickNav", oldFmt.getAsJsonObject("quickNav").get("enableQuickNav")); DataFixer buttonFixer = (oldFmt, newFmt) -> { -- cgit