WIP: TS3-Verification #54

Lixfel · 2020-01-18T08:19:07+01:00

Lixfel hat

2020-01-18 08:19:07 +01:00

kommentiert

Besitzer

Closes #17

Yaruma3341 hat

2020-01-18 19:10:33 +01:00

kommentiert

Erstmaliger Mitwirkender

public static void createConnection(ProxiedPlayer player, SteamwarUser user, String ts3Id){
TS3.create(user.getId(), ts3Id);
TS3Handler.getToVerify().remove(user.getId());
BungeeCore.send(player, Config.CHAT_PREFIX + "§aDeine Teamspeak-Verknüpfung wurde erfolgreich hergestellt");
try {
Client client = api.getClientByUId(ts3Id);
api.addClientToServerGroup(Config.TS3_QUERY_GROUPID, client.getDatabaseId());
}catch(TS3CommandFailedException e){
//ignored
}
}

Aufgrund der ZU VIELEN ÄNDERUNGEN, wurden mir nicht alle Datein angezeigt, sodass ich nicht direkt kommentieren konnte wie gewohnt. Also: Du catchst die Exception einfach weg.
So wie ich das verstehe, ist der Source innerhalb des Try catch Blocks zum setzen der Servergruppe. Wenn der Spieler jetzt aber offline ist (dadurch wird ja laut dir nur die Exception gewrofen, weshalb du sie auch einfach ignorierst, weil das ja durchaus passieren kann), dann wird der Source ja wohl nicht mehr fertig ausgeführt. Somit wird dem Spieler gesagt, er habe seinen TS3 Account mit MC verknüpft, obwohl das eventuell gar nicht der Fall ist und wir bekommen das aufgrund des Ignorierens der Exception auch nicht mit. Daher:
Das removen des Users aus der ToVerify Liste in den Try Catch verschieben, genauso wie die Nachricht. Das sollte dann entsprechend auch unter das Setzen der Gruppe, sodass das zu 100% durch ist, sonst machts ja keinen Sinn. In den Catch Block sollte dann eine Nachricht rein, dass beim Verknüpfen ein Problem aufgetreten ist.

public static void createConnection(ProxiedPlayer player, SteamwarUser user, String ts3Id){ TS3.create(user.getId(), ts3Id); TS3Handler.getToVerify().remove(user.getId()); BungeeCore.send(player, Config.CHAT_PREFIX + "§aDeine Teamspeak-Verknüpfung wurde erfolgreich hergestellt"); try { Client client = api.getClientByUId(ts3Id); api.addClientToServerGroup(Config.TS3_QUERY_GROUPID, client.getDatabaseId()); }catch(TS3CommandFailedException e){ //ignored } } Aufgrund der ZU VIELEN ÄNDERUNGEN, wurden mir nicht alle Datein angezeigt, sodass ich nicht direkt kommentieren konnte wie gewohnt. Also: Du catchst die Exception einfach weg. So wie ich das verstehe, ist der Source innerhalb des Try catch Blocks zum setzen der Servergruppe. Wenn der Spieler jetzt aber offline ist (dadurch wird ja laut dir nur die Exception gewrofen, weshalb du sie auch einfach ignorierst, weil das ja durchaus passieren kann), dann wird der Source ja wohl nicht mehr fertig ausgeführt. Somit wird dem Spieler gesagt, er habe seinen TS3 Account mit MC verknüpft, obwohl das eventuell gar nicht der Fall ist und wir bekommen das aufgrund des Ignorierens der Exception auch nicht mit. Daher: Das removen des Users aus der ToVerify Liste in den Try Catch verschieben, genauso wie die Nachricht. Das sollte dann entsprechend auch unter das Setzen der Gruppe, sodass das zu 100% durch ist, sonst machts ja keinen Sinn. In den Catch Block sollte dann eine Nachricht rein, dass beim Verknüpfen ein Problem aufgetreten ist.

Lixfel hat

2020-01-18 21:14:06 +01:00

kommentiert

Autor

Besitzer

Nein Yaruma, verknüpft ist der Acc schon (es gibt eine Zuordnung TS3-Account <-> Minecraft-Konto). Allerdings kann dann eben zu dem Zeitpunkt die TS3-Gruppe nicht gesetzt werden. Da muss ich dann aber eh noch mit H3MI Rücksprache halten, was dann wie Automaitsiert über den Bot abläuft (z.B. Gruppenzuteilung automatisch bei jedem TS-Join, automatische Teamchannelerstellung)

Lixfel hat den Titel von ~~TS3-Verification~~ zu WIP: TS3-Verification 2020-01-23 20:35:15 +01:00 geändert

AdmiralSeekrank hat

2020-03-04 10:17:45 +01:00

kommentiert

Besitzer

Wenn wir kein TS 3 bot zu Verifikation haben ist das pol request noch von nöten?

AdmiralSeekrank hat diesen Pull-Request

2020-03-04 10:17:45 +01:00

geschlossen

Lixfel hat diesen Pull-Request

2020-03-04 17:46:58 +01:00

wieder geöffnet