diff options
| author | terry%mozilla.org <> | 2000-02-17 22:49:33 +0000 |
|---|---|---|
| committer | terry%mozilla.org <> | 2000-02-17 22:49:33 +0000 |
| commit | c4246f55693582e74fd58ad5cbc2e67d8e41f652 (patch) | |
| tree | d0efcfa04de0d152630c3ba659b99d8a6fd5b729 /post_bug.cgi | |
| parent | e9a32920f47ce268e3835b12abccc9fb2e1dd8c6 (diff) | |
| download | bugs-c4246f55693582e74fd58ad5cbc2e67d8e41f652.tar bugs-c4246f55693582e74fd58ad5cbc2e67d8e41f652.tar.gz bugs-c4246f55693582e74fd58ad5cbc2e67d8e41f652.tar.bz2 bugs-c4246f55693582e74fd58ad5cbc2e67d8e41f652.tar.xz bugs-c4246f55693582e74fd58ad5cbc2e67d8e41f652.zip | |
Was crapping out if strictvaluechecks was turned on.
Diffstat (limited to 'post_bug.cgi')
| -rwxr-xr-x | post_bug.cgi | 34 |
1 files 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}) { |