Browse Source

Bug 26606: Correctly URI-encode query string in URL loaded after deleting an authority record

If you search for "this & that" and delete one of the results, after
it is deleted the search in the reloaded page is for "this " rather
than for "this & that". The template uses the url filter, which is
for filtering an entire URI where :, &, #, etc. should be left alone,
rather than the uri filter which is for filtering a URI component
where they should be encoded.

To test:
1) Go to Authorities > New authority > New from Z39.50/SRU
2) Search for the Author (corporate) this & that collective
3) Actions > Import, Save
4) From the detail for that authority, Edit > Edit as new (duplicate)
5) Edit the 110 field, add 2 at the end and Save
6) Search authorities for this & that
7) For one of the ones you created, Actions > Delete, confirm
8) The page that reloads should have a search for this & that, not
just for this

Signed-off-by: Didier Gautheron <didier.gautheron@biblibre.com>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
20.11.x
Phil Ringnalda 4 years ago
committed by Jonathan Druart
parent
commit
16e5a9d8dc
  1. 2
      koha-tmpl/intranet-tmpl/prog/en/includes/authorities_js.inc

2
koha-tmpl/intranet-tmpl/prog/en/includes/authorities_js.inc

@ -47,7 +47,7 @@ function confirm_deletion(id) {
+ "&excluding=[% excluding | html %]" + "&excluding=[% excluding | html %]"
+ "&operator=[% operator | html %]" + "&operator=[% operator | html %]"
+ "&orderby=[% orderby | html %]" + "&orderby=[% orderby | html %]"
+ "&value=[% value |url %]" + "&value=[% value |uri %]"
+ "&startfrom=[% startfrom | html %]" + "&startfrom=[% startfrom | html %]"
+ "&resultsperpage=[% resultsperpage | html %]" + "&resultsperpage=[% resultsperpage | html %]"
+ "&csrf_token=[% csrf_token | html %]"; + "&csrf_token=[% csrf_token | html %]";

Loading…
Cancel
Save