harish2704/node-job-manager
1
0
Fork 0
mirror of https://github.com/harish2704/node-job-manager.git synced 2026-09-10 13:11:08 +00:00
Code Issues Projects Releases Wiki Activity

Fix: Don't call onStopped immediatly on stop(). Instead call onStopped() after finishing all running tasks

Browse Source
This commit is contained in:
harish2704
2014-09-12 21:45:38 +05:30
parent 3e898725f3
commit 971f97e5f5
1 changed files with 3 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/JobManager.js
+3 -1
View File
@@ -90,6 +90,9 @@ JobManager.prototype.$doWork_ = function( cb ){
} }
} }
cb(); cb();
if( self.state == STATE.NOT_RUNNING ){
if( ( self.runningTasks == 1 ) && self.onStopped ) { self.onStopped(); }
}
self.returnToPool( worker ); self.returnToPool( worker );
}); });
}; };
@@ -124,7 +127,6 @@ JobManager.prototype.pause = function(){
JobManager.prototype.stop = function(){ JobManager.prototype.stop = function(){
this.state = STATE.NOT_RUNNING; this.state = STATE.NOT_RUNNING;
if( this.onStopped ){ this.onStopped(); }
}; };