From c5db208ab374b60301cdff107fd2d9f21dae4f95 Mon Sep 17 00:00:00 2001 From: "Harish.K" Date: Mon, 22 Sep 2014 15:24:35 +0530 Subject: [PATCH] Fix: no need to assign tasks before initial start of JobManager. tasks are loaded using onLoadmore at start time --- src/JobManager.js | 12 ++++++++++-- 1 file changed, 10 insertions(+), 2 deletions(-) diff --git a/src/JobManager.js b/src/JobManager.js index 872ce9c..01b1fd4 100644 --- a/src/JobManager.js +++ b/src/JobManager.js @@ -110,14 +110,22 @@ JobManager.prototype.$trigger = function (){ JobManager.prototype.start = function( ) { this.state = STATE.RUNNING; this.updateState(); - this.$trigger(); + var self = this; + if( this.tasks.length ) { + return this.$trigger(); + } + return this.$onLoadMore( function(){ + self.$trigger(); + }); }; -JobManager.prototype.$onLoadMore = function(){ +JobManager.prototype.$onLoadMore = function( cb ){ + cb = cb || function(){}; var self = this; self.isLoadingTakingPlace = true; self.onLoadMore(function(){ self.isLoadingTakingPlace = false; + return cb(); }); };