Fix SP-Devtime world loading crash due to missing server configs (#8757)

This commit is contained in:
Marc Hermans authored and GitHub committed 2022-06-17 11:38:41 -07:00
1 parent 2ae640b530
commit fa84c22662
3 files changed
+2 -8

No files matched your search

@@ -29,7 +29,6 @@ public class ForgeConfig {
public final DoubleValue zombieBabyChance;
public final BooleanValue treatEmptyTagsAsAir;
public final BooleanValue skipEmptyShapelessCheck;
public final BooleanValue fixAdvancementLoading;
@@ -74,11 +73,6 @@ public class ForgeConfig {
.translation("forge.configgui.treatEmptyTagsAsAir")
.define("treatEmptyTagsAsAir", false);
skipEmptyShapelessCheck = builder
.comment("Skip checking if an ingredient is empty during shapeless recipe deserialization to prevent complex ingredients from caching tags too early.")
.translation("forge.configgui.skipEmptyShapelessCheck")
.define("skipEmptyShapelessCheck", true);
fixAdvancementLoading = builder
.comment("Fix advancement loading to use a proper topological sort. This may have visibility side-effects and can thus be turned off if needed for data-pack compatibility.")
.translation("forge.configgui.fixAdvancementLoading")
@@ -846,7 +846,7 @@ public class ForgeConfigSpec extends UnmodifiableConfigWrapper<UnmodifiableConfi
{
Preconditions.checkNotNull(spec, "Cannot get config value before spec is built");
// TODO: Remove this dev-time check so this errors out on both production and dev
// This is dev-time-only in 1.18.2, to avoid breaking already published mods while forcing devs to fix their errors
// This is dev-time-only in 1.19.x, to avoid breaking already published mods while forcing devs to fix their errors
if (!FMLEnvironment.production)
{
// When the above if-check is removed, change message to "Cannot get config value before config is loaded"