Fix: Destroy should use detatch() not remove()

- This is to preserve user added listeners on destroy. If they want to
  remove their listeners to prevent memory leaks, then they should do
  so. But DataTables won't do it for them, as actually, you might want
  to keep your event listeners!

.datatables-commit-sync

@@ -1 +1 @@
-fb8a718af86bdcb880606e67c30bd118e3efd174
+d8efc98282583e4661bb38af9449f0fa4bb79996

media/js/jquery.dataTables.js

@@ -8548,24 +8548,26 @@
 				new _Api( settings ).columns().visible( true );
 			}
 	
-			// Blitz all DT events
+			// Blitz all `DT` namespaced events (these are internal events, the
+			// lowercase, `dt` events are user subscribed and they are responsible
+			// for removing them
 			jqWrapper.unbind('.DT').find(':not(tbody *)').unbind('.DT');
 			$(window).unbind('.DT-'+settings.sInstance);
 	
 			// When scrolling we had to break the table up - restore it
 			if ( table != thead.parentNode ) {
-				jqTable.children('thead').remove();
+				jqTable.children('thead').detach();
 				jqTable.append( thead );
 			}
 	
 			if ( tfoot && table != tfoot.parentNode ) {
-				jqTable.children('tfoot').remove();
+				jqTable.children('tfoot').detach();
 				jqTable.append( tfoot );
 			}
 	
 			// Remove the DataTables generated nodes, events and classes
-			jqTable.remove();
-			jqWrapper.remove();
+			jqTable.detach();
+			jqWrapper.detach();
 	
 			settings.aaSorting = [];
 			settings.aaSortingFixed = [];
@@ -8578,11 +8580,11 @@
 			);
 	
 			if ( settings.bJUI ) {
-				$('th span.'+classes.sSortIcon+ ', td span.'+classes.sSortIcon, thead).remove();
+				$('th span.'+classes.sSortIcon+ ', td span.'+classes.sSortIcon, thead).detach();
 				$('th, td', thead).each( function () {
 					var wrapper = $('div.'+classes.sSortJUIWrapper, this);
 					$(this).append( wrapper.contents() );
-					wrapper.remove();
+					wrapper.detach();
 				} );
 			}