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:30 +0000 (08:39 -0400)
commitcd281e488032c57d4a47c3913d5f887c3bca69d8
tree28710c4a923087268d86b6473abcc6db4c3ec700
parent0e6a2440a0d16b6248fcdac7b66d0c2b938a8078
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>
catalogue/search.pl
opac/opac-search.pl