Librarian requested we add for patrons the option to sort by author and title to patron check out history on the patron's detail screen in opac.
Does this bug refer to /cgi-bin/koha/opac-readingrecord.pl? That page can already sort by title. Do you mean sorting by title and author together?
Created attachment 37370 [details] [review] Bug 13865: Columns configuration - Orders search This patch adds the ability to hide columns on the orders search page (acqui/histsearch.pl). Test plan: Go on the orders search page and confirm you are able to hide columns. You can also play with the columns configuration admin page and confirm that the behaviors are consistent. Signed-off-by: Nicole <nicole@bywatersolutions.com>
Comment on attachment 37370 [details] [review] Bug 13865: Columns configuration - Orders search Sorry attached to the wrong bug.
(In reply to Owen Leonard from comment #1) Yes, this bug refers to /cgi-bin/koha/opac-readingrecord.pl It would be nice to have the availability for the patron to sort by author. For users who have read several titles by the same author, their list tends become to long. Having the ability to sort a column by author would be a great enhancement.
Is this still valid?
(In reply to Aleisha Amohia from comment #5) > Is this still valid? Yes. /cgi-bin/koha/opac-readingrecord.pl has a single column which contains title and author information, so sorting can't be done by author. If someone decides to take this on it'd be great to give the page a more thorough overhaul. The "Show last 50 items" versus "Show all items" options are strange. Why not just do real pagination?
Sorting by author would be a great addition to this feature on the OPAC.
Created attachment 59956 [details] [review] Bug 13685: Sorting by author in opac reading history To test: 1) Log into OPAC, go to your raeding history 2) Notice 'author' is in Title column, therefore unable to sort by author 3) Apply patch and refresh page 4) Notice author has now been moved into Author column 5) Confirm sorting by author works as expected and sorting of all other columns works as expected Sponsored-by: Catalyst IT
Might I also recommend doing the same kind of split for the summary page too? Not that affects this bug, but it might be another one to think about.
Created attachment 60134 [details] [review] Bug 13685: Sorting by author in opac reading history To test: 1) Log into OPAC, go to your raeding history 2) Notice 'author' is in Title column, therefore unable to sort by author 3) Apply patch and refresh page 4) Notice author has now been moved into Author column 5) Confirm sorting by author works as expected and sorting of all other columns works as expected Sponsored-by: Catalyst IT Signed-off-by: Mark Tompsett <mtompset@hotmail.com>
(In reply to M. Tompsett from comment #9) > Might I also recommend doing the same kind of split for the summary page > too? Not that affects this bug, but it might be another one to think about. Have implemented this fix in bug 18108
I do not think this patch answers to the initial need. What I understand is that we would like to add a new entry to the "order by" dropdown list, to order the checkout history by author. Your solution will not work if the patron has already checked out more than 50 items. Moreover it seems that we use to display the author close to the title, see "Your summary" tab > "Holds".
Created attachment 60222 [details] [review] Bug 13685: Sorting by author in opac reading history opac-readingrecord.pl was actually already set up to do this, the option to sort by author was just not put into the tt file. This patch adds that. To test: 1) Log into OPAC, go to your reading history 2) Confirm that 'Order by author' shows in the dropdown 3) Confirm it works as expected Sponsored-by: Catalyst IT
Strangely, orderbyauthor code was already in the perl script, but finding where it may have been deleted is difficult. But this fix is good. And as the layout doesn't change, I think bug 18108 could be closed (unless we foresee people signing out a large number of books at the same time, in which case we should add this drop-down). I was worried about consistency of appearance.
Created attachment 60226 [details] [review] [SIGNED-OFF] Bug 13685: Sorting by author in opac reading history opac-readingrecord.pl was actually already set up to do this, the option to sort by author was just not put into the tt file. This patch adds that. To test: 1) Log into OPAC, go to your reading history 2) Confirm that 'Order by author' shows in the dropdown 3) Confirm it works as expected Sponsored-by: Catalyst IT Signed-off-by: Mark Tompsett <mtompset@hotmail.com>
Created attachment 60464 [details] [review] Bug 13685: Sorting by author in opac reading history opac-readingrecord.pl was actually already set up to do this, the option to sort by author was just not put into the tt file. This patch adds that. To test: 1) Log into OPAC, go to your reading history 2) Confirm that 'Order by author' shows in the dropdown 3) Confirm it works as expected Sponsored-by: Catalyst IT Signed-off-by: Mark Tompsett <mtompset@hotmail.com> Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Pushed to master for 17.05, thanks Aleisha!