Fix(Datatables JS): Refactor turbolinks cache fix

hpi-swt2 · Mar 23, 2019 · 7c552fb · 7c552fb
1 parent a4f585d
commit 7c552fb
Showing 1 changed file with 4 additions and 6 deletions.
diff --git a/app/assets/javascripts/datatables.js b/app/assets/javascripts/datatables.js
@@ -21,14 +21,12 @@ $(document).on('turbolinks:load', function() {
   }
 });
 
-// https://stackoverflow.com/questions/41070556/how-can-i-prevent-duplicate-wrappers-on-a-jquery-datatable-when-navigating-back
 // Turbolinks cache fix
+// https://stackoverflow.com/questions/41070556
 $(document).on('turbolinks:before-cache', function() {
-  var dataTable = $($.fn.dataTable.tables(true)).DataTable();
-  if (dataTable !== null) {
-    dataTable.clear();
-    dataTable.destroy();
-    return dataTable = null;
+  var tables = $.fn.dataTable.tables(true);
+  if (tables.length > 0) {
+    $(tables).DataTable().destroy();
   }
 });