Bugzilla – Attachment 174779 Details for
Bug 38436
Adjust code for column visibility (after DataTables upgrade)
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 38436: Do not deal with 'Columns' button if no table settings
Bug-38436-Do-not-deal-with-Columns-button-if-no-ta.patch (text/plain), 2.18 KB, created by
Jonathan Druart
on 2024-11-19 10:21:33 UTC
(
hide
)
Description:
Bug 38436: Do not deal with 'Columns' button if no table settings
Filename:
MIME Type:
Creator:
Jonathan Druart
Created:
2024-11-19 10:21:33 UTC
Size:
2.18 KB
patch
obsolete
>From 4da36310ff3413f18da7ff5d910c3ddc863f2ae2 Mon Sep 17 00:00:00 2001 >From: Jonathan Druart <jonathan.druart@bugs.koha-community.org> >Date: Sat, 16 Nov 2024 08:05:05 +0100 >Subject: [PATCH] Bug 38436: Do not deal with 'Columns' button if no table > settings > >If the table does not have table settings we should not deal with the >column visibility and not display the 'Columns' button. > >Signed-off-by: Owen Leonard <oleonard@myacpl.org> >Signed-off-by: Paul Derscheid <paul.derscheid@lmscloud.de> >--- > koha-tmpl/intranet-tmpl/prog/js/datatables.js | 19 +++++++++++++------ > 1 file changed, 13 insertions(+), 6 deletions(-) > >diff --git a/koha-tmpl/intranet-tmpl/prog/js/datatables.js b/koha-tmpl/intranet-tmpl/prog/js/datatables.js >index f7d1ebb4a31..a17e183b2af 100644 >--- a/koha-tmpl/intranet-tmpl/prog/js/datatables.js >+++ b/koha-tmpl/intranet-tmpl/prog/js/datatables.js >@@ -668,12 +668,19 @@ function _dt_buttons(params){ > // But ideally should be retrieved using table.data() > let use_names = settings.bKohaColumnsUseNames; > let included_columns = []; >- if ( use_names ) { >- // bKohaColumnsUseNames is set, identify columns by their data-colname >- included_columns = table_settings.columns.filter(c => !c.cannot_be_toggled).map(c => "[data-colname='%s']".format(c.columnname)).join(','); >- } else { >- // Not set, columns are ordered the same than in the columns settings >- included_columns = table_settings.columns.map((c, i) => !c.cannot_be_toggled ? i : null).filter(i => i !== null); >+ if (table_settings) { >+ if (use_names) { >+ // bKohaColumnsUseNames is set, identify columns by their data-colname >+ included_columns = table_settings.columns >+ .filter(c => !c.cannot_be_toggled) >+ .map(c => "[data-colname='%s']".format(c.columnname)) >+ .join(","); >+ } else { >+ // Not set, columns are ordered the same than in the columns settings >+ included_columns = table_settings.columns >+ .map((c, i) => (!c.cannot_be_toggled ? i : null)) >+ .filter(i => i !== null); >+ } > } > if( included_columns.length > 0 ){ > buttons.push( >-- >2.34.1
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 38436
:
174587
|
174588
|
174589
|
174590
|
174591
|
174592
|
174593
|
174594
|
174595
|
174634
|
174641
|
174642
|
174688
|
174689
|
174690
|
174691
|
174692
|
174693
|
174694
|
174695
|
174696
|
174697
|
174698
|
174699
|
174700
|
174701
|
174702
|
174721
|
174722
|
174723
|
174724
|
174725
|
174726
|
174727
|
174728
|
174729
|
174730
|
174731
|
174732
|
174733
|
174734
|
174735
|
174736
|
174740
|
174741
|
174742
|
174743
|
174744
|
174745
|
174746
|
174747
|
174748
|
174749
|
174750
|
174751
|
174752
|
174753
|
174754
|
174755
|
174756
|
174766
|
174767
|
174768
|
174769
|
174770
|
174771
|
174772
|
174773
|
174774
|
174775
|
174776
|
174777
|
174778
| 174779 |
174780
|
174781
|
174782
|
174821
|
174967