Delete old repeating reminders first, then make the new one to hopefu…

…lly avoid edge cases where reminders aren't sent
HyacinthBots · Sep 15, 2022 · b522b8e · b522b8e
1 parent 4536e4c
commit b522b8e
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/src/main/kotlin/org/hyacinthbots/lilybot/extensions/util/Reminders.kt b/src/main/kotlin/org/hyacinthbots/lilybot/extensions/util/Reminders.kt
@@ -761,6 +761,7 @@ class Reminders : Extension() {
 
 				// Remove the old reminder from the database
 				if (it.repeating) {
+					RemindMeCollection().removeReminder(it.guildId, it.userId, it.id)
 					RemindMeCollection().setReminder(
 						Clock.System.now(),
 						it.guildId,
@@ -773,7 +774,6 @@ class Reminders : Extension() {
 						it.repeatingInterval,
 						it.id
 					)
-					RemindMeCollection().removeReminder(it.guildId, it.userId, it.id)
 				} else {
 					RemindMeCollection().removeReminder(it.guildId, it.userId, it.id)
 				}