refactor: move via stuff out to a separate class

Avoids blowing things up when ViaVersion isn't present.

Signed-off-by: TTtie <me@tttie.cz>

Author: TTtie

…n/feature/ModernServerLinksPlatform.java …rverlinks/ModernServerLinksPlatform.javaplatform/platform-modern/src/main/java/dev/pgm/community/platform/modern/feature/ModernServerLinksPlatform.java renamed to platform/platform-modern/src/main/java/dev/pgm/community/platform/modern/feature/serverlinks/ModernServerLinksPlatform.java platform/platform-modern/src/main/java/dev/pgm/community/platform/modern/feature/ModernServerLinksPlatform.java renamed to platform/platform-modern/src/main/java/dev/pgm/community/platform/modern/feature/serverlinks/ModernServerLinksPlatform.java

@@ -1,4 +1,4 @@
-package dev.pgm.community.platform.modern.feature;
+package dev.pgm.community.platform.modern.feature.serverlinks;
 
 import static dev.pgm.community.util.Supports.Variant.PAPER;
 

platform/platform-sportpaper/src/main/java/dev/pgm/community/platform/sportpaper/features/serverlinks/SpServerLinksPlatform.java

@@ -0,0 +1,35 @@
+package dev.pgm.community.platform.sportpaper.features.serverlinks;
+
+import static dev.pgm.community.util.Supports.Variant.SPORTPAPER;
+
+import dev.pgm.community.serverlinks.ServerLinksFeature;
+import dev.pgm.community.serverlinks.types.ServerLink;
+import dev.pgm.community.util.Supports;
+import java.util.List;
+import org.bukkit.entity.Player;
+
+@Supports(SPORTPAPER)
+public class SpServerLinksPlatform implements ServerLinksFeature.ServerLinksPlatform {
+  private static final boolean HAS_VIA = hasVia();
+
+  private static boolean hasVia() {
+    try {
+      Class.forName("com.viaversion.viaversion.api.Via");
+      return true;
+    } catch (ClassNotFoundException e) {
+      return false;
+    }
+  }
+
+  @Override
+  public boolean isSupported() {
+    return HAS_VIA;
+  }
+
+  @Override
+  public void sendToPlayer(Player player, List<ServerLink> serverLinks) {
+    if (HAS_VIA) {
+      ViaServerLinks.sendToPlayer(player, serverLinks);
+    }
+  }
+}

…aper/features/SpServerLinksPlatform.java …features/serverlinks/ViaServerLinks.javaplatform/platform-sportpaper/src/main/java/dev/pgm/community/platform/sportpaper/features/SpServerLinksPlatform.java renamed to platform/platform-sportpaper/src/main/java/dev/pgm/community/platform/sportpaper/features/serverlinks/ViaServerLinks.java platform/platform-sportpaper/src/main/java/dev/pgm/community/platform/sportpaper/features/SpServerLinksPlatform.java renamed to platform/platform-sportpaper/src/main/java/dev/pgm/community/platform/sportpaper/features/serverlinks/ViaServerLinks.java

@@ -1,6 +1,4 @@
-package dev.pgm.community.platform.sportpaper.features;
-
-import static dev.pgm.community.util.Supports.Variant.SPORTPAPER;
+package dev.pgm.community.platform.sportpaper.features.serverlinks;
 
 import com.viaversion.nbt.tag.Tag;
 import com.viaversion.viaversion.api.Via;
@@ -12,49 +10,29 @@
 import com.viaversion.viaversion.api.type.Types;
 import com.viaversion.viaversion.libs.gson.JsonParser;
 import com.viaversion.viaversion.libs.mcstructs.text.utils.JsonNbtConverter;
-import dev.pgm.community.serverlinks.ServerLinksFeature;
 import dev.pgm.community.serverlinks.types.ServerLink;
-import dev.pgm.community.util.Supports;
 import java.util.List;
 import net.kyori.adventure.text.Component;
 import net.kyori.adventure.text.serializer.gson.GsonComponentSerializer;
 import org.bukkit.entity.Player;
 
-@Supports(SPORTPAPER)
-public class SpServerLinksPlatform implements ServerLinksFeature.ServerLinksPlatform {
-  private final Protocol<?, ?, ?, ?> serverLinkProtocol = findServerLinkProtocol();
-  private static final boolean hasVia = hasVia();
-
-  private static boolean hasVia() {
-    try {
-      Class.forName("com.viaversion.viaversion.api.Via");
-      return true;
-    } catch (ClassNotFoundException e) {
-      return false;
-    }
-  }
+public class ViaServerLinks {
+  private static final Protocol<?, ?, ?, ?> serverLinkProtocol = findServerLinkProtocol();
 
-  @Override
-  public boolean isSupported() {
-    return hasVia;
-  }
-
-  @Override
-  public void sendToPlayer(Player player, List<ServerLink> serverLinks) {
-    if (hasVia && Via.getAPI().isInjected(player.getUniqueId())) {
-      UserConnection userConnection = Via.getAPI().getConnection(player.getUniqueId());
-      if (userConnection != null
-          && userConnection
-              .getProtocolInfo()
-              .protocolVersion()
-              .newerThanOrEqualTo(ProtocolVersion.v1_21)) {
-        PacketWrapper serverLinksPacket = createPacket(userConnection, serverLinks);
-        serverLinksPacket.scheduleSend(serverLinkProtocol.getClass());
-      }
+  public static void sendToPlayer(Player player, List<ServerLink> serverLinks) {
+    if (!Via.getAPI().isInjected(player.getUniqueId())) return;
+    UserConnection userConnection = Via.getAPI().getConnection(player.getUniqueId());
+    if (userConnection != null
+        && userConnection
+            .getProtocolInfo()
+            .protocolVersion()
+            .newerThanOrEqualTo(ProtocolVersion.v1_21)) {
+      PacketWrapper serverLinksPacket = createPacket(userConnection, serverLinks);
+      serverLinksPacket.scheduleSend(serverLinkProtocol.getClass());
     }
   }
 
-  private PacketWrapper createPacket(UserConnection conn, List<ServerLink> links) {
+  private static PacketWrapper createPacket(UserConnection conn, List<ServerLink> links) {
     var packetTypes = serverLinkProtocol.getPacketTypesProvider().mappedClientboundPacketTypes();
     var packetType = packetTypes.get(State.PLAY).typeByName("SERVER_LINKS");
     PacketWrapper packet = PacketWrapper.create(packetType, conn);
@@ -73,12 +51,12 @@ private PacketWrapper createPacket(UserConnection conn, List<ServerLink> links)
     return packet;
   }
 
-  private Tag toViaTag(Component component) {
+  private static Tag toViaTag(Component component) {
     return JsonNbtConverter.toNbt(
         JsonParser.parseString(GsonComponentSerializer.gson().serialize(component)));
   }
 
-  private Protocol<?, ?, ?, ?> findServerLinkProtocol() {
+  private static Protocol<?, ?, ?, ?> findServerLinkProtocol() {
     return Via.getManager()
         .getProtocolManager()
         .getProtocol(/* to */ ProtocolVersion.v1_21, /* from */ ProtocolVersion.v1_20_5);