From c4f9f7d473697fb7be8090613356ca64deb5e739 Mon Sep 17 00:00:00 2001 From: Garry Collum 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 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 (cherry picked from commit e1d948ecd69b802917272b2bc73e11fabc2a0e88) Signed-off-by: Chris Nighswonger (cherry picked from commit 07524b0de370211ebe5ff9fcb9f7077c633b0ef7) Signed-off-by: Chris Nighswonger --- reserve/request.pl | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/reserve/request.pl b/reserve/request.pl index a20ff80d17..1911c953b5 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