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> (cherry picked from commit16e5a9d8dc) Signed-off-by: Lucas Gass <lucas@bywatersolutions.com> (cherry picked from commit5436293590) Signed-off-by: Aleisha Amohia <aleishaamohia@hotmail.com> (cherry picked from commit3447a782c2) Signed-off-by: Victor Grousset/tuxayo <victor@tuxayo.net>
This commit is contained in:
parent
7d6fc1ec20
commit
53b7843ee9
|
|
@ -46,7 +46,7 @@ function confirm_deletion(id) {
|
|||
+ "&excluding=[% excluding | html %]"
|
||||
+ "&operator=[% operator | html %]"
|
||||
+ "&orderby=[% orderby | html %]"
|
||||
+ "&value=[% value |url %]"
|
||||
+ "&value=[% value |uri %]"
|
||||
+ "&startfrom=[% startfrom | html %]"
|
||||
+ "&resultsperpage=[% resultsperpage | html %]"
|
||||
+ "&csrf_token=[% csrf_token | html %]";
|
||||
|
|
|
|||
Loading…
Reference in New Issue