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 <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 2fe6305bd7)
Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>
This commit is contained in:
parent
58e831ce63
commit
0f86951b3c
|
|
@ -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 } );
|
||||
|
|
|
|||
Loading…
Reference in New Issue