https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32985 Christopher Brannon <cbrannon@cdalibrary.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Summary|DefaultPatronSearchFields |DefaultPatronSearchFields |not affecting all searches |not affecting all patron | |searches -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32985 --- Comment #2 from Christopher Brannon <cbrannon@cdalibrary.org> --- (In reply to Katrin Fischer from comment #1)

Have you tested with another field?

I think there is a shortcut on the cardnumber, so you can't probably exclude it from normal search. If there is a direct match on cardnumber, we open the record without displaying the results..

For the guarantor search there is no redirect.. so I guess it does not do that kind of check.

Yes, as I have said, I can search by name. So, if there is no consistency between this search and the regular patron search, to me that is a bug. This needs to be consistent. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32985 --- Comment #4 from Christopher Brannon <cbrannon@cdalibrary.org> --- (In reply to Katrin Fischer from comment #3)

I mean have you tried searching for a field that is not the cardnumber and not included in the pref.

My impression is that we might only have an exception for cardnumber in patron search as we do this as a first step to directly open the record for an exact match.

I understand. I have, and regular patron search works as expected. I've added fields we've needed to search by and the preference works. That exception you mention is understandable, but misleading because it does not work for all patron searching, as in guarantor searching. It needs to be consistent. If we are going to have the cardnumber searchable regardless of what is in DefaultPatronSearchFields, it should be mentioned on the preference, and work EVERYWHERE, not just some places. Otherwise we have oddballs like me trying to figure out what is wrong and filing bugs. :) -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32985 --- Comment #5 from Jonathan Druart <jonathan.druart+koha@gmail.com> --- Created attachment 150987 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=150987&action=edit Bug 32985: Do not search on cardnumber if not in DefaultPatronSearchFields -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32985 --- Comment #7 from Christopher Brannon <cbrannon@cdalibrary.org> --- (In reply to Jonathan Druart from comment #6)

This patch should fix the problem you are describing, but not sure it's something we want. This behaviour has been around for years now.

Lowering priority.

If it is not something you want, I'm not sure why you are providing it. A couple of options were presented. A) Make it consistent with how the preference works with everything else defined or not defined. B) Make it the exception, and list it as so in the description. Either way, it just needs to be consistent. There is no sane reason for having it work one way in some places, and another in other places. Let's just decide on a a consistent way of making this function, and help everyone to understand how it functions. That is all I ask. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.