fixed filter widget parsed data error - fixes issue #149

Mottie · Oct 14, 2012 · 5187750 · 5187750
1 parent a0e5229
commit 5187750
Show file tree

Hide file tree

Showing 6 changed files with 12 additions and 13 deletions.
diff --git a/README.markdown b/README.markdown
diff --git a/js/jquery.tablesorter.js b/js/jquery.tablesorter.js
@@ -1,5 +1,5 @@
 /*!
-* TableSorter 2.4.2 - Client-side table sorting with ease!
+* TableSorter 2.4.3 - Client-side table sorting with ease!
 * @requires jQuery v1.2.6+
 *
 * Copyright (c) 2007 Christian Bach

diff --git a/js/jquery.tablesorter.min.js b/js/jquery.tablesorter.min.js
diff --git a/js/jquery.tablesorter.widgets.js b/js/jquery.tablesorter.widgets.js
@@ -412,8 +412,8 @@ $.tablesorter.addWidget({
 										ff = val === '' ? true : !(wo.filter_startsWith ? s === 0 : s >= 0);
 									// Look for operators >, >=, < or <=
 									} else if (/^[<>]=?/.test(val)){
-										rg = $.tablesorter.formatFloat(xi.replace(reg[5], ''), table);
+										// xi may be numeric - see issue #149
-										if (isNaN(rg)) { rg = xi; }
+										rg = isNaN(xi) ? $.tablesorter.formatFloat(xi.replace(reg[5], ''), table) : $.tablesorter.formatFloat(xi, table);
 										s = $.tablesorter.formatFloat(val.replace(reg[5], '').replace(reg[6],''), table);
 										if (/>/.test(val)) { ff = />=/.test(val) ? rg >= s : rg > s; }
 										if (/</.test(val)) { ff = /<=/.test(val) ? rg <= s : rg < s; }