From e51c4771246999c64c00e76baf326318c69975ce 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 --- admin/aqcontract.pl | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/admin/aqcontract.pl b/admin/aqcontract.pl index 46954ea0266..c1bca4a4c5d 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.2