From 38446b23116444cadbcfa04758b108ec81b5f768 Mon Sep 17 00:00:00 2001 From: Kyle M Hall Date: Wed, 10 Oct 2018 11:13:37 -0400 Subject: [PATCH] Bug 21539: addorderiso2709.pl forces librarian to select a ccode and notforloan code when using MarcItemFieldsToOrder There is no reason that MarcItemFieldsToOrder should force librarians to select a ccode and notforloan value for each item. They are both optional fields. As with location, the first value should be an empty 'none' option. Test Plan: 1) Set up MarcItemFieldsToOrder, do *not* set mappings for ccode and notforloan 2) Add an order record to a basket that uses MarcItemFieldsToOrder 3) Note you must choose a ccode and nfl value 4) Apply this patch 5) Reload the page 6) Note you no longer have to select values for ccode and notforloan Signed-off-by: Daniel Gaghan Signed-off-by: Katrin Fischer Signed-off-by: Jonathan Druart (cherry picked from commit 0c79b7d93b96434ddf91c6ddabb1570836736a15) Signed-off-by: Lucas Gass (cherry picked from commit 5f6b557599b83e0230a3dca5b527924e883d17a3) Signed-off-by: Aleisha Amohia --- .../intranet-tmpl/prog/en/modules/acqui/addorderiso2709.tt | 2 ++ 1 file changed, 2 insertions(+) diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/addorderiso2709.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/addorderiso2709.tt index ebe364e3cb..1ae8d429d4 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/addorderiso2709.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/addorderiso2709.tt @@ -222,6 +222,7 @@
  • + [% FOREACH n IN notforloanloop %] [% IF n.code == item.notforloan %] -- 2.39.5