From: mkanat%kerio.com <>
Date: Thu, 17 Mar 2005 14:56:15 +0000 (+0000)
Subject: Bug 286501: Summarize time fails with "Can't bind reference" error
X-Git-Tag: bugzilla-2.19.3~130
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=9b837ad975360a2f26783581b4e2ff18a4ef3945;p=thirdparty%2Fbugzilla.git

Bug 286501: Summarize time fails with "Can't bind reference" error
Patch By Tomas Kopal <Tomas.Kopal@altap.cz> r=mkanat, a=justdave
---

diff --git a/Bugzilla/Bug.pm b/Bugzilla/Bug.pm
index 31b8f49441..86a1215522 100755
--- a/Bugzilla/Bug.pm
+++ b/Bugzilla/Bug.pm
@@ -643,7 +643,8 @@ sub AppendComment ($$$;$$$) {
     $dbh->do("UPDATE bugs SET delta_ts = ? WHERE bug_id = ?",
              undef, $timestamp, $bugid);
 }
-
+
+# This method is private and is not to be used outside of the Bug class.
 sub EmitDependList {
     my ($myfield, $targetfield, $bug_id) = (@_);
     my $dbh = Bugzilla->dbh;
diff --git a/summarize_time.cgi b/summarize_time.cgi
index ab8a6bf50e..6ea31fceb0 100755
--- a/summarize_time.cgi
+++ b/summarize_time.cgi
@@ -203,10 +203,10 @@ sub get_blocker_ids_unique {
 }
 
 sub get_blocker_ids_deep {
-    my ($bug_id, $ret) = @_;
-    my @deps = Bugzilla::Bug::EmitDependList("blocked", "dependson", $bug_id);
-    push @{$ret}, @deps;
-    foreach $bug_id (@deps) {
+    my ($bug_id, $ret) = @_;
+    my $deps = Bugzilla::Bug::EmitDependList("blocked", "dependson", $bug_id);
+    push @{$ret}, @$deps;
+    foreach $bug_id (@$deps) {
         get_blocker_ids_deep($bug_id, $ret);
     }
 }
@@ -238,7 +238,7 @@ sub query_work_by_buglist {
                       bugs.bug_id = longdescs.bug_id 
                       $date_bits } .
             $dbh->sql_group_by('longdescs.bug_id, profiles.login_name',
-                               'bugs.short_desc, bugs.bug_status') . qq{
+                'bugs.short_desc, bugs.bug_status, longdescs.bug_when') . qq{
                ORDER BY longdescs.bug_when};
     my $sth = $dbh->prepare($q);
     $sth->execute(@{$date_values});