Attachment 33534 Details for Bug 13232 – [PASSED QA] Bug 13232: Search patrons by a first letter should not redirect to a patron detail page

[patch] [PASSED QA] Bug 13232: Search patrons by a first letter should not redirect to a patron detail page

PASSED-QA-Bug-13232-Search-patrons-by-a-first-lett.patch (text/plain), 1.49 KB, created by Katrin Fischer on 2014-11-13 20:31:50 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Katrin Fischer

Created: 2014-11-13 20:31:50 UTC

Size: 1.49 KB

patch

obsolete

>From f9840e8b437eb20fab75f261f53e1df7e7177a8d Mon Sep 17 00:00:00 2001
>From: Jonathan Druart <jonathan.druart@biblibre.com>
>Date: Wed, 12 Nov 2014 10:02:15 +0100
>Subject: [PATCH] [PASSED QA] Bug 13232: Search patrons by a first letter
> should not redirect to a patron detail page
>
>Bug 12833 allows to find a patron with his cardnumber.
>But this won't never append if the firstletter parameter is given.
>
>Actually the firstletter param is the only one to take into account if
>it exists.
>
>Test plan:
>Search patrons by a first letter and verify that the feature is back.
>
>Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
>
>To reproduce the problem you need at least one borrower with a blank
>cardnumber
>
>Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
>works as descrobed, no problems found.
>---
> svc/members/search | 7 ++++++-
> 1 file changed, 6 insertions(+), 1 deletion(-)
>
>diff --git a/svc/members/search b/svc/members/search
>index e0b64a5..7359258 100755
>--- a/svc/members/search
>+++ b/svc/members/search
>@@ -58,7 +58,12 @@ foreach (grep {$_ =~ /^mDataProp/} keys %dt_params) {
> }
> 
> my $results;
>-if ( $searchfieldstype and $searchfieldstype eq 'standard' ) {
>+# 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
>+    and $searchfieldstype
>+    and $searchfieldstype eq 'standard' )
>+{
>     my $member = C4::Members::GetMember( cardnumber => $searchmember );
>     $results = {
>         iTotalRecords        => 1,
>-- 
>1.9.1

Actions: View | Diff | Splinter Review

Attachments on bug 13232: 33476 | 33508 | 33534