diff options
| author | bugreport%peshkin.net <> | 2005-10-08 13:58:54 +0200 |
|---|---|---|
| committer | bugreport%peshkin.net <> | 2005-10-08 13:58:54 +0200 |
| commit | b26634d776066be7e6e358bfff0bad4a25a27d51 (patch) | |
| tree | 797b1ae187c0bc4d44276cfd9c170a2a1d83fc4b | |
| parent | 1ba2db8af58cfbe8c521e83e9ca1f3468500f07f (diff) | |
| download | bugzilla-b26634d776066be7e6e358bfff0bad4a25a27d51.tar.gz bugzilla-b26634d776066be7e6e358bfff0bad4a25a27d51.tar.xz | |
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
| -rwxr-xr-x | process_bug.cgi | 6 | ||||
| -rw-r--r-- | template/en/default/bug/process/bugmail.html.tmpl | 12 |
2 files changed, 11 insertions, 7 deletions
diff --git a/process_bug.cgi b/process_bug.cgi index e98005a9c..af3ebc977 100755 --- a/process_bug.cgi +++ b/process_bug.cgi @@ -1914,9 +1914,9 @@ foreach my $id (@idlist) { $vars->{'id'} = $k; $vars->{'type'} = "dep"; - # Let the user know we checked to see if we should email notice - # of this change to users with a relationship to the dependent - # bug and who did and didn't receive email about it. + # Let the user (if he is able to see the bug) know we checked to see + # if we should email notice of this change to users with a relationship + # to the dependent bug and who did and didn't receive email about it. $template->process("bug/process/results.html.tmpl", $vars) || ThrowTemplateError($template->error()); $vars->{'header_done'} = 1; diff --git a/template/en/default/bug/process/bugmail.html.tmpl b/template/en/default/bug/process/bugmail.html.tmpl index a2a6e7353..a5d20e78d 100644 --- a/template/en/default/bug/process/bugmail.html.tmpl +++ b/template/en/default/bug/process/bugmail.html.tmpl @@ -62,11 +62,15 @@ [% BLOCK emails %] <b>[% description FILTER html %]:</b> - [% IF names.size > 0 %] - [%+ FOREACH name = names %] - [% name FILTER html %][% ", " UNLESS loop.last() %] + [% IF user.can_see_bug(mailing_bugid) %] + [% IF names.size > 0 %] + [%+ FOREACH name = names %] + [% name FILTER html %][% ", " UNLESS loop.last() %] + [% END %] + [% ELSE %] + no one [% END %] [% ELSE %] - no one + (list of e-mails not avaliable) [% END %] [% END %] |