From c5e9eebe87d8a44074e083823774599b628321ba Mon Sep 17 00:00:00 2001 From: Owen Leonard Date: Wed, 3 Jul 2024 15:43:30 +0000 Subject: [PATCH] Bug 37219: In OPAC item holds, separate copy number from radio button and add table configuration This patch changes the way items are displayed during the process of placing a single or multiple item-level holds in the OPAC. The item-selection radio button is now in a separate column from copy number, and the columns can be configured. Note: This patch includes indentation changes, so ignore whitespace when viewing the diff. To test, apply the patch and restart services. - If necessary, go to Administration -> Circulation and fine rules and enable item-level holds in one of the rules. - Log in to the OPAC and start the process of placing a hold on a title which falls under your configured circulation rule. - On the place hold page, click "Show more options" and "A specific item" to show the table of items. - The radio buttons and copy numbers should appear in separate columns. - Go to Administration -> Table settings -> OPAC -> opac_holds and change the table's configuration to hide one or more columns. - Return to the OPAC holds process and confirm that the table reflects your configuration changes. - Perform the same test with multiple holds (selecting multiple items in catalog search results). Sponsored-by: Athens County Public Libraries --- admin/columns_settings.yml | 26 ++ .../bootstrap/en/modules/opac-reserve.tt | 251 ++++++++++-------- 2 files changed, 160 insertions(+), 117 deletions(-) diff --git a/admin/columns_settings.yml b/admin/columns_settings.yml index af24061dcf..7b85919d33 100644 --- a/admin/columns_settings.yml +++ b/admin/columns_settings.yml @@ -2085,6 +2085,32 @@ modules: - columnname: link + opac_holds: + opac_holds_items: + columns: + - + cannot_be_toggled: 1 + cannot_be_modified: 1 + columnname: selection + - + columnname: copy_number + - + columnname: itemtype + - + columnname: barcode + - + columnname: home_library + - + columnname: current_library + - + columnname: collection + - + columnname: call_number + - + columnname: notes + - + columnname: information + serials: subscription-detail: orders: diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-reserve.tt b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-reserve.tt index 4451b46802..82b58a05b6 100644 --- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-reserve.tt +++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-reserve.tt @@ -5,6 +5,7 @@ [% USE Price %] [% USE ItemTypes %] [% USE AuthorisedValues %] +[% USE TablesSettings %] [% SET reserve_input_type = 'radio' %] [% IF ( Koha.Preference('DisplayMultiItemHolds') ) %] [% SET reserve_input_type = 'checkbox' %] @@ -378,123 +379,130 @@ [% IF bibitemloo.itemholdable %]
- - - - - - [% IF ( item_level_itypes ) %] - - [% END %] - - [% UNLESS ( singleBranchMode ) %] - - - [% END %] - [% IF ( itemdata_ccode ) %] - - [% END %] - - [% IF ( itemdata_enumchron ) %] - - [% END %] - - - - - - - [% SET unholdable_items = 0 %] - [% FOREACH itemLoo IN bibitemloo.itemLoop %] - [% IF ( itemLoo.available ) %] - [% IF ( itemLoo.checkout ) %] - - [% ELSE %] - - [% END %] - - [% ELSE %] - - [% END %] - + + + [% END # / FOREACH itemLoo IN bibitemloo.itemLoop%] + +
Select a specific item:
Copy numberItem typeBarcodeHome libraryLast locationCollectionCall numberVol infoNotesInformation
- - [% ELSE %] - [% SET unholdable_items = 1 %] - [% IF ( itemLoo.checkout ) %] -
+ [% IF unholdable_items %] + + + [% END %] +
[% END # / IF ( bibitemloo.itemholdable )%] @@ -516,6 +524,7 @@ [% BLOCK jsinclude %] [% INCLUDE 'calendar.inc' %] [% INCLUDE 'datatables.inc' %] +[% INCLUDE 'columns_settings.inc' %] [% END %] -- 2.39.2