Commit 206eb93d authored by endico%mozilla.org's avatar endico%mozilla.org

give more verbose error messages

parent 7661b31a
...@@ -135,6 +135,7 @@ if (defined $tree->[1][0]->{'maintainer'}) { ...@@ -135,6 +135,7 @@ if (defined $tree->[1][0]->{'maintainer'}) {
my $subject = "Bug import error: no maintainer"; my $subject = "Bug import error: no maintainer";
my $message = "Cannot import these bugs because no maintainer for "; my $message = "Cannot import these bugs because no maintainer for ";
$message .= "the exporting db is given.\n"; $message .= "the exporting db is given.\n";
$message .= "\n\nPlease re-open the original bug.\n";
$message .= "\n\n$xml"; $message .= "\n\n$xml";
my @to = (Param("maintainer")); my @to = (Param("maintainer"));
MailMessage ($subject, $message, @to); MailMessage ($subject, $message, @to);
...@@ -147,6 +148,7 @@ if (defined $tree->[1][0]->{'exporter'}) { ...@@ -147,6 +148,7 @@ if (defined $tree->[1][0]->{'exporter'}) {
} else { } else {
my $subject = "Bug import error: no exporter"; my $subject = "Bug import error: no exporter";
my $message = "Cannot import these bugs because no exporter is given.\n"; my $message = "Cannot import these bugs because no exporter is given.\n";
$message .= "\n\nPlease re-open the original bug.\n";
$message .= "\n\n$xml"; $message .= "\n\n$xml";
my @to = (Param("maintainer"), $maintainer); my @to = (Param("maintainer"), $maintainer);
MailMessage ($subject, $message, @to); MailMessage ($subject, $message, @to);
...@@ -158,6 +160,7 @@ if ( ! $exporterid ) { ...@@ -158,6 +160,7 @@ if ( ! $exporterid ) {
my $subject = "Bug import error: invalid exporter"; my $subject = "Bug import error: invalid exporter";
my $message = "The user <$tree->[1][0]->{'exporter'}> who tried to move\n"; my $message = "The user <$tree->[1][0]->{'exporter'}> who tried to move\n";
$message .= "bugs here does not have an account in this database.\n"; $message .= "bugs here does not have an account in this database.\n";
$message .= "\n\nPlease re-open the original bug.\n";
$message .= "\n\n$xml"; $message .= "\n\n$xml";
my @to = (Param("maintainer"), $maintainer); my @to = (Param("maintainer"), $maintainer);
MailMessage ($subject, $message, @to); MailMessage ($subject, $message, @to);
...@@ -170,6 +173,7 @@ if (defined $tree->[1][0]->{'urlbase'}) { ...@@ -170,6 +173,7 @@ if (defined $tree->[1][0]->{'urlbase'}) {
} else { } else {
my $subject = "Bug import error: invalid exporting database"; my $subject = "Bug import error: invalid exporting database";
my $message = "Cannot import these bugs because the name of the exporting db was not given.\n"; my $message = "Cannot import these bugs because the name of the exporting db was not given.\n";
$message .= "\n\nPlease re-open the original bug.\n";
$message .= "\n\n$xml"; $message .= "\n\n$xml";
my @to = (Param("maintainer"), $maintainer, $exporter); my @to = (Param("maintainer"), $maintainer, $exporter);
MailMessage ($subject, $message, @to); MailMessage ($subject, $message, @to);
...@@ -544,6 +548,7 @@ for (my $k=1 ; $k <= $bugqty ; $k++) { ...@@ -544,6 +548,7 @@ for (my $k=1 ; $k <= $bugqty ; $k++) {
$log .= "Bug $bug_fields{'bug_id'}\@$urlbase imported as bug $id.\n"; $log .= "Bug $bug_fields{'bug_id'}\@$urlbase imported as bug $id.\n";
} }
my $subject = "$bugqty bug(s) imported from $urlbase"; my $subject = "$bugqty bug(s) successfully moved from $urlbase to "
. Param("urlbase") ;
my @to = ($exporter); my @to = ($exporter);
MailMessage ($subject, $log, @to); MailMessage ($subject, $log, @to);
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