Bug 2808 - Hold queue not alphabetizing
Summary: Hold queue not alphabetizing
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Hold requests (show other bugs)
Version: Main
Hardware: All All
: PATCH-Sent (DO NOT USE) normal
Assignee: Owen Leonard
QA Contact:
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-11-18 02:09 UTC by Chris Cormack
Modified: 2012-10-25 23:03 UTC (History)
2 users (show)

See Also:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:


Attachments
Proposed fix (4.09 KB, patch)
2010-09-14 16:02 UTC, Owen Leonard
Details | Diff | Splinter Review
Revised fix (7.39 KB, patch)
2010-12-06 19:57 UTC, Owen Leonard
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Chris Cormack 2010-05-21 00:56:40 UTC


---- Reported by emily.funk@liblime.com 2008-11-18 14:09:48 ----

When you select a sort field on the Hold report, it doesn't alphabetize by title or patron.



---- Additional Comments From oleonard@myacpl.org 2009-02-10 09:07:23 ----

Are you referring to the "Holds to pull"/"Pending Holds" report at /cgi-bin/koha/circ/pendingreserves.pl? The Holds Queue report (/cgi-bin/koha/circ/view_holdsqueue.pl) doesn't have an option to sort.

The "Holds to pull" report has javascript-based table sorting enabled but it probably shouldn't since it's a report that is potentially very large.



---- Additional Comments From drdrulm@yahoo.com 2009-02-28 12:47:46 ----

(In reply to comment #1)
> Are you referring to the "Holds to pull"/"Pending Holds" report at
> /cgi-bin/koha/circ/pendingreserves.pl? The Holds Queue report
> (/cgi-bin/koha/circ/view_holdsqueue.pl) doesn't have an option to sort.
> 
> The "Holds to pull" report has javascript-based table sorting enabled but it
> probably shouldn't since it's a report that is potentially very large.
> 





---- Additional Comments From drdrulm@yahoo.com 2009-02-28 12:50:16 ----

Whoops- In reply to above. The pending holds can be pretty big. \

This report needs a branch selector with (none) as default. 




---- Additional Comments From cookie.wolfrom@liblime.com 2009-04-21 18:34:50 ----

holds awaiting pickup report - mostly used by smaller libraries - displays as being able to sort columns - name, date and title - but this function does not work - it would be helpful if this worked



--- Bug imported by chris@bigballofwax.co.nz 2010-05-21 00:56 UTC  ---

This bug was previously known as _bug_ 2808 at http://bugs.koha.org/cgi-bin/bugzilla3/show_bug.cgi?id=2808

Actual time not defined. Setting to 0.0
The original reporter of this bug does not have
   an account here. Reassigning to the person who moved
   it here: chris@bigballofwax.co.nz.
   Previous reporter was emily.funk@liblime.com.
CC member drdrulm@yahoo.com does not have an account here

Comment 1 Owen Leonard 2010-09-14 16:02:53 UTC Comment hidden (obsolete)
Comment 2 Owen Leonard 2010-12-06 19:57:56 UTC
Created attachment 2811 [details] [review]
Revised fix

Revised fix adds corrected sorting to the "Holds over" tab as well.
Comment 3 Chris Cormack 2010-12-12 07:32:15 UTC
Pushed to master please test
Comment 4 MJ Ray (software.coop) 2011-01-08 11:13:04 UTC
This bug is mentioned in:
Fix for Bug 2808,	Hold queue not alphabetizing http://lists.koha-community.org/pipermail/koha-patches/2010-December/013183.html
Fix for Bug 2808,	Hold queue not alphabetizing http://lists.koha-community.org/pipermail/koha-patches/2010-December/013184.html
Fix for Bug 2808,	Hold queue not alphabetizing http://lists.koha-community.org/pipermail/koha-patches/2010-December/013225.html