Cloned bugs will include duplicates in the CC list if the reporter is also CC'd to the bug

RESOLVED FIXED in Bugzilla 4.4

Status

()

Bugzilla
Creating/Changing Bugs
--
minor
RESOLVED FIXED
7 years ago
6 years ago

People

(Reporter: reed, Assigned: reed)

Tracking

4.0.2
Bugzilla 4.4
Bug Flags:
approval +

Details

(URL)

Attachments

(1 attachment)

(Assignee)

Description

7 years ago
Created attachment 585976 [details] [diff] [review]
patch - v1

Cloned bugs will include duplicates in the CC list if the reporter is also CC'd to the bug.
Attachment #585976 - Flags: review?(glob)

Comment 1

7 years ago
Duplicated entries will be automatically removed, so this is not an issue.
Resolving WONTFIX based on comment #1.

Gerv
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → WONTFIX
(Assignee)

Updated

6 years ago
Status: RESOLVED → REOPENED
Resolution: WONTFIX → ---
(Assignee)

Updated

6 years ago
Status: REOPENED → ASSIGNED
Comment on attachment 585976 [details] [diff] [review]
patch - v1

r=glob

while it's a cosmetic fix, it looks ugly to have the duplicates in that list.
Attachment #585976 - Flags: review?(glob) → review+
Flags: approval?

Comment 4

6 years ago
Comment on attachment 585976 [details] [diff] [review]
patch - v1

>+    if ($cloned_bug->reporter->id != $user->id &&
>+        none { $_ eq $cloned_bug->reporter->login } @{$cloned_bug->cc}) {

To follow our Bugzilla guidelines, please move && at the beginning of the next line, and { on its own line:

if (foo
    && bar)
{
    ....
}

Updated

6 years ago
Flags: approval? → approval+
Target Milestone: --- → Bugzilla 4.4
(Assignee)

Comment 5

6 years ago
Committing to: bzr+ssh://bzr.mozilla.org/bugzilla/trunk/
modified enter_bug.cgi
Committed revision 8289.
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago6 years ago
Resolution: --- → FIXED

Updated

6 years ago
Blocks: 819330
You need to log in before you can comment on or make changes to this bug.