diff options
author | lpsolit%gmail.com <> | 2006-07-17 07:53:08 +0200 |
---|---|---|
committer | lpsolit%gmail.com <> | 2006-07-17 07:53:08 +0200 |
commit | 146d47f6c92fc7f64ca940ca4629bf8997195b58 (patch) | |
tree | 6453e97f0a65641fa8c04f10ff4e06d994b7c3a9 /enter_bug.cgi | |
parent | fe1d0e18e5f58c0a3d1c59941e736928b154aea7 (diff) | |
download | bugzilla-146d47f6c92fc7f64ca940ca4629bf8997195b58.tar.gz bugzilla-146d47f6c92fc7f64ca940ca4629bf8997195b58.tar.xz |
Bug 238819: enter_bug.cgi should offer ASSIGNED as a possible initial state - Patch by Frédéric Buclin <LpSolit@gmail.com> r=vladd a=justdave
Diffstat (limited to 'enter_bug.cgi')
-rwxr-xr-x | enter_bug.cgi | 31 |
1 files changed, 17 insertions, 14 deletions
diff --git a/enter_bug.cgi b/enter_bug.cgi index 1315ca927..512650a7e 100755 --- a/enter_bug.cgi +++ b/enter_bug.cgi @@ -438,24 +438,27 @@ if ( Bugzilla->params->{'usetargetmilestone'} ) { } } +# Construct the list of allowable statuses. +# +# * If the product requires votes to confirm: +# users with privs : NEW + ASSI + UNCO +# users with no privs: UNCO +# +# * If the product doesn't require votes to confirm: +# users with privs : NEW + ASSI +# users with no privs: NEW (as these users cannot reassign +# bugs to them, it doesn't make sense +# to let them mark bugs as ASSIGNED) -# List of status values for drop-down. my @status; - -# Construct the list of allowable values. There are three cases: -# -# case values -# product does not have confirmation NEW -# confirmation, user cannot confirm UNCONFIRMED -# confirmation, user can confirm NEW, UNCONFIRMED. - +if ($user->in_group('editbugs') || $user->in_group('canconfirm')) { + @status = ('NEW', 'ASSIGNED'); +} +elsif (!$product->votes_to_confirm) { + @status = ('NEW'); +} if ($product->votes_to_confirm) { - if (UserInGroup("editbugs") || UserInGroup("canconfirm")) { - push(@status, "NEW"); - } push(@status, 'UNCONFIRMED'); -} else { - push(@status, "NEW"); } $vars->{'bug_status'} = \@status; |