Changeset 9216

Timestamp:

10/21/2019 06:00:38 AM (5 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 );
+
 }