Bug 5945

Summary: can't search patron's email anymore
Product: Koha Reporter: Nicole C. Engard <nengard>
Component: PatronsAssignee: Ian Walls <koha.sekjal>
Status: CLOSED FIXED QA Contact: Marcel de Rooy <m.de.rooy>
Severity: major    
Priority: P5 - low CC: gmcharlt, julian.maurice, koha.sekjal, kohapatch, savitra.sirohi
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Small patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Bug Depends on:    
Bug Blocks: 6253    
Attachments: Proposed Patch
signed off patch

Description Nicole C. Engard 2011-03-24 10:22:55 UTC
Just last week (i think) we were talking about the email address for patron's being searchable. I tested it and documented it and today it is no longer searchable. Can we put that functionality back in?
Comment 1 Nicole C. Engard 2011-08-03 23:43:40 UTC
Had anyone had a chance to look at this - now we're pretty far away from this being functional so I don't know how easy it is to find. But when I reported this it had worked just a week or two before.

Nicole
Comment 2 Nicole C. Engard 2011-09-07 16:06:26 UTC
*** Bug 6808 has been marked as a duplicate of this bug. ***
Comment 3 Nicole C. Engard 2011-09-07 16:06:59 UTC
from another bug report:


Amit Gupta 2011-09-02 08:35:07 UTC

Nicole,
but the existing code is comment.

#($results)=Search(\@searchpatron,{surname=>1,firstname=>1},[$from,$to],undef,["firstname","surname","email","othernames"]
 ) if (@searchpatron);

So it is not possible to search by patron email id.
Comment 4 Ian Walls 2011-10-18 16:56:46 UTC
I will write the fix for this once bug 6253 is committed, as it will change the underlying landscape for this.  Should be a one-liner to add 'email' as a valid search field.
Comment 5 Ian Walls 2011-11-04 10:36:09 UTC Comment hidden (obsolete)
Comment 6 Julian Maurice 2011-11-05 10:20:28 UTC
Created attachment 6223 [details] [review]
signed off patch

Works as expected
Comment 7 Ian Walls 2011-11-05 10:58:11 UTC
Assigning to Marcel for QA.
Comment 8 Marcel de Rooy 2011-11-07 07:47:29 UTC
Small patch; passing QA.
Comment 9 Marcel de Rooy 2011-11-10 12:34:40 UTC
Has been pushed: commit c3abbedebedd34948b664a4d2a6b34dc4863802d !