chore: removed screenposition, removed static_method from click handling

Author: MichaelHillcox

common/src/main/java/dev/ftb/mods/ftblibrary/math/ScreenPosition.java

@@ -504,17 +504,6 @@ public final void openGui() {
         getMinecraft().setScreen(new ScreenWrapper(this));
     }
 
-    /**
-     * This is poorly named, so let's move over to getWindow()
-     *
-     * @deprecated use {@link #getWindow()} instead
-     */
-    @Deprecated
-    @Override
-    public final Window getScreen() {
-        return getMinecraft().getWindow();
-    }
-
     @Override
     public Window getWindow() {
         return getMinecraft().getWindow();

common/src/main/java/dev/ftb/mods/ftblibrary/ui/BaseScreen.java

@@ -17,18 +17,15 @@
 import net.minecraft.util.Util;
 import org.jetbrains.annotations.Nullable;
 
-import java.lang.invoke.MethodHandle;
-import java.lang.invoke.MethodHandles;
-import java.lang.invoke.MethodType;
 import java.net.URI;
-import java.util.*;
+import java.util.ArrayList;
+import java.util.List;
+import java.util.Objects;
 import java.util.function.BooleanSupplier;
 
 public class ClientUtils {
     public static final BooleanSupplier IS_CLIENT_OP = () -> Minecraft.getInstance().player != null && Minecraft.getInstance().player.permissions().hasPermission(Permissions.COMMANDS_GAMEMASTER);
     public static final List<Runnable> RUN_LATER = new ArrayList<>();
-    private static final MethodType EMPTY_METHOD_TYPE = MethodType.methodType(void.class);
-    private static final HashMap<String, Optional<MethodHandle>> staticMethodCache = new HashMap<>();
 
     public static void execClientCommand(String command, boolean printChat) {
         if (!command.isEmpty() && Minecraft.getInstance().player != null) {
@@ -103,27 +100,6 @@ public static boolean handleClick(String scheme, String path) {
                 execClientCommand(path, false);
                 return true;
             }
-            case "static_method" -> {
-                return staticMethodCache.computeIfAbsent(path, k -> {
-                    var s = path.split(":", 2);
-                    try {
-                        Class<?> cls = Class.forName(s[0]);
-                        return Optional.ofNullable(MethodHandles.publicLookup().findStatic(cls, s[1], EMPTY_METHOD_TYPE));
-                    } catch (ClassNotFoundException | NoSuchMethodException | IllegalAccessException |
-                             ArrayIndexOutOfBoundsException ex) {
-                        logHandleClickFailure(scheme, path, ex);
-                        return Optional.empty();
-                    }
-                }).map(handle -> {
-                    try {
-                        handle.invoke();
-                        return true;
-                    } catch (Throwable ex) {
-                        logHandleClickFailure(scheme, path, ex);
-                        return false;
-                    }
-                }).orElse(false);
-            }
             case "custom" -> {
                 return trySendCustomClickEvent(path);
             }