[Koha-bugs] [Bug 15930] DataTables patron search defaulting to 'starts_with' and not getting correct parameters to parse multiple word searches

bugzilla-daemon at bugs.koha-community.org bugzilla-daemon at bugs.koha-community.org
Tue Apr 5 14:25:12 CEST 2016


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

Blou <philippe.blouin at inlibro.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |In Discussion
                 CC|                            |philippe.blouin at inlibro.com

--- Comment #10 from Blou <philippe.blouin at inlibro.com> ---
see also bug 16197

So, this discussion is about "what should be the default behavior of the
function", which brings "what is the expected behavior for the users", which
could means "The users of previous versions" or "normal internet users used to
certain UI behaviors", which are different.

The function works.  It allows for 
- splitting the words and looking for their usages somewhere
- using the word "as-is", including the spaces/commas
It does not allow for
- splitting the words and using each as a start with.  Not a loss, it's mostly
useless

My opinion is that it should be resolved in two phases:
1) Add the argument "contains" to each call that need it.  Even if it's all of
them, it helps readability of complex functions.
2) Add the possibility of quotes, at which points files like request.pl must
NOT do their own s =~ and leave everything to the call.

The 1) is very important to move fast.  Right now it is a bug.  However you
define it, this is a regression to the users.  Let's fix that in master before
3.24, and in the next patch for 3.22.

Then let someone develop the quote thing. It's a nice feature and matches users
expectations, which are set by big search engines, not by us.

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


More information about the Koha-bugs mailing list