Commit 2ce90918 authored by Albert Ting's avatar Albert Ting Committed by Frédéric Buclin

Bug 422256: email_in.pl should send an email if user matching fails or returns too many results

r/a=mkanat
parent 61ead83b
......@@ -169,22 +169,29 @@ sub parse_mail {
return \%fields;
}
sub post_bug {
sub check_email_fields {
my ($fields) = @_;
debug_print('Posting a new bug...');
my $user = Bugzilla->user;
my ($retval, $non_conclusive_fields) =
Bugzilla::User::match_field({
'assigned_to' => { 'type' => 'single' },
'qa_contact' => { 'type' => 'single' },
'cc' => { 'type' => 'multi' }
'cc' => { 'type' => 'multi' },
'newcc' => { 'type' => 'multi' }
}, $fields, MATCH_SKIP_CONFIRM);
if ($retval != USER_MATCH_SUCCESS) {
ThrowUserError('user_match_too_many', {fields => $non_conclusive_fields});
}
}
sub post_bug {
my ($fields) = @_;
debug_print('Posting a new bug...');
my $user = Bugzilla->user;
check_email_fields($fields);
my $bug = Bugzilla::Bug->create($fields);
debug_print("Created bug " . $bug->id);
......@@ -225,6 +232,8 @@ sub process_bug {
$fields{'removecc'} = 1;
}
check_email_fields(\%fields);
my $cgi = Bugzilla->cgi;
foreach my $field (keys %fields) {
$cgi->param(-name => $field, -value => $fields{$field});
......
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