From 05cdc48fd6c549f5f67af2562f9c5f889b63fa50 Mon Sep 17 00:00:00 2001 From: Owen Leonard Date: Wed, 17 Jan 2024 11:58:03 +0000 Subject: [PATCH] Bug 35820: Fix table id on hold ratios page This patch corrects the ID on the table showing holds on the holds ratios page so that it matches the DataTables initialization and configuration. I'm not aware of any bug with the functionality, but consistency is good. The patch also wraps the table in a "page-section" div, an update which includes indentation changes. To test, apply the patch and if necessary, place multiple holds on a title so that the hold ratios page will display results. - Go to Circulation -> Hold ratios. - The table of hold ratios should look correct. - Click the "Configure" button at the top of the table. - On the table settings page, change the default visibility of some columns. - Return to the hold ratios page to confirm that the correct columns are hidden by default. Signed-off-by: David Nind Signed-off-by: Marcel de Rooy Signed-off-by: Katrin Fischer --- .../prog/en/modules/circ/reserveratios.tt | 180 +++++++++--------- 1 file changed, 91 insertions(+), 89 deletions(-) diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/reserveratios.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/reserveratios.tt index 0a80196eaa..89ac009270 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/reserveratios.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/reserveratios.tt @@ -49,94 +49,96 @@
[% sql | html %]
[% IF ( reserveloop ) %] - - - - - - - - - - - - - - - - - - [% FOREACH reserveloo IN reserveloop %] - - - - - - - - - - - - - - [% END %] - -
HoldsItemsHold ratioTitleHome librariesHolding librariesLocationItypeCollectionCall numbersItems needed

[% reserveloo.reservecount | html %]

[% reserveloo.itemcount | html %]

[% reserveloo.thisratio | html %] - [% INCLUDE 'biblio-title.inc' biblio=reserveloo link = 1 %][% IF ( reserveloo.author ) %] by [% reserveloo.author | html %][% END %] - -
    - [% FOREACH homebranch IN reserveloo.homebranch_list %] -
  • [% Branches.GetName ( homebranch ) | html %]
  • - [% END %] -
-
-
    - [% FOREACH holdingbranch IN reserveloo.holdingbranch_list %] -
  • [% Branches.GetName ( holdingbranch ) | html %]
  • - [% END %] -
-
-
    - [% FOREACH location IN reserveloo.location %] -
  • [% AuthorisedValues.GetDescriptionByKohaField( kohafield => 'items.location', authorised_value => location ) | html %]
  • - [% END %] -
-
-
    - [% FOREACH itype IN reserveloo.itype %] -
  • [% ItemTypes.GetDescription( itype ) | html %]
  • - [% END %] -
-
-
    - [% FOREACH ccode IN reserveloo.ccode %] -
  • [% AuthorisedValues.GetDescriptionByKohaField( kohafield => 'items.ccode', authorised_value => ccode ) | html %]
  • - [% END %] -
-
-
    - [% FOREACH listcall IN reserveloo.listcall %] -
  • [% listcall | html %]
  • - [% END %] -
-
-

- [% IF ( reserveloo.thisratio_atleast1 ) %] - [% IF ( CAN_user_acquisition && basketno && booksellerid ) %] - [% reserveloo.copies_to_buy | html %] to order - [% ELSE %] - [% reserveloo.copies_to_buy | html %] to order - [% END %] - [% IF ( reserveloo.pendingorders ) %]
[% reserveloo.pendingorders | html %] pending[% END %] - [% ELSE %] - 0 to order - [% END %] -

-
- [% ELSE %] - No items found. - [% END %] +
+ + + + + + + + + + + + + + + + + + [% FOREACH reserveloo IN reserveloop %] + + + + + + + + + + + + + + [% END %] + +
HoldsItemsHold ratioTitleHome librariesHolding librariesLocationItypeCollectionCall numbersItems needed

[% reserveloo.reservecount | html %]

[% reserveloo.itemcount | html %]

[% reserveloo.thisratio | html %] + [% INCLUDE 'biblio-title.inc' biblio=reserveloo link = 1 %][% IF ( reserveloo.author ) %] by [% reserveloo.author | html %][% END %] + +
    + [% FOREACH homebranch IN reserveloo.homebranch_list %] +
  • [% Branches.GetName ( homebranch ) | html %]
  • + [% END %] +
+
+
    + [% FOREACH holdingbranch IN reserveloo.holdingbranch_list %] +
  • [% Branches.GetName ( holdingbranch ) | html %]
  • + [% END %] +
+
+
    + [% FOREACH location IN reserveloo.location %] +
  • [% AuthorisedValues.GetDescriptionByKohaField( kohafield => 'items.location', authorised_value => location ) | html %]
  • + [% END %] +
+
+
    + [% FOREACH itype IN reserveloo.itype %] +
  • [% ItemTypes.GetDescription( itype ) | html %]
  • + [% END %] +
+
+
    + [% FOREACH ccode IN reserveloo.ccode %] +
  • [% AuthorisedValues.GetDescriptionByKohaField( kohafield => 'items.ccode', authorised_value => ccode ) | html %]
  • + [% END %] +
+
+
    + [% FOREACH listcall IN reserveloo.listcall %] +
  • [% listcall | html %]
  • + [% END %] +
+
+

+ [% IF ( reserveloo.thisratio_atleast1 ) %] + [% IF ( CAN_user_acquisition && basketno && booksellerid ) %] + [% reserveloo.copies_to_buy | html %] to order + [% ELSE %] + [% reserveloo.copies_to_buy | html %] to order + [% END %] + [% IF ( reserveloo.pendingorders ) %]
[% reserveloo.pendingorders | html %] pending[% END %] + [% ELSE %] + 0 to order + [% END %] +

+
+
+[% ELSE %] + No items found. +[% END %] @@ -202,7 +204,7 @@