]> git.koha-community.org Git - koha.git/commit
Bug 37395: Fix columns visibility on Vue tables
authorJonathan Druart <jonathan.druart@bugs.koha-community.org>
Tue, 24 Sep 2024 12:20:59 +0000 (14:20 +0200)
committerKatrin Fischer <katrin.fischer@bsz-bw.de>
Wed, 23 Oct 2024 17:14:50 +0000 (19:14 +0200)
commit1a8542f05dbde8476d634a0da1ac19386358c7d4
tree131e975b9bc1b346f02864fa3efb24d0985c69db
parent07cf21e6b915d34d64dc96995f6df8fce50c6798
Bug 37395: Fix columns visibility on Vue tables

On bug 33066 we reworked a lot of datatables.js to make the JS function
reusable from the Vue component (and especially the KohaTable
component). I guess the regression comes from this change, but I am
surprised that we didn't catch it when testing.

The was a mistake for _dt_visibility, we didn't pass the node. Actually
we cannot pass it as we do not have it yet.

This "node" variable is only used when bKohaColumnsUseNames is passed
(ie. we do not rely on the order of the columns but we need to rely on
their data-colname attributes).

There is then a limitation here: we cannot use bKohaColumnsUseNames from
Vue, but we do not yet.

An ideal fix would be too big for now, especially for an hypothetic use
case.

Test plan:
Confirm that the "Columns" button are now back for the tables of the Vue
apps (ERM, Preservation) and that the settings are taken into account
(hide by default, etc.)

Signed-off-by: Olivier V <olivier.vezina@inLibro.com>
Signed-off-by: Pedro Amorim <pedro.amorim@ptfs-europe.com>
Signed-off-by: Katrin Fischer <katrin.fischer@bsz-bw.de>
koha-tmpl/intranet-tmpl/prog/js/datatables.js