From 42f2d0b1791c18a64a97b2f72d44872be13944d8 Mon Sep 17 00:00:00 2001 From: Jonathan Druart Date: Tue, 31 Mar 2015 15:29:49 +0200 Subject: [PATCH] Bug 8399: Display tab number on the mandatory field alert (authority) MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit A wrong variable scope was used here, the number value is contained in the BIG_LOOP (note for later: change this variable name...). Test plan: 1/ Define 1+ mandatory fields for an authority type 2/ Create an authority and don't fill the mandatory fields 3/ Try to save: the popup alert should display the number of the tab. Signed-off-by: Andreas Hedström Mace Signed-off-by: Katrin Fischer Signed-off-by: Tomas Cohen Arazi --- .../intranet-tmpl/prog/en/modules/authorities/authorities.tt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) 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 de0d2ab056..d6b3a2ed01 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/authorities/authorities.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/authorities/authorities.tt @@ -62,7 +62,7 @@ function AreMandatoriesNotOk(){ [% END %] [% FOREACH subfield_loo IN innerloo.subfield_loop %] [% IF ( subfield_loo.mandatory ) %]mandatories.push("[% subfield_loo.id %]"); - tab.push("[% subfield_loo.number %]"); + tab.push("[% BIG_LOO.number %]"); label.push("[% subfield_loo.marc_lib |replace("'", "\'") |replace('"', '\"') |replace('\n', '\\n') |replace('\r', '\\r') %]"); [% END %] [% END %] -- 2.39.5