From f5369c05ac30387f05c5ac9126363118cd4e665e Mon Sep 17 00:00:00 2001
From: Karl Hudgell <karl.hudgell@bjss.com>
Date: Tue, 5 Nov 2024 14:34:53 +0000
Subject: [PATCH] fix sorting on accounts page

---
 templates/user_accounts.html | 15 +++++++++++----
 1 file changed, 11 insertions(+), 4 deletions(-)

diff --git a/templates/user_accounts.html b/templates/user_accounts.html
index 0c7c6b7..b88babf 100644
--- a/templates/user_accounts.html
+++ b/templates/user_accounts.html
@@ -90,20 +90,27 @@
     <script src="https://cdn.datatables.net/responsive/2.2.9/js/dataTables.responsive.min.js"></script>
     <script>
         $(document).ready(function() {
-            $.fn.dataTable.ext.type.order['date-eu'] = function(data) {
+            // Extend DataTables date sorting for DD/MM/YYYY format
+            $.fn.dataTable.ext.type.order['date-eu-pre'] = function(data) {
                 const parts = data.split('/');
-                return new Date(parts[2], parts[1] - 1, parts[0]).getTime();
+                // Check if the data is in the correct format before processing
+                if (parts.length === 3) {
+                    return new Date(parts[2], parts[1] - 1, parts[0]).getTime();
+                }
+                return data; // return as is if the format is unexpected
             };
-
+    
+            // Initialize DataTable with custom sorting
             $('#accountsTable').DataTable({
                 "searching": true,
                 "ordering": true,
                 "responsive": true,
                 "columnDefs": [
-                    { "type": "date-eu", "targets": 4 }
+                    { "type": "date-eu", "targets": 4 } // Use custom date-eu type for the date column
                 ]
             });
         });
     </script>
+    
 </body>
 </html>