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)
committerKatrin Fischer <katrin.fischer@bsz-bw.de>
Thu, 27 Jun 2024 09:49:55 +0000 (11:49 +0200)
commit11f5db2641ef87fab981bfa273827b8309125440
tree7bd8b6176285f59cd919ed09a70218727da36711
parent2fe6305bd7e6886d4003947bdebad2544de15f59
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>
admin/auth_tag_structure.pl
admin/marctagstructure.pl