Bug 5674 - Staff client - placing holds - patron name sort not ignoring letter case
Summary: Staff client - placing holds - patron name sort not ignoring letter case
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Hold requests (show other bugs)
Version: 3.6
Hardware: All All
: P5 - low minor (vote)
Assignee: Garry Collum
QA Contact: Bugs List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-01-31 15:51 UTC by bchurch
Modified: 2013-12-05 19:59 UTC (History)
5 users (show)

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


Attachments
Proposed patch (1.07 KB, patch)
2012-02-01 17:46 UTC, Garry Collum
Details | Diff | Splinter Review
[Signed Off] Bug 5674: patron name sort ignoring case when placing holds. (1.28 KB, patch)
2012-02-02 05:12 UTC, Liz Rea
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
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.