diff --git a/mozilla/webtools/bugzilla/Bugzilla/Bug.pm b/mozilla/webtools/bugzilla/Bugzilla/Bug.pm index 5e8392d9fd9..f09386a0c1c 100644 --- a/mozilla/webtools/bugzilla/Bugzilla/Bug.pm +++ b/mozilla/webtools/bugzilla/Bugzilla/Bug.pm @@ -1363,18 +1363,15 @@ sub editable_bug_fields { return sort(@fields); } -# This method is private and is not to be used outside of the Bug class. +# XXX - When Bug::update() will be implemented, we should make this routine +# a private method. sub EmitDependList { my ($myfield, $targetfield, $bug_id) = (@_); my $dbh = Bugzilla->dbh; - my $list_ref = - $dbh->selectcol_arrayref( - "SELECT dependencies.$targetfield - FROM dependencies, bugs - WHERE dependencies.$myfield = ? - AND bugs.bug_id = dependencies.$targetfield - ORDER BY dependencies.$targetfield", - undef, ($bug_id)); + my $list_ref = $dbh->selectcol_arrayref( + "SELECT $targetfield FROM dependencies + WHERE $myfield = ? ORDER BY $targetfield", + undef, $bug_id); return $list_ref; } diff --git a/mozilla/webtools/bugzilla/process_bug.cgi b/mozilla/webtools/bugzilla/process_bug.cgi index 965e75a9f1d..c7ba981ff2a 100755 --- a/mozilla/webtools/bugzilla/process_bug.cgi +++ b/mozilla/webtools/bugzilla/process_bug.cgi @@ -1273,11 +1273,8 @@ sub SnapShotBug { sub SnapShotDeps { - my ($i, $target, $me) = (@_); - my $dbh = Bugzilla->dbh; - my $list = $dbh->selectcol_arrayref(qq{SELECT $target FROM dependencies - WHERE $me = ? ORDER BY $target}, - undef, $i); + my ($bug_id, $target, $me) = (@_); + my $list = Bugzilla::Bug::EmitDependList($me, $target, $bug_id); return join(',', @$list); }