fix: more bugs in execution engine

Author: titivermeesch

build.gradle

@@ -9,7 +9,7 @@ java {
 }
 
 group = 'me.playbosswar.com'
-version = '8.16.2'
+version = '8.16.3'
 description = 'CommandTimer'
 
 repositories {
@@ -74,7 +74,7 @@ publishing {
         maven(MavenPublication) {
             groupId = 'me.playbosswar.com'
             artifactId = 'commandtimer'
-            version = '8.16.2'
+            version = '8.16.3'
 
             from components.java
         }

java17-build.gradle

@@ -10,7 +10,7 @@ java {
 
 
 group = 'me.playbosswar.com'
-version = '8.16.2'
+version = '8.16.3'
 description = 'CommandTimer'
 
 repositories {
@@ -63,7 +63,7 @@ publishing {
         maven(MavenPublication) {
             groupId = 'me.playbosswar.com'
             artifactId = 'commandtimer-java17'
-            version = '8.16.2'
+            version = '8.16.3'
 
             from components.java
         }

java21-build.gradle

@@ -10,7 +10,7 @@ java {
 
 
 group = 'me.playbosswar.com'
-version = '8.16.2'
+version = '8.16.3'
 description = 'CommandTimer'
 
 repositories {
@@ -67,7 +67,7 @@ publishing {
         maven(MavenPublication) {
             groupId = 'me.playbosswar.com'
             artifactId = 'commandtimer-java21'
-            version = '8.16.2'
+            version = '8.16.3'
             from components.java
         }
     }

src/main/java/me/playbosswar/com/tasks/TasksManager.java

@@ -210,17 +210,21 @@ public void populateScheduleForTask(Task task) {
                 .filter(scheduledTask -> scheduledTask.getTask().getId().equals(task.getId()))
                 .map(ScheduledTask::getDate).max(ZonedDateTime::compareTo).orElse(null);
 
+        boolean hadNoScheduledTasks = (latestScheduledDate == null);
+        ZonedDateTime lastExecuted = task.getLastExecuted().toInstant().atZone(ZoneId.systemDefault());
+        ZonedDateTime now = ZonedDateTime.now();
         if (latestScheduledDate == null) {
-            ZonedDateTime now = ZonedDateTime.now();
-            ZonedDateTime lastExecuted = task.getLastExecuted().toInstant().atZone(ZoneId.systemDefault());
-            latestScheduledDate = lastExecuted.isAfter(now) ? lastExecuted : now;
+            latestScheduledDate = lastExecuted;
         }
 
         if (maxToSchedule <= 0) {
             return;
         }
 
         if (!task.getTimes().isEmpty()) {
+            if (latestScheduledDate.isBefore(now)) {
+                latestScheduledDate = now;
+            }
             List<TaskTime> rangeTimes = new ArrayList<>();
             List<TaskTime> nonRangeTimes = new ArrayList<>();
 
@@ -383,14 +387,33 @@ public void populateScheduleForTask(Task task) {
             return;
         }
 
+        long intervalSeconds = task.getInterval().toSeconds();
+        if (intervalSeconds <= 0) intervalSeconds = 1;
+        
+        ZonedDateTime nextExpectedExecution = lastExecuted.plusSeconds(intervalSeconds);
+        
+        if (hadNoScheduledTasks && nextExpectedExecution.isBefore(now)) {
+            if (task.getDays().contains(nextExpectedExecution.toLocalDate().getDayOfWeek())) {
+                scheduledTasks.add(new ScheduledTask(task, nextExpectedExecution));
+                maxToSchedule--;
+            }
+        }
+        
         int i = 1;
+        ZonedDateTime startDate = latestScheduledDate;
+        
         while (maxToSchedule > 0) {
-            ZonedDateTime date = latestScheduledDate.plusSeconds(i * task.getInterval().toSeconds());
+            ZonedDateTime date = startDate.plusSeconds(i * intervalSeconds);
             if (!task.getDays().contains(date.getDayOfWeek())) {
                 i++;
                 continue;
             }
 
+            if (date.isBefore(now)) {
+                i++;
+                continue;
+            }
+
             scheduledTasks.add(new ScheduledTask(task, date));
             maxToSchedule--;
             i++;

src/main/resources/plugin.yml

@@ -1,6 +1,6 @@
 main: me.playbosswar.com.CommandTimerPlugin
 name: "CommandTimer"
-version: "8.16.2"
+version: "8.16.3"
 description: "Schedule commands like you want"
 author: PlayBossWar
 api-version: 1.13