Bug 1286987 - When the cisco-spark keyword is added to an existing bug, please have the BMO bot send info to Spark

1 files changed, 8 insertions, 4 deletions

diff --git a/extensions/Push/lib/Connector/Spark.pm b/extensions/Push/lib/Connector/Spark.pm
index a8b5bf056..548b2ab54 100644
--- a/extensions/Push/lib/Connector/Spark.pm
+++ b/extensions/Push/lib/Connector/Spark.pm
@@ -21,7 +21,7 @@ use Bugzilla::Util qw(correct_urlbase);
 
 use JSON qw(decode_json encode_json);
 use LWP::UserAgent;
-use List::MoreUtils qw(none);
+use List::MoreUtils qw(any);
 
 sub options {
     return (
@@ -68,10 +68,14 @@ sub should_send {
         return 1;
     }
     else {
-        # send status and resolution updates
         foreach my $change (@{ $data->{event}->{changes} }) {
-            return 1 if $change->{field} eq 'bug_status'
-                || $change->{field} eq 'resolution';
+            # send status and resolution updates
+            return 1 if $change->{field} eq 'bug_status' || $change->{field} eq 'resolution';
+            # also send if the right keyword has been added to this bug
+            if ($change->{field} eq 'keywords' && $change->{added}) {
+                my @added = split(/, /, $change->{added});
+                return 1 if any { $_ eq 'cisco-spark' } @added;
+            }
         }
     }