diff options
author | David Lawrence <dkl@mozilla.com> | 2016-05-31 23:29:31 +0200 |
---|---|---|
committer | David Lawrence <dkl@mozilla.com> | 2016-05-31 23:29:31 +0200 |
commit | e31c0e9c22197ba938c6f4a853347edf54931700 (patch) | |
tree | ff9cd88a72c9e5cd8d969df2938be745a7bad5c7 /extensions/BugModal/lib | |
parent | e1e8135e1a703adbb17fffcd8da9297286ba6877 (diff) | |
download | bugzilla-e31c0e9c22197ba938c6f4a853347edf54931700.tar.gz bugzilla-e31c0e9c22197ba938c6f4a853347edf54931700.tar.xz |
Bug 1262459 - restyle the "this bug has been marked as a duplicate" comment
Diffstat (limited to 'extensions/BugModal/lib')
-rw-r--r-- | extensions/BugModal/lib/ActivityStream.pm | 15 |
1 files changed, 11 insertions, 4 deletions
diff --git a/extensions/BugModal/lib/ActivityStream.pm b/extensions/BugModal/lib/ActivityStream.pm index b3b8bbaa6..c9e053934 100644 --- a/extensions/BugModal/lib/ActivityStream.pm +++ b/extensions/BugModal/lib/ActivityStream.pm @@ -178,6 +178,12 @@ sub _add_comments_to_stream { $comment->{collapsed} = 1; $comment->{collapsed_reason} = 'empty'; } + # If comment type is resolved as duplicate, do not add '...marked as duplicate...' string to comment body + if ($comment->type == CMT_DUPE_OF) { + $comment->set_type(0); + # Skip if user did not supply comment also + next if $comment->body eq ''; + } _add_comment_to_stream($stream, date_str_to_time($comment->creation_ts), $comment->author->id, $comment); } @@ -345,20 +351,21 @@ sub _add_duplicates_to_stream { SELECT longdescs.who, UNIX_TIMESTAMP(bug_when), " . $dbh->sql_date_format('bug_when') . ", + type, extra_data FROM longdescs INNER JOIN profiles ON profiles.userid = longdescs.who - WHERE bug_id = ? AND type = ? + WHERE bug_id = ? AND (type = ? OR type = ?) ORDER BY bug_when "); - $sth->execute($bug->id, CMT_HAS_DUPE); + $sth->execute($bug->id, CMT_HAS_DUPE, CMT_DUPE_OF); - while (my($who, $time, $when, $dupe_id) = $sth->fetchrow_array) { + while (my($who, $time, $when, $type, $dupe_id) = $sth->fetchrow_array) { _add_activity_to_stream($stream, $time, $who, { who => Bugzilla::User->new({ id => $who, cache => 1 }), when => $when, changes => [{ - fieldname => 'duplicate', + fieldname => ($type == CMT_HAS_DUPE ? 'has_dupe' : 'dupe_of'), added => $dupe_id, buglist => 1, }], |