From 811966be3bbc72b591da459170e77511043419ff Mon Sep 17 00:00:00 2001 From: "lpsolit%gmail.com" <> Date: Sun, 8 Oct 2006 00:54:57 +0000 Subject: Bug 355841: Bug::check_can_change_field() incorrectly updates $PrivilegesRequired - Patch by Frédéric Buclin r=mkanat a=justdave 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(-) (limited to 'Bugzilla') 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.3-24-g4f1b