src/main/java/de/steev/Tridentwar/Handlers/FileHandler.java

@@ -1,6 +1,6 @@
-package de.steev.Tridentwar.handlers;
+package de.steev.Tridentwar.Handlers;
 
-import de.steev.Tridentwar.manager.GameManager;
+import de.steev.Tridentwar.Manager.GameManager;
 import org.bukkit.configuration.file.FileConfiguration;
 import org.bukkit.configuration.file.YamlConfiguration;
 import org.bukkit.entity.Player;

src/main/java/de/steev/Tridentwar/States/Aborting.java

@@ -1,7 +1,7 @@
 package de.steev.Tridentwar.States;
 
-import de.steev.Tridentwar.manager.GameManager;
+import de.steev.Tridentwar.Manager.GameManager;
-import de.steev.Tridentwar.manager.GameState;
+import de.steev.Tridentwar.Manager.GameState;
 import org.bukkit.Bukkit;
 import org.bukkit.ChatColor;
 import org.bukkit.scheduler.BukkitRunnable;

src/main/java/de/steev/Tridentwar/States/Active.java

@@ -1,21 +1,14 @@
 package de.steev.Tridentwar.States;
 
-import de.steev.Tridentwar.manager.GameManager;
+import de.steev.Tridentwar.Manager.GameManager;
 import org.bukkit.Bukkit;
 import org.bukkit.GameMode;
 import org.bukkit.entity.Player;
-import org.bukkit.scoreboard.DisplaySlot;
-import org.bukkit.scoreboard.Objective;
-import org.bukkit.scoreboard.Scoreboard;
-import org.bukkit.scoreboard.ScoreboardManager;
 
 public class Active {
     private final GameManager gameManager;
-    private final ScoreboardManager manager;
-    private boolean isInitialized = false;
 
     public Active(GameManager gameManager) {
-        this.manager = Bukkit.getScoreboardManager();
         this.gameManager = gameManager;
     }
 
@@ -25,26 +18,9 @@ public class Active {
         this.gameManager.getPlayerManager().setPlayersHealth(20F);
         this.gameManager.getPlayerManager().setAlive(Bukkit.getOnlinePlayers().size());
         this.gameManager.getPlayerManager().giveKits();
-
-        Scoreboard board = manager.getNewScoreboard();
-        Objective obj = board.registerNewObjective("Tridentwar", "dummy", "SLPNetwork");
-        obj.setDisplaySlot(DisplaySlot.SIDEBAR);
-
         for (Player p : Bukkit.getOnlinePlayers()) {
-
+            this.gameManager.getPlayerManager().setKills(p, 0);
-            if (!isInitialized) {
+            this.gameManager.getScoreBoardManager().updateScoreBoard(p, this.gameManager.getPlayerManager().getAlive(), this.gameManager.getPlayerManager().getKills(p));
-                this.gameManager.getPlayerManager().setKills(p, 0);
-            }
-
-            obj.getScore("=-=-=-=-=-=-=-=-=-=-=").setScore(5);
-            obj.getScore(this.gameManager.getPlugin().config.getString("scoreboard.ingame.player") + " " + p.getDisplayName()).setScore(4);
-            obj.getScore(this.gameManager.getPlugin().config.getString("scoreboard.ingame.alive") + " " + this.gameManager.getPlayerManager().getAlive()).setScore(3);
-            obj.getScore(this.gameManager.getPlugin().config.getString("scoreboard.ingame.kills") + " " + this.gameManager.getPlayerManager().getKills(p)).setScore(2);
-            obj.getScore("").setScore(1);
-            obj.getScore(this.gameManager.getPlugin().config.getString("scoreboard.ingame.server")).setScore(0);
-
-            p.setScoreboard(board);
-
         }
     }
 }

src/main/java/de/steev/Tridentwar/States/Lobby.java

@@ -1,6 +1,6 @@
 package de.steev.Tridentwar.States;
 
-import de.steev.Tridentwar.manager.GameManager;
+import de.steev.Tridentwar.Manager.GameManager;
 import org.bukkit.Bukkit;
 import org.bukkit.ChatColor;
 import org.bukkit.GameMode;

src/main/java/de/steev/Tridentwar/States/Starting.java

