From 9c07acd0cca79b703a7d979814ef627872d72a35 Mon Sep 17 00:00:00 2001 From: David Gustafsson Date: Thu, 26 Sep 2019 11:50:14 +0200 Subject: [PATCH] Bug 23680: Incorrect conversion of Perl boolean to js A Perl boolean value is on the server side converted to a JS string (resulting in the values '0' or '1). Since the string '0' is a truthy value in JS this incorrectly triggers window.close() on the client in many instances, for example when adding a new item. To test: 1. Go to a bibliographic record in the staff client. 2. Expand the "New" menu and open "New item" in new tab. 3. The new tab will immediately close. 4. Apply the patch. 5. Perform steps 1-2 again. 6. The tab should now remain open. Sponsored-by: Gothenburg University Library Signed-off-by: David Nind Signed-off-by: Katrin Fischer Signed-off-by: Martin Renvoize (cherry picked from commit b488527ebeb3b335854f2290aa1c6cf80f464eb0) Signed-off-by: Fridolin Somers --- .../prog/en/includes/str/cataloging_additem.inc | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/koha-tmpl/intranet-tmpl/prog/en/includes/str/cataloging_additem.inc b/koha-tmpl/intranet-tmpl/prog/en/includes/str/cataloging_additem.inc index 97cadb9148..57e389e8a7 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/includes/str/cataloging_additem.inc +++ b/koha-tmpl/intranet-tmpl/prog/en/includes/str/cataloging_additem.inc @@ -3,7 +3,11 @@ var searchid = '[% searchid | html %]'; var biblionumber = '[% biblionumber | html %]'; var frameworkcode = '[% frameworkcode | html %]'; - var popup = '[% popup | html %]'; + [% IF popup %] + var popup = true; + [% ELSE %] + var popup = false; + [% END %] var op = '[% op | html %]'; var LABEL_EDIT_ITEM = _("Edit item"); var LABEL_DELETE_ITEM = _("Delete item"); -- 2.39.5