fix(pat-structure): Fix ordering logic.

thet · thet · commit 57cd7ddc8883 · 2026-03-10T17:22:16.000+01:00
The logic to set orderable columns in the structure pattern was quite wrong: - The very first and second-to-last columns were set to no ordering. This PR fixes it to the very first (selection checkboxes) and very last column (actions column). - The very first column was set for a default, ascending ordering. This went unnoticed, as the checkbox cells all have the same order information. This PR sets no initial ordering, as we want the order position in the container set as the default ordering. Fixes: #1551
diff --git a/src/pat/structure/js/views/table.js b/src/pat/structure/js/views/table.js
@@ -78,11 +78,14 @@ export default BaseView.extend({
         // Ignore first column and the last one (activeColumns.length + 1)
         // Do not show paginator, search or information, we only want column sorting
         const datatables_options = {
-            order: [0, "asc"],
+            // Don't apply initial ordering.
+            order: [],
             columnDefs: [
                 {
+                    // The first (selection checkboxes) and last (actions)
+                    // columns should not be sortable.
                     orderable: false,
-                    targets: [0, this.app.activeColumns.length + 1],
+                    targets: [0, this.app.activeColumns.length + 2],
                 },
             ],
             paging: false,
