https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38462 Phil Ringnalda <phil@chetcolibrary.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=38462 --- Comment #1 from Phil Ringnalda <phil@chetcolibrary.org> --- Created attachment 174636 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=174636&action=edit Bug 38462: Remove unused code for pagination in OPAC authority search The remote branch for bug 13618 started out when OPAC authority search did its own pagination in a template block, then bug 2735 replaced that with pagination_bar, the template var passed from the script, then at some point before bug 13618 landed the second time, a rebase error put the template block back in. And since it's not at all obvious it is dead code unless you look for where "pagination" rather than "pagination_bar" is used, we've kept updating it, and bug 36032 thought the way it used >> instead of > for next ought to be fixed while one that was actually used was fixed. Time to get rid of it, before we 'fix' it again. Test plan: 1. Without the patch, in the OPAC go to Authority Search and do a search that will return pages of results, like the letter a 2. Note that the button for the next page uses ">" rather than the "Next >>" in the block this patch removes 3. Apply patch, click the browser Back button, refresh page 4. Verify that when you search for something that returns more than one page of results, the page navigation hasn't changed from what it was before the patch, telling you that what the patch removes was unused Sponsored-by: Chetco Community Public Library -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38462 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #174636|0 |1 is obsolete| | --- Comment #2 from David Nind <david@davidnind.com> --- Created attachment 175884 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=175884&action=edit Bug 38462: Remove unused code for pagination in OPAC authority search The remote branch for bug 13618 started out when OPAC authority search did its own pagination in a template block, then bug 2735 replaced that with pagination_bar, the template var passed from the script, then at some point before bug 13618 landed the second time, a rebase error put the template block back in. And since it's not at all obvious it is dead code unless you look for where "pagination" rather than "pagination_bar" is used, we've kept updating it, and bug 36032 thought the way it used >> instead of > for next ought to be fixed while one that was actually used was fixed. Time to get rid of it, before we 'fix' it again. Test plan: 1. Without the patch, in the OPAC go to Authority Search and do a search that will return pages of results, like the letter a 2. Note that the button for the next page uses ">" rather than the "Next >>" in the block this patch removes 3. Apply patch, click the browser Back button, refresh page 4. Verify that when you search for something that returns more than one page of results, the page navigation hasn't changed from what it was before the patch, telling you that what the patch removes was unused Sponsored-by: Chetco Community Public Library 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=38462 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Version(s)| |25.05.00 released in| | Status|Passed QA |Pushed to main -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38462 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Text to go in the| |This removes unused code release notes| |for OPAC authority search | |results pagination. (There | |are no visible changes for | |patrons.) CC| |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=38462 --- Comment #5 from Paul Derscheid <paul.derscheid@lmscloud.de> --- Nice work everyone! Pushed to 24.11.x for 24.11.03 -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38462 Wainui Witika-Park <wainuiwitikapark@catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Pushed to stable |Needs documenting --- Comment #7 from Wainui Witika-Park <wainuiwitikapark@catalyst.net.nz> --- Not backporting to 22.11.x as it wasn't backported to 24.05.x (or 23.XX) -- You are receiving this mail because: You are watching all bug changes.