Fixed a NPE and changed colors usage

Author: bilektugrul

src/main/java/io/github/bilektugrul/simpleservertools/AsyncUserSaveThread.java

@@ -1,6 +1,7 @@
 package io.github.bilektugrul.simpleservertools;
 
 import io.github.bilektugrul.simpleservertools.users.UserManager;
+import org.bukkit.ChatColor;
 import org.bukkit.scheduler.BukkitRunnable;
 
 import java.io.IOException;
@@ -20,7 +21,7 @@ public AsyncUserSaveThread(SST plugin) {
     }
 
     public void start() {
-        logger.info("§aAsync user saving thread is starting...");
+        logger.info(ChatColor.GREEN + "Async user saving thread is starting...");
         int i = plugin.getConfig().getInt("auto-save-interval");
         runTaskTimerAsynchronously(plugin, 2400, (i * 60L) * 20);
     }

src/main/java/io/github/bilektugrul/simpleservertools/SST.java

@@ -33,7 +33,7 @@
 import io.github.bilektugrul.simpleservertools.utils.Utils;
 import io.github.bilektugrul.simpleservertools.utils.VaultManager;
 import org.bukkit.Bukkit;
-import org.bukkit.command.ConsoleCommandSender;
+import org.bukkit.ChatColor;
 import org.bukkit.entity.Player;
 import org.bukkit.plugin.PluginManager;
 import org.bukkit.plugin.java.JavaPlugin;
