From 192bd3f79e237f75c5e9c1b96950b6b0d53768e0 Mon Sep 17 00:00:00 2001
From: Owen Leonard <oleonard@myacpl.org>
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
---
 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.2