show explicit error when duration format is invalid

Author: zefir-git

src/main/java/pro/cloudnode/smp/smpcore/Messages.java

@@ -619,6 +619,13 @@ public Messages() {
         return MiniMessage.miniMessage().deserialize(Objects.requireNonNull(config.getString("error.duration-zero-or-less")));
     }
 
+    public @NotNull Component invalidDuration(final @NotNull String duration) {
+        return MiniMessage.miniMessage().deserialize(
+                Objects.requireNonNull(config.getString("error.invalid-duration")),
+                Placeholder.unparsed("duration", duration)
+        );
+    }
+
     public record SubCommandArgument(@NotNull String name, boolean required) {
         public @NotNull Component component() {
             return required ? SMPCore.messages().subCommandArgumentRequired(name) : SMPCore.messages()

src/main/java/pro/cloudnode/smp/smpcore/command/BanCommand.java

@@ -33,10 +33,12 @@ public boolean run(@NotNull CommandSender sender, @NotNull String label, @NotNul
 
         final @Nullable String durationArg = args.length > 1 ? args[1] : null;
         @Nullable Duration duration = null;
-        if (durationArg != null) try {
+        if (durationArg != null && durationArg.matches("(?i)^PT\\d.*")) try {
             duration = Duration.parse(durationArg);
         }
-        catch (DateTimeParseException ignored) {}
+        catch (DateTimeParseException ignored) {
+            return sendMessage(sender, SMPCore.messages().invalidDuration(durationArg));
+        }
 
         if (duration != null && (duration.isNegative() || duration.isZero()))
             return sendMessage(sender, SMPCore.messages().errorDurationZeroOrLess());

src/main/resources/messages.yml

@@ -117,3 +117,4 @@ error:
   demote-leader: <red>(!) You cannot demote the nation leader.</red>
   demote-citizen: <red>(!) Don't make this <i>citizen</i> unworthy of the nation! (To remove from nation, use <click:suggest_command:/nation citizens kick ><gray>/nation citizens kick</gray></click>.)</red>
   duration-zero-or-less: <red>(!) Duration must be greater than zero.</red>
+  invalid-duration: <red>(!) Invalid duration format <gray>‘<duration></gray>’.</red>