@@ -66,7 +66,7 @@ public class SST extends JavaPlugin {
     public void onEnable() {
         saveDefaultConfig();
         logger = getLogger();
-        logger.info("§aSimpleServerTools v" + getDescription().getVersion() + " is being enabled. Thanks for using SST!");
+        logger.info(ChatColor.GREEN + "SimpleServerTools v" + getDescription().getVersion() + " is being enabled. Thanks for using SST!");
         pluginManager = getServer().getPluginManager();
         placeholderManager = new CustomPlaceholderManager(this);
         languageManager = new LanguageManager(this);
@@ -81,12 +81,12 @@ public void onEnable() {
         if (pluginManager.isPluginEnabled("PlaceholderAPI")) {
             new PAPIPlaceholders(this).register();
         } else {
-            logger.warning("§cPlaceholderAPI is not installed. You should check it out.");
+            logger.warning(ChatColor.RED + "PlaceholderAPI is not installed. You should check it out.");
         }
         if (pluginManager.isPluginEnabled("Vault")) {
             vaultManager = new VaultManager(this);
         } else {
-            logger.warning("§cVault is not installed. Some features may not work.");
+            logger.warning(ChatColor.RED + "Vault is not installed. Some features may not work.");
         }
         announcementManager = new AnnouncementManager(this);
         for (Player looped : Bukkit.getOnlinePlayers()) {
@@ -129,11 +129,11 @@ public void onEnable() {
             asyncUserSaveThread = new AsyncUserSaveThread(this);
         }
         if (Utils.getBoolean("metrics-enabled")) {
-            logger.info("§aEnabling metrics...");
+            logger.info(ChatColor.GREEN + "Enabling metrics...");
             new Metrics(this, 11344);
         }
         checkUpdate();
-        logger.info("§aEnabling process is done, enjoy!");
+        logger.info(ChatColor.GREEN + "Enabling process is done, enjoy!");
     }
 
     @Override
@@ -147,7 +147,7 @@ public void onDisable() {
         warpManager.saveWarps();
         spawnManager.saveSpawn();
         maintenanceManager.save();
-        logger.info("§aEverything has been saved.");
+        logger.info(ChatColor.GREEN + "Everything has been saved.");
     }
 
     public void checkUpdate() {
@@ -165,16 +165,16 @@ public void checkUpdate() {
 
             if (result.getNewestVersion().contains("b")) {
                 if (getConfig().getBoolean("updates.notify-beta-versions", true)) {
-                    logger.info("§aFound a new beta version available: v" + result.getNewestVersion());
-                    logger.info("§aDownload it on SpigotMC:");
-                    logger.info("§b https://www.spigotmc.org/resources/simpleservertools-1-8-8-1-16-5.92388/");
+                    logger.info(ChatColor.GREEN + "Found a new beta version available: v" + result.getNewestVersion());
+                    logger.info(ChatColor.GREEN + "Download it on SpigotMC:");
+                    logger.info(ChatColor.BLUE + "https://www.spigotmc.org/resources/simpleservertools-1-8-8-1-16-5.92388/");
                 }
                 return;
             }
 
-            logger.info("§aFound a new version available: v" + result.getNewestVersion());
-            logger.info("§aDownload it SpigotMC:");
-            logger.info("§ahttps://www.spigotmc.org/resources/simpleservertools-1-8-8-1-16-5.92388/");
+            logger.info(ChatColor.GREEN + "Found a new version available: v" + result.getNewestVersion());
+            logger.info(ChatColor.GREEN + "Download it SpigotMC:");
+            logger.info(ChatColor.BLUE + "https://www.spigotmc.org/resources/simpleservertools-1-8-8-1-16-5.92388/");
 
         });
 
@@ -235,7 +235,7 @@ public void checkAndLoadPacketListener() {
     }
 
     public void reload(boolean first) {
-        logger.info("§aReloading configurations from disk...");
+        logger.info(ChatColor.GREEN + "Reloading configurations from disk...");
         reloadConfig();
         checkAndLoadPacketListener();
         placeholderManager.load();

src/main/java/io/github/bilektugrul/simpleservertools/commands/PingCommand.java

@@ -12,6 +12,7 @@ public class PingCommand implements CommandExecutor {
 
     @Override
     public boolean onCommand(@NotNull CommandSender sender, @NotNull Command command, @NotNull String label, @NotNull String[] args) {
+
         if (!sender.hasPermission("sst.ping")) {
             sender.sendMessage(Utils.getMessage("no-permission", sender));
             return true;
@@ -23,7 +24,9 @@ public boolean onCommand(@NotNull CommandSender sender, @NotNull Command command
             pingPlayer = Bukkit.getPlayer(args[0]);
         } else if (sender instanceof Player) {
             pingPlayer = (Player) sender;
-        } else {
+        }
+
+        if (pingPlayer == null) {
             sender.sendMessage(Utils.getMessage("ping.not-found", sender));
             return true;
         }

src/main/java/io/github/bilektugrul/simpleservertools/features/language/LanguageManager.java

@@ -2,6 +2,7 @@
 
 import io.github.bilektugrul.simpleservertools.SST;
 import me.despical.commons.configuration.ConfigUtils;
+import org.bukkit.ChatColor;
 import org.bukkit.configuration.file.FileConfiguration;
 
 import java.io.File;
@@ -25,7 +26,7 @@ public void loadLanguage() {
         try {
             language = ConfigUtils.getConfig(plugin, "language" + s + "messages_" + languageString);
         } catch (IllegalArgumentException ignored) {
-            plugin.getLogger().warning("§cYou have chosen a non-existent language. Please check our Spigot page and use one of available languages. Plugin will use EN language.");
+            plugin.getLogger().warning(ChatColor.RED + "You have chosen a non-existent language. Please check our Spigot page and use one of available languages. Plugin will use EN language.");
             languageString = "en";
             language = ConfigUtils.getConfig(plugin, "language" + s + "messages_" + languageString);
         }

src/main/java/io/github/bilektugrul/simpleservertools/features/spawn/SpawnManager.java

@@ -9,6 +9,7 @@
 import io.github.bilektugrul.simpleservertools.utils.Utils;
 import me.despical.commons.configuration.ConfigUtils;
 import org.bukkit.Bukkit;
+import org.bukkit.ChatColor;
 import org.bukkit.Location;
 import org.bukkit.command.CommandSender;
 import org.bukkit.configuration.file.FileConfiguration;
@@ -75,7 +76,7 @@ public TeleportSettings getSettings() {
 
     public void saveSpawn() {
         if (spawn != null) {
-            plugin.getLogger().info("§aSaving spawn...");
+            plugin.getLogger().info(ChatColor.GREEN + "Saving spawn...");
             spawnFile.set("spawn.location", spawn.getLocation());
             ConfigUtils.saveConfig(plugin, spawnFile, "spawn");
         }

src/main/java/io/github/bilektugrul/simpleservertools/utils/Utils.java

@@ -61,7 +61,7 @@ public static String getString(FileConfiguration file, String string, CommandSen
     public static String replacePlaceholders(String msg, CommandSender from, boolean replacePersonalPlaceholders, boolean replacePAPI) {
         boolean isPlayer = from instanceof Player;
         if (msg == null) {
-            plugin.getLogger().warning("§cYour language file[s] is/are corrupted or old. Please reset or update them.");
+            plugin.getLogger().warning(org.bukkit.ChatColor.RED + "Your language file[s] is/are corrupted or old. Please reset or update them.");
             return "";
         }
         msg = placeholderManager.replacePlaceholders(ChatColor.translateAlternateColorCodes('&', msg));