From c4246f55693582e74fd58ad5cbc2e67d8e41f652 Mon Sep 17 00:00:00 2001 From: "terry%mozilla.org" <> Date: Thu, 17 Feb 2000 22:49:33 +0000 Subject: Was crapping out if strictvaluechecks was turned on. --- post_bug.cgi | 34 +++++++++++++++++----------------- 1 file changed, 17 insertions(+), 17 deletions(-) diff --git a/post_bug.cgi b/post_bug.cgi index 18b579119..df5e78e77 100755 --- a/post_bug.cgi +++ b/post_bug.cgi @@ -73,23 +73,6 @@ if (!defined $::FORM{'short_desc'} || trim($::FORM{'short_desc'}) eq "") { exit; } -if ( Param("strictvaluechecks") ) { - GetVersionTable(); - CheckFormField(\%::FORM, 'reporter'); - CheckFormField(\%::FORM, 'product', \@::legal_product); - CheckFormField(\%::FORM, 'version', \@{$::versions{$::FORM{'product'}}}); - CheckFormField(\%::FORM, 'rep_platform', \@::legal_platform); - CheckFormField(\%::FORM, 'bug_severity', \@::legal_severity); - CheckFormField(\%::FORM, 'priority', \@::legal_priority); - CheckFormField(\%::FORM, 'op_sys', \@::legal_opsys); - CheckFormFieldDefined(\%::FORM, 'assigned_to'); - CheckFormField(\%::FORM, 'bug_status', \@::legal_bug_status); - CheckFormFieldDefined(\%::FORM, 'bug_file_loc'); - CheckFormField(\%::FORM, 'component', - \@{$::components{$::FORM{'product'}}}); - CheckFormFieldDefined(\%::FORM, 'comment'); -} - my $forceAssignedOK = 0; if ($::FORM{'assigned_to'} eq "") { SendSQL("select initialowner from components where program=" . @@ -136,6 +119,23 @@ if (!exists $::FORM{'bug_status'}) { } +if ( Param("strictvaluechecks") ) { + GetVersionTable(); + CheckFormField(\%::FORM, 'reporter'); + CheckFormField(\%::FORM, 'product', \@::legal_product); + CheckFormField(\%::FORM, 'version', \@{$::versions{$::FORM{'product'}}}); + CheckFormField(\%::FORM, 'rep_platform', \@::legal_platform); + CheckFormField(\%::FORM, 'bug_severity', \@::legal_severity); + CheckFormField(\%::FORM, 'priority', \@::legal_priority); + CheckFormField(\%::FORM, 'op_sys', \@::legal_opsys); + CheckFormFieldDefined(\%::FORM, 'assigned_to'); + CheckFormField(\%::FORM, 'bug_status', \@::legal_bug_status); + CheckFormFieldDefined(\%::FORM, 'bug_file_loc'); + CheckFormField(\%::FORM, 'component', + \@{$::components{$::FORM{'product'}}}); + CheckFormFieldDefined(\%::FORM, 'comment'); +} + my @used_fields; foreach my $f (@bug_fields) { if (exists $::FORM{$f}) { -- cgit v1.2.1