From 97505287f89aa7f4ba4584d241218abfd73a025e Mon Sep 17 00:00:00 2001 From: "justdave%syndicomm.com" <> Date: Tue, 28 Aug 2001 16:19:16 +0000 Subject: [PATCH] Re-fix for bug 87769: the original patch checked in for this prevented any votes from being changed at all. This corrects that so votes can be changed again while still trapping the error the original patch was designed to catch. Patch by Christoper Aillon r= justdave@syndicomm.com --- doeditvotes.cgi | 32 +++++++++++++++++--------------- 1 file changed, 17 insertions(+), 15 deletions(-) diff --git a/doeditvotes.cgi b/doeditvotes.cgi index 40d6390dd9..a80f4ae403 100755 --- a/doeditvotes.cgi +++ b/doeditvotes.cgi @@ -41,21 +41,23 @@ my @buglist = grep {/^[1-9][0-9]*$/} keys(%::FORM); # If no bugs are in the buglist, let's make sure the user gets notified # that their votes will get nuked if they continue. -if ((0 == @buglist) && (! defined $::FORM{'delete_all_votes'})) { - print "Content-type: text/html\n\n"; - PutHeader("Remove your votes?"); - print "

You are about to remove all of your bug votes. Are you sure you wish to remove your vote from every bug you've voted on?

"; - print qq{
\n}; - print qq{

Yes

\n}; - print qq{

No

\n}; - print qq{

Review your votes

\n}; - print qq{

\n}; - PutFooter(); - exit(); -} -elsif ($::FORM{'delete_all_votes'} == 0) { - print "Location: showvotes.cgi\n\n"; - exit(); +if ((0 == @buglist) { + if (! defined $::FORM{'delete_all_votes'})) { + print "Content-type: text/html\n\n"; + PutHeader("Remove your votes?"); + print "

You are about to remove all of your bug votes. Are you sure you wish to remove your vote from every bug you've voted on?

"; + print qq{
\n}; + print qq{

Yes

\n}; + print qq{

No

\n}; + print qq{

Review your votes

\n}; + print qq{

\n}; + PutFooter(); + exit(); + } + elsif ($::FORM{'delete_all_votes'} == 0)) { + print "Location: showvotes.cgi\n\n"; + exit(); + } } # Call ValidateBugID on each bug ID to make sure it is a positive -- 2.47.2