moving to real milestones...
-> myself & I have a patch
What about a subroutine ErrorExit(Title, ErrMsg) for these lines: + print "Content-type: text/html\n\n"; + PutHeader($Title); + print "<p>$ErrMsg<p>\n"; + PutFooter(); + exit; This could be useful elsewhere, too...
I'm not sure that it needs to validate the bug number/UID against the database... I think it'd probably be enough that it made sure it was a number. But I suppose taking the validation the next step does allow for better error messages, and it does only validate the one that "matters". So, all in all, I'd say r=jake
*** Bug 39537 has been marked as a duplicate of this bug. ***
Created attachment 33749 [details] [diff] [review] uses DisplayError function to simplify display of errors
Jake, could you re-review my new patch?
Using the Param("errorhtml")... nice touch :) r=jake
Moving to Bugzilla product