SteamWar/BungeeCore
Archiviert
13
2

Remove Redundant Message when switching Servers #337

Zusammengeführt
Lixfel hat 4 Commits von FabricModSender nach master 2022-04-05 18:39:03 +02:00 zusammengeführt
2 geänderte Dateien mit 6 neuen und 0 gelöschten Zeilen
Nur Änderungen aus Commit cb52e0bc38 werden angezeigt - Alle Commits anzeigen

Datei anzeigen

@ -42,6 +42,8 @@ public class Fabric extends BasicListener {
private final Set<String> neededMods = new HashSet<>();
public static List<ProxiedPlayer> checkedPlayers = new LinkedList<>();
zOnlyKroks markierte diese Unterhaltung als gelöst Veraltet
Veraltet
Review

final, und bitte keine LinkedList! WTF! Ein (Hash)Set sollte das sein.

final, und bitte keine LinkedList! WTF! Ein (Hash)Set sollte das sein.
{
neededMods.add("java");
neededMods.add("minecraft");
@ -61,6 +63,8 @@ public class Fabric extends BasicListener {
ProxiedPlayer player = (ProxiedPlayer) e.getSender();
SteamwarUser user = SteamwarUser.get(player.getUniqueId());
if(checkedPlayers.contains(player)) return;
zOnlyKroks markierte diese Unterhaltung als gelöst Veraltet
Veraltet
Review

man kann ja schon noch prüfen, nur dann halt keine Nachricht mehr.

man kann ja schon noch prüfen, nur dann halt keine Nachricht mehr.
List<Mod> mods = new LinkedList<>();
byte[] data = e.getData();
@ -108,6 +112,7 @@ public class Fabric extends BasicListener {
}
if(Utils.handleMods(player,mods)) {
checkedPlayers.add(player);
Storage.fabricPlayers.remove(player);
Message.send("MODIFICATION_CHECK_SUCCESS", player);
}

Datei anzeigen

@ -58,6 +58,7 @@ public class ModLoaderBlocker extends BasicListener {
@EventHandler
public void onDisconnect(PlayerDisconnectEvent e){
Fabric.checkedPlayers.remove(e.getPlayer());
Storage.fabricPlayers.remove(e.getPlayer());
}