From 3d41c25c08a12ab38af6a2636121501bc12a68bf Mon Sep 17 00:00:00 2001
From: Baptiste Wojtkowski <baptiste.wojtkowski@biblibre.com>
Date: Fri, 7 Aug 2026 09:56:07 +0000
Subject: [PATCH] Bug 43245: Redirect user if no email was provided

TO TEST:
0 - Apply patch
1 - Make sure OPACResetPassword is activated
2 - Go to that good old Henry Acevedo's page and reset their password
3 - Notice you have been redirected with an error message telling you
have no email

Signed-off-by: David Nind <david@davidnind.com>
Signed-off-by: Andrew Fuerste Henry <andrew@bywatersolutions.com>
Signed-off-by: Pedro Amorim <pedro.amorim@openfifth.co.uk>
---
 koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember.tt | 4 ++++
 members/notices.pl                                            | 3 +++
 2 files changed, 7 insertions(+)

diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember.tt
index fcd30b063d..ff127995ce 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember.tt
@@ -83,6 +83,10 @@
                 <h3>Unable to renew patron</h3>
                 <p>They must have a guarantor</p>
             [% END %]
+            [% IF ( error == 'NO_EMAIL_FOR_RECOVERY' ) %]
+                <h3>Unable to send recovery email</h3>
+                <p>No email address provided</p>
+            [% END %]
         </div>
     [% END %]
 
diff --git a/members/notices.pl b/members/notices.pl
index f563bfc1cc..ff5e7687cb 100755
--- a/members/notices.pl
+++ b/members/notices.pl
@@ -131,6 +131,9 @@ if ( $op eq 'send_password_reset' ) {
 
         # send staff initiated password recovery
         SendPasswordRecoveryEmail( $patron, $emailaddr, 1 );
+    } else {
+        print $input->redirect(
+            "/cgi-bin/koha/members/moremember.pl?borrowernumber=$borrowernumber&error=NO_EMAIL_FOR_RECOVERY");
     }
 
     # redirect to self to avoid form submission on refresh