[Koha-bugs] [Bug 20334] Elasticsearch - Option for escaping slashes in search queries

[bugzilla-daemon at bugs.koha-community.org]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20334

--- Comment #12 from David Gustafsson <glasklas at gmail.com> ---
(In reply to Katrin Fischer from comment #11)
> Hi David,
> 
> 1) Can you please add an entry for the new system preference in sysprefs.sql?

I added an entry in the atomicupdate directory, has there been a change so it
should be added in sysprefs.sql instead?

> 2) I wonder, is this something where picking a sensible default behaviour
> could make sense? Honestly I am more than a little confused by the different
> options, but I can confirm that searching for something containing / won't
> work right now, which is a problem. For example, it will break when someone
> copies a MARC record for search. Example:
> 
> Dutch : an essential grammar / William Z. Shetter and Inge Van der
> Cruysse-Van Antwerpen

I changed the default to "Escape", from "Don't escape" since that's probably
what most people want (think very few actually uses regexps in search strings).
I also expanded the explanation on the system-preferences page.

> 3) The system preference doesn't mention that it applies to Elasticsearch
> only - I think we need to add this information.

I added this info on the system-preferences page.

> 4) My sample search: Dutch / William doesn't work with any of the settings.
> What am I missing?

I think the reason might be that if you included the ":" it will be interpreted
as a field query (this could be the subject for another bug). Try just: an
essential grammar / William Z. Shetter, for example.

-- 
You are receiving this mail because:
You are watching all bug changes.