feat: made jobList key named by timeframe

Alex-Werner · Jul 24, 2023 · c0b05c0 · c0b05c0
1 parent 9768beb
commit c0b05c0
Showing 1 changed file with 7 additions and 5 deletions.
diff --git a/src/Kronos.js b/src/Kronos.js
@@ -7,7 +7,7 @@ class Kronos extends EventEmitter {
   constructor(){
     super();
 
-    this.jobList = [];
+    this.jobList = {};
   }
   subscribe(timeframe){
     let cronRule;
@@ -47,15 +47,17 @@ class Kronos extends EventEmitter {
     const start = false;
     const utcOffset = moment().utcOffset();
     const job = new CronJob(cronRule, onTick, onComplete, start, null, null, null, utcOffset);
-    this.jobList.push(job);
-    job.start();
+    if(!this.jobList[timeframe]){
+      this.jobList[timeframe] = job;
+    }
+    this.jobList[timeframe].start();
     this.emit('SUBSCRIPTIONS', {type:'SUBSCRIBED', payload:{timeframe}});
   }
   unsubscribeAll(){
-    this.jobList.forEach((job)=>{
+    Object.entries(this.jobList).forEach(([,job])=>{
       job.stop();
     });
-    this.jobList = [];
+    this.jobList = {};
   }
 };
 module.exports = Kronos;