fixed permission check to change other player's tags

Author: Maloschnikow

LICENSE_FABRIC

@@ -9,7 +9,7 @@ yarn_mappings=1.21.4+build.8
 loader_version=0.16.10
 
 # Mod Properties
-mod_version=1.0.0
+mod_version=1.0.1
 maven_group=maloschnikow.playertags
 archives_base_name=player-tags
 

gradle.properties

@@ -4,6 +4,7 @@
 import com.mojang.brigadier.context.CommandContext;
 import com.mojang.brigadier.exceptions.CommandSyntaxException;
 
+import maloschnikow.playertags.WrongPermissionLevelMessage;
 import net.minecraft.command.argument.EntityArgumentType;
 import net.minecraft.scoreboard.Team;
 import net.minecraft.server.command.ServerCommandSource;
@@ -17,6 +18,12 @@ public int run(CommandContext<ServerCommandSource> context) throws CommandSyntax
 
         //get arguments
         ServerPlayerEntity player = EntityArgumentType.getPlayer(context, "player");
+        ServerCommandSource commandSource = context.getSource();
+
+        //check if player has permission to remove other players tag
+        if( (!commandSource.hasPermissionLevel(2)) && player != commandSource.getPlayer() ) {
+            throw new CommandSyntaxException(null, new WrongPermissionLevelMessage(2, "remove another player's tag"));
+        }
 
         Team playerTeam = player.getServer().getScoreboard().getTeam(player.getUuidAsString());
         player.getServer().getScoreboard().removeTeam(playerTeam);

src/main/java/maloschnikow/playertags/commands/CommandCustomPlayerTagRemove.java

@@ -39,7 +39,7 @@ public int run(CommandContext<ServerCommandSource> context) throws CommandSyntax
 
 
         //check if player has permission to change other players tag
-        if( !commandSource.hasPermissionLevel(2) && player != commandSource.getPlayer() ) {
+        if( (!commandSource.hasPermissionLevel(2)) && player != commandSource.getPlayer() ) {
             throw new CommandSyntaxException(null, new WrongPermissionLevelMessage(2, "change another player's tag"));
         }