From ee2606ce6ded7a0c52b7f1b6d2541dcfa8edbed2 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Joonas=20Kylm=C3=A4l=C3=A4?= Date: Tue, 24 Nov 2020 16:17:58 +0200 Subject: [PATCH] Bug 20447: (follow-up) Correctly list all holdings frameworks in editor The Template Toolkit expects a ref to array and we passed the array instead. It caused the listing of holdings frameworks not to show when there was more than one holdings framework defined. To test: 1) Apply patch 2) Make 3 copies of the default HLD framework 3) Go to a biblio page and create new holding 4) Click Settings in the editor page and notice all 3 frameworks are now listed in the drop-down menu --- cataloguing/addholding.pl | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/cataloguing/addholding.pl b/cataloguing/addholding.pl index fe2b0cd43d..67cfdedc38 100755 --- a/cataloguing/addholding.pl +++ b/cataloguing/addholding.pl @@ -690,8 +690,9 @@ $template->param( author => $biblio->author ); +my @frameworks = Koha::BiblioFrameworks->search({}, { order_by => ['frameworktext'] }); $template->param( - frameworks => Koha::BiblioFrameworks->search({}, { order_by => ['frameworktext'] }), + frameworks => \@frameworks, popup => $mode, frameworkcode => $frameworkcode, itemtype => $frameworkcode, -- 2.11.0