Added Multi Leader Feature #196

zOnlyKroks · 2021-04-11T15:46:33+02:00

zOnlyKroks hat

2021-04-11 15:46:33 +02:00

kommentiert

Mitglied

Keine Beschreibung angegeben.

zOnlyKroks hat 1 Commit 2021-04-11 15:46:35 +02:00 hinzugefügt

Added Multi Leader Feature f3a6269331

YoyoNow hat 2021-04-11 16:01:25 +02:00 Änderungen angefragt

YoyoNow hat einen Kommentar hinterlassen

Besitzer

Ist dieser Branch schonmal getestet?

src/de/steamwar/bungeecore/commands/TeamCommand.java Veraltet

						
				@ -419,3 +430,1 @@

				                sb.append(playerName(SteamwarUser.get(i))).append(" ");

				            }

				            Message.sendPrefixless("TEAM_INFO_MEMBER", player, sb.toString());

				        List<SteamwarUser> users = team.getMembers().stream().map(userId -> SteamwarUser.get(userId)).collect(Collectors.toList());

YoyoNow hat

2021-04-11 15:59:50 +02:00

kommentiert

Besitzer

Warum wurde hier beim map nicht auch 'SteamwarUser::get' verwendet, was oben schon verwendet wurde?

zOnlyKroks markierte diese Unterhaltung als gelöst

src/de/steamwar/bungeecore/commands/TeamCommand.java Veraltet

						
				@ -432,1 +448,4 @@

				    private String getMemberList(List<SteamwarUser> users, boolean leaders) {

				        StringBuilder sb = new StringBuilder();

				        for(SteamwarUser user  : users) {

YoyoNow hat

2021-04-11 16:00:21 +02:00

kommentiert

Besitzer

Hier ist ein doppelspace nach 'user'

zOnlyKroks markierte diese Unterhaltung als gelöst

zOnlyKroks hat

2021-04-12 12:30:11 +02:00

kommentiert

Autor

Mitglied

Habe nicht die möglichkeit das Ding zu testen :D

zOnlyKroks hat 1 Commit 2021-04-12 12:32:12 +02:00 hinzugefügt

Fixed Formating of Code bc39e6f307

zOnlyKroks hat 1 Commit 2021-04-12 12:33:15 +02:00 hinzugefügt

Used Lambda 2bbcb283b4

zOnlyKroks hat ein Review von YoyoNow 2021-04-12 12:33:24 +02:00 angefragt

zOnlyKroks hat

2021-04-12 14:20:34 +02:00

kommentiert

Autor

Mitglied

Habe die Fehler jetzt "behoben".

Lixfel hat 2021-04-12 17:48:18 +02:00 Änderungen angefragt

src/de/steamwar/bungeecore/commands/TeamCommand.java Veraltet

						
				@ -297,3 +315,3 @@

				        }

				        if (team.getTeamLeader() == target.getId()) {

				        if (user.isLeader()) {

Lixfel hat

2021-04-12 17:44:22 +02:00

kommentiert

Besitzer

Hier muss nach dem target geprüft werden (glaube ich). Bitte nochmal gedanklich durchgehen.

zOnlyKroks hat

2021-04-13 07:23:12 +02:00

kommentiert

Autor

Mitglied

sollte gefixt sein

zOnlyKroks markierte diese Unterhaltung als gelöst

zOnlyKroks hat 1 Commit 2021-04-13 07:23:02 +02:00 hinzugefügt

Fixed team remove command dc49ddff4d

zOnlyKroks hat ein Review von Lixfel 2021-04-13 07:23:27 +02:00 angefragt

Lixfel hat 1 Commit 2021-04-13 07:25:35 +02:00 hinzugefügt

Fix leader f59350bab7

Signed-off-by: Lixfel <agga-games@gmx.de>

Lixfel hat die Änderungen 2021-04-13 07:25:40 +02:00 genehmigt

Lixfel hat dieses Issue

2021-04-13 07:26:24 +02:00

aus einem Commit referenziert

Merge pull request 'Added Multi Leader Feature' (#196) from Multi_Leader into master

Lixfel hat Commit e4106df682 in master 2021-04-13 07:26:24 +02:00 manuell gemerged

Lixfel löschte die Branch Multi_Leader

2021-04-13 07:27:57 +02:00

Dieses Repo ist archiviert. Du kannst Pull-Requests nicht kommentieren.

Keine Reviewer

Kein Label

Bug

Codeverbesserung

Einsteiger Freundlich

Überprüfung notwendig

Verbesserung

Zu Beobachten

Kein Meilenstein

Niemand zuständig

3 Beteiligte

Fällig am

Das Fälligkeitsdatum ist ungültig oder außerhalb des zulässigen Bereichs. Bitte verwende das Format „jjjj-mm-tt“.

Kein Fälligkeitsdatum gesetzt.

Abhängigkeiten

Keine Abhängigkeiten gesetzt.

Referenz: SteamWar/BungeeCore#196