From f7625e84c757ef51cca72da01cddf2c5344801d0 Mon Sep 17 00:00:00 2001 From: Jonathan Druart Date: Wed, 1 Oct 2014 10:46:14 +0200 Subject: [PATCH] Bug 14562: All patrons returned if 1 has a cardnumber set to an empty string (Fix conflict with bug 12833) MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit This patch is a backport of (Bug 12648 is in 3.20.x, not 3.18.x): commit 481479fe627dcbabb3ef66df0724cc3fe75b110c Bug 12648: Fix conflict with bug 12833 Test plan: Set the cardnumber of one of the patron to an empty string Do a patron search and use filters The search will redirect to the patron detail page of the patron with the empty string Signed-off-by: Joonas Kylmälä Signed-off-by: Katrin Fischer Signed-off-by: Liz Rea --- svc/members/search | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/svc/members/search b/svc/members/search index f8c725bfc1..60695ecb0d 100755 --- a/svc/members/search +++ b/svc/members/search @@ -60,7 +60,8 @@ foreach (grep {$_ =~ /^mDataProp/} keys %dt_params) { my $results; # If the user filled a term, maybe it's a cardnumber. # This cannot be the case if a first letter is given. -if ( not $firstletter +if ( $searchmember + and not $firstletter and $searchfieldstype and $searchfieldstype eq 'standard' ) { -- 2.39.5