Bug 5674

Summary: Staff client - placing holds - patron name sort not ignoring letter case
Product: Koha Reporter: bchurch
Component: Hold requestsAssignee: Garry Collum <gcollum>
Status: CLOSED FIXED QA Contact: Bugs List <koha-bugs>
Severity: minor    
Priority: P5 - low CC: gmcharlt, nengard, paul.poulain, smoreland, wizzyrea
Version: 3.6   
Hardware: All   
OS: All   
See Also: http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5609
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Proposed patch
[Signed Off] Bug 5674: patron name sort ignoring case when placing holds.

Description bchurch 2011-01-31 15:51:30 UTC
When you are placing holds for patrons in the staff client, patron names do not sort correctly if patron names are entered in upper case.

To recreate this issue, do the following:
1. Add two test patron records, one with the name "Smith, Joseph" and the other "SMITH, THOMAS".
2. Search for a title in the staff client and click Holds to open the request.pl window.
3. Do patron search for the name "Smith" and you will see that SMITH, THOMAS sorts before "Smith, Joseph".
Comment 1 Garry Collum 2012-02-01 17:46:34 UTC Comment hidden (obsolete)
Comment 2 Liz Rea 2012-02-02 05:12:25 UTC
Created attachment 7418 [details] [review]
[Signed Off] Bug 5674: patron name sort ignoring case when placing holds.

Fixes the patron name sorting when placing holds.

Testing - See Bug 5674.
Create two patrons.  One named SMITH, THOMAS and the other Smith, John.
Search for an item to place on request.  Do a patron search for Smith.

Signed-off-by: Liz Rea <wizzyrea@gmail.com>
Confirm both the original bug and the fix, sorting is correct now.
Patch does not change status of tests - prove t t/db_dependent xt no different from master.
Comment 3 Paul Poulain 2012-02-03 15:40:19 UTC
QA comment: tiny patch, nothing to say
Comment 4 Jared Camins-Esakov 2012-05-23 12:59:02 UTC
This fix was included in the 3.6.x branch prior to 3.6.4.