From cf0876d071d6f8b1a38f790c5449917e93bc8d6f Mon Sep 17 00:00:00 2001 From: Emily Lamancusa Date: Fri, 14 Jun 2024 10:29:34 -0400 Subject: [PATCH] Bug 37091: Correct op parameter in systempreferences.pl To test: 1. Go to Administration > System Preferences > Local Use 2. Attempt to delete a local system preference --> The confirmation page displays a blank name and value 3. Click "Yes, delete" --> The system preference is still there 4. Apply patch, restart_all, and reload page 5. Attempt to delete the system preference again --> The confirmation page displays correct information 6. Click "Yes, delete" --> The system preference is deleted successfully Signed-off-by: David Nind --- admin/systempreferences.pl | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/admin/systempreferences.pl b/admin/systempreferences.pl index 609a06eda4..816714034d 100755 --- a/admin/systempreferences.pl +++ b/admin/systempreferences.pl @@ -339,7 +339,7 @@ if ( $op eq 'add_form' ) { exit; ################## 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' ) { my $value = C4::Context->preference($searchfield); $template->param( searchfield => $searchfield, -- 2.39.2