]> 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)
committerLucas Gass <lucas@bywatersolutions.com>
Fri, 25 Oct 2024 15:23:24 +0000 (15:23 +0000)
commit33fb560cf10272bf557c3fd0b0f2b6fcebc3923f
tree449b93da26fc989d86c363ac877a204958e4cb4f
parent695c411f65d63a09ef949191adc32cf4f86cc990
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>
(cherry picked from commit 6eb4d5804b006f16f6876dcfd9b4b6b036bba10d)
Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>
koha-tmpl/intranet-tmpl/prog/en/modules/authorities/searchresultlist.tt