summaryrefslogtreecommitdiffstats
path: root/Bugzilla
diff options
context:
space:
mode:
authorDylan William Hardison <dylan@hardison.net>2014-09-16 23:38:36 +0200
committerDylan William Hardison <dylan@hardison.net>2014-09-20 02:43:36 +0200
commitc516e35a1e444387512e9a49afb4b20b76d36375 (patch)
treedc737491507804c12fb827b256a8243d6257ca43 /Bugzilla
parentfb7ed7baca8c4321549f8142ffab4d6c9eefd391 (diff)
downloadbugzilla-c516e35a1e444387512e9a49afb4b20b76d36375.tar.gz
bugzilla-c516e35a1e444387512e9a49afb4b20b76d36375.tar.xz
Bug 1067410 - Modification time wrong for deleted flags in review schema
Diffstat (limited to 'Bugzilla')
-rw-r--r--Bugzilla/Flag.pm11
1 files changed, 10 insertions, 1 deletions
diff --git a/Bugzilla/Flag.pm b/Bugzilla/Flag.pm
index ea83ae5d9..f0cb830b5 100644
--- a/Bugzilla/Flag.pm
+++ b/Bugzilla/Flag.pm
@@ -484,7 +484,7 @@ sub update {
}
# BMO - provide a hook which passes the flag object
- Bugzilla::Hook::process('flag_updated', {flag => $self, changes => $changes});
+ Bugzilla::Hook::process('flag_updated', {flag => $self, changes => $changes, timestamp => $timestamp});
return $changes;
}
@@ -540,6 +540,10 @@ sub update_flags {
# These flags have been deleted.
foreach my $old_flag (values %old_flags) {
$class->notify(undef, $old_flag, $self, $timestamp);
+
+ # BMO - provide a hook which passes the timestamp,
+ # because that isn't passed to remove_from_db().
+ Bugzilla::Hook::process('flag_deleted', {flag => $old_flag, timestamp => $timestamp});
$old_flag->remove_from_db();
}
@@ -637,6 +641,11 @@ sub force_retarget {
# Track deleted attachment flags.
push(@removed, $class->snapshot([$flag])) if $flag->attach_id;
$class->notify(undef, $flag, $bug || $flag->bug);
+
+ # BMO - provide a hook which passes the timestamp,
+ # because that isn't passed to remove_from_db().
+ my ($timestamp) = $dbh->selectrow_array('SELECT LOCALTIMESTAMP(0)');
+ Bugzilla::Hook::process('flag_deleted', {flag => $flag, timestamp => $timestamp});
$flag->remove_from_db();
}
}