From e1d948ecd69b802917272b2bc73e11fabc2a0e88 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 --- 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.20.1