From d7958942a691a719be4d3433b5a24817c8646f9a Mon Sep 17 00:00:00 2001 From: "mkanat%kerio.com" <> Date: Sun, 20 Feb 2005 16:02:06 +0000 Subject: Bug 280502: Replace "INTERVAL" with Bugzilla::DB function call Patch By Tomas Kopal r=mkanat, a=justdave --- Bugzilla/Search.pm | 2 +- Bugzilla/Token.pm | 3 ++- contrib/sendunsentbugmail.pl | 4 +++- sanitycheck.cgi | 10 ++++++---- userprefs.cgi | 3 ++- whine.pl | 19 ++++++++++++------- 6 files changed, 26 insertions(+), 15 deletions(-) diff --git a/Bugzilla/Search.pm b/Bugzilla/Search.pm index 82cbb9aa6..a1058fbe5 100644 --- a/Bugzilla/Search.pm +++ b/Bugzilla/Search.pm @@ -906,7 +906,7 @@ sub init { $unitinterval = 'YEAR'; } my $cutoff = "DATE_SUB(NOW(), " . - "INTERVAL $quantity $unitinterval)"; + $dbh->sql_interval("$quantity $unitinterval"); my $assigned_fieldid = &::GetFieldID('assigned_to'); push(@supptables, "LEFT JOIN longdescs comment_$table " . "ON comment_$table.who = bugs.assigned_to " . diff --git a/Bugzilla/Token.pm b/Bugzilla/Token.pm index 4c70d4bad..73f074185 100644 --- a/Bugzilla/Token.pm +++ b/Bugzilla/Token.pm @@ -119,7 +119,8 @@ sub IssuePasswordToken { LEFT JOIN tokens ON tokens.userid = profiles.userid AND tokens.tokentype = 'password' - AND tokens.issuedate > DATE_SUB(NOW(), INTERVAL 10 MINUTE) + AND tokens.issuedate > DATE_SUB(NOW(), " . + $dbh->sql_interval('10 MINUTE') . ") WHERE login_name = $quotedloginname"); my ($userid, $toosoon) = &::FetchSQLData(); diff --git a/contrib/sendunsentbugmail.pl b/contrib/sendunsentbugmail.pl index ffe892e08..b17289592 100644 --- a/contrib/sendunsentbugmail.pl +++ b/contrib/sendunsentbugmail.pl @@ -29,8 +29,10 @@ require "CGI.pl"; use Bugzilla::Constants; use Bugzilla::BugMail; +my $dbh = Bugzilla->dbh; SendSQL("SELECT bug_id FROM bugs WHERE lastdiffed < delta_ts AND - delta_ts < date_sub(now(), INTERVAL 30 minute) ORDER BY bug_id"); + delta_ts < date_sub(now(), " . $dbh->sql_interval('30 minute') . + " ORDER BY bug_id"); my @list; while (MoreSQLData()) { push (@list, FetchOneColumn()); diff --git a/sanitycheck.cgi b/sanitycheck.cgi index 1696a3e7a..f58995d26 100755 --- a/sanitycheck.cgi +++ b/sanitycheck.cgi @@ -144,7 +144,8 @@ if (defined $cgi->param('cleangroupsnow')) { # to get the groups up to date. # If any page starts taking longer than one hour to load, this interval # should be revised. - SendSQL("SELECT MAX(last_changed) FROM groups WHERE last_changed < NOW() - INTERVAL 1 HOUR"); + SendSQL("SELECT MAX(last_changed) FROM groups WHERE last_changed < NOW() - " . + $dbh->sql_interval('1 HOUR')); (my $cutoff) = FetchSQLData(); Status("Cutoff is $cutoff"); SendSQL("SELECT COUNT(*) FROM user_group_map"); @@ -175,7 +176,8 @@ if (defined $cgi->param('rescanallBugMail')) { Status("OK, now attempting to send unsent mail"); SendSQL("SELECT bug_id FROM bugs WHERE lastdiffed < delta_ts AND - delta_ts < date_sub(now(), INTERVAL 30 minute) ORDER BY bug_id"); + delta_ts < date_sub(now(), " . $dbh->sql_interval('30 minute') . + " ORDER BY bug_id"); my @list; while (MoreSQLData()) { push (@list, FetchOneColumn()); @@ -738,8 +740,8 @@ Status("Checking for unsent mail"); SendSQL("SELECT bug_id " . "FROM bugs WHERE lastdiffed < delta_ts AND ". - "delta_ts < date_sub(now(), INTERVAL 30 minute) ". - "ORDER BY bug_id"); + "delta_ts < date_sub(now(), " . $dbh->sql_interval('30 minute') . + " ORDER BY bug_id"); while (@row = FetchSQLData()) { my ($id) = (@row); diff --git a/userprefs.cgi b/userprefs.cgi index a1239ca8f..72f7232ae 100755 --- a/userprefs.cgi +++ b/userprefs.cgi @@ -54,7 +54,8 @@ sub DoAccount { $vars->{'realname'} = FetchSQLData(); if(Param('allowemailchange')) { - SendSQL("SELECT tokentype, issuedate + INTERVAL 3 DAY, eventdata + SendSQL("SELECT tokentype, issuedate + " . $dbh->sql_interval('3 DAY') . + ", eventdata FROM tokens WHERE userid = $userid AND tokentype LIKE 'email%' diff --git a/whine.pl b/whine.pl index 997e4a58a..52fbe4e82 100755 --- a/whine.pl +++ b/whine.pl @@ -168,7 +168,8 @@ while (my ($schedule_id, $day, $time) = $sched_h->fetchrow_array) { elsif ($time >= $now_hour) { # set it to today + number of hours $sth = $dbh->prepare( "UPDATE whine_schedules " . - "SET run_next=DATE_ADD(CURRENT_DATE(), INTERVAL ? HOUR) " . + "SET run_next=DATE_ADD(CURRENT_DATE(), " . + $dbh->sql_interval('? HOUR') . ") " . "WHERE id=?"); $sth->execute($time, $schedule_id); } @@ -177,7 +178,7 @@ while (my ($schedule_id, $day, $time) = $sched_h->fetchrow_array) { my $nextdate = &get_next_date($day); $sth = $dbh->prepare( "UPDATE whine_schedules " . "SET run_next=" . - "DATE_ADD(?, INTERVAL ? HOUR) " . + "DATE_ADD(?, " . $dbh->sql_interval('? HOUR') . ") " . "WHERE id=?"); $sth->execute($nextdate, $time, $schedule_id); } @@ -192,7 +193,8 @@ while (my ($schedule_id, $day, $time) = $sched_h->fetchrow_array) { my $target_time = ($time =~ /^\d+$/) ? $time : 0; $sth = $dbh->prepare( "UPDATE whine_schedules " . - "SET run_next=DATE_ADD(?, INTERVAL ? HOUR) " . + "SET run_next=DATE_ADD(?, " . + $dbh->sql_interval('? HOUR') . ") " . "WHERE id=?"); $sth->execute($target_date, $target_time, $schedule_id); } @@ -590,7 +592,8 @@ sub reset_timer { my $nextdate = &get_next_date($run_day); $sth = $dbh->prepare( "UPDATE whine_schedules " . - "SET run_next=DATE_ADD(?, INTERVAL ? HOUR) " . + "SET run_next=DATE_ADD(?, " . + $dbh->sql_interval('? HOUR') . ") " . "WHERE id=?"); $sth->execute($nextdate, $target_time, $schedule_id); return; @@ -601,8 +604,9 @@ sub reset_timer { if ($minute_offset > 0) { $sth = $dbh->prepare("UPDATE whine_schedules " . "SET run_next = " . - "DATE_SUB(DATE_ADD(NOW(), INTERVAL ? MINUTE), " . - "INTERVAL SECOND(NOW()) SECOND) " . + "DATE_SUB(DATE_ADD(NOW(), " . + $dbh->sql_interval('? MINUTE') . "), " . + $dbh->sql_interval('SECOND(NOW()) SECOND') . ") " . "WHERE id=?"); $sth->execute($minute_offset, $schedule_id); } else { @@ -683,7 +687,8 @@ sub get_next_date { } # Get a date in whatever format the database will accept - $sth = $dbh->prepare("SELECT DATE_ADD(CURRENT_DATE(), INTERVAL ? DAY)"); + $sth = $dbh->prepare("SELECT DATE_ADD(CURRENT_DATE(), " . + $dbh->sql_interval('? DAY') . ")"); $sth->execute($add_days); return $sth->fetch->[0]; } -- cgit v1.2.3-24-g4f1b