Bug 10281: searching for a search domain fails
authorJared Camins-Esakov <jcamins@cpbibliography.com>
Sat, 18 May 2013 13:09:23 +0000 (09:09 -0400)
committerJared Camins-Esakov <jcamins@cpbibliography.com>
Sun, 19 May 2013 12:39:46 +0000 (08:39 -0400)
commit43e0400728901bef61b40866e7007cac7680523b
tree60eaaad35a59a46822ec29a73dc46d9db36a456d
parentdc7ad9b673587f53fa6cee119c3cd0bebabb0669
Bug 10281: searching for a search domain fails

For reasons I cannot fathom, the split() in handling multi-branch
limits was not coming up with a valid search group code. Replacing
the split() with a substr() and creating the CGI parameter as a string
rather than as an arrayref fixes the problem. This problem may not
affect all installations, since I tested this exact feature just under
two months ago and it worked fine, and none of the relevant code has
been changed since then that I can see.

To test:
1) Create search group, and add at least one library to it, in
   /cgi-bin/koha/admin/branches.pl
2) Apply patch
3) Try doing a search limited to your search group, making sure that
   the search will match items that belong to a library in the search
   group
4) Sign off

Signed-off-by: Magnus Enger <magnus@enger.priv.no>
I have failed to recreate the problem on three different dev installs,
both on Ubuntu and Debian, but the current patch does not break
anything as far as I can tell, so I'm signing off.

I tested with two libraries in the same search domain, with each
library owning a different book by the same author. Searching for
the author in
- all libraries,
- individual libraries and
- the search domain that contains both libraries
all return the expected results.

Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
All tests and QA script pass.
I couldn't reproduce the problem, but didn't find any regressions.
Signed-off-by: Jared Camins-Esakov <jcamins@cpbibliography.com>
(cherry picked from commit cd281e488032c57d4a47c3913d5f887c3bca69d8)

Signed-off-by: Jared Camins-Esakov <jcamins@cpbibliography.com>
catalogue/search.pl
opac/opac-search.pl