Version 2.1.5.1 that fixes NPE when replacing the alias placeholder

Author: jamezrin

Main Plugin/pom.xml

@@ -11,7 +11,7 @@
 
     <name>PlayerBalancer Plugin</name>
     <artifactId>playerbalancer-plugin</artifactId>
-    <version>2.1.5</version>
+    <version>2.1.5.1</version>
 
     <build>
         <finalName>PlayerBalancer</finalName>

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

@@ -109,12 +109,25 @@ public void execute(CommandSender sender, String[] args) {
                                             .create());
                                 }
 
-                                sender.sendMessage(new ComponentBuilder("Alias: ")
-                                        .color(ChatColor.GRAY)
-                                        .append(String.valueOf(section.getProps().getAlias()))
-                                        .color(ChatColor.AQUA)
-                                        .create()
-                                );
+                                if (section.getProps().getAlias() != null) {
+                                    sender.sendMessage(new ComponentBuilder("Alias: ")
+                                            .color(ChatColor.GRAY)
+                                            .append("\"")
+                                            .color(ChatColor.AQUA)
+                                            .append(section.getProps().getAlias())
+                                            .color(ChatColor.RED)
+                                            .append("\"")
+                                            .color(ChatColor.AQUA)
+                                            .create()
+                                    );
+                                } else {
+                                    sender.sendMessage(new ComponentBuilder("Alias: ")
+                                            .color(ChatColor.GRAY)
+                                            .append("None")
+                                            .color(ChatColor.AQUA)
+                                            .create()
+                                    );
+                                }
 
                                 sender.sendMessage(new ComponentBuilder("Position: ")
                                         .color(ChatColor.GRAY)

Main Plugin/src/main/java/com/jaimemartz/playerbalancer/connection/ConnectionIntent.java

@@ -13,6 +13,8 @@
 import java.util.List;
 import java.util.concurrent.TimeUnit;
 
+import static com.jaimemartz.playerbalancer.utils.MessageUtils.safeNull;
+
 //TODO I don't like this, improve it
 public abstract class ConnectionIntent {
     protected final PlayerBalancer plugin;
@@ -26,7 +28,7 @@ public ConnectionIntent(PlayerBalancer plugin, ProxiedPlayer player, ProviderTyp
 
         MessageUtils.send(player, plugin.getSettings().getMessagesProps().getConnectingMessage(),
                 (str) -> str.replace("{section}", section.getName())
-                        .replace("{alias}", section.getProps().getAlias())
+                        .replace("{alias}", safeNull(section.getProps().getAlias()))
         );
 
         //Prevents removing servers from the section
@@ -48,7 +50,7 @@ public ConnectionIntent(PlayerBalancer plugin, ProxiedPlayer player, ProviderTyp
                         MessageUtils.send(player, plugin.getSettings().getMessagesProps().getConnectedMessage(),
                                 (str) -> str.replace("{server}", target.getName())
                                         .replace("{section}", section.getName())
-                                        .replace("{alias}", section.getProps().getAlias())
+                                        .replace("{alias}", safeNull(section.getProps().getAlias()))
                         );
                     }
                 });

Main Plugin/src/main/java/com/jaimemartz/playerbalancer/utils/MessageUtils.java

@@ -26,8 +26,11 @@ public static String revertColor(String string) {
         return string.replace(ChatColor.COLOR_CHAR, '&');
     }
 
-    public static String safePrint(String string) {
-        return string.replace(ChatColor.COLOR_CHAR, '\u00A7');
+    public static String safeNull(String string) {
+        if (string == null) {
+            return "Undefined";
+        }
+        return string;
     }
 
     private MessageUtils() {}

Main Plugin/src/main/resources/default.conf

@@ -229,7 +229,7 @@ features {
 
     rules {
       general-lobbies {
-        special.permission=other-lobby-section,
+        special.permission=other-lobby-section
       }
     }
   }