Commit 0e1b77bf authored by terry%mozilla.org's avatar terry%mozilla.org

Don't do dmose's paranoid checking of option values in the query page.

parent 0a856720
...@@ -270,6 +270,8 @@ sub navigation_header { ...@@ -270,6 +270,8 @@ sub navigation_header {
} }
$::CheckOptionValues = 1;
sub make_options { sub make_options {
my ($src,$default,$isregexp) = (@_); my ($src,$default,$isregexp) = (@_);
my $last = ""; my $last = "";
...@@ -294,7 +296,7 @@ sub make_options { ...@@ -294,7 +296,7 @@ sub make_options {
} }
} }
if (!$found && $default ne "") { if (!$found && $default ne "") {
if ( Param("strictvaluechecks") && if ( Param("strictvaluechecks") && $::CheckOptionValues &&
($default ne $::dontchange) && ($default ne "-All-") && ($default ne $::dontchange) && ($default ne "-All-") &&
($default ne "DUPLICATE") ) { ($default ne "DUPLICATE") ) {
print "Possible bug database corruption has been detected. " . print "Possible bug database corruption has been detected. " .
......
...@@ -26,6 +26,11 @@ use strict; ...@@ -26,6 +26,11 @@ use strict;
require "CGI.pl"; require "CGI.pl";
$::CheckOptionValues = 0; # It's OK if we have some bogus things in the
# pop-up lists here, from a remembered query
# that is no longer quite valid. We don't
# want to crap out in the query page.
# Shut up misguided -w warnings about "used only once": # Shut up misguided -w warnings about "used only once":
use vars @::legal_resolution, use vars @::legal_resolution,
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment