From c134776d601d888924f2040d11387b6d4ba50359 Mon Sep 17 00:00:00 2001 From: Tomas Cohen Arazi Date: Mon, 1 Jul 2019 13:16:42 -0300 Subject: [PATCH] Bug 23246: Only jump into the 'images' tab HTML5MediaEnabled is set to This patch makes jumping into the third tab happen only if HTML5MediaEnabled is set to 'both' or 'staff'. To test: - Have HTML5MediaEnabled set to 'opac' or 'not at all' - Open the detail page of a record with no holdings => FAIL: The 'images' tab is selected - Apply this patch - Reload => SUCCESS: The 'holdings' tab is selected as it should be. - Sign off :-D Sponsored-by: America Numismatic Society Signed-off-by: Maryse Simard --- koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt index 6af2687..50f93c7 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt @@ -896,9 +896,11 @@ $(document).ready(function() { $('#bibliodetails').tabs(); - [% IF count == 0 %] - $('#bibliodetails').tabs("option", "active", 3); - [% END %] + [% IF count == 0 and + ( Koha.Preference('HTML5MediaEnabled') == 'staff' or + Koha.Preference('HTML5MediaEnabled') == 'both' ) %] + $('#bibliodetails').tabs("option", "active", 3); + [% END %] $('#search-form').focus(); $('.thumbnails > li > .remove').click(function() { var result = confirm(_("Are you sure you want to delete this cover image?")); -- 2.7.4