]> git.koha-community.org Git - koha.git/commit
do some validation of the KohaOpacRecentSearches cookie
authorGalen Charlton <gmc@esilibrary.com>
Sun, 28 Jul 2013 02:48:44 +0000 (02:48 +0000)
committerJared Camins-Esakov <jcamins@cpbibliography.com>
Mon, 29 Jul 2013 12:47:11 +0000 (08:47 -0400)
commitb7cbf8b53101824243b32ead096f6756400cc479
tree12c90ee8b1eefde46fdde1d1848a16034ceef940
parent73fc8f439fcaeb790f96dffe0a8f99aed194e311
do some validation of the KohaOpacRecentSearches cookie

Add validation of the value of the KohaOpacRecentSearches.  In
particular, this patch avoids the generation of an internal server
error when the OPAC is presented with an old cookie that uses the
old Storable-based serialization.

This patch also moves parsing of the cookie value into a
new routine in C4::Auth, ParseSearchHistoryCookie, and adds
a test case.

To test (in conjunction with the previous patch):

Exercise the OPAC search history functionality, after
turning on the EnableOpacSearchHistory syspref:

- As an anonymous user, conduct a variety of searches,
  including ones that include non-ASCII characters
- Check the search history and verify that all searches
  are listed
- Apply this patch and the previous one.
- Do *not* clear the KohaOpacRecentSearches cookie
- Check the search history and verify that no searches
  are listed any more
- As an anonymous user, conduct a variety of searches,
  including ones that include non-ASCII characters
- Check the search history and verify that all searches
  are listed
- Log into the OPAC
- Verify that current and past searches are listed in
  search history.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
(cherry picked from commit 2932860374670ac2656261db15e3375c6ed2e9f1)

Signed-off-by: Jared Camins-Esakov <jcamins@cpbibliography.com>
C4/Auth.pm
opac/opac-search-history.pl
opac/opac-search.pl
t/Auth_ParseSearchHistoryCookie.t [new file with mode: 0644]