Fix permission problems

Author: bilektugrul

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

@@ -21,7 +21,12 @@ public boolean onCommand(@NotNull CommandSender sender, @NotNull Command command
         Player feedPlayer;
 
         if (args.length >= 1) {
-            feedPlayer = Bukkit.getPlayer(args[0]);
+            if (sender.hasPermission("sst.feed.others")) {
+                feedPlayer = Bukkit.getPlayer(args[0]);
+            } else {
+                sender.sendMessage(Utils.getMessage("no-permission", sender));
+                return true;
+            }
         } else if (sender instanceof Player) {
             feedPlayer = (Player) sender;
         } else {

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

@@ -42,11 +42,19 @@ public boolean onCommand(@NotNull CommandSender sender, @NotNull Command command
     }
 
     public void change(CommandSender from, Player flightPlayer, boolean newMode) {
+
+        boolean isSame = from.equals(flightPlayer);
+
+        if (!isSame && !from.hasPermission("sst.fly.others")) {
+            from.sendMessage(Utils.getMessage("no-permission", from));
+            return;
+        }
+
         flightPlayer.setAllowFlight(newMode);
         flightPlayer.setFlying(newMode);
         flightPlayer.sendMessage(Utils.getMessage("fly.toggled", flightPlayer)
                 .replace("%flymode%", Utils.getMessage("fly.modes." + newMode, from)));
-        if (!from.equals(flightPlayer)) {
+        if (!isSame) {
             from.sendMessage(Utils.getMessage("fly.toggled-other", from)
                     .replace("%other%", flightPlayer.getName())
                     .replace("%flymode%", Utils.getMessage("fly.modes." + newMode, from)));

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

@@ -47,6 +47,14 @@ public boolean onCommand(@NotNull CommandSender sender, @NotNull Command command
     }
 
     public void change(CommandSender from, Player godPlayer, User godUser, boolean newMode) {
+
+        boolean isSame = from.equals(godPlayer);
+
+        if (!isSame && !from.hasPermission("sst.god.others")) {
+            from.sendMessage(Utils.getMessage("no-permission", from));
+            return;
+        }
+
         godUser.setGod(newMode);
         godPlayer.sendMessage(Utils.getMessage("god.toggled", godPlayer)
                 .replace("%godmode%", Utils.getMessage("god.modes." + newMode, from)));

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

@@ -18,10 +18,15 @@ public boolean onCommand(@NotNull CommandSender sender, @NotNull Command command
             return true;
         }
 
-        Player healPlayer = null;
+        Player healPlayer;
 
         if (args.length >= 1) {
-            healPlayer = Bukkit.getPlayer(args[0]);
+            if (sender.hasPermission("sst.heal.others")) {
+                healPlayer = Bukkit.getPlayer(args[0]);
+            } else {
+                sender.sendMessage(Utils.getMessage("no-permission", sender));
+                return true;
+            }
         } else if (sender instanceof Player) {
             healPlayer = (Player) sender;
         } else {

src/main/java/io/github/bilektugrul/simpleservertools/commands/gamemode/GamemodeCommand.java

@@ -65,7 +65,7 @@ private GameMode matchGameMode(String mode) {
     }
 
     private boolean canChange(CommandSender sender, GameMode gamemode) {
-        return sender.hasPermission("sst.gamemode.*") || sender.hasPermission("sst.gamemode." + gamemode.name().toLowerCase(Locale.ROOT));
+        return sender.hasPermission("sst.gamemode.all") || sender.hasPermission("sst.gamemode.*") || sender.hasPermission("sst.gamemode." + gamemode.name().toLowerCase(Locale.ROOT));
     }
 
 }