#140 Make sur to handle 401 responses of charts queries

TheHive-Project · Mar 7, 2017 · 567f0a5 · 567f0a5
1 parent 94c342c
commit 567f0a5
Show file tree

Hide file tree

Showing 4 changed files with 12 additions and 6 deletions.
diff --git a/ui/app/scripts/directives/charts/donut-chart.js b/ui/app/scripts/directives/charts/donut-chart.js
@@ -1,6 +1,6 @@
 (function() {
     'use strict';
-    angular.module('theHiveDirectives').directive('donutChart', function(StatSrv, $state, $base64) {
+    angular.module('theHiveDirectives').directive('donutChart', function(StatSrv, $state, $base64, AlertSrv) {
         return {
             restrict: 'E',
             scope: {
@@ -74,6 +74,8 @@
                             }
                         };
 
+                    }, function(err) {
+                        AlertSrv.error('donutChart', err.data, err.status);
                     });
                 };
 

diff --git a/ui/app/scripts/directives/charts/duration-over-time-chart.js b/ui/app/scripts/directives/charts/duration-over-time-chart.js
@@ -1,6 +1,6 @@
 (function() {
     'use strict';
-    angular.module('theHiveDirectives').directive('durationOverTimeChart', function($http, $interpolate, ChartSrv) {
+    angular.module('theHiveDirectives').directive('durationOverTimeChart', function($http, $interpolate, ChartSrv, AlertSrv) {
         return {
             restrict: 'E',
             scope: {
@@ -127,8 +127,8 @@
                             }
                         };
 
-                    }, function(response) {
-                        console.error(response);
+                    }, function(err) {
+                        AlertSrv.error('durationOverTimeChart', err.data, err.status);
                     });
                 };
 

diff --git a/ui/app/scripts/directives/charts/histo-chart.js b/ui/app/scripts/directives/charts/histo-chart.js
@@ -1,6 +1,6 @@
 (function() {
     'use strict';
-    angular.module('theHiveDirectives').directive('histoChart', function($http, ChartSrv) {
+    angular.module('theHiveDirectives').directive('histoChart', function($http, ChartSrv, AlertSrv) {
         return {
             restrict: 'E',
             scope: {
@@ -88,6 +88,8 @@
                                 enabled: scope.options.zoom || false
                             }
                         };
+                    }, function(err) {
+                        AlertSrv.error('histoChart', err.data, err.status);
                     });
                 };
 

diff --git a/ui/app/scripts/directives/charts/metric-histo-chart.js b/ui/app/scripts/directives/charts/metric-histo-chart.js
@@ -1,6 +1,6 @@
 (function() {
     'use strict';
-    angular.module('theHiveDirectives').directive('metricHistoChart', function($http, $interpolate, MetricsCacheSrv, ChartSrv) {
+    angular.module('theHiveDirectives').directive('metricHistoChart', function($http, $interpolate, MetricsCacheSrv, ChartSrv, AlertSrv) {
         return {
             restrict: 'E',
             scope: {
@@ -123,6 +123,8 @@
 
 
                         scope.chart = chart;
+                    }, function(err) {
+                        AlertSrv.error('metricHistoChart', err.data, err.status);
                     });
                 };