From 41bf4e28bd32d5234b3c643a6c007cc444b86ab4 Mon Sep 17 00:00:00 2001
From: Allan Jardine <allan.jardine@sprymedia.co.uk>
Date: Sat, 25 Jun 2011 07:40:30 +0100
Subject: [PATCH] Fix: Columns which are sortable but hidden have events
 attached to them for sorting, but this event handler is not removed when the
 column is made visible again for fnDestroy. Simply need to change the order
 of how the destroy is done - i.e. remove events just after the columns are
 made visible, not before - 5497

---
 media/js/jquery.dataTables.js | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/media/js/jquery.dataTables.js b/media/js/jquery.dataTables.js
index 4386768f..947fd14e 100644
--- a/media/js/jquery.dataTables.js
+++ b/media/js/jquery.dataTables.js
@@ -2138,9 +2138,6 @@
 			/* Flag to note that the table is currently being destoryed - no action should be taken */
 			oSettings.bDestroying = true;
 			
-			/* Blitz all DT events */
-			$(oSettings.nTableWrapper).find('*').andSelf().unbind('.DT');
-			
 			/* Restore hidden columns */
 			for ( i=0, iLen=oSettings.aoColumns.length ; i<iLen ; i++ )
 			{
@@ -2150,6 +2147,9 @@
 				}
 			}
 			
+			/* Blitz all DT events */
+			$(oSettings.nTableWrapper).find('*').andSelf().unbind('.DT');
+			
 			/* If there is an 'empty' indicator row, remove it */
 			$('tbody>tr>td.'+oSettings.oClasses.sRowEmpty, oSettings.nTable).parent().remove();