SteamWar/BungeeCore
Archiviert
13
2

Add Team showcase in tablist #438

Zusammengeführt
Lixfel hat 5 Commits von TeamTablist nach master 2022-10-24 17:44:16 +02:00 zusammengeführt
Besitzer
Keine Beschreibung angegeben.
YoyoNow hat 1 Commit 2022-10-24 17:16:46 +02:00 hinzugefügt
Add Team showcase in tablist
Alle Prüfungen waren erfolgreich
SteamWarCI Build successful
70a8d8e3ed
YoyoNow hat 1 Commit 2022-10-24 17:17:36 +02:00 hinzugefügt
Merge branch 'master' into TeamTablist
Alle Prüfungen waren erfolgreich
SteamWarCI Build successful
3606e75ab6
Lixfel hat 2022-10-24 17:22:28 +02:00 Änderungen angefragt
Lixfel hat einen Kommentar hinterlassen
Besitzer

Musst du glaube noch für TablistBuild implementieren.

Musst du glaube noch für TablistBuild implementieren.
@ -77,0 +79,4 @@
players.stream().map(item -> {
ProxiedPlayer p = BungeeCord.getInstance().getPlayer(item.getUuid());
if (p == null) return item;
if (SteamwarUser.get(p).getTeam() != team) return item;
Besitzer

Statt sich erst den proxiedplayer und dann davon den SWUser zu holen solltest du wenn direkt den SWUser holen (mit der UUID)

Statt sich erst den proxiedplayer und dann davon den SWUser zu holen solltest du wenn direkt den SWUser holen (mit der UUID)
Autor
Besitzer

Stimmt, danke für die Info, das vergesse ich irgendwie immer.

Stimmt, danke für die Info, das vergesse ich irgendwie immer.
Autor
Besitzer

Aber ich brauche den player um das Item zu erzeugen.

Aber ich brauche den player um das Item zu erzeugen.
Besitzer

SWUser.get(ProxiedPlayer) ist highly deprecated. Nutz trotzdem die uuid.

SWUser.get(ProxiedPlayer) ist highly deprecated. Nutz trotzdem die uuid.
Autor
Besitzer

So dann jetzt richtig oder?

So dann jetzt richtig oder?
Lixfel markierte diese Unterhaltung als gelöst
YoyoNow hat 1 Commit 2022-10-24 17:27:23 +02:00 hinzugefügt
Add Team showcase in tablist
Alle Prüfungen waren erfolgreich
SteamWarCI Build successful
f2a8cb064e
YoyoNow hat 1 Commit 2022-10-24 17:30:06 +02:00 hinzugefügt
Add Team showcase in tablist
Alle Prüfungen waren erfolgreich
SteamWarCI Build successful
fe4280f9e3
Lixfel hat 2022-10-24 17:40:14 +02:00 Änderungen angefragt
Lixfel hat einen Kommentar hinterlassen
Besitzer

Jetzt ist natürlich noch die Code-Duplication ungeil, könntest du das in eine static TablistServer-Methode auslagern?

Jetzt ist natürlich noch die Code-Duplication ungeil, könntest du das in eine static TablistServer-Methode auslagern?
@ -64,3 +66,2 @@
for(ServerInfo info : servers) {
(server == info ? direct : tablist).addAll(players.get(info));
int team = SteamwarUser.get(player).getTeam();
Besitzer

.

.
Lixfel markierte diese Unterhaltung als gelöst
@ -74,3 +76,3 @@
}
items.addAll(players);
int team = SteamwarUser.get(player).getTeam();
Besitzer

hier auch noch das issue.

hier auch noch das issue.
Lixfel markierte diese Unterhaltung als gelöst
YoyoNow hat 1 Commit 2022-10-24 17:43:00 +02:00 hinzugefügt
Add Team showcase in tablist
Alle Prüfungen waren erfolgreich
SteamWarCI Build successful
79bd9598c7
Lixfel hat die Änderungen 2022-10-24 17:44:04 +02:00 genehmigt
Lixfel hat Commit 67a7b60bdc in master 2022-10-24 17:44:16 +02:00 gemerged
Lixfel löschte die Branch TeamTablist 2022-10-24 17:44:23 +02:00
Dieses Repo ist archiviert. Du kannst Pull-Requests nicht kommentieren.
Keine Beschreibung angegeben.