From 06c875631f7f76bc53de53621c635b97008855fe Mon Sep 17 00:00:00 2001 From: Fridolin Somers Date: Thu, 19 Oct 2023 09:17:57 -1000 Subject: [PATCH] Bug 35110: Authorities editor with JS error when only one tab Exactly like Bug 34251 for biblios : When using the an authorities framework with all fields in the same tab, there is thrown a JS error in selectTab( "#tab0XX_panel" ); This is because no nav tabs are build in authorities/authorities.tt as BIG_LOOP has a length(size) of 1 Test plan : 1) Edit an authority framework, for example 'Meeting Name', to set all sufields in tab 0 You may use SQL : UPDATE auth_subfield_structure SET tab=0 WHERE authtypecode='MEETI_NAME' Don't forget to restart memcached 2) Go to 'Authorities' 3) Click on 'New authority' then 'Meeting Name' 4) Check you don't see any JavaScript error 5) Check you can save Signed-off-by: David Nind Signed-off-by: Katrin Fischer --- .../intranet-tmpl/prog/en/modules/authorities/authorities.tt | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/authorities/authorities.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/authorities/authorities.tt index 4e8a81ed39..cbae792256 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/authorities/authorities.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/authorities/authorities.tt @@ -129,6 +129,10 @@ }); function selectTab( tablink ){ + /* return if no tabs displayed (BIG_LOOP.size <= 1) */ + if ($(".toolbar-tabs-container .nav-tabs li").length === 0){ + return; + } let a = $("a[href='" + tablink + "']"); $(".toolbar-tabs-container .nav-tabs li").removeClass("selected"); a.tab("show").parent().addClass("selected"); -- 2.30.2