555207 - When resolving as duplicate, do not add reporter to cc list of master bug if reporter is disabled

Status: RESOLVED WONTFIX

(Bugzilla :: Creating/Changing Bugs, enhancement)

Description

[Kent Rogers [:mrbball]]

User-Agent:       Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.0.13) Gecko/2009073021 Firefox/3.0.13
Build Identifier: 3.4.4

When marking one bug as a duplicate of another, the reporter of the
duplicate is generally added to the cc list of the master (if they are
allowed to see the master and they were not the reporter of the master).
However, adding the reporter of the duplicate to the cc list of the
master seems inappropriate if that reporter has been disabled.

The process of adding the reporter of the duplicate to the cc list of
the master should either:

1. Be skipped in the same way it is skipped if the same user is the
   reporter of both duplicate and master.
OR
2. Prompt the user making the change for confirmation, in the same way
   they are prompted for confirmation if the reporter of the duplicate
   cannot see the master bug.

I prefer the first option, since I don't foresee a situation where a
disabled user should ever be added to the CC list (at least not on my
local installation).

Reproducible: Always

Comment 1

[Kent Rogers [:mrbball]]

Attachment: v1

Comment 2

[Max Kanat-Alexander]

There are situations in which disabled users should be added to the CC list of bugs--mostly if somebody is watching that disabled user. If the user has bugmail disabled, it should be generally harmless to have them added to the CC, and it still allows watchers to receive email.

Comment 3

[Kent Rogers [:mrbball]]

So are you saying you want to leave it functioning completely as-is or
would you willing to have it prompt the user, as in case #2 (reporter of
duplicate can't see master)?  Configurable?

This should be a fairly rare occurrence.

Comment 4

[Max Kanat-Alexander]

  Um, I really appreciate the patch, but I think I'd prefer to leave it as-is for now. I bet it'd be a really simple extension to write, though, if you wanted.

Comment 5

[Kent Rogers [:mrbball]]

Thanks, but it's way not worth the hassle of an extension.