From 89f3e67a1d58b1060b9c2f22370173d1c483c723 Mon Sep 17 00:00:00 2001 From: Owen Leonard Date: Tue, 9 Jul 2024 18:40:33 +0000 Subject: [PATCH] Bug 36276: Fix editing of identity provider and domains This patch fixes identity_providers.pl to remove the "cud-" prefix for two operations which are not POSTed: editing identity providers and editing domains. To test, apply the patch and restart services. - Go to Administration -> Identity providers. - If necessary, create an identity provider and then click "Manage domains" -> New identity provider domain to create a new domain. - From the list of domains, click "Edit" - The edit form should appear with all the correct details prefilled. - Check that the page title, breadcrumbs, and page header all show the correct domain. - Make a change to the domain and confirm that the changes are saved. - From the list of identity providers, click "Edit" - This form should also load correctly. - Submitted changes should be saved correctly. Sponsored-by: Athens County Public Libraries Signed-off-by: David Nind Signed-off-by: David Cook Signed-off-by: Katrin Fischer (cherry picked from commit 32578e4f724d56ebb8905f6523b665026eea8e7a) Signed-off-by: Lucas Gass --- admin/identity_providers.pl | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/admin/identity_providers.pl b/admin/identity_providers.pl index 27622ae8dc..74416f6aa7 100755 --- a/admin/identity_providers.pl +++ b/admin/identity_providers.pl @@ -155,7 +155,7 @@ elsif ( $domain_ops && $op eq 'cud-add' ) { # list servers after adding $op = 'list'; } -elsif ( !$domain_ops && $op eq 'cud-edit_form' ) { +elsif ( !$domain_ops && $op eq 'edit_form' ) { if ( $identity_provider ) { $template->param( @@ -171,7 +171,7 @@ elsif ( !$domain_ops && $op eq 'cud-edit_form' ) { }; } } -elsif ( $domain_ops && $op eq 'cud-edit_form' ) { +elsif ( $domain_ops && $op eq 'edit_form' ) { my $identity_provider_domain_id = $input->param('identity_provider_domain_id'); my $identity_provider_domain; -- 2.39.5