Check if a chunk is loaded while replacing. May fix crash 2 in #22

DaFuqs · Nov 4, 2021 · bdccb37 · bdccb37
1 parent fb31e52
commit bdccb37
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/src/main/java/de/dafuqs/lootcrates/worldgen/LootCratesWorldgenReplacer.java b/src/main/java/de/dafuqs/lootcrates/worldgen/LootCratesWorldgenReplacer.java
@@ -283,7 +283,7 @@ public static void tick(MinecraftServer server) {
             for (LootCrateReplacementPosition replacementPosition : list) {
                 try {
                     ServerWorld serverWorld = server.getWorld(replacementPosition.worldKey);
-                    if (serverWorld != null) {
+                    if (serverWorld != null && serverWorld.isChunkLoaded(replacementPosition.blockPos)) {
                         BlockEntity blockEntity;
                         try {
                             blockEntity = serverWorld.getBlockEntity(replacementPosition.blockPos);
@@ -339,6 +339,8 @@ public static void tick(MinecraftServer server) {
                                 }
                             }
                         }
+                    } else {
+                        LootCrates.LOGGER.error("[LootCrates] Error while replacing a container with loot table '" + replacementPosition.lootTable + "' in the world '" + replacementPosition.worldKey + "' at '" + replacementPosition.blockPos + "' ). The chunk at that pos is not loaded.");
                     }
                 } catch (Exception e) {
                     LootCrates.LOGGER.error("[LootCrates] Error while replacing a container with loot table '" + replacementPosition.lootTable + "' in the world '" + replacementPosition.worldKey + "' at '" + replacementPosition.blockPos + "' ) + " + e.getLocalizedMessage());