From 5799f4bed2918dd21d2d4d4606c5b2f6aa2651d9 Mon Sep 17 00:00:00 2001 From: Jonathan Druart Date: Fri, 9 Jun 2023 16:36:43 +0200 Subject: [PATCH] Bug 33973: Restore sorting on Vue tables Regression from the KohaTable Vue component added by bug 33066. _dt_default_ajax expects options.columns to contain the columns. Test plan: Add some agreements, go the the agreement list view and confirm that you can now sort the agreements using the header columns. Note that the CSS is not correct yet. My understanding is that the DataTable Vue component keep the 'sorting' class on the th which display the grey up/down icon over the blue icon which indicate the current sort column. Signed-off-by: Pedro Amorim Signed-off-by: Katrin Fischer Signed-off-by: Tomas Cohen Arazi (cherry picked from commit 7990f3e74142ab20c813534ea615586965524415) Signed-off-by: Martin Renvoize --- koha-tmpl/intranet-tmpl/prog/js/vue/components/KohaTable.vue | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/koha-tmpl/intranet-tmpl/prog/js/vue/components/KohaTable.vue b/koha-tmpl/intranet-tmpl/prog/js/vue/components/KohaTable.vue index 6637980251..59376f00af 100644 --- a/koha-tmpl/intranet-tmpl/prog/js/vue/components/KohaTable.vue +++ b/koha-tmpl/intranet-tmpl/prog/js/vue/components/KohaTable.vue @@ -44,7 +44,7 @@ export default { ajax: { url: typeof this.url === "function" ? this.url() : this.url, ..._dt_default_ajax({ - options: this.options, + options: { ...this.options, columns: this.columns }, default_filters: this.default_filters, }), }, -- 2.39.2