[Bug 29820] New: Print summary just show 20 items
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29820 Bug ID: 29820 Summary: Print summary just show 20 items Change sponsored?: --- Product: Koha Version: 21.11 Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Circulation Assignee: koha-bugs@lists.koha-community.org Reporter: mhby87@gmail.com QA Contact: testopia@bugs.koha-community.org CC: gmcharlt@gmail.com, kyle.m.hall@gmail.com Created attachment 129190 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=129190&action=edit List print summary After upgrade from 21.05 to 21.11, print summary for check out items just show 20 rows only. For example, patron borrow 500 item, but print summary show 20 items list. How to show all check out items in patron print summary. Thank you -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29820 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Version|21.11 |master Severity|enhancement |normal Depends on| |28450 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28450 [Bug 28450] Make Account summary print tables configurable -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29820 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Assignee|koha-bugs@lists.koha-commun |oleonard@myacpl.org |ity.org | Status|NEW |ASSIGNED -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29820 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Patch complexity|--- |Trivial 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=29820 --- Comment #1 from Owen Leonard <oleonard@myacpl.org> --- Created attachment 129202 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=129202&action=edit Bug 29820: Set paging option using boolean instead of string Bug 28450, "Make Account summary print tables configurable," added DataTables to the print summary view. The updated page includes the wrong option: "paging": "false", It should be: "paging": false, Because DataTables expects that option to be boolean (true or false). To test, apply the patch and check out to a patron who has more than 20 checkouts and more than 20 holds. - From the toolbar, click Print -> Print summary. - On the acount summary page, confirm that the "Items checked out" and "Pending holds" tables show ALL entries, not just the first 20. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29820 --- Comment #2 from Mohd Hafiz Yusoff <mhby87@gmail.com> --- Created attachment 129209 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=129209&action=edit List print summary after patch Patch tested by me -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29820 Mohd Hafiz Yusoff <mhby87@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off --- Comment #3 from Mohd Hafiz Yusoff <mhby87@gmail.com> --- Created attachment 129202 [details] [review] Bug 29820: Set paging option using boolean instead of string Bug 28450, "Make Account summary print tables configurable," added DataTables to the print summary view. The updated page includes the wrong option: "paging": "false", It should be: "paging": false, Because DataTables expects that option to be boolean (true or false). To test, apply the patch and check out to a patron who has more than 20 checkouts and more than 20 holds. - From the toolbar, click Print -> Print summary. - On the acount summary page, confirm that the "Items checked out" and "Pending holds" tables show ALL entries, not just the first 20. Signed-off-by: Mohd Hafiz Yusoff <mhby87@gmail.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29820 Katrin Fischer <katrin.fischer@bsz-bw.de> 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=29820 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #129202|0 |1 is obsolete| | --- Comment #4 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Created attachment 129220 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=129220&action=edit Bug 29820: Set paging option using boolean instead of string Bug 28450, "Make Account summary print tables configurable," added DataTables to the print summary view. The updated page includes the wrong option: "paging": "false", It should be: "paging": false, Because DataTables expects that option to be boolean (true or false). To test, apply the patch and check out to a patron who has more than 20 checkouts and more than 20 holds. - From the toolbar, click Print -> Print summary. - On the acount summary page, confirm that the "Items checked out" and "Pending holds" tables show ALL entries, not just the first 20. Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29820 --- Comment #5 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Thx for the quick fix, Owen. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29820 Fridolin Somers <fridolin.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Version(s)| |22.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=29820 --- Comment #6 from Fridolin Somers <fridolin.somers@biblibre.com> --- Pushed to master for 22.05, thanks to everybody involved 🦄 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29820 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |kyle@bywatersolutions.com Version(s)|22.05.00 |22.05.00,21.11.03 released in| | Status|Pushed to master |Pushed to stable --- Comment #7 from Kyle M Hall <kyle@bywatersolutions.com> --- Pushed to 21.11.x for 21.11.03 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29820 Andrew Fuerste-Henry <andrew@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Pushed to stable |RESOLVED CC| |andrew@bywatersolutions.com Resolution|--- |FIXED --- Comment #8 from Andrew Fuerste-Henry <andrew@bywatersolutions.com> --- Missing dependency, not backported to 21.05 -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon@bugs.koha-community.org