]> git.ipfire.org Git - thirdparty/bugzilla.git/commitdiff
Bug 313806: Whine E-mails do not "Order" the Queries Correctly - Patch by Frédéric...
authorlpsolit%gmail.com <>
Tue, 7 Feb 2006 17:31:22 +0000 (17:31 +0000)
committerlpsolit%gmail.com <>
Tue, 7 Feb 2006 17:31:22 +0000 (17:31 +0000)
whine.pl

index 96356f1fab5addd9ea00a3b40de6a6dfb8b715a0..e584646f5b270a9d43a4a47653dd93ba200e30be 100755 (executable)
--- a/whine.pl
+++ b/whine.pl
@@ -51,7 +51,7 @@ my @seen_schedules = ();
 # allow the database to keep their SQL compiled.
 my $sth_run_queries =
     $dbh->prepare("SELECT " .
-                  "id, query_name, title, onemailperbug " .
+                  "query_name, title, onemailperbug " .
                   "FROM whine_queries " .
                   "WHERE eventid=? " .
                   "ORDER BY sortkey");
@@ -429,18 +429,19 @@ sub run_queries {
     my $return_queries = [];
 
     $sth_run_queries->execute($args->{'eventid'});
-    my $queries = {};
+    my @queries = ();
     for (@{$sth_run_queries->fetchall_arrayref}) {
-        $queries->{$_->[0]} = {
-            'name'          => $_->[1],
-            'title'         => $_->[2],
-            'onemailperbug' => $_->[3],
-            'bugs'          => [],
-        };
+        push(@queries,
+            {
+              'name'          => $_->[0],
+              'title'         => $_->[1],
+              'onemailperbug' => $_->[2],
+              'bugs'          => [],
+            }
+        );
     }
 
-    for my $query_id (keys %{$queries}) {
-        my $thisquery = $queries->{$query_id};
+    foreach my $thisquery (@queries) {
         next unless $thisquery->{'name'};   # named query is blank
 
         my $savedquery = get_query($thisquery->{'name'}, $args->{'author'});