moving to real milestones...
-> myself & I have a patch
Created attachment 33256 [details] [diff] [review]
validates user, bug_id, and voteon parameters
What about a subroutine ErrorExit(Title, ErrMsg) for these lines:
+ print "Content-type: text/html\n\n";
+ print "<p>$ErrMsg<p>\n";
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 :)
Moving to Bugzilla product