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

Status: RESOLVED FIXED

(Bugzilla :: Creating/Changing Bugs, defect)

Description

[Reed Loden [:reed]]

Attachment: patch - v1

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

Comment 1

[Frédéric Buclin]

Duplicated entries will be automatically removed, so this is not an issue.

Comment 2

[Gervase Markham [:gerv]]

Resolving WONTFIX based on comment #1.

Gerv

Comment 3

[:glob ✱]

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.

Comment 4

[Frédéric Buclin]

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)
{
    ....
}

Comment 5

[Reed Loden [:reed]]

Committing to: bzr+ssh://bzr.mozilla.org/bugzilla/trunk/
modified enter_bug.cgi
Committed revision 8289.