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

---
 showdependencytree.cgi | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

(limited to 'showdependencytree.cgi')

diff --git a/showdependencytree.cgi b/showdependencytree.cgi
index 1886126b5..5efa0bc9c 100755
--- a/showdependencytree.cgi
+++ b/showdependencytree.cgi
@@ -51,7 +51,7 @@ my $dbh = Bugzilla->switch_to_shadow_db();
 # bug that the user is authorized to access.
 my $id = $cgi->param('id');
 ValidateBugID($id);
-my $current_bug = new Bugzilla::Bug($id, $user->id);
+my $current_bug = new Bugzilla::Bug($id);
 
 my $hide_resolved = $cgi->param('hide_resolved') ? 1 : 0;
 
@@ -120,7 +120,7 @@ sub GenerateTree {
         # its sub-tree if we haven't already done so (which happens
         # when bugs appear in dependency trees multiple times).
         if (!$bugs->{$dep_id}) {
-            $bugs->{$dep_id} = new Bugzilla::Bug($dep_id, $user->id);
+            $bugs->{$dep_id} = new Bugzilla::Bug($dep_id);
             GenerateTree($dep_id, $relationship, $depth+1, $bugs, $ids);
         }
 
@@ -129,6 +129,7 @@ sub GenerateTree {
         # wants the tree to go, and if the dependency isn't resolved 
         # (if we're ignoring resolved dependencies).
         if (!$bugs->{$dep_id}->{'error'}
+            && Bugzilla->user->can_see_bug($dep_id)
             && (!$maxdepth || $depth <= $maxdepth) 
             && ($bugs->{$dep_id}->{'isopened'} || !$hide_resolved))
         {
-- 
cgit v1.2.3-24-g4f1b