Changeset 1189

Timestamp:

01/25/2015 09:30:37 AM (10 years ago)

Author:

nacin

Message:

Slack: Avoid sending to the channel false which ends up being the default channel for the webhook. (Typically #core.)

File:

• sites/trunk/common/includes/slack/trac/comment-handler.php (modified) (1 diff)

sites/trunk/common/includes/slack/trac/comment-handler.php

@@ -19 +19 @@
 
         $this->generate_payload();
-        $this->send->send( $this->trac->get_firehose_channel() );
+        $firehose = $this->trac->get_firehose_channel();
+
+        // We still want to process this payload even if no firehose channel
+        // is specified, as we may use it to process mentions and such.
+        if ( $firehose ) {
+            $this->send->send( $firehose );
+        }
     }