Bug 452519: Fix timezones in emails - Patch by Frédéric Buclin <LpSolit@gmail.com> r=wicked a=LpSolit
git-svn-id: svn://10.0.0.236/trunk@255775 18797224-902f-48f8-a5cc-f745e15eee43
This commit is contained in:
@@ -74,7 +74,7 @@ sub issue_new_user_account_token {
|
||||
my ($token, $token_ts) = _create_token(undef, 'account', $login_name);
|
||||
|
||||
$vars->{'email'} = $login_name . Bugzilla->params->{'emailsuffix'};
|
||||
$vars->{'token_ts'} = $token_ts;
|
||||
$vars->{'expiration_ts'} = ctime($token_ts + MAX_TOKEN_AGE * 86400);
|
||||
$vars->{'token'} = $token;
|
||||
|
||||
my $message;
|
||||
@@ -103,10 +103,7 @@ sub IssueEmailChangeToken {
|
||||
|
||||
$vars->{'oldemailaddress'} = $old_email . $email_suffix;
|
||||
$vars->{'newemailaddress'} = $new_email . $email_suffix;
|
||||
|
||||
$vars->{'max_token_age'} = MAX_TOKEN_AGE;
|
||||
$vars->{'token_ts'} = $token_ts;
|
||||
|
||||
$vars->{'expiration_ts'} = ctime($token_ts + MAX_TOKEN_AGE * 86400);
|
||||
$vars->{'token'} = $token;
|
||||
$vars->{'emailaddress'} = $old_email . $email_suffix;
|
||||
|
||||
@@ -151,8 +148,10 @@ sub IssuePasswordToken {
|
||||
|
||||
$vars->{'token'} = $token;
|
||||
$vars->{'emailaddress'} = $user->email;
|
||||
$vars->{'max_token_age'} = MAX_TOKEN_AGE;
|
||||
$vars->{'token_ts'} = $token_ts;
|
||||
$vars->{'expiration_ts'} = ctime($token_ts + MAX_TOKEN_AGE * 86400);
|
||||
# The user is not logged in (else he wouldn't request a new password).
|
||||
# So we have to pass this information to the template.
|
||||
$vars->{'timezone'} = $user->timezone;
|
||||
|
||||
my $message = "";
|
||||
$template->process("account/password/forgotten-password.txt.tmpl",
|
||||
@@ -233,6 +232,9 @@ sub Cancel {
|
||||
$vars->{'token'} = $token;
|
||||
$vars->{'tokentype'} = $tokentype;
|
||||
$vars->{'issuedate'} = $issuedate;
|
||||
# The user is probably not logged in.
|
||||
# So we have to pass this information to the template.
|
||||
$vars->{'timezone'} = $user->timezone;
|
||||
$vars->{'eventdata'} = $eventdata;
|
||||
$vars->{'cancelaction'} = $cancelaction;
|
||||
|
||||
|
||||
@@ -37,7 +37,7 @@ to [% Param('maintainer') %] if you suspect foul play.
|
||||
Token: [% token %]
|
||||
Token Type: [% tokentype %]
|
||||
User: [% emailaddress %]
|
||||
Issue Date: [% issuedate %]
|
||||
Issue Date: [% issuedate FILTER time("%Y-%m-%d %H:%M:%S %Z", timezone) %]
|
||||
Event Data: [% eventdata %]
|
||||
Canceled Because: [% PROCESS cancelactionmessage %]
|
||||
|
||||
|
||||
@@ -20,7 +20,6 @@
|
||||
|
||||
[% PROCESS global/variables.none.tmpl %]
|
||||
|
||||
[% expiration_ts = token_ts + (max_token_age * 86400) %]
|
||||
From: [% Param('mailfrom') %]
|
||||
To: [% emailaddress %]
|
||||
Subject: [% terms.Bugzilla %] Change Email Address Request
|
||||
@@ -38,5 +37,5 @@ this request, visit the following link:
|
||||
|
||||
[%+ urlbase %]token.cgi?t=[% token FILTER url_quote %]&a=cxlem
|
||||
|
||||
If you do nothing, the request will lapse after [%+ max_token_age %] days
|
||||
(on [%+ time2str("%B %o, %Y at %H:%M %Z", expiration_ts) %]).
|
||||
If you do nothing, the request will lapse after [% constants.MAX_TOKEN_AGE %] days
|
||||
(on [% expiration_ts FILTER time("%B %e, %Y at %H:%M %Z") %]).
|
||||
|
||||
@@ -25,7 +25,6 @@
|
||||
|
||||
[% PROCESS global/variables.none.tmpl %]
|
||||
|
||||
[% expiration_ts = token_ts + (max_token_age * 86400) %]
|
||||
From: [% Param('mailfrom') %]
|
||||
To: [% emailaddress %]
|
||||
Subject: [% terms.Bugzilla %] Change Email Address Request
|
||||
@@ -43,5 +42,5 @@ this request, visit the following link:
|
||||
[%+ urlbase %]token.cgi?t=[% token FILTER url_quote %]&a=cxlem
|
||||
|
||||
If you do nothing, and [%+ newemailaddress %] confirms this request,
|
||||
the change will be made permanent after [%+ max_token_age %] days
|
||||
(on [%+ time2str("%B %o, %Y at %H:%M %Z", expiration_ts) %]).
|
||||
the change will be made permanent after [% constants.MAX_TOKEN_AGE %] days
|
||||
(on [% expiration_ts FILTER time("%B %e, %Y at %H:%M %Z") %]).
|
||||
|
||||
@@ -16,7 +16,7 @@
|
||||
[%# INTERFACE:
|
||||
# token: string. The token to be used in the user account creation.
|
||||
# email: email address of the user account.
|
||||
# date: creation date of the token.
|
||||
# expiration_ts: expiration date of the token.
|
||||
#%]
|
||||
|
||||
[% title = BLOCK %]Create a new user account for '[% email FILTER html %]'[% END %]
|
||||
@@ -24,7 +24,6 @@
|
||||
title = title
|
||||
onload = "document.forms['confirm_account_form'].realname.focus();" %]
|
||||
|
||||
[% expiration_ts = date + (constants.MAX_TOKEN_AGE * 86400) %]
|
||||
<div>
|
||||
To create your account, you must enter a password in the form below.
|
||||
Your email address and Real Name (if provided) will be shown with
|
||||
@@ -60,7 +59,7 @@
|
||||
|
||||
<p>
|
||||
This account will not be created if this form is not completed by
|
||||
<u>[%+ time2str("%B %o, %Y at %H:%M %Z", expiration_ts) %]</u>.
|
||||
<u>[% expiration_ts FILTER time("%B %e, %Y at %H:%M %Z") %]</u>.
|
||||
</p>
|
||||
|
||||
<p>
|
||||
|
||||
@@ -15,13 +15,12 @@
|
||||
|
||||
[%# INTERFACE:
|
||||
# token: random string used to authenticate the transaction.
|
||||
# token_ts: creation date of the token.
|
||||
# expiration_ts: expiration date of the token.
|
||||
# email: email address of the new account.
|
||||
#%]
|
||||
|
||||
[% PROCESS global/variables.none.tmpl %]
|
||||
|
||||
[% expiration_ts = token_ts + (constants.MAX_TOKEN_AGE * 86400) %]
|
||||
From: [% Param('mailfrom') %]
|
||||
To: [% email %]
|
||||
Subject: [% terms.Bugzilla %]: confirm account creation
|
||||
@@ -31,11 +30,11 @@ X-Bugzilla-Type: admin
|
||||
using your email address ([% email %]).
|
||||
|
||||
To continue creating an account using this email address, visit the
|
||||
following link by [%+ time2str("%B %o, %Y at %H:%M %Z", expiration_ts) %]:
|
||||
following link by [% expiration_ts FILTER time("%B %e, %Y at %H:%M %Z") %]:
|
||||
|
||||
[%+ urlbase %]token.cgi?t=[% token FILTER url_quote %]&a=request_new_account
|
||||
|
||||
If you did not receive this email before [%+ time2str("%B %o, %Y at %H:%M %Z", expiration_ts) %] or
|
||||
If you did not receive this email before [% expiration_ts FILTER time("%B %e, %Y at %H:%M %Z") %] or
|
||||
you wish to create an account using a different email address you can begin
|
||||
again by going to:
|
||||
|
||||
|
||||
@@ -20,7 +20,6 @@
|
||||
|
||||
[% PROCESS global/variables.none.tmpl %]
|
||||
|
||||
[% expiration_ts = token_ts + (max_token_age * 86400) %]
|
||||
From: [% Param('mailfrom') %]
|
||||
To: [% emailaddress %]
|
||||
Subject: [% terms.Bugzilla %] Change Password Request
|
||||
@@ -36,6 +35,6 @@ this request, visit the following link:
|
||||
|
||||
[%+ urlbase %]token.cgi?t=[% token FILTER url_quote %]&a=cxlpw
|
||||
|
||||
If you do nothing, the request will lapse after [%+ max_token_age +%] days (at
|
||||
precisely [%+ time2str("%H:%M on the %o of %B, %Y", expiration_ts) -%]) or when you
|
||||
If you do nothing, the request will lapse after [% constants.MAX_TOKEN_AGE %] days
|
||||
(on [% expiration_ts FILTER time("%B %e, %Y at %H:%M %Z", timezone) %]) or when you
|
||||
log in successfully.
|
||||
|
||||
@@ -37,6 +37,7 @@ use Bugzilla::Error;
|
||||
use Bugzilla::Token;
|
||||
use Bugzilla::User;
|
||||
|
||||
use Date::Format;
|
||||
use Date::Parse;
|
||||
|
||||
my $dbh = Bugzilla->dbh;
|
||||
@@ -351,7 +352,7 @@ sub request_create_account {
|
||||
my (undef, $date, $login_name) = Bugzilla::Token::GetTokenData($token);
|
||||
$vars->{'token'} = $token;
|
||||
$vars->{'email'} = $login_name . Bugzilla->params->{'emailsuffix'};
|
||||
$vars->{'date'} = str2time($date);
|
||||
$vars->{'expiration_ts'} = ctime(str2time($date) + MAX_TOKEN_AGE * 86400);
|
||||
|
||||
# When 'ssl' equals 'always' or 'authenticated sessions',
|
||||
# we want this form to always be over SSL.
|
||||
|
||||
Reference in New Issue
Block a user