From fb9ad768b29b1eb7f98607fa9b2915978e453791 Mon Sep 17 00:00:00 2001
From: "burnus%gmx.de" <>
Date: Mon, 17 Feb 2003 00:11:45 +0000
Subject: Bug 135820 - token cancellation message are not user-friendly r=gerv,
 a=justdave

---
 CGI.pl                                            |  2 +-
 template/en/default/account/cancel-token.txt.tmpl | 28 +++++++++++++++++------
 2 files changed, 22 insertions(+), 8 deletions(-)

diff --git a/CGI.pl b/CGI.pl
index eb74862fa..4765a7a00 100644
--- a/CGI.pl
+++ b/CGI.pl
@@ -414,7 +414,7 @@ sub confirm_login {
             use Token;
             my $token = Token::HasPasswordToken($userid);
             while ( $token ) {
-                Token::Cancel($token, "user logged in");
+                Token::Cancel($token, 'user_logged_in');
                 $token = Token::HasPasswordToken($userid);
             }
         }
diff --git a/template/en/default/account/cancel-token.txt.tmpl b/template/en/default/account/cancel-token.txt.tmpl
index b1545f6d3..be926d5e2 100644
--- a/template/en/default/account/cancel-token.txt.tmpl
+++ b/template/en/default/account/cancel-token.txt.tmpl
@@ -21,11 +21,12 @@
   #%]
 From: bugzilla-admin-daemon
 To: [% emailaddress %]
-Subject: [% tokentype %] token cancelled
+Subject: [% PROCESS subject %]
+
+A request was cancelled from [% remoteaddress %].
 
-A token was cancelled from [% remoteaddress %].  
 If you did not request this, it could be either an honest 
-mistake or the result of a malicious hack attempt.  
+mistake or someone attempting to break into your Bugzilla account.
 
 Take a look at the information below and forward this email 
 to [% maintainer %] if you suspect foul play.
@@ -38,6 +39,16 @@ to [% maintainer %] if you suspect foul play.
 Cancelled Because:
 [% PROCESS cancelactionmessage %]
 
+[% BLOCK subject %]
+  [% IF tokentype == 'password' %]
+    Password change request cancelled
+  [% ELSIF tokentype == 'emailnew' OR tokentype == 'emailold' %]
+    Email change request cancelled
+  [% ELSE %]
+    [% tokentype %] token cancelled
+  [% END %]
+[% END %]
+
 [% BLOCK cancelactionmessage %]
   [% IF    cancelaction == 'account_exists' %]
     Account [% email %] already exists.
@@ -57,16 +68,19 @@ Cancelled Because:
     to [% new_email %] has been cancelled.
 
   [% ELSIF cancelaction == 'password_change_canceled' %]
-    The user requested cancellation.
+    You have requested cancellation.
+
+  [% ELSIF cancelaction == 'user_logged_in' %]
+    You have logged in.
 
   [% ELSIF cancelaction == 'wrong_token_for_changing_passwd' %]
-    The user tried to use the token to change the password.
+    You have tried to use the token to change the password.
 
   [% ELSIF cancelaction == 'wrong_token_for_cancelling_email_change' %]
-    The user tried to use the token to cancel the email address change.
+    You have tried to use the token to cancel the email address change.
 
   [% ELSIF cancelaction == 'wrong_token_for_confirming_email_change' %]
-    The user tried to use the token to confirm the email address change.
+    You have tried to use the token to confirm the email address change.
 
   [% ELSE %]
     [%# Give sensible error if the cancel-token function is used incorrectly.
-- 
cgit v1.2.3-24-g4f1b