Fixed more NPEs and some minor changes

Author: bilektugrul

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

@@ -15,6 +15,8 @@ public boolean onCommand(@NotNull CommandSender sender, @NotNull Command command
         if (sender instanceof Player && sender.hasPermission("sst.disposal")) {
             Player player = (Player) sender;
             player.openInventory(Bukkit.getServer().createInventory(player, 54, Utils.getMessage("disposal.title", player)));
+        } else {
+            sender.sendMessage(Utils.getMessage("no-permission", sender));
         }
         return true;
     }

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

@@ -18,7 +18,7 @@ public boolean onCommand(@NotNull CommandSender sender, @NotNull Command command
             return true;
         }
 
-        Player feedPlayer;
+        Player feedPlayer = null;
 
         if (args.length >= 1) {
             if (sender.hasPermission("sst.feed.others")) {
@@ -29,7 +29,9 @@ public boolean onCommand(@NotNull CommandSender sender, @NotNull Command command
             }
         } else if (sender instanceof Player) {
             feedPlayer = (Player) sender;
-        } else {
+        }
+
+        if (feedPlayer == null) {
             sender.sendMessage(Utils.getMessage("feed.not-found", sender));
             return true;
         }

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

@@ -18,7 +18,7 @@ public boolean onCommand(@NotNull CommandSender sender, @NotNull Command command
             return true;
         }
 
-        Player healPlayer;
+        Player healPlayer = null;
 
         if (args.length >= 1) {
             if (sender.hasPermission("sst.heal.others")) {
@@ -29,7 +29,9 @@ public boolean onCommand(@NotNull CommandSender sender, @NotNull Command command
             }
         } else if (sender instanceof Player) {
             healPlayer = (Player) sender;
-        } else {
+        }
+
+        if (healPlayer == null) {
             sender.sendMessage(Utils.getMessage("heal.not-found", sender));
             return true;
         }

src/main/java/io/github/bilektugrul/simpleservertools/commands/tpa/TPACommand.java

@@ -77,6 +77,8 @@ public boolean onCommand(@NotNull CommandSender sender, @NotNull Command command
             } else {
                 p.sendMessage(Utils.getMessage("tpa.not-found", p));
             }
+        } else {
+            sender.sendMessage(Utils.getMessage("no-permission", sender));
         }
         return true;
     }

src/main/resources/language/messages_en.yml

@@ -219,7 +219,7 @@ messages:
 
   clear-chat:
     lines: 100
-    cleared: '%prefix% &3Chat has been cleaned by &b%executor% &3.' # if you set this message to '', it won't be broadcasted.
+    cleared: '%prefix% &3Chat has been cleaned by &b%executor%&3.' # if you set this message to '', it won't be broadcasted.
     for-staffs:
       enabled: true # true = staffs' chat won't be cleaned.