Fix the comparator when dealing with nulls.
Turns out neither 'x' < 0 nor 'x' > 0. However, 'x' > '0', and '0' < 5, and
!('0' > 5). So basically, if it's undefined, order it like '0' instead of 0.
diff --git a/render.js b/render.js
index daf84ce..36b816b 100644
--- a/render.js
+++ b/render.js
@@ -991,9 +991,11 @@
var keycol = keycols[keyi][0], invert = keycols[keyi][1];
var av = a[keycol], bv = b[keycol];
if (grid.types[keycol] === T_NUM) {
- av = parseFloat(av) || 0;
- bv = parseFloat(bv) || 0;
+ av = parseFloat(av);
+ bv = parseFloat(bv);
}
+ av = av || '0'; // ensure consistent ordering given NaN and undefined
+ bv = bv || '0';
if (av < bv) {
return -1 * invert;
} else if (av > bv) {
