From cc69d134483d1e10423475735b1084535dd075b7 Mon Sep 17 00:00:00 2001
From: "mkanat%bugzilla.org" <>
Date: Sat, 12 Aug 2006 06:45:07 +0000
Subject: Bug 348057: Move the checks for bug visibility out of
 Bugzilla::Bug->new Patch By Max Kanat-Alexander <mkanat@bugzilla.org>
 r=LpSolit, a=myk

---
 process_bug.cgi | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

(limited to 'process_bug.cgi')

diff --git a/process_bug.cgi b/process_bug.cgi
index 5669c017f..b26352d22 100755
--- a/process_bug.cgi
+++ b/process_bug.cgi
@@ -141,7 +141,7 @@ scalar(@idlist) || ThrowUserError("no_bugs_chosen");
 # Build a bug object using $cgi->param('id') as ID.
 # If there are more than one bug changed at once, the bug object will be
 # empty, which doesn't matter.
-my $bug = new Bugzilla::Bug(scalar $cgi->param('id'), $whoid);
+my $bug = new Bugzilla::Bug(scalar $cgi->param('id'));
 
 # Make sure form param 'dontchange' is defined so it can be compared to easily.
 $cgi->param('dontchange','') unless defined $cgi->param('dontchange');
@@ -190,7 +190,7 @@ foreach my $field ("dependson", "blocked") {
             # throw an error if any of the changed bugs are not visible.
             ValidateBugID($id);
             if (Bugzilla->params->{"strict_isolation"}) {
-                my $deltabug = new Bugzilla::Bug($id, $user->id);
+                my $deltabug = new Bugzilla::Bug($id);
                 if (!$user->can_edit_product($deltabug->{'product_id'})) {
                     $vars->{'field'} = $field;
                     ThrowUserError("illegal_change_deps", $vars);
@@ -527,7 +527,7 @@ if ($action eq Bugzilla->params->{'move-button-text'}) {
     my @bugs;
     # First update all moved bugs.
     foreach my $id (@idlist) {
-        my $bug = new Bugzilla::Bug($id, $whoid);
+        my $bug = new Bugzilla::Bug($id);
         push(@bugs, $bug);
 
         $sth->execute($timestamp, $id);
@@ -1346,7 +1346,7 @@ if ($prod_changed && Bugzilla->params->{"strict_isolation"}) {
 foreach my $id (@idlist) {
     my $query = $basequery;
     my @bug_values = @values;
-    my $old_bug_obj = new Bugzilla::Bug($id, $whoid);
+    my $old_bug_obj = new Bugzilla::Bug($id);
 
     if ($cgi->param('knob') eq 'reassignbycomponent') {
         # We have to check whether the bug is moved to another product
@@ -1897,7 +1897,7 @@ foreach my $id (@idlist) {
     # and then generate any necessary bug activity entries by seeing 
     # what has changed since before we wrote out the new values.
     #
-    my $new_bug_obj = new Bugzilla::Bug($id, $whoid);
+    my $new_bug_obj = new Bugzilla::Bug($id);
     my @newvalues = SnapShotBug($id);
     my %newhash;
     $i = 0;
@@ -2096,7 +2096,7 @@ if ($action eq 'next_bug') {
     }
     if ($next_bug) {
         if (detaint_natural($next_bug) && Bugzilla->user->can_see_bug($next_bug)) {
-            my $bug = new Bugzilla::Bug($next_bug, $whoid);
+            my $bug = new Bugzilla::Bug($next_bug);
             ThrowCodeError("bug_error", { bug => $bug }) if $bug->error;
 
             $vars->{'bugs'} = [$bug];
@@ -2110,7 +2110,7 @@ if ($action eq 'next_bug') {
     }
 } elsif ($action eq 'same_bug') {
     if (Bugzilla->user->can_see_bug($cgi->param('id'))) {
-        my $bug = new Bugzilla::Bug($cgi->param('id'), $whoid);
+        my $bug = new Bugzilla::Bug($cgi->param('id'));
         ThrowCodeError("bug_error", { bug => $bug }) if $bug->error;
 
         $vars->{'bugs'} = [$bug];
-- 
cgit v1.2.3-24-g4f1b