Renamings from CR

by marko

Author: Zabuzard

application/src/main/java/org/togetherjava/tjbot/commands/reminder/RemindCommand.java

@@ -33,15 +33,15 @@
  */
 public final class RemindCommand extends SlashCommandAdapter {
     private static final String COMMAND_NAME = "remind";
-    private static final String WHEN_AMOUNT_OPTION = "when-amount";
-    private static final String WHEN_UNIT_OPTION = "when-unit";
+    private static final String TIME_AMOUNT_OPTION = "time-amount";
+    private static final String TIME_UNIT_OPTION = "time-unit";
     private static final String CONTENT_OPTION = "content";
 
-    private static final int MIN_WHEN_AMOUNT = 1;
-    private static final int MAX_WHEN_AMOUNT = 1_000;
-    private static final List<String> WHEN_UNITS =
+    private static final int MIN_TIME_AMOUNT = 1;
+    private static final int MAX_TIME_AMOUNT = 1_000;
+    private static final List<String> TIME_UNITS =
             List.of("minutes", "hours", "days", "weeks", "months", "years");
-    private static final Period MAX_WHEN_PERIOD = Period.ofYears(3);
+    private static final Period MAX_TIME_PERIOD = Period.ofYears(3);
     private static final int MAX_PENDING_REMINDERS_PER_USER = 100;
 
     private final Database database;
@@ -57,12 +57,12 @@ public RemindCommand(@NotNull Database database) {
 
         // TODO As soon as JDA offers date/time selector input, this should also offer
         // "/remind at" next to "/remind in" and use subcommands then
-        OptionData whenAmount = new OptionData(OptionType.INTEGER, WHEN_AMOUNT_OPTION,
-                "when to remind you, the amount of the time period (e.g. [5] weeks)", true)
-                    .setRequiredRange(MIN_WHEN_AMOUNT, MAX_WHEN_AMOUNT);
-        OptionData whenUnit = new OptionData(OptionType.STRING, WHEN_UNIT_OPTION,
-                "when to remind you, the unit of the time period (e.g. 5 [weeks])", true);
-        WHEN_UNITS.forEach(unit -> whenUnit.addChoice(unit, unit));
+        OptionData whenAmount = new OptionData(OptionType.INTEGER, TIME_AMOUNT_OPTION,
+                "period to remind you in, the amount of time (e.g. [5] weeks)", true)
+                    .setRequiredRange(MIN_TIME_AMOUNT, MAX_TIME_AMOUNT);
+        OptionData whenUnit = new OptionData(OptionType.STRING, TIME_UNIT_OPTION,
+                "period to remind you in, the unit of time (e.g. 5 [weeks])", true);
+        TIME_UNITS.forEach(unit -> whenUnit.addChoice(unit, unit));
 
         getData().addOptions(whenUnit, whenAmount)
             .addOption(OptionType.STRING, CONTENT_OPTION, "what to remind you about", true);
@@ -72,8 +72,8 @@ public RemindCommand(@NotNull Database database) {
 
     @Override
     public void onSlashCommand(@NotNull SlashCommandEvent event) {
-        int whenAmount = Math.toIntExact(event.getOption(WHEN_AMOUNT_OPTION).getAsLong());
-        String whenUnit = event.getOption(WHEN_UNIT_OPTION).getAsString();
+        int whenAmount = Math.toIntExact(event.getOption(TIME_AMOUNT_OPTION).getAsLong());
+        String whenUnit = event.getOption(TIME_UNIT_OPTION).getAsString();
         String content = event.getOption(CONTENT_OPTION).getAsString();
 
         Instant when = parseWhen(whenAmount, whenUnit);
@@ -117,15 +117,15 @@ public void onSlashCommand(@NotNull SlashCommandEvent event) {
 
     private static boolean handleIsWhenWithinLimits(@NotNull Instant when,
             @NotNull Interaction event) {
-        ZonedDateTime maxWhen = ZonedDateTime.now(ZoneOffset.UTC).plus(MAX_WHEN_PERIOD);
+        ZonedDateTime maxWhen = ZonedDateTime.now(ZoneOffset.UTC).plus(MAX_TIME_PERIOD);
 
         if (when.atZone(ZoneOffset.UTC).isBefore(maxWhen)) {
             return true;
         }
 
         event
             .reply("The reminder is set too far in the future. The maximal allowed period is '%s'."
-                .formatted(MAX_WHEN_PERIOD))
+                .formatted(MAX_TIME_PERIOD))
             .setEphemeral(true)
             .queue();