From 3321a21580047a12832f47e16220c2c3ee0c7d1b Mon Sep 17 00:00:00 2001 From: "gerv%gerv.net" <> Date: Fri, 26 Apr 2002 13:13:40 +0000 Subject: Bug 140124 - PuntTryAgain must die. Patch by gerv, 2xr=myk. --- CGI.pl | 23 +++++------------------ 1 file changed, 5 insertions(+), 18 deletions(-) (limited to 'CGI.pl') diff --git a/CGI.pl b/CGI.pl index f9892f7c6..c0e38c50a 100644 --- a/CGI.pl +++ b/CGI.pl @@ -228,14 +228,11 @@ sub CheckFormField (\%$;\@) { SendSQL("SELECT description FROM fielddefs WHERE name=" . SqlQuote($fieldname)); my $result = FetchOneColumn(); if ($result) { - PuntTryAgain("A legal $result was not set."); + ThrowCodeError("A legal $result was not set.", undef, "abort"); } else { - PuntTryAgain("A legal $fieldname was not set."); - print Param("browserbugmessage"); + ThrowCodeError("A legal $fieldname was not set.", undef, "abort"); } - PutFooter(); - exit 0; } } @@ -965,9 +962,11 @@ sub DisplayError { # For "this shouldn't happen"-type places in the code. # $vars->{'variables'} is a reference to a hash of useful debugging info. sub ThrowCodeError { - ($vars->{'error'}, $vars->{'variables'}) = (@_); + ($vars->{'error'}, $vars->{'variables'}, my $unlock_tables) = (@_); $vars->{'title'} = "Code Error"; + SendSQL("UNLOCK TABLES") if $unlock_tables; + # We may optionally log something to file here. print "Content-type: text/html\n\n" if !$vars->{'header_done'}; @@ -1016,18 +1015,6 @@ END exit; } -# PuntTryAgain is deprecated. Use UserError with the unlock_tables parameter. -sub PuntTryAgain ($) { - ($vars->{'error'}) = (@_); - - SendSQL("UNLOCK TABLES"); - - $vars->{'header_done'} = "true"; - $template->process("global/user-error.html.tmpl", $vars) - || ThrowTemplateError($template->error()); - exit; -} - sub CheckIfVotedConfirmed { my ($id, $who) = (@_); SendSQL("SELECT bugs.votes, bugs.bug_status, products.votestoconfirm, " . -- cgit v1.2.3-24-g4f1b