]> git.koha-community.org Git - koha.git/commit
Bug 38056: Search term after deleting an authority shouldn't be URI encoded
authorPhil Ringnalda <phil@chetcolibrary.org>
Wed, 2 Oct 2024 02:01:39 +0000 (19:01 -0700)
committerKatrin Fischer <katrin.fischer@bsz-bw.de>
Mon, 21 Oct 2024 13:10:29 +0000 (15:10 +0200)
commit6eb4d5804b006f16f6876dcfd9b4b6b036bba10d
tree4546a7d46451973373527c2bbd65ba51a59d86a8
parentcbf6acbe7a07f8ed2bb1c3763c6ec155f3eaf80b
Bug 38056: Search term after deleting an authority shouldn't be URI encoded

Now that deleting an authority record uses a form, rather than building up a
URL in JavaScript, using the filter | uri on the value (confusingly named
value) which passes along the search terms to reload after deleting results
in a bad search. It should instead use the filter | html.

Test plan:
 1. Without the patch, top menu - Authorities
 2. The default search, Main heading ($a only), Authority type Default,
    Operator contains, Order by Heading A-Z will work fine. You need to a
    search which will return more than one of the same thing (not hard in
    ktd) - for MARC21 search for a professional
 3. You should get five results. For one of them, Actions - Delete - Confirm
    that you meant to click it
 4. You now have a%20professional in the search box, and search results for
    whatever ElasticSearch thinks a% means. It's certainly not professional -
    you can change that term to any random string and get the same results
 5. Apply patch, top menu - Authorities
 6. Repeat steps 2 and 3, but this time after deleting another result,
    you will get a professional without the %20 in the search box, and
    your results will be the remaining Addison-Wesley professional computing
    series authorities

Sponsored-by: Chetco Community Public Library
Signed-off-by: David Nind <david@davidnind.com>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Signed-off-by: Katrin Fischer <katrin.fischer@bsz-bw.de>
koha-tmpl/intranet-tmpl/prog/en/modules/authorities/searchresultlist.tt