Bug 234540: "Take bug" on create attachment screen missed an API change to…

Bug 234540: "Take bug" on create attachment screen missed an API change to BugMail which caused it not to mail the previous bug owner about the change. r=gerv, a=justdave
parent ee5f2884
...@@ -885,7 +885,7 @@ sub insert ...@@ -885,7 +885,7 @@ sub insert
} }
# Assign the bug to the user, if they are allowed to take it # Assign the bug to the user, if they are allowed to take it
my $forcecc = ""; my $owner = "";
if ($::FORM{'takebug'} && UserInGroup("editbugs")) { if ($::FORM{'takebug'} && UserInGroup("editbugs")) {
SendSQL("select NOW()"); SendSQL("select NOW()");
...@@ -902,7 +902,7 @@ sub insert ...@@ -902,7 +902,7 @@ sub insert
my @newvalues = ($::userid, "ASSIGNED", "", DBID_to_name($::userid)); my @newvalues = ($::userid, "ASSIGNED", "", DBID_to_name($::userid));
# Make sure the person we are taking the bug from gets mail. # Make sure the person we are taking the bug from gets mail.
$forcecc = $oldvalues[3]; $owner = $oldvalues[3];
@oldvalues = map(SqlQuote($_), @oldvalues); @oldvalues = map(SqlQuote($_), @oldvalues);
@newvalues = map(SqlQuote($_), @newvalues); @newvalues = map(SqlQuote($_), @newvalues);
...@@ -930,7 +930,8 @@ sub insert ...@@ -930,7 +930,8 @@ sub insert
} }
# Define the variables and functions that will be passed to the UI template. # Define the variables and functions that will be passed to the UI template.
$vars->{'mailrecipients'} = { 'changer' => $::COOKIE{'Bugzilla_login'} }; $vars->{'mailrecipients'} = { 'changer' => $::COOKIE{'Bugzilla_login'},
'owner' => $owner };
$vars->{'bugid'} = $::FORM{'bugid'}; $vars->{'bugid'} = $::FORM{'bugid'};
$vars->{'attachid'} = $attachid; $vars->{'attachid'} = $attachid;
$vars->{'description'} = $description; $vars->{'description'} = $description;
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment