From 2fe6305bd7e6886d4003947bdebad2544de15f59 Mon Sep 17 00:00:00 2001 From: Phil Ringnalda Date: Fri, 21 Jun 2024 11:38:10 -0700 Subject: [PATCH] Bug 37152: Can't delete an Acquisitions contract admin/aqcontract.pl expects the op cud-delete-confirm, but what's sent is the op delete-confirm because it's just a GET of the page that will actually do a cud op once it is confirmed. Test plan: 1. Without the patch, Acquisitions - Vendor search for My Vendor 2. Left Sidebar menu - Contracts - Add a contract 3. Fill in Name, Start date, End date, Save 4. On the line for the contract, click Delete 5. Note that the page that loads has no button for "Yes, delete contract" (or for anything else) 6. Apply patch, restart_all 7. Acquisitions - Vendor search for My Vendor 8. Left Sidebar menu, Contracts, click Delete, then Yes, delete contract 9. Vendor search for My Vendor, Left Sidebar Contracts, verify the contract is gone Signed-off-by: Emily Lamancusa Signed-off-by: Nick Clemens Signed-off-by: Katrin Fischer --- admin/aqcontract.pl | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/admin/aqcontract.pl b/admin/aqcontract.pl index 46954ea026..c1bca4a4c5 100755 --- a/admin/aqcontract.pl +++ b/admin/aqcontract.pl @@ -129,7 +129,7 @@ elsif ( $op eq 'cud-add_validate' ) { # END $OP eq ADD_VALIDATE } #DELETE_CONFIRM: called by default form, used to confirm deletion of data in DB -elsif ( $op eq 'cud-delete_confirm' ) { +elsif ( $op eq 'delete_confirm' ) { $template->param( delete_confirm => 1 ); my $contract = GetContract( { contractnumber => $contractnumber } ); -- 2.39.5