Fix double-jailing bug and add admin jail time checking

BUG FIXES:
- Prevent jailing already-jailed players (was overwriting jail data)
- Show clear error with remaining time: 'Player is already jailed! Remaining time: X. Use /jail release first.'

NEW FEATURE:
- Add /jail time <player> for admins to check any player's jail time
- Regular players can only use /jail time (self-only)
- Admins need oldschooljail.jail permission to check others
- Clear error messages for permissions and non-jailed players

DOCUMENTATION:
- Updated README with new /jail time <player> command
- Added permission explanation for admin jail time checking

This prevents data corruption and gives admins better jail management tools.

Author: FugLong

README.md

@@ -35,7 +35,8 @@ A classic jail mod for Minecraft Fabric servers (1.21.8+) - just like the good o
 
 - `/jail release <player>` - Release a player from jail early and teleport them back to where they were
 
-- `/jail time` - Check remaining jail time (available to jailed players)
+- `/jail time` - Check your own remaining jail time (available to jailed players)
+- `/jail time <player>` - Check any player's jail time (admin only - requires `oldschooljail.jail` permission)
 
 ### Admin Commands
 - `/jail set <jail_name>` - Set a jail at your current location
@@ -53,6 +54,7 @@ The mod uses the Fabric Permissions API and supports any permissions plugin that
 - `oldschooljail.delete` - Allows deleting jails
 - `oldschooljail.immune` - Makes a player immune to being jailed
 - `oldschooljail.time` - Allows using `/jail time` (granted to all by default)
+- `oldschooljail.jail` - Also allows using `/jail time <player>` to check other players' jail time
 
 **Note**: If no permissions plugin is installed, the mod falls back to OP level 2 for admin commands.
 

src/main/java/com/oldschooljail/command/JailCommand.java

@@ -77,7 +77,10 @@ public static void register(CommandDispatcher<ServerCommandSource> dispatcher) {
 
 			// /jail time
 			.then(CommandManager.literal("time")
-				.executes(JailCommand::checkJailTime))
+				.executes(JailCommand::checkJailTime)
+				// /jail time <player>
+				.then(CommandManager.argument("player", EntityArgumentType.player())
+					.executes(JailCommand::checkPlayerJailTime)))
 		);
 	}
 
@@ -136,6 +139,16 @@ private static int executeJail(ServerCommandSource source, ServerPlayerEntity ta
 			return 0;
 		}
 
+		// Check if target is already jailed
+		JailedPlayersData jailedData = OldSchoolJailMod.getJailedPlayersData();
+		if (jailedData.isJailed(target.getUuid())) {
+			JailedPlayer existingJail = jailedData.getJailedPlayer(target.getUuid());
+			long remaining = existingJail.getRemainingTimeSeconds();
+			source.sendError(Text.literal("§c" + target.getName().getString() + " is already jailed! " +
+				"Remaining time: " + formatTime(remaining) + ". Use /jail release first."));
+			return 0;
+		}
+		
 		JailConfig config = OldSchoolJailMod.getConfig();
 		long timeInSeconds = config.convertToSeconds(time);
 
@@ -147,7 +160,6 @@ private static int executeJail(ServerCommandSource source, ServerPlayerEntity ta
 		}
 
 		JailData jailData = OldSchoolJailMod.getJailData();
-		JailedPlayersData jailedData = OldSchoolJailMod.getJailedPlayersData();
 
 		Jail jail = jailData.getJail(jailName);
 		if (jail == null) {
@@ -333,6 +345,30 @@ private static int checkJailTime(CommandContext<ServerCommandSource> context) th
 		return 1;
 	}
 
+	private static int checkPlayerJailTime(CommandContext<ServerCommandSource> context) throws CommandSyntaxException {
+		ServerCommandSource source = context.getSource();
+		
+		// Check permission - only admins can check other players' jail time
+		if (!PermissionUtil.hasPermission(source, PermissionUtil.JAIL_PLAYER)) {
+			source.sendError(Text.literal("§cYou don't have permission to check other players' jail time!"));
+			return 0;
+		}
+		
+		ServerPlayerEntity target = EntityArgumentType.getPlayer(context, "player");
+		JailedPlayersData jailedData = OldSchoolJailMod.getJailedPlayersData();
+		JailedPlayer jailed = jailedData.getJailedPlayer(target.getUuid());
+		
+		if (jailed == null) {
+			source.sendError(Text.literal("§c" + target.getName().getString() + " is not jailed!"));
+			return 0;
+		}
+		
+		long remaining = jailed.getRemainingTimeSeconds();
+		source.sendFeedback(() -> Text.literal("§e" + target.getName().getString() + " will be released in: " + formatTime(remaining)), false);
+		
+		return 1;
+	}
+	
 	private static void teleportToJail(ServerPlayerEntity player, Jail jail) {
 		// Get the world
 		MinecraftServer server = player.getServer();