Revert "Move pre-transformation checks to SkyblockAddonsSetup"

biscuut · biscuut · commit f01dfe73ce44 · 2020-07-26T03:48:42.000-04:00
This reverts commit 4d41865
diff --git a/src/main/java/codes/biscuit/skyblockaddons/asm/hooks/GuiNewChatHook.java b/src/main/java/codes/biscuit/skyblockaddons/asm/hooks/GuiNewChatHook.java
@@ -1,7 +1,7 @@
 package codes.biscuit.skyblockaddons.asm.hooks;
 
 import codes.biscuit.skyblockaddons.SkyblockAddons;
-import codes.biscuit.skyblockaddons.tweaker.SkyblockAddonsSetup;
+import codes.biscuit.skyblockaddons.tweaker.SkyblockAddonsTransformer;
 import net.minecraft.client.Minecraft;
 import net.minecraft.command.ICommandSender;
 import net.minecraft.util.IChatComponent;
@@ -11,7 +11,7 @@ public class GuiNewChatHook {
     public static String getUnformattedText(IChatComponent iChatComponent) {
         SkyblockAddons main = SkyblockAddons.getInstance();
         ICommandSender player = Minecraft.getMinecraft().thePlayer;
-        if (main != null && SkyblockAddonsSetup.isDeobfuscatedEnvironment() || (player != null && player.getName().equals("Biscut"))) {
+        if (main != null && SkyblockAddonsTransformer.isDeobfuscated() || (player != null && player.getName().equals("Biscut"))) {
             return iChatComponent.getFormattedText(); // makes it easier for debugging
         }
         return iChatComponent.getUnformattedText();
diff --git a/src/main/java/codes/biscuit/skyblockaddons/asm/hooks/SoundManagerHook.java b/src/main/java/codes/biscuit/skyblockaddons/asm/hooks/SoundManagerHook.java
@@ -1,7 +1,7 @@
 package codes.biscuit.skyblockaddons.asm.hooks;
 
 import codes.biscuit.skyblockaddons.SkyblockAddons;
-import codes.biscuit.skyblockaddons.tweaker.SkyblockAddonsSetup;
+import codes.biscuit.skyblockaddons.tweaker.SkyblockAddonsTransformer;
 import net.minecraft.client.audio.ISound;
 import net.minecraft.client.audio.SoundCategory;
 import net.minecraft.client.audio.SoundManager;
@@ -19,7 +19,7 @@ public static float getNormalizedVolume(SoundManager soundManager, ISound sound,
         if (main != null && main.getUtils() != null && main.getUtils().isPlayingSound()) {
             return 1;
         } else {
-            if (SkyblockAddonsSetup.isUsingLabyModClient()) { // There are no access transformers in LabyMod.
+            if (SkyblockAddonsTransformer.isLabymodClient()) { // There are no access transformers in labymod.
                 try {
                     if (getNormalizedVolume == null) {
                         getNormalizedVolume = soundManager.getClass().getDeclaredMethod("a", ISound.class, SoundPoolEntry.class, SoundCategory.class);
diff --git a/src/main/java/codes/biscuit/skyblockaddons/asm/hooks/TileEntityEnderChestRendererHook.java b/src/main/java/codes/biscuit/skyblockaddons/asm/hooks/TileEntityEnderChestRendererHook.java
@@ -1,7 +1,7 @@
 package codes.biscuit.skyblockaddons.asm.hooks;
 
 import codes.biscuit.skyblockaddons.SkyblockAddons;
-import codes.biscuit.skyblockaddons.tweaker.SkyblockAddonsSetup;
+import codes.biscuit.skyblockaddons.tweaker.SkyblockAddonsTransformer;
 import codes.biscuit.skyblockaddons.core.Feature;
 import codes.biscuit.skyblockaddons.core.Location;
 import codes.biscuit.skyblockaddons.utils.ColorCode;
@@ -32,7 +32,7 @@ public static void bindTexture(TileEntityEnderChestRenderer tileEntityEnderChest
     }
 
     private static void bindRightTexture(TileEntityEnderChestRenderer tileEntityEnderChestRenderer, ResourceLocation resourceLocation) {
-        if (SkyblockAddonsSetup.isUsingLabyModClient()) { // There are no access transformers in LabyMod.
+        if (SkyblockAddonsTransformer.isLabymodClient()) { // There are no access transformers in labymod.
             try {
                 if (bindTexture == null) {
                     bindTexture = tileEntityEnderChestRenderer.getClass().getSuperclass().getDeclaredMethod("a", ResourceLocation.class);
diff --git a/src/main/java/codes/biscuit/skyblockaddons/asm/utils/TransformerClass.java b/src/main/java/codes/biscuit/skyblockaddons/asm/utils/TransformerClass.java
@@ -1,6 +1,6 @@
 package codes.biscuit.skyblockaddons.asm.utils;
 
-import codes.biscuit.skyblockaddons.tweaker.SkyblockAddonsSetup;
+import codes.biscuit.skyblockaddons.tweaker.SkyblockAddonsTransformer;
 
 public enum TransformerClass {
 
@@ -71,7 +71,7 @@ public enum TransformerClass {
         this.seargeClass = seargeClass;
         this.notchClass18 = notchClass18;
 
-        if (SkyblockAddonsSetup.isDeobfuscatedEnvironment() || !SkyblockAddonsSetup.isRuntimeDeobfuscationEnabled()) {
+        if (SkyblockAddonsTransformer.isDeobfuscated() || !SkyblockAddonsTransformer.isUsingNotchMappings()) {
             name = seargeClass;
         } else {
             name = notchClass18;
@@ -93,7 +93,7 @@ public String getName() {
     }
 
     public String getTransformerName() {
-        if (SkyblockAddonsSetup.isUsingLabyModClient()) return notchClass18;
+        if (SkyblockAddonsTransformer.isLabymodClient()) return notchClass18;
 
         return seargeClass.replaceAll("/", ".");
     }
diff --git a/src/main/java/codes/biscuit/skyblockaddons/asm/utils/TransformerField.java b/src/main/java/codes/biscuit/skyblockaddons/asm/utils/TransformerField.java
@@ -1,6 +1,6 @@
 package codes.biscuit.skyblockaddons.asm.utils;
 
-import codes.biscuit.skyblockaddons.tweaker.SkyblockAddonsSetup;
+import codes.biscuit.skyblockaddons.tweaker.SkyblockAddonsTransformer;
 import org.objectweb.asm.Opcodes;
 import org.objectweb.asm.tree.FieldInsnNode;
 
@@ -52,10 +52,10 @@ public enum TransformerField {
     TransformerField(String deobfName, String seargeName, String notchName18, String type) {
         this.type = type;
 
-        if (SkyblockAddonsSetup.isDeobfuscatedEnvironment()) {
+        if (SkyblockAddonsTransformer.isDeobfuscated()) {
             name = deobfName;
         } else {
-            if (SkyblockAddonsSetup.isRuntimeDeobfuscationEnabled()) {
+            if (SkyblockAddonsTransformer.isUsingNotchMappings()) {
                 name = notchName18;
             } else {
                 name = seargeName;
diff --git a/src/main/java/codes/biscuit/skyblockaddons/asm/utils/TransformerMethod.java b/src/main/java/codes/biscuit/skyblockaddons/asm/utils/TransformerMethod.java
@@ -1,6 +1,6 @@
 package codes.biscuit.skyblockaddons.asm.utils;
 
-import codes.biscuit.skyblockaddons.tweaker.SkyblockAddonsSetup;
+import codes.biscuit.skyblockaddons.tweaker.SkyblockAddonsTransformer;
 import org.objectweb.asm.Opcodes;
 import org.objectweb.asm.tree.MethodInsnNode;
 import org.objectweb.asm.tree.MethodNode;
@@ -158,11 +158,11 @@ public enum TransformerMethod {
     }
 
     TransformerMethod(String deobfMethod, String seargeMethod, String notchMethod18, String seargeDescription, String notchDescription, boolean ioException) {
-        if (SkyblockAddonsSetup.isDeobfuscatedEnvironment()) {
+        if (SkyblockAddonsTransformer.isDeobfuscated()) {
             name = deobfMethod;
             description = seargeDescription;
         } else {
-            if (SkyblockAddonsSetup.isRuntimeDeobfuscationEnabled()) {
+            if (SkyblockAddonsTransformer.isUsingNotchMappings()) {
                 name = notchMethod18;
                 description = notchDescription;
             } else {
diff --git a/src/main/java/codes/biscuit/skyblockaddons/features/tabtimers/TabEffectManager.java b/src/main/java/codes/biscuit/skyblockaddons/features/tabtimers/TabEffectManager.java
@@ -3,7 +3,7 @@
 import codes.biscuit.skyblockaddons.SkyblockAddons;
 import codes.biscuit.skyblockaddons.core.Feature;
 import codes.biscuit.skyblockaddons.gui.buttons.ButtonLocation;
-import codes.biscuit.skyblockaddons.tweaker.SkyblockAddonsSetup;
+import codes.biscuit.skyblockaddons.tweaker.SkyblockAddonsTransformer;
 import codes.biscuit.skyblockaddons.utils.RomanNumeralParser;
 import lombok.Getter;
 import net.minecraft.client.Minecraft;
@@ -128,7 +128,7 @@ public void updatePotionEffects() {
     private static IChatComponent getFooter() {
         GuiPlayerTabOverlay guiTab = Minecraft.getMinecraft().ingameGUI.getTabList();
 
-        if (SkyblockAddonsSetup.isUsingLabyModClient()) { // There are no access transformers in LabyMod.
+        if (SkyblockAddonsTransformer.isLabymodClient()) { // There are no access transformers in labymod.
             try {
                 if (footer == null) {
                     footer = guiTab.getClass().getDeclaredField("h");
diff --git a/src/main/java/codes/biscuit/skyblockaddons/tweaker/SkyblockAddonsLoadingPlugin.java b/src/main/java/codes/biscuit/skyblockaddons/tweaker/SkyblockAddonsLoadingPlugin.java
@@ -20,7 +20,7 @@ public String getModContainerClass() {
 
     @Override
     public String getSetupClass() {
-        return SkyblockAddonsSetup.class.getName();
+        return null;
     }
 
     @Override
diff --git a/src/main/java/codes/biscuit/skyblockaddons/tweaker/SkyblockAddonsSetup.java b/src/main/java/codes/biscuit/skyblockaddons/tweaker/SkyblockAddonsSetup.java
diff --git a/src/main/java/codes/biscuit/skyblockaddons/tweaker/SkyblockAddonsTransformer.java b/src/main/java/codes/biscuit/skyblockaddons/tweaker/SkyblockAddonsTransformer.java
@@ -15,10 +15,42 @@
 import java.io.File;
 import java.io.FileOutputStream;
 import java.io.IOException;
+import java.lang.reflect.Field;
 import java.util.Collection;
+import java.util.Map;
 
+@SuppressWarnings("unchecked")
 public class SkyblockAddonsTransformer implements IClassTransformer {
-    private final Logger logger = LogManager.getLogger("SkyblockAddons Transformer");
+
+    private static boolean LABYMOD_CLIENT;
+    private static boolean DEOBFUSCATED;
+
+    static {
+        DEOBFUSCATED = false;
+        boolean foundLaunchClass = false;
+        try {
+            // DEOBFUSCATED = (boolean)Launch.blackboard.get("fml.deobfuscatedEnvironment");
+            Class<?> launch = Class.forName("net.minecraft.launchwrapper.Launch");
+            Field blackboardField = launch.getField("blackboard");
+            Map<String,Object> blackboard = (Map<String, Object>) blackboardField.get(null);
+            DEOBFUSCATED = (boolean) blackboard.get("fml.deobfuscatedEnvironment");
+            foundLaunchClass = true;
+        } catch (ClassNotFoundException | NoSuchFieldException | IllegalAccessException ex) {
+            // If the class doesn't exist, its probably just obfuscated labymod client, so leave it false.
+        }
+
+        LABYMOD_CLIENT = false;
+        try {
+            Class.forName("net.labymod.api.LabyModAddon"); // Try to find a labymod class.
+            LABYMOD_CLIENT = !foundLaunchClass; // If the launch class is also found, they are probably using labymod for forge and not the client.
+        } catch (ClassNotFoundException ex) {
+            // They just aren't using labymod.
+        }
+    }
+
+    private static boolean USING_NOTCH_MAPPINGS = !DEOBFUSCATED;
+
+    private Logger logger = LogManager.getLogger("SkyblockAddons Transformer");
     private final Multimap<String, ITransformer> transformerMap = ArrayListMultimap.create();
 
     public SkyblockAddonsTransformer() {
@@ -112,4 +144,16 @@ private void outputBytecode(String transformedName, ClassWriter writer) {
             e.printStackTrace();
         }
     }
+
+    public static boolean isDeobfuscated() {
+        return DEOBFUSCATED;
+    }
+
+    public static boolean isLabymodClient() {
+        return LABYMOD_CLIENT;
+    }
+
+    public static boolean isUsingNotchMappings() {
+        return USING_NOTCH_MAPPINGS;
+    }
 }

Original file line number	Diff line number	Diff line change
`@@ -20,7 +20,7 @@ public String getModContainerClass() {`
`20`	`20`
`21`	`21`	`@Override`
`22`	`22`	`public String getSetupClass() {`
`23`		`- return SkyblockAddonsSetup.class.getName();`
	`23`	`+ return null;`
`24`	`24`	`}`
`25`	`25`
`26`	`26`	`@Override`