diff options
author | Byron Jones <glob@mozilla.com> | 2014-06-11 17:21:53 +0200 |
---|---|---|
committer | Byron Jones <glob@mozilla.com> | 2014-06-11 17:21:53 +0200 |
commit | 39953ccace47ab8669e6e0d6d4bb397fec7d19ec (patch) | |
tree | cba6dfda0e698f4b996c3037a7df7a4a9f4e9f1a /Bugzilla/Attachment.pm | |
parent | 8690811cafbac9f588faac26044f6d428354f811 (diff) | |
download | bugzilla-39953ccace47ab8669e6e0d6d4bb397fec7d19ec.tar.gz bugzilla-39953ccace47ab8669e6e0d6d4bb397fec7d19ec.tar.xz |
Bug 978146: activity entry when setting flags isn't split across multiple rows
Diffstat (limited to 'Bugzilla/Attachment.pm')
-rw-r--r-- | Bugzilla/Attachment.pm | 10 |
1 files changed, 3 insertions, 7 deletions
diff --git a/Bugzilla/Attachment.pm b/Bugzilla/Attachment.pm index 1c8f872e7..c4c1b28aa 100644 --- a/Bugzilla/Attachment.pm +++ b/Bugzilla/Attachment.pm @@ -903,16 +903,12 @@ sub update { } # Record changes in the activity table. - my $sth = $dbh->prepare('INSERT INTO bugs_activity (bug_id, attach_id, who, bug_when, - fieldid, removed, added) - VALUES (?, ?, ?, ?, ?, ?, ?)'); - + require Bugzilla::Bug; foreach my $field (keys %$changes) { my $change = $changes->{$field}; $field = "attachments.$field" unless $field eq "flagtypes.name"; - my $fieldid = get_field_id($field); - $sth->execute($self->bug_id, $self->id, $user->id, $timestamp, - $fieldid, $change->[0], $change->[1]); + Bugzilla::Bug::LogActivityEntry($self->bug_id, $field, $change->[0], + $change->[1], $user->id, $timestamp, undef, $self->id); } if (scalar(keys %$changes)) { |