summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorbugreport%peshkin.net <>2005-10-08 13:58:54 +0200
committerbugreport%peshkin.net <>2005-10-08 13:58:54 +0200
commitb26634d776066be7e6e358bfff0bad4a25a27d51 (patch)
tree797b1ae187c0bc4d44276cfd9c170a2a1d83fc4b
parent1ba2db8af58cfbe8c521e83e9ca1f3468500f07f (diff)
downloadbugzilla-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-xprocess_bug.cgi6
-rw-r--r--template/en/default/bug/process/bugmail.html.tmpl12
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 %]