Bug 1396957 - Synchronize CC list on security bugs over to revisions as subscribers (#230)

* Bug 1396957 - Synchronize CC list on security bugs over to revisions as subscribers

* - Updated to only make subscriber changes if no groups have changed.
- This cuts down on the amount of custom policy changes in the revision
activity.

* Bug 1396957 - Synchronize CC list on security bugs over to revisions as subscribers

* - Updated to only make subscriber changes if no groups have changed.
- This cuts down on the amount of custom policy changes in the revision
activity.

* Bug 1396042 - Implement logging for the BMO push connector

* Bug 1397747 - Move _get_all_group_names to Bugzilla::Config::Common::_get_all_group_names, and remove copy-pasted code

1 files changed, 18 insertions, 1 deletions

diff --git a/extensions/PhabBugz/lib/Util.pm b/extensions/PhabBugz/lib/Util.pm
index becd5bb29..9e2cac149 100644
--- a/extensions/PhabBugz/lib/Util.pm
+++ b/extensions/PhabBugz/lib/Util.pm
@@ -38,6 +38,7 @@ our @EXPORT = qw(
     make_revision_public
     request
     set_project_members
+    set_revision_subscribers
 );
 
 sub get_revisions_by_ids {
@@ -202,7 +203,7 @@ sub edit_revision_policy {
 
     if (@$subscribers) {
         push(@{ $data->{transactions} }, {
-            type  => 'subscribers.add',
+            type  => 'subscribers.set',
             value => $subscribers
         });
     }
@@ -210,6 +211,22 @@ sub edit_revision_policy {
     return request('differential.revision.edit', $data);
 }
 
+sub set_revision_subscribers {
+    my ($revision_phid, $subscribers) = @_;
+
+    my $data = {
+        transactions => [
+            {
+                type  => 'subscribers.set',
+                value => $subscribers
+            }
+        ],
+        objectIdentifier => $revision_phid
+    };
+
+    return request('differential.revision.edit', $data);
+}
+
 sub add_comment_to_revision {
     my ($revision_phid, $comment) = @_;