diff options
| author |   bbaetz%student.usyd.edu.au <> | 2002-02-05 08:20:07 +0000 |
|---|---|---|
| committer | bbaetz%student.usyd.edu.au <> | 2002-02-05 08:20:07 +0000 |
| commit | 9612f5b87b1503cdfc56c8170bead7cd47de783e (patch) | |
| tree | d73242de3891ed50719cf2308f9aee8098ec1106 /post_bug.cgi | |
| parent | Bug 122418 pt2 - obsoleting a patch from the create attachment screen gave (diff) | |
| download | bugzilla-9612f5b87b1503cdfc56c8170bead7cd47de783e.tar.gz bugzilla-9612f5b87b1503cdfc56c8170bead7cd47de783e.tar.bz2 bugzilla-9612f5b87b1503cdfc56c8170bead7cd47de783e.zip | |
Bug 119715 - strictvaluechecks should always be enabled
r=justdave, gerv
Diffstat (limited to 'post_bug.cgi')
| -rwxr-xr-x | post_bug.cgi | 32 |
1 files changed, 15 insertions, 17 deletions
diff --git a/post_bug.cgi b/post_bug.cgi index 361cba848..fa3fd075f 100755 --- a/post_bug.cgi +++ b/post_bug.cgi @@ -142,23 +142,21 @@ if (!exists $::FORM{'target_milestone'}) { $::FORM{'target_milestone'} = FetchOneColumn(); } -if ( Param("strictvaluechecks") ) { - GetVersionTable(); - CheckFormField(\%::FORM, 'product', \@::legal_product); - CheckFormField(\%::FORM, 'version', \@{$::versions{$::FORM{'product'}}}); - CheckFormField(\%::FORM, 'target_milestone', - \@{$::target_milestone{$::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'); -} +GetVersionTable(); +CheckFormField(\%::FORM, 'product', \@::legal_product); +CheckFormField(\%::FORM, 'version', \@{$::versions{$::FORM{'product'}}}); +CheckFormField(\%::FORM, 'target_milestone', + \@{$::target_milestone{$::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) { |