diff options
author | 2005-07-19 21:33:40 +0000 | |
---|---|---|
committer | 2005-07-19 21:33:40 +0000 | |
commit | 20e0d1f73fdaf8954941ce65d0a976d1c620e954 (patch) | |
tree | 032ee3afc2d8c1724dfdfa23870d7f5f157819f7 /collectstats.pl | |
parent | Bug 300532: Update editversions.cgi to use routines from Version.pm and Produ... (diff) | |
download | bugzilla-20e0d1f73fdaf8954941ce65d0a976d1c620e954.tar.gz bugzilla-20e0d1f73fdaf8954941ce65d0a976d1c620e954.tar.bz2 bugzilla-20e0d1f73fdaf8954941ce65d0a976d1c620e954.zip |
Bug 300709: Avoid the use of SELECT * - Patch by Frédéric Buclin <LpSolit@gmail.com> r=glob a=myk
Diffstat (limited to 'collectstats.pl')
-rwxr-xr-x | collectstats.pl | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/collectstats.pl b/collectstats.pl index eca072e61..a12c85586 100755 --- a/collectstats.pl +++ b/collectstats.pl @@ -181,11 +181,11 @@ FIN } sub calculate_dupes { - SendSQL("SELECT * FROM duplicates"); + my $dbh = Bugzilla->dbh; + my $rows = $dbh->selectall_arrayref("SELECT dupe_of, dupe FROM duplicates"); my %dupes; my %count; - my @row; my $key; my $changed = 1; @@ -203,9 +203,8 @@ sub calculate_dupes { # Create a hash with key "a bug number", value "bug which that bug is a # direct dupe of" - straight from the duplicates table. - while (@row = FetchSQLData()) { - my $dupe_of = shift @row; - my $dupe = shift @row; + foreach my $row (@$rows) { + my ($dupe_of, $dupe) = @$row; $dupes{$dupe} = $dupe_of; } |