SteamWar/FightSystem
Archiviert
13
1

Changed Fight Scoreboard to SWScoreboard #194

Manuell gemergt
Lixfel hat 7 Commits von scoreboard-reimplement nach master 2020-11-17 09:29:41 +01:00 zusammengeführt
Besitzer

Closes: #189

Closes: #189
Lixfel hat 2020-10-10 20:40:01 +02:00 Änderungen angefragt
@ -57,3 +46,1 @@
//noinspection deprecation
scoreboard.registerNewObjective("AAA", "BBB");
objective.setDisplaySlot(DisplaySlot.SIDEBAR);
Bukkit.getOnlinePlayers().forEach(player -> SWScoreboard.removeScoreboard(player));
Besitzer

Man sollte das Scoreboard nicht regelmäßig bei jedem Spieler neu setzen, das macht das SWScoreboard schon von selbst.

Man sollte das Scoreboard nicht regelmäßig bei jedem Spieler neu setzen, das macht das SWScoreboard schon von selbst.
Lixfel hat 2020-10-15 09:24:45 +02:00 Änderungen angefragt
@ -123,0 +125,4 @@
if(index > 2)
return;
index = 0;
FightTeam team = getIndexDisplay();
Besitzer

Das schaut mir katastrophal verbuggt aus, da dürfte nie das korrekte Scoreboard angezeigt werden!

Das schaut mir katastrophal verbuggt aus, da dürfte nie das korrekte Scoreboard angezeigt werden!
Besitzer

Es wird wohl immer einmal kein Scoreboard angezeigt werden weil Index = 0 bedeutet die getIndexDisplay() Methode gibt null zurück. Dies sollte behoben werden.

Es wird wohl immer einmal kein Scoreboard angezeigt werden weil `Index = 0` bedeutet die `getIndexDisplay()` Methode gibt `null` zurück. Dies sollte behoben werden.
Besitzer

Der Fehler ist nun behoben, bzw um 13:00 am gleichen Tag wurde dieser Fehler noch behoben, deswegen habe ich den Pull-Request jetzt nach einer guten Zeit drauf gucken genehmigt

Der Fehler ist nun behoben, bzw um 13:00 am gleichen Tag wurde dieser Fehler noch behoben, deswegen habe ich den Pull-Request jetzt nach einer guten Zeit drauf gucken genehmigt
YoyoNow hat die Änderungen 2020-10-25 10:14:24 +01:00 genehmigt
Besitzer

Wurde dieser Branch (erfolgreich) getestet?

Wurde dieser Branch (erfolgreich) getestet?
Besitzer

@Chaoscaot Bitte Merge-Konflikt beheben und einmal in einer Arena testen!

@Chaoscaot Bitte Merge-Konflikt beheben und einmal in einer Arena testen!
Autor
Besitzer

Tested!

Tested!
Lixfel hat 2020-11-17 07:29:36 +01:00 Änderungen angefragt
@ -44,3 +46,2 @@
private static final Scoreboard scoreboard = Objects.requireNonNull(Bukkit.getScoreboardManager()).getMainScoreboard();
private static final Objective objective;
private static int index = 0;
private static ScoreBoardObjective objective;
Besitzer

Fände es besser, direkt titel und scores zu speichern (siehe unten)

Fände es besser, direkt titel und scores zu speichern (siehe unten)
@ -52,2 +49,2 @@
else
objective = scoreboard.getObjective("AAA");
public static void init(){
Bukkit.getPluginManager().registerEvents(new FightScoreboard(), FightSystem.getPlugin());
Besitzer

Listener sollen im Ordner Listener sein, in diesem Fall solltest du wsl. einfach im ConnectionListener im JonListener einen Aufruf hierher machen.

Listener sollen im Ordner Listener sein, in diesem Fall solltest du wsl. einfach im ConnectionListener im JonListener einen Aufruf hierher machen.
@ -66,2 +54,2 @@
else
generalScoreboard();
public static Scoreboard getScoreboard() {
return Bukkit.getScoreboardManager().getMainScoreboard();
Besitzer

Hast du darauf geachtet, dass die Teammitglieder immer noch keinen Nametag über sich haben & in der passenden Farbe rechts angezeigt werden? Aufgrund der derzeitigen SWScoreboard-Implementierung (welche ich überarbeiten möchte) habe ich da nämlich so meine Befürchtungen.

Hast du darauf geachtet, dass die Teammitglieder immer noch keinen Nametag über sich haben & in der passenden Farbe rechts angezeigt werden? Aufgrund der derzeitigen SWScoreboard-Implementierung (welche ich überarbeiten möchte) habe ich da nämlich so meine Befürchtungen.
Autor
Besitzer

Die Nametags werden gehided und der Name hat auch die richtige Farbe in der Sidebar.

Die Nametags werden gehided und der Name hat auch die richtige Farbe in der Sidebar.
@ -122,3 +121,2 @@
objective.getScore(key).setScore(value);
if(Config.recording())
RecordSystem.scoreboardData(key, value);
objective.scores.forEach(RecordSystem::scoreboardData);
Besitzer

Du sendest jeden Score doppelt raus.

Du sendest jeden Score doppelt raus.
@ -134,3 +131,4 @@
return null;
}
private static void nextIndexDisplay() {
Besitzer

Die Funktion nextIndexDisplay sollte sich mit der Funktion getIndexDisplay zusammenführen lassen.

Die Funktion nextIndexDisplay sollte sich mit der Funktion getIndexDisplay zusammenführen lassen.
@ -137,0 +142,4 @@
objective = generalScoreboard();
}
private static class ScoreBoardObjective {
Besitzer

Die Extraklasse als Container für Titel und Scores finde ich nicht wirklich nötig & fügt nur unnötig Komplexität hinzu

Die Extraklasse als Container für Titel und Scores finde ich nicht wirklich nötig & fügt nur unnötig Komplexität hinzu
@ -137,0 +159,4 @@
public void addScore(String string, int i) {
if(Config.recording())
RecordSystem.scoreboardData(string, i);
Besitzer

Hier zweite Scoresendung.

Hier zweite Scoresendung.
YoyoNow hat die Änderungen 2020-11-17 09:27:03 +01:00 genehmigt
Lixfel hat diesen Pull-Request 2020-11-17 09:29:41 +01:00 geschlossen
Lixfel löschte die Branch scoreboard-reimplement 2020-11-17 09:29:54 +01:00
Autor
Besitzer

Da war noch eine NPE drin...

Da war noch eine NPE drin...
Besitzer

Dann fix sie jetzt!

Dann fix sie jetzt!
Autor
Besitzer

Ist schon im Branch

Ist schon im Branch
Dieses Repo ist archiviert. Du kannst Pull-Requests nicht kommentieren.
Keine Beschreibung angegeben.