@@ -1,8 +1,8 @@
 package de.steev.Tridentwar.States;
 
-import de.steev.Tridentwar.manager.GameManager;
+import de.steev.Tridentwar.Manager.GameManager;
-import de.steev.Tridentwar.manager.GameState;
+import de.steev.Tridentwar.Manager.GameState;
-import de.steev.Tridentwar.tasks.GameStartCountdownTask;
+import de.steev.Tridentwar.Tasks.GameStartCountdownTask;
 import org.bukkit.Bukkit;
 import org.bukkit.ChatColor;
 

src/main/java/de/steev/Tridentwar/States/Stopping.java

@@ -1,7 +1,7 @@
 package de.steev.Tridentwar.States;
 
-import de.steev.Tridentwar.manager.GameManager;
+import de.steev.Tridentwar.Manager.GameManager;
-import de.steev.Tridentwar.manager.GameState;
+import de.steev.Tridentwar.Manager.GameState;
 import org.bukkit.Bukkit;
 import org.bukkit.ChatColor;
 import org.bukkit.scheduler.BukkitRunnable;

src/main/java/de/steev/Tridentwar/States/Waiting.java

@@ -1,7 +1,7 @@
 package de.steev.Tridentwar.States;
 
-import de.steev.Tridentwar.manager.GameManager;
+import de.steev.Tridentwar.Manager.GameManager;
-import de.steev.Tridentwar.tasks.LobbyWaitingTask;
+import de.steev.Tridentwar.Tasks.LobbyWaitingTask;
 import org.bukkit.Bukkit;
 import org.bukkit.ChatColor;
 

src/main/java/de/steev/Tridentwar/States/Won.java

@@ -1,7 +1,7 @@
 package de.steev.Tridentwar.States;
 
-import de.steev.Tridentwar.manager.GameManager;
+import de.steev.Tridentwar.Manager.GameManager;
-import de.steev.Tridentwar.manager.GameState;
+import de.steev.Tridentwar.Manager.GameState;
 import org.bukkit.Bukkit;
 import org.bukkit.ChatColor;
 import org.bukkit.GameMode;

src/main/java/de/steev/Tridentwar/Tridentwar.java

@@ -1,8 +1,8 @@
 package de.steev.Tridentwar;
 
-import de.steev.Tridentwar.commands.TridentwarCommand;
+import de.steev.Tridentwar.Commands.TridentwarCommand;
-import de.steev.Tridentwar.listeners.*;
+import de.steev.Tridentwar.Listeners.*;
-import de.steev.Tridentwar.manager.GameManager;
+import de.steev.Tridentwar.Manager.GameManager;
 import org.bukkit.configuration.file.FileConfiguration;
 import org.bukkit.configuration.file.YamlConfiguration;
 import org.bukkit.entity.Player;

src/main/java/de/steev/Tridentwar/commands/TridentwarCommand.java

@@ -1,7 +1,7 @@
-package de.steev.Tridentwar.commands;
+package de.steev.Tridentwar.Commands;
 
-import de.steev.Tridentwar.manager.GameManager;
+import de.steev.Tridentwar.Manager.GameManager;
-import de.steev.Tridentwar.manager.GameState;
+import de.steev.Tridentwar.Manager.GameState;
 import org.bukkit.Bukkit;
 import org.bukkit.ChatColor;
 import org.bukkit.GameMode;

src/main/java/de/steev/Tridentwar/listeners/EntityDamageListener.java

@@ -1,7 +1,7 @@
-package de.steev.Tridentwar.listeners;
+package de.steev.Tridentwar.Listeners;
 
-import de.steev.Tridentwar.manager.GameManager;
+import de.steev.Tridentwar.Manager.GameManager;
-import de.steev.Tridentwar.manager.GameState;
+import de.steev.Tridentwar.Manager.GameState;
 import org.bukkit.ChatColor;
 import org.bukkit.GameMode;
 import org.bukkit.Sound;

src/main/java/de/steev/Tridentwar/listeners/JoinListener.java

@@ -1,7 +1,7 @@
-package de.steev.Tridentwar.listeners;
+package de.steev.Tridentwar.Listeners;
 
