Changeset 9216

Timestamp:

10/21/2019 06:00:38 AM (7 years ago)

Author:

dd32

Message:

Slack: Broadcast /here messages in subchannels such as #core-privacy into thei parent channel (#core).

The message into the parent channel shouldn't contain any pinging content or be itself a here broadcast, and is trimmed to 100char if needed.

Fixes #4747.

File:

• sites/trunk/common/includes/slack/announce/lib.php (modified) (1 diff)

sites/trunk/common/includes/slack/announce/lib.php

@@ -110 +110 @@
         // By sending the channel ID, we can post to private groups.
         $send->send( $data['channel_id'] );
+
+        // Broadcast this message as a non-@here to the "parent" channel too.
+        list( $parent_channel, ) = explode( '-', $channel, 2 );
+        if (
+                // Skip for private groups.
+                'privategroup' === $parent_channel ||
+                // If this message was posted in the "parent" channel, nothing to do.
+                $parent_channel === $channel ||
+                // Is it an actual channel? Assume that there'll always be at least one whitelisted user for the parent channel
+                ! get_whitelist_for_channel( $parent_channel )
+        ) {
+                return;
+        }
+
+        $text = $data['text'];
+        // Remove any @here or @channel
+        $text = str_ireplace( [ '@here', '@channel', '@group' ], '', $text );
+        if ( mb_strlen( $text ) > 103 ) {
+                $text = mb_subsr( $text, 0, 100 ) . '...';
+        }
+
+        $send->set_text( 'In #' . $channel . ': ' . $text );
+        $send->send( '#' . $parent_channel );
+
 }