From b0b4c4fce29ca6ea88675165f2e7f6100609e9e7 Mon Sep 17 00:00:00 2001 From: "lpsolit%gmail.com" <> Date: Wed, 5 Apr 2006 04:52:36 +0000 Subject: Bug 331343: The timestamp is quoted twice in Bugzilla::Flag::modify() - Patch by Frédéric Buclin r=mkanat a=justdave MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- Bugzilla/Flag.pm | 10 ++++------ 1 file changed, 4 insertions(+), 6 deletions(-) (limited to 'Bugzilla') diff --git a/Bugzilla/Flag.pm b/Bugzilla/Flag.pm index 7b074887a..a3929a970 100644 --- a/Bugzilla/Flag.pm +++ b/Bugzilla/Flag.pm @@ -437,6 +437,8 @@ sub process { # Use the date/time we were given if possible (allowing calling code # to synchronize the comment's timestamp with those of other records). + # XXX - we shouldn't quote the timestamp here, but this would involve + # many changes in this file. $timestamp = ($timestamp ? &::SqlQuote($timestamp) : "NOW()"); # Take a snapshot of flags before any changes. @@ -607,10 +609,6 @@ sub modify { my ($cgi, $timestamp) = @_; my $setter = Bugzilla->user; - # Use the date/time we were given if possible (allowing calling code - # to synchronize the comment's timestamp with those of other records). - my $sql_timestamp = ($timestamp ? &::SqlQuote($timestamp) : "NOW()"); - # Extract a list of flags from the form data. my @ids = map(/^flag-(\d+)$/ ? $1 : (), $cgi->param()); @@ -683,7 +681,7 @@ sub modify { SET setter_id = " . $setter->id . ", requestee_id = NULL , status = '$status' , - modification_date = $sql_timestamp , + modification_date = $timestamp , is_active = 1 WHERE id = $flag->{'id'}"); @@ -724,7 +722,7 @@ sub modify { SET setter_id = " . $setter->id . ", requestee_id = $requestee_id , status = '$status' , - modification_date = $sql_timestamp , + modification_date = $timestamp , is_active = 1 WHERE id = $flag->{'id'}"); -- cgit v1.2.3-24-g4f1b