From f27c48786235933cde9680d3fbec6321917a6e4e Mon Sep 17 00:00:00 2001 From: Allan Jardine Date: Thu, 12 May 2011 20:46:33 +0100 Subject: [PATCH] Fix: fnSetColumnVis was broken when used with deferred rendering due to the offset calculation done with the huge flat array of TD nodes. The fix is to simplify this somewhat and get an array for each row (when it is available) and process that using the optional parameter for _fnGetTdNodes. This doesn't result any any extra computation time, other than the addional time to call the function (but not to execute it), since it is just moving where the loop is. --- media/js/jquery.dataTables.js | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/media/js/jquery.dataTables.js b/media/js/jquery.dataTables.js index 80aa86bb..9abe445e 100644 --- a/media/js/jquery.dataTables.js +++ b/media/js/jquery.dataTables.js @@ -2085,12 +2085,11 @@ else { /* Remove a column from display */ - anTds = _fnGetTdNodes( oSettings ); for ( i=0, iLen=oSettings.aoData.length ; i