I just pulled master a few minutes ago, went to the patron module and clicked 'E' it gave me a patron named John Stewart - that was the only result. I do have patrons with last names that start with E - like mine. I think the alphabet filters on the patron search page are not working as expected.
Here's a video: http://screencast.com/t/z7UCKT57Olp
Created attachment 33476 [details] [review] 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.
You can only reproduce it if you have a patron with a cardnumber==""
Created attachment 33508 [details] [review] 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
Created attachment 33534 [details] [review] [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.
Patch pushed to master. Thanks Jonathan!