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

webtools/bugzilla/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;

webtools/bugzilla/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 %]