[Bug 35820] New: Table on Hold ratios page at circ/reserveratios.pl has wrong id
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35820 Bug ID: 35820 Summary: Table on Hold ratios page at circ/reserveratios.pl has wrong id Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: trivial Priority: P5 - low Component: Templates Assignee: oleonard@myacpl.org Reporter: jrobb@sekls.org QA Contact: testopia@bugs.koha-community.org The table at circ/reserveratios.pl currently has an id of "holdst". The table settings page says that table id should be "holds-ratios". "holdst" is the id for the holds queue table at circ/view_holdsqueue.pl. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35820 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35820 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Patch complexity|--- |Small patch Status|ASSIGNED |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35820 --- Comment #1 from Owen Leonard <oleonard@myacpl.org> --- Created attachment 161096 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=161096&action=edit 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. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35820 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35820 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #161096|0 |1 is obsolete| | --- Comment #2 from David Nind <david@davidnind.com> --- Created attachment 161102 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=161102&action=edit 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 <david@davidnind.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35820 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |david@davidnind.com --- Comment #3 from David Nind <david@davidnind.com> --- Testing notes using KTD: 1. Place four holds on a record. 2. To get the report to display I needed to: 2.1 Edit /etc/koha/sites/kohadev/koha-conf.xml 2.2 Change strict mode to 0: <strict_sql_modes>0</strict_sql_modes> 2.3 Restart everything 2.4 Otherwise, I get the group by error trace: CGI::Compile::ROOT::kohadevbox_koha_circ_reserveratios_2epl::reserveratios_2epl(): DBI Exception: DBD::mysql::st execute failed: 'koha_kohadev.reserves.reservedate' isn't in GROUP BY at /usr/share/perl5/CGI/Emulate/PSGI.pm line 30 at /usr/share/perl5/DBIx/Class/Exception.pm line 77 Additional things noted - both before and after the patch: 1. Row alignment - these columns are in the center, whereas all the other columns are aligned to the top of the row: - Holds - Items - Items needed 2. The item type column is labelled 'Itype', should be labelled as 'Item type'. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35820 Marcel de Rooy <m.de.rooy@rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35820 Marcel de Rooy <m.de.rooy@rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #161102|0 |1 is obsolete| | --- Comment #4 from Marcel de Rooy <m.de.rooy@rijksmuseum.nl> --- Created attachment 161206 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=161206&action=edit 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 <david@davidnind.com> Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35820 Marcel de Rooy <m.de.rooy@rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- QA Contact|testopia@bugs.koha-communit |m.de.rooy@rijksmuseum.nl |y.org | CC| |m.de.rooy@rijksmuseum.nl -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35820 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Version(s)| |24.05.00 released in| | Status|Passed QA |Pushed to master -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35820 --- Comment #5 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Pushed for 24.05! Well done everyone, thank you! -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35820 Fridolin Somers <fridolin.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |fridolin.somers@biblibre.co | |m Version(s)|24.05.00 |24.05.00,23.11.03 released in| | Status|Pushed to master |Pushed to stable --- Comment #6 from Fridolin Somers <fridolin.somers@biblibre.com> --- Pushed to 23.11.x for 23.11.03 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35820 Lucas Gass <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |lucas@bywatersolutions.com Status|Pushed to stable |RESOLVED Resolution|--- |FIXED --- Comment #7 from Lucas Gass <lucas@bywatersolutions.com> --- Doesn't apply cleanly for 23.05.x, no backport -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon@bugs.koha-community.org