diff options
author | Dylan William Hardison <dylan@hardison.net> | 2018-03-29 23:56:46 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2018-03-29 23:56:46 +0200 |
commit | 427de4c955b23390972d6981db22e07c4c3b2ddf (patch) | |
tree | adae495a9e85c4a25f94727c82289a467f4b069e /Bugzilla/Error.pm | |
parent | 1463e5c9c868be02643fded1f0beea0e3a2d6bdb (diff) | |
download | bugzilla-427de4c955b23390972d6981db22e07c4c3b2ddf.tar.gz bugzilla-427de4c955b23390972d6981db22e07c4c3b2ddf.tar.xz |
Bug 1441244 - prevent compounding error messages in tests
Diffstat (limited to 'Bugzilla/Error.pm')
-rw-r--r-- | Bugzilla/Error.pm | 10 |
1 files changed, 4 insertions, 6 deletions
diff --git a/Bugzilla/Error.pm b/Bugzilla/Error.pm index e7a99dba0..d67571848 100644 --- a/Bugzilla/Error.pm +++ b/Bugzilla/Error.pm @@ -38,15 +38,14 @@ sub _in_eval { sub _throw_error { my ($name, $error, $vars) = @_; - my $dbh = Bugzilla->dbh; $vars ||= {}; - $vars->{error} = $error; # Make sure any transaction is rolled back (if supported). # If we are within an eval(), do not roll back transactions as we are # eval'uating some test on purpose. - $dbh->bz_rollback_transaction() if ($dbh->bz_in_transaction() && !_in_eval()); + my $dbh = eval { Bugzilla->dbh }; + $dbh->bz_rollback_transaction() if ($dbh && $dbh->bz_in_transaction() && !_in_eval()); my $datadir = bz_locations()->{'datadir'}; # If a writable $datadir/errorlog exists, log error details there. @@ -191,10 +190,9 @@ sub ThrowCodeError { sub ThrowTemplateError { my ($template_err) = @_; - my $dbh = Bugzilla->dbh; - + my $dbh = eval { Bugzilla->dbh }; # Make sure the transaction is rolled back (if supported). - $dbh->bz_rollback_transaction() if $dbh->bz_in_transaction(); + $dbh->bz_rollback_transaction() if $dbh && $dbh->bz_in_transaction(); if (blessed($template_err) && $template_err->isa('Template::Exception')) { my $type = $template_err->type; |