From 8de3f8a8f62d475f1c2d65ad171473c9bd9d9f2d Mon Sep 17 00:00:00 2001
From: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Date: Wed, 31 Jan 2024 11:42:57 +0100
Subject: [PATCH] Bug 34478: Add 'op' to opac-messaging

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
---
 koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-messaging.tt | 2 +-
 opac/opac-messaging.pl                                     | 3 ++-
 2 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-messaging.tt b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-messaging.tt
index 33564e2443..1fa2975e9f 100644
--- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-messaging.tt
+++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-messaging.tt
@@ -39,7 +39,7 @@
                         [% END %]
                         <form action="/cgi-bin/koha/opac-messaging.pl" method="post" name="opacmessaging">
                             [% INCLUDE 'csrf-token.inc' %]
-                            <input type="hidden" name="modify" value="yes" />
+                            <input type="hidden" name="op" value="cud-modify" />
 
                             [% IF Koha.Preference( 'EnhancedMessagingPreferencesOPAC' )  %]
                                 <table class="table table-bordered table-condensed table-striped">
diff --git a/opac/opac-messaging.pl b/opac/opac-messaging.pl
index dac74addc4..a3eab4d74f 100755
--- a/opac/opac-messaging.pl
+++ b/opac/opac-messaging.pl
@@ -51,8 +51,9 @@ my $patron = Koha::Patrons->find( $borrowernumber ); # FIXME and if borrowernumb
 my $messaging_options;
 $messaging_options = C4::Members::Messaging::GetMessagingOptions() if $opac_messaging;
 
-if ( defined $query->param('cud-modify') && $query->param('cud-modify') eq 'yes' ) {
+my $op = $query->param('op') || q{};
 
+if ( $op eq 'cud-modify' ) {
     if( $opac_messaging ) {
         my $sms = $query->param('SMSnumber');
         my $sms_provider_id = $query->param('sms_provider_id');
-- 
2.39.5