-import de.steev.Tridentwar.manager.GameManager;
+import de.steev.Tridentwar.Manager.GameManager;
-import de.steev.Tridentwar.manager.GameState;
+import de.steev.Tridentwar.Manager.GameState;
 import org.bukkit.Effect;
 import org.bukkit.GameMode;
 import org.bukkit.Material;

src/main/java/de/steev/Tridentwar/listeners/LeaveListener.java

@@ -1,7 +1,7 @@
-package de.steev.Tridentwar.listeners;
+package de.steev.Tridentwar.Listeners;
 
-import de.steev.Tridentwar.manager.GameManager;
+import de.steev.Tridentwar.Manager.GameManager;
-import de.steev.Tridentwar.manager.GameState;
+import de.steev.Tridentwar.Manager.GameState;
 import org.bukkit.GameMode;
 import org.bukkit.event.EventHandler;
 import org.bukkit.event.Listener;

src/main/java/de/steev/Tridentwar/listeners/PlayerDamageListener.java

@@ -1,7 +1,7 @@
-package de.steev.Tridentwar.listeners;
+package de.steev.Tridentwar.Listeners;
 
-import de.steev.Tridentwar.manager.GameManager;
+import de.steev.Tridentwar.Manager.GameManager;
-import de.steev.Tridentwar.manager.GameState;
+import de.steev.Tridentwar.Manager.GameState;
 import org.bukkit.ChatColor;
 import org.bukkit.GameMode;
 import org.bukkit.Sound;

src/main/java/de/steev/Tridentwar/listeners/ProjectileHitListener.java

@@ -1,7 +1,7 @@
-package de.steev.Tridentwar.listeners;
+package de.steev.Tridentwar.Listeners;
 
-import de.steev.Tridentwar.manager.GameManager;
+import de.steev.Tridentwar.Manager.GameManager;
-import de.steev.Tridentwar.manager.GameState;
+import de.steev.Tridentwar.Manager.GameState;
 import org.bukkit.Material;
 import org.bukkit.Sound;
 import org.bukkit.entity.Player;

src/main/java/de/steev/Tridentwar/listeners/ProjectileLaunchListener.java

@@ -1,9 +1,9 @@
-package de.steev.Tridentwar.listeners;
+package de.steev.Tridentwar.Listeners;
 
 import de.steev.Tridentwar.Tridentwar;
-import de.steev.Tridentwar.manager.GameManager;
+import de.steev.Tridentwar.Manager.GameManager;
-import de.steev.Tridentwar.manager.GameState;
+import de.steev.Tridentwar.Manager.GameState;
-import de.steev.Tridentwar.tasks.TridentResetTask;
+import de.steev.Tridentwar.Tasks.TridentResetTask;
 import org.bukkit.entity.Player;
 import org.bukkit.event.EventHandler;
 import org.bukkit.event.Listener;

src/main/java/de/steev/Tridentwar/manager/GameManager.java

@@ -1,13 +1,17 @@
-package de.steev.Tridentwar.manager;
+package de.steev.Tridentwar.Manager;
 
-import de.steev.Tridentwar.handlers.FileHandler;
+import de.steev.Tridentwar.Handlers.FileHandler;
 import de.steev.Tridentwar.States.*;
 import de.steev.Tridentwar.Tridentwar;
-import de.steev.Tridentwar.tasks.GameStartCountdownTask;
+import de.steev.Tridentwar.Tasks.GameStartCountdownTask;
-import de.steev.Tridentwar.tasks.LobbyWaitingTask;
+import de.steev.Tridentwar.Tasks.LobbyWaitingTask;
 import org.bukkit.*;
+import org.bukkit.entity.Player;
+import org.bukkit.scheduler.BukkitRunnable;
+import org.checkerframework.checker.units.qual.A;
 
 import java.io.File;
