fix timeout duration check
All checks were successful
continuous-integration/drone/push Build is passing
All checks were successful
continuous-integration/drone/push Build is passing
This commit is contained in:
@@ -182,12 +182,15 @@ public class UserPunishment
|
|||||||
case KICK -> punishmentAction = guild.kick(mentioned);
|
case KICK -> punishmentAction = guild.kick(mentioned);
|
||||||
case TIMEOUT ->
|
case TIMEOUT ->
|
||||||
{
|
{
|
||||||
if (args != null)
|
// Ensure a duration argument is provided at index 1 (after mention/user)
|
||||||
|
if (args == null || args.length <= 1)
|
||||||
{
|
{
|
||||||
String durationStr = args[1];
|
return new MessageResponse("Sorry, but the specified duration is invalid!", null);
|
||||||
duration = FormatUtil.parseDuration(durationStr);
|
|
||||||
}
|
}
|
||||||
|
|
||||||
|
String durationStr = args[1];
|
||||||
|
duration = FormatUtil.parseDuration(durationStr);
|
||||||
|
|
||||||
boolean isDurationValid = true;
|
boolean isDurationValid = true;
|
||||||
|
|
||||||
if (duration == null) isDurationValid = false;
|
if (duration == null) isDurationValid = false;
|
||||||
@@ -197,7 +200,7 @@ public class UserPunishment
|
|||||||
if (minTimeoutDuration.compareTo(duration) > 0) isDurationValid = false;
|
if (minTimeoutDuration.compareTo(duration) > 0) isDurationValid = false;
|
||||||
}
|
}
|
||||||
|
|
||||||
if (duration == null || !isDurationValid)
|
if (!isDurationValid)
|
||||||
{
|
{
|
||||||
// todo nicer looking with emojis
|
// todo nicer looking with emojis
|
||||||
return new MessageResponse("Sorry, but the specified duration is invalid!", null);
|
return new MessageResponse("Sorry, but the specified duration is invalid!", null);
|
||||||
|
Reference in New Issue
Block a user