Bug 151714 - user with no canconfirm permission should not get option to

mark bugs they reported as NEW r=jouni, myk
parent 96235373
...@@ -275,7 +275,9 @@ sub show_bug { ...@@ -275,7 +275,9 @@ sub show_bug {
|| $::userid == $bug{'qa_contact'} || $::userid == $bug{'qa_contact'}
|| $::userid == $bug{'assigned_to'} || $::userid == $bug{'assigned_to'}
|| UserInGroup("editbugs"); || UserInGroup("editbugs");
$user{'canconfirm'} = ($::userid == 0) || UserInGroup("canconfirm"); $user{'canconfirm'} = ($::userid == 0)
|| UserInGroup("canconfirm")
|| UserInGroup("editbugs");
# Bug states # Bug states
$bug{'isunconfirmed'} = ($bug{'bug_status'} eq $::unconfirmedstate); $bug{'isunconfirmed'} = ($bug{'bug_status'} eq $::unconfirmedstate);
......
...@@ -371,7 +371,7 @@ ...@@ -371,7 +371,7 @@
[% knum = 1 %] [% knum = 1 %]
[% IF bug.bug_status == "UNCONFIRMED" && [% IF bug.bug_status == "UNCONFIRMED" &&
(user.canedit || user.canconfirm) %] user.canconfirm %]
<input type="radio" name="knob" value="confirm"> <input type="radio" name="knob" value="confirm">
Confirm bug (change status to <b>NEW</b>) Confirm bug (change status to <b>NEW</b>)
<br> <br>
...@@ -380,7 +380,7 @@ ...@@ -380,7 +380,7 @@
[% IF user.canedit %] [% IF user.canedit %]
[% IF bug.isopened %] [% IF bug.isopened %]
[% IF bug.bug_status != "ASSIGNED" %] [% IF bug.bug_status != "ASSIGNED" && user.canconfirm %]
<input type="radio" name="knob" value="accept"> <input type="radio" name="knob" value="accept">
Accept bug ( Accept bug (
[% "confirm bug, " IF bug.isunconfirmed %]change [% "confirm bug, " IF bug.isunconfirmed %]change
...@@ -424,7 +424,7 @@ ...@@ -424,7 +424,7 @@
}" }"
value="[% bug.assigned_to_email FILTER html %]"> value="[% bug.assigned_to_email FILTER html %]">
<br> <br>
[% IF bug.isunconfirmed %] [% IF bug.isunconfirmed && user.canconfirm %]
&nbsp;&nbsp;&nbsp;&nbsp;<input type="checkbox" name="andconfirm"> &nbsp;&nbsp;&nbsp;&nbsp;<input type="checkbox" name="andconfirm">
and confirm bug (change status to <b>NEW</b>) and confirm bug (change status to <b>NEW</b>)
<br> <br>
...@@ -436,7 +436,7 @@ ...@@ -436,7 +436,7 @@
[% "and QA contact" IF useqacontact %] [% "and QA contact" IF useqacontact %]
of selected component of selected component
<br> <br>
[% IF bug.isunconfirmed %] [% IF bug.isunconfirmed && user.canconfirm %]
&nbsp;&nbsp;&nbsp;&nbsp;<input type="checkbox" name="compconfirm"> &nbsp;&nbsp;&nbsp;&nbsp;<input type="checkbox" name="compconfirm">
and confirm bug (change status to <b>NEW</b>) and confirm bug (change status to <b>NEW</b>)
<br> <br>
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment