Fix tooltip content when null included - #1631

c3.js

@@ -3891,19 +3891,21 @@
             orderAsc = $$.isOrderAsc();
 
         if (config.data_groups.length === 0) {
-            d.sort(function(a,b){
-                return orderAsc ? a.value - b.value : b.value - a.value;
+            d.sort(function(a, b){
+                var v1 = a ? a.value : null, v2 = b ? b.value : null;
+                return orderAsc ? v1 - v2 : v2 - v1;
             });
         } else {
             var ids = $$.orderTargets($$.data.targets).map(function (i) {
                 return i.id;
             });
             d.sort(function(a, b) {
-                if (a.value > 0 && b.value > 0) {
-                    return orderAsc ? ids.indexOf(a.id) - ids.indexOf(b.id) : ids.indexOf(b.id) - ids.indexOf(a.id);
-                } else {
-                    return orderAsc ? a.value - b.value : b.value - a.value;
+                var v1 = a ? a.value : null, v2 = b ? b.value : null;
+                if (v1 > 0 && v2 > 0) {
+                    v1 = a ? ids.indexOf(a.id) : null;
+                    v2 = b ? ids.indexOf(b.id) : null;
                 }
+                return orderAsc ? v1 - v2 : v2 - v1;
             });
         }
 

src/tooltip.js

@@ -33,19 +33,21 @@ c3_chart_internal_fn.getTooltipContent = function (d, defaultTitleFormat, defaul
         orderAsc = $$.isOrderAsc();
 
     if (config.data_groups.length === 0) {
-        d.sort(function(a,b){
-            return orderAsc ? a.value - b.value : b.value - a.value;
+        d.sort(function(a, b){
+            var v1 = a ? a.value : null, v2 = b ? b.value : null;
+            return orderAsc ? v1 - v2 : v2 - v1;
         });
     } else {
         var ids = $$.orderTargets($$.data.targets).map(function (i) {
             return i.id;
         });
         d.sort(function(a, b) {
-            if (a.value > 0 && b.value > 0) {
-                return orderAsc ? ids.indexOf(a.id) - ids.indexOf(b.id) : ids.indexOf(b.id) - ids.indexOf(a.id);
-            } else {
-                return orderAsc ? a.value - b.value : b.value - a.value;
+            var v1 = a ? a.value : null, v2 = b ? b.value : null;
+            if (v1 > 0 && v2 > 0) {
+                v1 = a ? ids.indexOf(a.id) : null;
+                v2 = b ? ids.indexOf(b.id) : null;
             }
+            return orderAsc ? v1 - v2 : v2 - v1;
         });
     }