Bug 20876: The form_serialized_itype cookie is not used and should be removed
authorOwen Leonard <oleonard@myacpl.org>
Mon, 4 Jun 2018 14:40:03 +0000 (14:40 +0000)
committerNick Clemens <nick@bywatersolutions.com>
Fri, 22 Jun 2018 16:28:51 +0000 (16:28 +0000)
This patch removes references to a form_serialized_itype cookie which
appears in the code as part of the process of storing information used
in the generation of the "Return to the last advanced search" link.

To test, apply the patch and perform an advanced search in the OPAC
using several search options including a limit by item type. From the
search results page, follow the "Return to the last advanced search"
link. The advanced search form should appear with all the search options
you submitted before.

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
koha-tmpl/opac-tmpl/bootstrap/en/includes/opac-bottom.inc
koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-advsearch.tt

index 763d090..735cc89 100644 (file)
@@ -253,7 +253,6 @@ $(document).ready(function() {
     if($('#searchsubmit').length) {
         $(document).on("click", '#searchsubmit', function(e) {
             jQuery.removeCookie("form_serialized", { path: '/'});
-            jQuery.removeCookie("form_serialized_itype", { path: '/'});
             jQuery.removeCookie("form_serialized_limits", { path: '/'});
             jQuery.removeCookie("num_paragraph", { path: '/'});
             jQuery.removeCookie("search_path_code", { path: '/'});
index 5eca3d2..620d83b 100644 (file)
@@ -421,7 +421,6 @@ $(document).ready(function() {
     [% ELSE %]
         //Clear all form cookies
         jQuery.removeCookie("form_serialized", { path: '/'});
-        jQuery.removeCookie("form_serialized_itype", { path: '/'});
         jQuery.removeCookie("form_serialized_limits", { path: '/'});
         jQuery.removeCookie("num_paragraph", { path: '/'});
         jQuery.removeCookie("search_path_code", { path: '/'});