You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kylin.apache.org by zh...@apache.org on 2015/07/13 13:04:50 UTC
incubator-kylin git commit: KYLIN-888,
fix loading more jobs button lost issue
Repository: incubator-kylin
Updated Branches:
refs/heads/0.7-staging f116f2c41 -> 1c4ab4fea
KYLIN-888,fix loading more jobs button lost issue
Project: http://git-wip-us.apache.org/repos/asf/incubator-kylin/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-kylin/commit/1c4ab4fe
Tree: http://git-wip-us.apache.org/repos/asf/incubator-kylin/tree/1c4ab4fe
Diff: http://git-wip-us.apache.org/repos/asf/incubator-kylin/diff/1c4ab4fe
Branch: refs/heads/0.7-staging
Commit: 1c4ab4fea1aea3257fd57eab30984bff7abecaca
Parents: f116f2c
Author: jiazhong <ji...@ebay.com>
Authored: Mon Jul 13 19:00:57 2015 +0800
Committer: jiazhong <ji...@ebay.com>
Committed: Mon Jul 13 19:03:08 2015 +0800
----------------------------------------------------------------------
webapp/app/js/controllers/job.js | 3 +--
webapp/app/js/model/jobListModel.js | 6 ++++--
2 files changed, 5 insertions(+), 4 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/incubator-kylin/blob/1c4ab4fe/webapp/app/js/controllers/job.js
----------------------------------------------------------------------
diff --git a/webapp/app/js/controllers/job.js b/webapp/app/js/controllers/job.js
index 812316e..2303ce5 100644
--- a/webapp/app/js/controllers/job.js
+++ b/webapp/app/js/controllers/job.js
@@ -80,11 +80,10 @@ KylinApp
};
$scope.state.loading = true;
- var defer = $q.defer();
return JobList.list(jobRequest).then(function (resp) {
$scope.state.loading = false;
defer.resolve(resp);
- defer.promise;
+ return defer.promise;
});
}
http://git-wip-us.apache.org/repos/asf/incubator-kylin/blob/1c4ab4fe/webapp/app/js/model/jobListModel.js
----------------------------------------------------------------------
diff --git a/webapp/app/js/model/jobListModel.js b/webapp/app/js/model/jobListModel.js
index 61c5cc3..7dc8428 100755
--- a/webapp/app/js/model/jobListModel.js
+++ b/webapp/app/js/model/jobListModel.js
@@ -35,8 +35,10 @@ KylinApp.service('JobList', function (JobService, $q) {
_this.jobs[id] = job;
}
});
-
- defer.resolve(jobs.length);
+ _this.jobs = _this.jobs.concat(jobs);
+ defer.resolve(_this.jobs.length);
+ },function(){
+ defer.reject("failed to load jobs");
});
return defer.promise;