From: jocuri%softhome.net <> Date: Tue, 6 Jul 2004 04:19:34 +0000 (+0000) Subject: Patch for bug 249798: improve SQL capitalization in sanitycheck.cgi; r=joel, a=justdave. X-Git-Tag: bugzilla-2.18rc1~28 X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=184130db836eb33c0d6d454d55763d9b6a3695f4;p=thirdparty%2Fbugzilla.git Patch for bug 249798: improve SQL capitalization in sanitycheck.cgi; r=joel, a=justdave. --- diff --git a/sanitycheck.cgi b/sanitycheck.cgi index 51b4990252..7e860dd357 100755 --- a/sanitycheck.cgi +++ b/sanitycheck.cgi @@ -97,18 +97,18 @@ PutHeader("Bugzilla Sanity Check"); if (defined $cgi->param('rebuildvotecache')) { Status("OK, now rebuilding vote cache."); - SendSQL("lock tables bugs write, votes read"); - SendSQL("update bugs set votes = 0, delta_ts=delta_ts"); - SendSQL("select bug_id, sum(vote_count) from votes group by bug_id"); + SendSQL("LOCK TABLES bugs WRITE, votes READ"); + SendSQL("UPDATE bugs SET votes = 0, delta_ts = delta_ts"); + SendSQL("SELECT bug_id, SUM(vote_count) FROM votes GROUP BY bug_id"); my %votes; while (@row = FetchSQLData()) { my ($id, $v) = (@row); $votes{$id} = $v; } foreach my $id (keys %votes) { - SendSQL("update bugs set votes = $votes{$id}, delta_ts=delta_ts where bug_id = $id"); + SendSQL("UPDATE bugs SET votes = $votes{$id}, delta_ts = delta_ts WHERE bug_id = $id"); } - SendSQL("unlock tables"); + SendSQL("UNLOCK TABLES"); Status("Vote cache has been rebuilt."); } @@ -436,7 +436,7 @@ sub AlertBadVoteCache { $offervotecacherebuild = 1; } -SendSQL("SELECT bug_id,votes,keywords FROM bugs " . +SendSQL("SELECT bug_id, votes, keywords FROM bugs " . "WHERE votes != 0 OR keywords != ''"); my %votes; @@ -454,7 +454,7 @@ while (@row = FetchSQLData()) { } Status("Checking cached vote counts"); -SendSQL("select bug_id, sum(vote_count) from votes group by bug_id"); +SendSQL("SELECT bug_id, SUM(vote_count) FROM votes GROUP BY bug_id"); while (@row = FetchSQLData()) { my ($id, $v) = (@row);