Bugzilla – Attachment 108935 Details for
Bug 21539
addorderiso2709.pl forces librarian to select a ccode and notforloan code when using MarcItemFieldsToOrder
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 21539: addorderiso2709.pl forces librarian to select a ccode and notforloan code when using MarcItemFieldsToOrder
Bug-21539-addorderiso2709pl-forces-librarian-to-se.patch (text/plain), 2.56 KB, created by
Katrin Fischer
on 2020-08-22 16:22:54 UTC
(
hide
)
Description:
Bug 21539: addorderiso2709.pl forces librarian to select a ccode and notforloan code when using MarcItemFieldsToOrder
Filename:
MIME Type:
Creator:
Katrin Fischer
Created:
2020-08-22 16:22:54 UTC
Size:
2.56 KB
patch
obsolete
>From 709d7be349b8d520f9c715f7490590ef815bc20a Mon Sep 17 00:00:00 2001 >From: Kyle M Hall <kyle@bywatersolutions.com> >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 <daniel.gaghan@pueblolibrary.org> > >Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> >--- > koha-tmpl/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 @@ > </li> > > <li><label for="ccode_item_[% item.item_id | html %]">ccode</label><select id="ccode_item_[% item.item_id | html %]" name="ccode_[% item.biblio_count | html %]"> >+ <option value=""> </option> > [% FOREACH ccodeloo IN ccodeloop %] > [% IF ( ccodeloo.code ) == (item.ccode) %]<option value="[% ccodeloo.code | html %]" selected="selected">[% ccodeloo.description | html %]</option>[% ELSE %]<option value="[% ccodeloo.code | html %]">[% ccodeloo.description | html %]</option>[% END %] > [% END %] >@@ -229,6 +230,7 @@ > </li> > > <li><label for="notforloan_item_[% item.item_id | html %]">notforloan</label><select id="notforloan_item_[% item.item_id | html %]" name="notforloan_[% item.biblio_count | html %]"> >+ <option value=""> </option> > [% FOREACH n IN notforloanloop %] > [% IF n.code == item.notforloan %] > <option value="[% n.code | html %]" selected="selected">[% n.description | html %]</option> >-- >2.11.0
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 21539
:
80361
|
108553
| 108935