Selaa lähdekoodia

don't calculate cumulative value if series overrides stack to false

Mitsuhiro Tanda 10 vuotta sitten
vanhempi
commit
caccacf52b

+ 2 - 0
public/app/panels/graph/graph.tooltip.js

@@ -58,6 +58,8 @@ function ($) {
         if (scope.panel.stack) {
         if (scope.panel.stack) {
           if (scope.panel.tooltip.value_type === 'individual') {
           if (scope.panel.tooltip.value_type === 'individual') {
             value = series.data[hoverIndex][1];
             value = series.data[hoverIndex][1];
+          } else if (!series.stack) {
+            value = series.data[hoverIndex][1];
           } else {
           } else {
             last_value += series.data[hoverIndex][1];
             last_value += series.data[hoverIndex][1];
             value = last_value;
             value = last_value;

+ 18 - 2
public/test/specs/graph-tooltip-specs.js

@@ -73,8 +73,8 @@ define([
   describeSharedTooltip("steppedLine false, stack true, individual false", function(ctx) {
   describeSharedTooltip("steppedLine false, stack true, individual false", function(ctx) {
     ctx.setup(function() {
     ctx.setup(function() {
       ctx.data = [
       ctx.data = [
-        { data: [[10, 15], [12, 20]], },
-        { data: [[10, 2], [12, 3]], }
+        { data: [[10, 15], [12, 20]], stack: true },
+        { data: [[10, 2], [12, 3]], stack: true }
       ];
       ];
       ctx.scope.panel.stack = true;
       ctx.scope.panel.stack = true;
       ctx.pos = { x: 11 };
       ctx.pos = { x: 11 };
@@ -86,6 +86,22 @@ define([
 
 
   });
   });
 
 
+  describeSharedTooltip("steppedLine false, stack true, individual false, series stack false", function(ctx) {
+    ctx.setup(function() {
+      ctx.data = [
+        { data: [[10, 15], [12, 20]], stack: true },
+        { data: [[10, 2], [12, 3]], stack: false }
+      ];
+      ctx.scope.panel.stack = true;
+      ctx.pos = { x: 11 };
+    });
+
+    it('should not show stacked value', function() {
+      expect(ctx.results[1].value).to.be(2);
+    });
+
+  });
+
   describeSharedTooltip("steppedLine false, stack true, individual true", function(ctx) {
   describeSharedTooltip("steppedLine false, stack true, individual true", function(ctx) {
     ctx.setup(function() {
     ctx.setup(function() {
       ctx.data = [
       ctx.data = [