352625 - "You are receiving this mail because" messages cannot be localized

Status: RESOLVED FIXED

(Bugzilla :: Email Notifications, defect)

Description

[Max Kanat-Alexander]

There's a whole block of code in Bugzilla::BugMail that has the reasons you're received the mail. All that stuff should be inside the template instead.

Comment 1

[Max Kanat-Alexander]

Attachment: v1

This is a pretty simple patch.

The Filesystem.pm change is sort of related to this bug, and I'll sign off on it as module owner. (Basically, it was just something we were missing from Filesystem.pm.)

Comment 2

[Myk Melez [:myk] [@mykmelez]]

Comment on attachment 238394 [details] [diff] [review]
v1


>+You are the watching QA contact of the [% terms.bug %].

the watching -> watching the

Otherwise looks good.  Just fix that on checkin.

Comment 3

[Max Kanat-Alexander]

Checking in Bugzilla/BugMail.pm;
/cvsroot/mozilla/webtools/bugzilla/Bugzilla/BugMail.pm,v  <--  BugMail.pm
new revision: 1.92; previous revision: 1.91
done
Checking in Bugzilla/Install/Filesystem.pm;
/cvsroot/mozilla/webtools/bugzilla/Bugzilla/Install/Filesystem.pm,v  <--  Filesystem.pm
new revision: 1.15; previous revision: 1.14
done
Checking in template/en/default/email/newchangedmail.txt.tmpl;
/cvsroot/mozilla/webtools/bugzilla/template/en/default/email/newchangedmail.txt.tmpl,v  <--  newchangedmail.txt.tmpl
new revision: 1.2; previous revision: 1.1
done