Before bug 9811, if you searched for "John Doe" and only 1 patron with this firstname/surname existed, the patron detail page was displayed. Now the result lists all patrons with ( firstname=john or doe ) or ( surname=john or doe). Which is quite annoying when you expect only 1 result.
Created attachment 29799 [details] [review] Bug 12595: Search a patron by his/her firstname and surname Before bug 9811, if you searched for "John Doe" and only 1 patron with this firstname/surname existed, the patron detail page was displayed. Now the result lists all patrons with ( firstname=john or doe ) or ( surname=john or doe). Which is quite annoying when you expect only 1 result. Test plan: 0/ Create some patrons like: "John Doe", "Jane Doe", "John Smith". 1/ Search for "John Doe" in the patron search box 2/ Verify you got 2 results 3/ Apply the patch 4/ Search for "John Doe" in the patron search box 5/ Verify you are redirected to the right patron detail page.
Created attachment 29801 [details] [review] Bug 12595: Search a patron by his/her firstname and surname Before bug 9811, if you searched for "John Doe" and only 1 patron with this firstname/surname existed, the patron detail page was displayed. Now the result lists all patrons with ( firstname=john or doe ) or ( surname=john or doe). Which is quite annoying when you expect only 1 result. Test plan: 0/ Create some patrons like: "John Doe", "Jane Doe", "John Smith". 1/ Search for "John Doe" in the patron search box 2/ Verify you got 2 results 3/ Apply the patch 4/ Search for "John Doe" in the patron search box 5/ Verify you are redirected to the right patron detail page. Signed-off-by: Dobrica Pavlinusic <dpavlin@rot13.org>
Created attachment 29825 [details] [review] [PASSED QA] Bug 12595: Search a patron by his/her firstname and surname Before bug 9811, if you searched for "John Doe" and only 1 patron with this firstname/surname existed, the patron detail page was displayed. Now the result lists all patrons with ( firstname=john or doe ) or ( surname=john or doe). Which is quite annoying when you expect only 1 result. Test plan: 0/ Create some patrons like: "John Doe", "Jane Doe", "John Smith". 1/ Search for "John Doe" in the patron search box 2/ Verify you got 2 results 3/ Apply the patch 4/ Search for "John Doe" in the patron search box 5/ Verify you are redirected to the right patron detail page. Signed-off-by: Dobrica Pavlinusic <dpavlin@rot13.org> Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Created attachment 29852 [details] [review] Bug 12595: (regression tests) Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
Created attachment 29855 [details] [review] Bug 12595: (regression tests) Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
Created attachment 29857 [details] [review] Bug 12595: (regression tests) Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
Created attachment 29858 [details] [review] Bug 12595: (regression tests) Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com> Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Patch pushed to master. Thanks Jonathan!