+import java.util.Objects;
 
 public class GameManager {
     private final Tridentwar plugin;

src/main/java/de/steev/Tridentwar/manager/GameState.java

@@ -1,4 +1,4 @@
-package de.steev.Tridentwar.manager;
+package de.steev.Tridentwar.Manager;
 
 public enum GameState {
     LOBBY, WAITING, STARTING, ACTIVE, WON, RESTARTING, STOPPING, ABORTING;

src/main/java/de/steev/Tridentwar/manager/MessageManager.java

@@ -1,4 +1,4 @@
-package de.steev.Tridentwar.manager;
+package de.steev.Tridentwar.Manager;
 
 import net.md_5.bungee.api.ChatMessageType;
 import net.md_5.bungee.api.chat.TextComponent;

src/main/java/de/steev/Tridentwar/manager/PlayerManager.java

@@ -1,4 +1,4 @@
-package de.steev.Tridentwar.manager;
+package de.steev.Tridentwar.Manager;
 
 import com.google.common.io.ByteArrayDataOutput;
 import com.google.common.io.ByteStreams;

src/main/java/de/steev/Tridentwar/manager/ScoreBoardManager.java

@@ -1,11 +1,9 @@
-package de.steev.Tridentwar.manager;
+package de.steev.Tridentwar.Manager;
 
 import org.bukkit.Bukkit;
 import org.bukkit.entity.Player;
 import org.bukkit.scoreboard.*;
 
-// TODO: consider building a scoreboard builder
-// TODO: reworking scoreboard logic to make it possible to replace the entire scoreboard
 public class ScoreBoardManager {
     private GameManager gameManager;
     private final ScoreboardManager manager;
@@ -14,10 +12,7 @@ public class ScoreBoardManager {
      * Manages Scoreboards created by the plugin
      * @param gameManager the local gamemanager instance
      */
-    public ScoreBoardManager(GameManager gameManager) {
+    public ScoreBoardManager(GameManager gameManager) { this.manager = Bukkit.getScoreboardManager(); }
-        this.gameManager = gameManager;
-        this.manager = Bukkit.getScoreboardManager();
-    }
 
     /**
      * Creates and sets a scoreboard for a player
@@ -26,18 +21,26 @@ public class ScoreBoardManager {
      * @param kills how many kills the player got
      */
     public void createScoreBoard(Player player, int alive, int kills) {
+        // Create board instance
         Scoreboard board = manager.getNewScoreboard();
+
+        // Create Board Objective
         Objective obj = board.registerNewObjective("Tridentwar", "dummy", "SLPNetwork");
         obj.setDisplaySlot(DisplaySlot.SIDEBAR);
 
-        obj.getScore("=-=-=-=-=-=-=-=-=-=-=").setScore(5);
+        // Create Scores
-        obj.getScore(this.gameManager.getPlugin().config.getString("scoreboard.ingame.player") + " " + player.getDisplayName()).setScore(4);
+        Score splitter = obj.getScore("=-=-=-=-=-=-=-=-=-=-=");
-        obj.getScore(this.gameManager.getPlugin().config.getString("scoreboard.ingame.alive") + " " + alive).setScore(3);
+        splitter.setScore(5);
-        obj.getScore(this.gameManager.getPlugin().config.getString("scoreboard.ingame.kills") + " " + kills).setScore(2);
+        Score playerName = obj.getScore("Player: " + player.getDisplayName());
-        obj.getScore("").setScore(1);
+        playerName.setScore(4);
-        obj.getScore(this.gameManager.getPlugin().config.getString("scoreboard.ingame.server")).setScore(0);
+        Score aliveScore = obj.getScore("Alive: " + alive);
-
+        aliveScore.setScore(3);
-        player.setScoreboard(board); // **Scoreboard setzen**
+        Score killsScore = obj.getScore("Kills: " + kills);
+        killsScore.setScore(2);
+        Score blankScore = obj.getScore("");
+        blankScore.setScore(1);
+        Score hostScore = obj.getScore("play.slpnetwork.de");
+        hostScore.setScore(0);
     }
 
     /**
@@ -47,32 +50,32 @@ public class ScoreBoardManager {
      * @param kills how many kills the player got
      * TODO: add time
      */
-    public void updateScoreBoard(Player player, int alive, int kills) {
+    public void updateScoreBoard(Player player, int alive, int kills){
+        // Grab Scoreboard from Player
         Scoreboard board = player.getScoreboard();
 
-        // Falls der Spieler kein eigenes Scoreboard hat, erstelle eins
+        // Detect if a Scoreboard has been created
-        if (board == null || board.getObjective(DisplaySlot.SIDEBAR) == null) {
+        if(board == null) {
+            // If there is none, create one
             createScoreBoard(player, alive, kills);
-            player.setScoreboard(board);
+        } else {
-            return;
+            // Fetch objective from existing scoreboard
+            Objective obj = board.getObjective(DisplaySlot.SIDEBAR);
+
+            // Replace Scores present on the Screen
+            Score splitter = obj.getScore("=-=-=-=-=-=-=-=-=-=-=");
+            splitter.setScore(5);
+            Score playerName = obj.getScore("Player: " + player.getDisplayName());
+            playerName.setScore(4);
+            Score aliveScore = obj.getScore("Alive: " + alive);
+            aliveScore.setScore(3);
+            Score killsScore = obj.getScore("Kills: " + kills);
+            killsScore.setScore(2);
+            Score blankScore = obj.getScore("");
+            blankScore.setScore(1);
+            Score hostScore = obj.getScore("play.slpnetwork.de");
+            hostScore.setScore(0);
         }
-
-        Objective obj = board.getObjective(DisplaySlot.SIDEBAR);
-
-        if (obj == null) { // Falls das Objective aus irgendeinem Grund null ist
-            obj = board.registerNewObjective("Tridentwar", "dummy", "SLPNetwork");
-            obj.setDisplaySlot(DisplaySlot.SIDEBAR);
-        }
-
-        // Scores aktualisieren
-        obj.getScore("=-=-=-=-=-=-=-=-=-=-=").setScore(5);
-        obj.getScore(this.gameManager.getPlugin().config.getString("scoreboard.ingame.player") + " " + player.getDisplayName()).setScore(4);
-        obj.getScore(this.gameManager.getPlugin().config.getString("scoreboard.ingame.alive") + " " + alive).setScore(3);
-        obj.getScore(this.gameManager.getPlugin().config.getString("scoreboard.ingame.kills") + " " + kills).setScore(2);
-        obj.getScore("").setScore(1);
-        obj.getScore(this.gameManager.getPlugin().config.getString("scoreboard.ingame.server")).setScore(0);
-
-        player.setScoreboard(board);
     }
 
     /**

src/main/java/de/steev/Tridentwar/manager/SoundManager.java

@@ -1,4 +1,4 @@
-package de.steev.Tridentwar.manager;
+package de.steev.Tridentwar.Manager;
 
 import org.bukkit.Bukkit;
 import org.bukkit.Sound;

src/main/java/de/steev/Tridentwar/manager/TridentManager.java

@@ -1,6 +1,6 @@
-package de.steev.Tridentwar.manager;
+package de.steev.Tridentwar.Manager;
 
-import de.steev.Tridentwar.tasks.TridentResetTask;
+import de.steev.Tridentwar.Tasks.TridentResetTask;
 import org.bukkit.entity.Player;
 
 import java.util.HashMap;

src/main/java/de/steev/Tridentwar/tasks/GameStartCountdownTask.java

@@ -1,21 +1,13 @@
-package de.steev.Tridentwar.tasks;
+package de.steev.Tridentwar.Tasks;
 
-import de.steev.Tridentwar.manager.GameManager;
+import de.steev.Tridentwar.Manager.GameManager;
-import de.steev.Tridentwar.manager.GameState;
+import de.steev.Tridentwar.Manager.GameState;
 import org.bukkit.Bukkit;
-import org.bukkit.entity.Player;
 import org.bukkit.scheduler.BukkitRunnable;
-import org.bukkit.scoreboard.DisplaySlot;
-import org.bukkit.scoreboard.Objective;
-import org.bukkit.scoreboard.Scoreboard;
-import org.bukkit.scoreboard.ScoreboardManager;
 
 public class GameStartCountdownTask extends BukkitRunnable {
     private GameManager gameManager;
-    private final ScoreboardManager manager;
-
     public GameStartCountdownTask (GameManager gameManager) {
-        this.manager = Bukkit.getScoreboardManager();
         this.gameManager = gameManager;
     }
 
@@ -23,22 +15,6 @@ public class GameStartCountdownTask extends BukkitRunnable {
 
     @Override
     public void run() {
-
-        Scoreboard board = manager.getNewScoreboard();
-
-        Objective obj = board.registerNewObjective("Tridentwar", "dummy", "SLPNetwork");
-        obj.setDisplaySlot(DisplaySlot.SIDEBAR);
-
-        for (Player player : Bukkit.getOnlinePlayers()){
-            obj.getScore("=-=-=-=-=-=-=-=-=-=-=").setScore(4);
-            obj.getScore(this.gameManager.getPlugin().config.getString("scoreboard.lobby.player") + " " + player.getDisplayName()).setScore(3);
-            obj.getScore(this.gameManager.getPlugin().config.getString("scoreboard.lobby.timeleft") + " " + timeLeft).setScore(2);
-            obj.getScore("").setScore(1);
-            obj.getScore(this.gameManager.getPlugin().config.getString("scoreboard.ingame.server")).setScore(0);
-
-            player.setScoreboard(board);
-        }
-
         timeLeft--;
         if(timeLeft <= 0) {
             cancel();

src/main/java/de/steev/Tridentwar/tasks/LobbyWaitingTask.java

@@ -1,23 +1,15 @@
-package de.steev.Tridentwar.tasks;
+package de.steev.Tridentwar.Tasks;
 
-import de.steev.Tridentwar.manager.GameManager;
+import de.steev.Tridentwar.Manager.GameManager;
-import de.steev.Tridentwar.manager.GameState;
+import de.steev.Tridentwar.Manager.GameState;
 import org.bukkit.Bukkit;
 import org.bukkit.ChatColor;
-import org.bukkit.entity.Player;
 import org.bukkit.scheduler.BukkitRunnable;
-import org.bukkit.scoreboard.DisplaySlot;
-import org.bukkit.scoreboard.Objective;
-import org.bukkit.scoreboard.Scoreboard;
-import org.bukkit.scoreboard.ScoreboardManager;
 
 public class LobbyWaitingTask extends BukkitRunnable {
     private GameManager gameManager;
-    private final ScoreboardManager manager;
-
     public LobbyWaitingTask(GameManager gameManager) {
         this.gameManager = gameManager;
-        this.manager = Bukkit.getScoreboardManager();
         System.out.println("DEBUG: Lobby is Waiting");
     }
 
@@ -25,21 +17,6 @@ public class LobbyWaitingTask extends BukkitRunnable {
 
     @Override
     public void run() {
-        Scoreboard board = manager.getNewScoreboard();
-
-        Objective obj = board.registerNewObjective("Tridentwar", "dummy", "SLPNetwork");
-        obj.setDisplaySlot(DisplaySlot.SIDEBAR);
-
-        for (Player player : Bukkit.getOnlinePlayers()){
-            obj.getScore("=-=-=-=-=-=-=-=-=-=-=").setScore(4);
-            obj.getScore(this.gameManager.getPlugin().config.getString("scoreboard.lobby.player") + " " + player.getDisplayName()).setScore(3);
-            obj.getScore(this.gameManager.getPlugin().config.getString("scoreboard.lobby.timeleft") + " " + timeLeft).setScore(2);
-            obj.getScore("").setScore(1);
-            obj.getScore(this.gameManager.getPlugin().config.getString("scoreboard.ingame.server")).setScore(0);
-
-            player.setScoreboard(board);
-        }
-
         timeLeft--;
         this.gameManager.getMessageManager().broadCastXPBar(ChatColor.translateAlternateColorCodes('&', this.gameManager.getPlugin().languageDataConfig.getString("game.countdown") + timeLeft));
         if (timeLeft > 10) {
@@ -71,11 +48,4 @@ public class LobbyWaitingTask extends BukkitRunnable {
             }
         }
     }
-
-    /**
-     * calls cancel to stop task
-     */
-    public void stopTask() {
-        cancel();
-    }
 }

src/main/java/de/steev/Tridentwar/tasks/TridentResetTask.java

@@ -1,6 +1,6 @@
-package de.steev.Tridentwar.tasks;
+package de.steev.Tridentwar.Tasks;
 
-import de.steev.Tridentwar.manager.GameManager;
+import de.steev.Tridentwar.Manager.GameManager;
 import org.bukkit.Material;
 import org.bukkit.entity.Entity;
 import org.bukkit.entity.Player;