From 07524b0de370211ebe5ff9fcb9f7077c633b0ef7 Mon Sep 17 00:00:00 2001
From: Garry Collum <gcollum@gmail.com>
Date: Wed, 1 Feb 2012 12:35:27 -0500
Subject: [PATCH] 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.

Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
(cherry picked from commit e1d948ecd69b802917272b2bc73e11fabc2a0e88)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
---
 reserve/request.pl | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/reserve/request.pl b/reserve/request.pl
index 21183da4a2..24557f4c4c 100755
--- a/reserve/request.pl
+++ b/reserve/request.pl
@@ -192,9 +192,9 @@ if ($borrowerslist) {
 
     foreach my $borrower (
         sort {
-                $a->{surname}
-              . $a->{firstname} cmp $b->{surname}
-              . $b->{firstname}
+                uc($a->{surname}
+              . $a->{firstname}) cmp uc($b->{surname}
+              . $b->{firstname})
         } @{$borrowerslist}
       )
     {
-- 
2.39.5