Commit b26634d7 authored by bugreport%peshkin.net's avatar bugreport%peshkin.net

Bug 28398 Gotta be more careful when emailing state changes of dependent bugs

Patch by André Batosti <batosti@async.com.br> r=lpsolit, a=justdave
parent 1ba2db8a
...@@ -1914,9 +1914,9 @@ foreach my $id (@idlist) { ...@@ -1914,9 +1914,9 @@ foreach my $id (@idlist) {
$vars->{'id'} = $k; $vars->{'id'} = $k;
$vars->{'type'} = "dep"; $vars->{'type'} = "dep";
# Let the user know we checked to see if we should email notice # Let the user (if he is able to see the bug) know we checked to see
# of this change to users with a relationship to the dependent # if we should email notice of this change to users with a relationship
# bug and who did and didn't receive email about it. # to the dependent bug and who did and didn't receive email about it.
$template->process("bug/process/results.html.tmpl", $vars) $template->process("bug/process/results.html.tmpl", $vars)
|| ThrowTemplateError($template->error()); || ThrowTemplateError($template->error());
$vars->{'header_done'} = 1; $vars->{'header_done'} = 1;
......
...@@ -62,11 +62,15 @@ ...@@ -62,11 +62,15 @@
[% BLOCK emails %] [% BLOCK emails %]
<b>[% description FILTER html %]:</b> <b>[% description FILTER html %]:</b>
[% IF names.size > 0 %] [% IF user.can_see_bug(mailing_bugid) %]
[%+ FOREACH name = names %] [% IF names.size > 0 %]
[% name FILTER html %][% ", " UNLESS loop.last() %] [%+ FOREACH name = names %]
[% name FILTER html %][% ", " UNLESS loop.last() %]
[% END %]
[% ELSE %]
no one
[% END %] [% END %]
[% ELSE %] [% ELSE %]
no one (list of e-mails not avaliable)
[% END %] [% END %]
[% END %] [% END %]
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment