]> git.koha-community.org Git - koha.git/commit
Bug 37152: Deleting framework tags expects a cud-delete-confirm op instead of delete...
authorPhil Ringnalda <phil@chetcolibrary.org>
Fri, 21 Jun 2024 18:53:25 +0000 (11:53 -0700)
committerLucas Gass <lucas@bywatersolutions.com>
Thu, 18 Jul 2024 19:52:37 +0000 (19:52 +0000)
commit4c4126bdb72ec54fb1bb177a19332994e821402b
treeba64e2133ddf0ff06300b23fcc976065edebfd1e
parent0f86951b3c25d387f2339ba1b5d1355f482b6686
Bug 37152: Deleting framework tags expects a cud-delete-confirm op instead of delete-confirm

The scripts for managing biblio and authority framework tag editing are
expecting an op of cud-delete-confirm, but the op that is passed is the
correct thing, delete-confirm, because it's a GET of a page where the
confirmation will happen, not the later cud- operation. You can only see
that the expected op is wrong because the confirmation page doesn't get
the description of the tag.

Test plan:
 1. Without the patch, Administration - MARC bibliographic framework
 2. Default framework Actions menu, MARC structure
 3. In the row for 025, Actions menu, Delete
 4. Note that the page which loads shows "Tag: 025" and "Description:"
    without actually having a description
 5. Administration - Authority types
 6. Default authority type Actions menu, MARC structure
 7. In the row for 046, Actions menu, Delete
 8. Note that the page that loads shows "Tag: 046" without the tag description
 9. Apply patch, restart_all
10. With the patch, repeat steps 1-8, but this time note that you see the
    description of the tag, not just the number. For both, click Yes, delete,
    and verify that the tag is deleted.

Signed-off-by: Emily Lamancusa <emily.lamancusa@montgomerycountymd.gov>
Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
Signed-off-by: Katrin Fischer <katrin.fischer@bsz-bw.de>
(cherry picked from commit 11f5db2641ef87fab981bfa273827b8309125440)
Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>
admin/auth_tag_structure.pl
admin/marctagstructure.pl