Fixed the fallback principal option and renamed it to something better

Author: jamezrin

src/main/java/com/jaimemartz/playerbalancer/commands/FallbackCommand.java

@@ -88,8 +88,10 @@ public ServerSection getSection(ProxiedPlayer player) {
 
             return target;
         } else {
-            if (plugin.getSettings().getGeneralProps().isFallbackPrincipal()) {
+            if (plugin.getSettings().getBalancerProps().isDefaultPrincipal()) {
                 return plugin.getSectionManager().getPrincipal();
+            } else {
+                MessageUtils.send(player, messages.getUnavailableServerMessage());
             }
         }
 

src/main/java/com/jaimemartz/playerbalancer/commands/ManageCommand.java

@@ -190,6 +190,10 @@ public void execute(CommandSender sender, String[] args) {
                                                         .color(ChatColor.WHITE)
                                                         .append("\"")
                                                         .color(ChatColor.AQUA)
+                                                        .append("\nAddress: ")
+                                                        .color(ChatColor.GRAY)
+                                                        .append(server.getAddress().toString())
+                                                        .color(ChatColor.AQUA)
                                                         .create()))
                                                 .append(String.format(" (%d/%d) ",
                                                         status.getOnline(),

src/main/java/com/jaimemartz/playerbalancer/listener/ServerKickListener.java

@@ -124,8 +124,10 @@ private ServerSection getSection(ProxiedPlayer player, ServerInfo from) {
 
             return target;
         } else {
-            if (plugin.getSettings().getGeneralProps().isFallbackPrincipal()) {
+            if (plugin.getSettings().getBalancerProps().isDefaultPrincipal()) {
                 return plugin.getSectionManager().getPrincipal();
+            } else {
+                MessageUtils.send(player, messages.getUnavailableServerMessage());
             }
         }
 

src/main/java/com/jaimemartz/playerbalancer/ping/StatusManager.java

@@ -28,7 +28,8 @@ public void start() {
         stopped = false;
         tactic = plugin.getSettings().getServerCheckerProps().getTactic();
         plugin.getLogger().info(String.format("Starting the ping task, the interval is %s",
-                plugin.getSettings().getServerCheckerProps().getInterval()));
+                plugin.getSettings().getServerCheckerProps().getInterval()
+        ));
 
         task = plugin.getProxy().getScheduler().schedule(plugin, () -> {
             storage.forEach((k, v) -> v.setOutdated(true));

src/main/java/com/jaimemartz/playerbalancer/settings/props/GeneralProps.java

@@ -17,9 +17,6 @@ public class GeneralProps {
     @Setting(value = "redis-bungee")
     private boolean redisBungee;
 
-    @Setting(value = "fallback-principal")
-    private boolean fallbackPrincipal;
-
     @Setting
     private String version;
 
@@ -55,14 +52,6 @@ public void setRedisBungee(boolean redisBungee) {
         this.redisBungee = redisBungee;
     }
 
-    public boolean isFallbackPrincipal() {
-        return fallbackPrincipal;
-    }
-
-    public void setFallbackPrincipal(boolean fallbackPrincipal) {
-        this.fallbackPrincipal = fallbackPrincipal;
-    }
-
     public String getVersion() {
         return version;
     }
@@ -78,7 +67,6 @@ public String toString() {
                 ", silent=" + silent +
                 ", autoReload=" + autoReload +
                 ", redisBungee=" + redisBungee +
-                ", fallbackPrincipal=" + fallbackPrincipal +
                 ", version='" + version + '\'' +
                 '}';
     }

src/main/java/com/jaimemartz/playerbalancer/settings/props/features/BalancerProps.java

@@ -12,6 +12,9 @@ public class BalancerProps {
     @Setting(value = "principal-section")
     private String principalSectionName;
 
+    @Setting(value = "default-principal")
+    private boolean defaultPrincipal;
+
     @Setting(value = "dummy-sections")
     private List<String> dummySectionNames;
 
@@ -64,6 +67,14 @@ public void setShowPlayers(boolean showPlayers) {
         this.showPlayers = showPlayers;
     }
 
+    public boolean isDefaultPrincipal() {
+        return defaultPrincipal;
+    }
+
+    public void setDefaultPrincipal(boolean defaultPrincipal) {
+        this.defaultPrincipal = defaultPrincipal;
+    }
+
     @Override
     public String toString() {
         return "BalancerProps{" +