Bommels05/Mozilla
Bommels05/Mozilla
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Bug 592480: Only one person is added to CC list when multiple are specified

Browse Source 
r=dkl a=LpSolit


git-svn-id: svn://10.0.0.236/trunk@261160 18797224-902f-48f8-a5cc-f745e15eee43
This commit is contained in:
mkanat%bugzilla.org 2010-09-01 21:19:16 +00:00
parent e2cd27a01c
commit 8f8ab07a31
2 changed files with 10 additions and 14 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
mozilla/webtools/bugzilla/.bzrrev
View File

@ -1 +1 @@
7456
7457

22
mozilla/webtools/bugzilla/process_bug.cgi
View File

@ -287,37 +287,33 @@ foreach my $dep_field (qw(dependson blocked)) {
}
}
# Formulate the CC data into two arrays of users involved in this CC change.
my (@cc_add, @cc_remove);
if (defined $cgi->param('newcc')
or defined $cgi->param('addselfcc')
or defined $cgi->param('removecc')
or defined $cgi->param('masscc'))
{
my (@cc_add, @cc_remove);
# If masscc is defined, then we came from buglist and need to either add or
# remove cc's... otherwise, we came from show_bug and may need to do both.
my ($cc_add, $cc_remove) = "";
if (defined $cgi->param('masscc')) {
if ($cgi->param('ccaction') eq 'add') {
$cc_add = $cgi->param('masscc');
@cc_add = $cgi->param('masscc');
} elsif ($cgi->param('ccaction') eq 'remove') {
$cc_remove = $cgi->param('masscc');
@cc_remove = $cgi->param('masscc');
}
} else {
$cc_add = $cgi->param('newcc');
# We came from bug_form which uses a select box to determine what cc's
@cc_add = $cgi->param('newcc');
push(@cc_add, Bugzilla->user) if $cgi->param('addselfcc');
# We came from show_bug which uses a select box to determine what cc's
# need to be removed...
if ($cgi->param('removecc') && $cgi->param('cc')) {
$cc_remove = join(",", $cgi->param('cc'));
@cc_remove = $cgi->param('cc');
}
}
push(@cc_add, split(/[\s,]+/, $cc_add)) if $cc_add;
push(@cc_add, Bugzilla->user) if $cgi->param('addselfcc');
push(@cc_remove, split(/[\s,]+/, $cc_remove)) if $cc_remove;
$set_all_fields{cc} = { add => \@cc_add, remove => \@cc_remove };
}
$set_all_fields{cc} = { add => \@cc_add, remove => \@cc_remove };
# Fields that can only be set on one bug at a time.
if (defined $cgi->param('id')) {