Commit e0318906 authored by lpsolit%gmail.com's avatar lpsolit%gmail.com

Bug 370926: Incorrect mail sent when unsetting requestee on review request -…

Bug 370926: Incorrect mail sent when unsetting requestee on review request - Patch by Fré©ric Buclin <LpSolit@gmail.com> r=myk a=LpSolit
parent effbef93
...@@ -748,6 +748,7 @@ sub modify { ...@@ -748,6 +748,7 @@ sub modify {
# Now update the flag object with its new values. # Now update the flag object with its new values.
$flag->{'setter'} = $setter; $flag->{'setter'} = $setter;
$flag->{'requestee'} = undef; $flag->{'requestee'} = undef;
$flag->{'requestee_id'} = undef;
$flag->{'status'} = $status; $flag->{'status'} = $status;
# Send an email notifying the relevant parties about the fulfillment, # Send an email notifying the relevant parties about the fulfillment,
...@@ -764,11 +765,13 @@ sub modify { ...@@ -764,11 +765,13 @@ sub modify {
if ($requestee_email) { if ($requestee_email) {
$requestee_id = login_to_id($requestee_email); $requestee_id = login_to_id($requestee_email);
$flag->{'requestee'} = new Bugzilla::User($requestee_id); $flag->{'requestee'} = new Bugzilla::User($requestee_id);
$flag->{'requestee_id'} = $requestee_id;
} }
else { else {
# If the status didn't change but we only removed the # If the status didn't change but we only removed the
# requestee, we have to clear the requestee field. # requestee, we have to clear the requestee field.
$flag->{'requestee'} = undef; $flag->{'requestee'} = undef;
$flag->{'requestee_id'} = undef;
} }
# Update the database with the changes. # Update the database with the changes.
......
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