From 811966be3bbc72b591da459170e77511043419ff Mon Sep 17 00:00:00 2001 From: "lpsolit%gmail.com" <> Date: Sun, 8 Oct 2006 00:54:57 +0000 Subject: =?UTF-8?q?Bug=20355841:=20Bug::check=5Fcan=5Fchange=5Ffield()=20i?= =?UTF-8?q?ncorrectly=20updates=20$PrivilegesRequired=20-=20Patch=20by=20F?= =?UTF-8?q?r=C3=A9d=C3=A9ric=20Buclin=20=20r=3Dmkanat?= =?UTF-8?q?=20a=3Djustdave?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- Bugzilla/Bug.pm | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/Bugzilla/Bug.pm b/Bugzilla/Bug.pm index 825ec4149..be4b2a037 100755 --- a/Bugzilla/Bug.pm +++ b/Bugzilla/Bug.pm @@ -1825,7 +1825,7 @@ sub check_can_change_field { && $oldvalue eq 'UNCONFIRMED' && is_open_state($newvalue))) { - $PrivilegesRequired = 3; + $$PrivilegesRequired = 3; return $user->in_group('canconfirm'); } @@ -1854,24 +1854,24 @@ sub check_can_change_field { # in that case we will have already returned 1 above # when checking for the assignee of the bug. if ($field eq 'assigned_to') { - $PrivilegesRequired = 2; + $$PrivilegesRequired = 2; return 0; } # - change the QA contact if ($field eq 'qa_contact') { - $PrivilegesRequired = 2; + $$PrivilegesRequired = 2; return 0; } # - change the target milestone if ($field eq 'target_milestone') { - $PrivilegesRequired = 2; + $$PrivilegesRequired = 2; return 0; } # - change the priority (unless he could have set it originally) if ($field eq 'priority' && !Bugzilla->params->{'letsubmitterchoosepriority'}) { - $PrivilegesRequired = 2; + $$PrivilegesRequired = 2; return 0; } @@ -1882,7 +1882,7 @@ sub check_can_change_field { # If we haven't returned by this point, then the user doesn't # have the necessary permissions to change this field. - $PrivilegesRequired = 1; + $$PrivilegesRequired = 1; return 0; } -- cgit v1.2.1