diff options
| -rwxr-xr-x | post_bug.cgi | 14 |
1 files changed, 11 insertions, 3 deletions
diff --git a/post_bug.cgi b/post_bug.cgi index fa3fd075f..7152e37c6 100755 --- a/post_bug.cgi +++ b/post_bug.cgi @@ -39,7 +39,6 @@ sub sillyness { $zz = %::COOKIE; $zz = %::components; $zz = %::versions; - $zz = @::legal_bug_status; $zz = @::legal_opsys; $zz = @::legal_platform; $zz = @::legal_priority; @@ -122,7 +121,12 @@ if (Param("useqacontact")) { } if (exists $::FORM{'bug_status'}) { - if (!UserInGroup("canedit") && !UserInGroup("canconfirm")) { + # Ignore the given status, so that we can set it to UNCONFIRMED + # or NEW, depending on votestoconfirm if either the given state was + # unconfirmed (so that a user can't override the below check), or if + # the user doesn't have permission to change the default status anyway + if ($::FORM{'bug_status'} == $::unconfirmedstate + || (!UserInGroup("canedit") && !UserInGroup("canconfirm"))) { delete $::FORM{'bug_status'}; } } @@ -142,6 +146,10 @@ if (!exists $::FORM{'target_milestone'}) { $::FORM{'target_milestone'} = FetchOneColumn(); } +if (!Param('letsubmitterchoosepriority')) { + $::FORM{'priority'} = Param{'defaultpriority'}; +} + GetVersionTable(); CheckFormField(\%::FORM, 'product', \@::legal_product); CheckFormField(\%::FORM, 'version', \@{$::versions{$::FORM{'product'}}}); @@ -152,7 +160,7 @@ 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); +CheckFormField(\%::FORM, 'bug_status', [$::unconfirmedstate, 'NEW']); CheckFormFieldDefined(\%::FORM, 'bug_file_loc'); CheckFormField(\%::FORM, 'component', \@{$::components{$::FORM{'product'}}}); |