diff options
| author | dklawren <dklawren@users.noreply.github.com> | 2018-03-23 16:15:51 +0100 |
|---|---|---|
| committer | Dylan William Hardison <dylan@hardison.net> | 2018-03-23 16:15:51 +0100 |
| commit | 7ed1891fb2b81514d090969aece07a8e0be894f1 (patch) | |
| tree | 1ae821ee03ed6bd1c9b5b41c8069a1729c8947e7 /extensions/PhabBugz | |
| parent | 5e590c931ab384fc294d6a0f1ab3fdf66fcc8029 (diff) | |
| download | bugzilla-7ed1891fb2b81514d090969aece07a8e0be894f1.tar.gz bugzilla-7ed1891fb2b81514d090969aece07a8e0be894f1.tar.xz | |
Bug 1446974 - Revert changes of the subscribers list on secure revisions
Diffstat (limited to 'extensions/PhabBugz')
| -rw-r--r-- | extensions/PhabBugz/lib/Feed.pm | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/extensions/PhabBugz/lib/Feed.pm b/extensions/PhabBugz/lib/Feed.pm index 323681ebc..006ab04f3 100644 --- a/extensions/PhabBugz/lib/Feed.pm +++ b/extensions/PhabBugz/lib/Feed.pm @@ -229,10 +229,12 @@ sub process_revision_change { my $secure_project_phid = get_project_phid('secure-revision'); $revision->add_project($secure_project_phid); - - my $subscribers = get_bug_role_phids($bug); - $revision->set_subscribers($subscribers); } + + # Subscriber list of the private revision should always match + # the bug roles such as assignee, qa contact, and cc members. + my $subscribers = get_bug_role_phids($bug); + $revision->set_subscribers($subscribers); } my ($timestamp) = Bugzilla->dbh->selectrow_array("SELECT NOW()"); |