Skip to content

Commit

Permalink
#312 Add 520 error handling to dashboard list and view routes
Browse files Browse the repository at this point in the history
  • Loading branch information
nadouani committed Dec 1, 2017
1 parent d7d7253 commit fffd084
Show file tree
Hide file tree
Showing 2 changed files with 16 additions and 3 deletions.
17 changes: 14 additions & 3 deletions ui/app/scripts/app.js
Original file line number Diff line number Diff line change
Expand Up @@ -300,20 +300,31 @@ angular.module('thehive', ['ngAnimate', 'ngMessages', 'ngSanitize', 'ui.bootstra
controller: 'DashboardViewCtrl',
controllerAs: '$vm',
resolve: {
dashboard: function(DashboardSrv, $stateParams, $q) {
dashboard: function(NotificationSrv, DashboardSrv, $stateParams, $q) {
var defer = $q.defer();

DashboardSrv.get($stateParams.id)
.then(function(response) {
defer.resolve(response.data);
}, function(err) {
NotificationSrv.error('DashboardViewCtrl', err.data, err.status);
defer.reject(err);
});

return defer.promise;
},
metadata: function(DashboardSrv) {
return DashboardSrv.getMetadata();
metadata: function($q, DashboardSrv, NotificationSrv) {
var defer = $q.defer();

DashboardSrv.getMetadata()
.then(function(response) {
defer.resolve(response);
}, function(err) {
NotificationSrv.error('DashboardViewCtrl', err.data, err.status);
defer.reject(err);
});

return defer.promise;
}
}
});
Expand Down
2 changes: 2 additions & 0 deletions ui/app/scripts/controllers/dashboard/DashboardsCtrl.js
Original file line number Diff line number Diff line change
Expand Up @@ -58,6 +58,8 @@
this.load = function() {
DashboardSrv.list().then(function(response) {
self.dashboards = response.data;
}, function(err){
NotificationSrv.error('DashboardsCtrl', err.data, err.status);
});
};

Expand Down

0 comments on commit fffd084

Please sign in to comment.