Fix addTask to insert new tasks in correct order (fixes #2)

jaclarke · Jul 22, 2019 · 6e8a1fe · 6e8a1fe
1 parent f9ec76f
commit 6e8a1fe
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "cronosjs",
-  "version": "1.2.0",
+  "version": "1.2.1",
   "description": "A cron based task scheduler for node and the browser, with extended syntax and timezone support.",
   "keywords": [
     "cron",

diff --git a/src/scheduler.ts b/src/scheduler.ts
@@ -8,7 +8,7 @@ let runningTimer: number | null = null
 function addTask(task: CronosTask) {
   if (task['_timestamp']) {
     const insertIndex = scheduledTasks.findIndex(t => (t['_timestamp'] || 0) < (task['_timestamp'] || 0))
-    if (insertIndex >= 0) scheduledTasks.splice(insertIndex + 1, 0, task)
+    if (insertIndex >= 0) scheduledTasks.splice(insertIndex, 0, task)
     else scheduledTasks.push(task)
   }
 }