From da2ef99c8f9e07675f15491153a7717f0a4c553c Mon Sep 17 00:00:00 2001
From: Estecka <estecka@galacsys.com>
Date: Sat, 2 Mar 2024 22:28:49 +0100
Subject: [PATCH] Fix Cloth class being searched for by mixins (#8)

---
 changelog.md                                    |  6 ++++++
 gradle.properties                               |  2 +-
 .../tk/estecka/packrulemenus/PackRuleMenus.java | 17 ++++++++++++++++-
 .../packrulemenus/mixin/OptionScreenMixin.java  | 15 ++-------------
 4 files changed, 25 insertions(+), 15 deletions(-)

diff --git a/changelog.md b/changelog.md
index c67c8c6..8936e33 100644
--- a/changelog.md
+++ b/changelog.md
@@ -10,3 +10,9 @@
 ## 1.1
 - Clicking "proceed" on the feature flag warning screen will automatically exit the world, back to the title screen. A reload is no longer triggered.
 - "Proceed" on the warning screens can no longer be clicked without ticking the checkbox.
+
+## 1.2
+### 1.2.0
+- Integration for Cloth Gamerules screen.
+### 1.2.1
+- Fix Mixin searching for ClotGamerules classes when not installed.
diff --git a/gradle.properties b/gradle.properties
index c05a1b6..c99f441 100644
--- a/gradle.properties
+++ b/gradle.properties
@@ -12,7 +12,7 @@ loader_version=0.15.7
 fabric_version=0.87.2+1.19.4
 
 # Mod Properties
-mod_version=1.2.0
+mod_version=1.2.1
 maven_group=tk.estecka.packrulemenus
 archives_base_name=packrule-menus
 
diff --git a/src/main/java/tk/estecka/packrulemenus/PackRuleMenus.java b/src/main/java/tk/estecka/packrulemenus/PackRuleMenus.java
index 4e988e0..d3d0235 100644
--- a/src/main/java/tk/estecka/packrulemenus/PackRuleMenus.java
+++ b/src/main/java/tk/estecka/packrulemenus/PackRuleMenus.java
@@ -1,9 +1,24 @@
 package tk.estecka.packrulemenus;
 
+import java.util.Optional;
+import java.util.function.Consumer;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
+import net.fabricmc.loader.api.FabricLoader;
+import net.minecraft.client.MinecraftClient;
+import net.minecraft.client.gui.screen.Screen;
+import net.minecraft.client.gui.screen.world.EditGameRulesScreen;
+import net.minecraft.world.GameRules;
+import tk.estecka.clothgamerules.api.ClothGamerulesScreenFactory;
 
 public class PackRuleMenus
 {
 	static public final Logger LOGGER = LoggerFactory.getLogger("Pack-Rule Menus");
-}
\ No newline at end of file
+
+	static public Screen CreateGameruleScreen(Screen parent, GameRules rules, Consumer<Optional<GameRules>> saveConsumer){
+		if (FabricLoader.getInstance().isModLoaded("cloth-gamerules"))
+			return ClothGamerulesScreenFactory.CreateScreen(parent, rules, saveConsumer);
+		else
+			return new EditGameRulesScreen(rules, saveConsumer.andThen( __->MinecraftClient.getInstance().setScreen(parent) ));
+	}
+}
diff --git a/src/main/java/tk/estecka/packrulemenus/mixin/OptionScreenMixin.java b/src/main/java/tk/estecka/packrulemenus/mixin/OptionScreenMixin.java
index 0d41186..2f0dd49 100644
--- a/src/main/java/tk/estecka/packrulemenus/mixin/OptionScreenMixin.java
+++ b/src/main/java/tk/estecka/packrulemenus/mixin/OptionScreenMixin.java
@@ -1,8 +1,6 @@
 package tk.estecka.packrulemenus.mixin;
 
 import java.util.Collection;
-import java.util.Optional;
-import java.util.function.Consumer;
 import java.util.function.Supplier;
 import org.spongepowered.asm.mixin.Mixin;
 import org.spongepowered.asm.mixin.Shadow;
@@ -13,13 +11,11 @@
 import org.spongepowered.asm.mixin.injection.callback.CallbackInfo;
 import com.llamalad7.mixinextras.sugar.Local;
 import it.unimi.dsi.fastutil.booleans.BooleanConsumer;
-import net.fabricmc.loader.api.FabricLoader;
 import net.minecraft.client.gui.screen.MessageScreen;
 import net.minecraft.client.gui.screen.Screen;
 import net.minecraft.client.gui.screen.TitleScreen;
 import net.minecraft.client.gui.screen.option.OptionsScreen;
 import net.minecraft.client.gui.screen.pack.PackScreen;
-import net.minecraft.client.gui.screen.world.EditGameRulesScreen;
 import net.minecraft.client.gui.widget.ButtonWidget;
 import net.minecraft.client.gui.widget.GridWidget;
 import net.minecraft.resource.DataConfiguration;
@@ -32,7 +28,6 @@
 import net.minecraft.util.Identifier;
 import net.minecraft.util.WorldSavePath;
 import net.minecraft.world.GameRules;
-import tk.estecka.clothgamerules.api.ClothGamerulesScreenFactory;
 import tk.estecka.packrulemenus.GenericWarningScreen;
 import tk.estecka.packrulemenus.PackRuleMenus;
 
@@ -58,7 +53,8 @@ abstract public class OptionScreenMixin
 			final GameRules worldRules = server.getOverworld().getGameRules();
 			adder.add(createButton(
 				Text.translatable("selectWorld.gameRules"),
-				() -> CreateGameruleScreen(
+				() -> PackRuleMenus.CreateGameruleScreen(
+					this,
 					worldRules.copy(),
 					optRules -> optRules.ifPresent(r -> worldRules.setAllValues(r, server))
 				)
@@ -76,13 +72,6 @@ abstract public class OptionScreenMixin
 			));
 		}
 	}
-	
-	private Screen CreateGameruleScreen(GameRules rules, Consumer<Optional<GameRules>> saveConsumer){
-		if (FabricLoader.getInstance().isModLoaded("cloth-gamerules"))
-			return ClothGamerulesScreenFactory.CreateScreen(this, rules, saveConsumer);
-		else
-			return new EditGameRulesScreen(rules, saveConsumer.andThen( __->RevertScreen() ));
-	}
 
 	private void	RevertScreen(){
 		client.setScreen((OptionsScreen)(Object)this);