Bug 1070317 - Bugzilla::Flag's attribute modification_date is affected by the…

Bug 1070317 - Bugzilla::Flag's attribute modification_date is affected by the user's timezone and differs from the database copy after a call to $flag->update() r=dkl, a=justdave

Bug 1070317 - Bugzilla::Flag's attribute modification_date is affected by the…
f5a7c7d2 · Dylan William Hardison · 141b8b9e · f5a7c7d2
Commit f5a7c7d2 authored Sep 30, 2014 by Dylan William Hardison
Hide whitespace changes
Inline Side-by-side

Showing with 3 additions and 2 deletions

Flag.pm Bugzilla/Flag.pm +3 -2

No files found.
--- a/Bugzilla/Flag.pm
+++ b/Bugzilla/Flag.pm
@@ -454,14 +454,15 @@ sub create {
 sub update {
    my $self = shift;
    my $dbh = Bugzilla->dbh;
-    my $timestamp = shift || $dbh->selectrow_array('SELECT NOW()');
+    my $timestamp = shift || $dbh->selectrow_array('SELECT LOCALTIMESTAMP(0)');

    my $changes = $self->SUPER::update(@_);

    if (scalar(keys %$changes)) {
        $dbh->do('UPDATE flags SET modification_date = ? WHERE id = ?',
                 undef, ($timestamp, $self->id));
-        $self->{'modification_date'} = format_time($timestamp, '%Y.%m.%d %T');
+        $self->{'modification_date'} =
+          format_time($timestamp, '%Y.%m.%d %T', Bugzilla->local_timezone);
        Bugzilla->memcached->clear({ table => 'flags', id => $self->id });
    }
    return $changes;