Changeset 12738 for sites/trunk/common/includes/slack/announce/lib.php

Timestamp:

07/13/2023 03:45:24 AM (3 years ago)

Author:

dd32

Message:

Slack: Announce: When determining parent channels, accept that some parents may have parents.

Followup to [12722]. Reverts [12737].

File:

• sites/trunk/common/includes/slack/announce/lib.php (modified) (2 diffs)

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

@@ -140 +140 @@
             $root = 'core';
             break;
-        case 'mentorship':  // Such as #mentorship-cohort-july-2023
+        case 'mentorship': // Such as #mentorship-cohort-july-2023
             $root = 'contributor-mentorship';
             break;
@@ -173 +173 @@
     }
 
-    return $parent_channels ?: false;
+    // Recurse, in case the parent channel has a parent channel.
+    // ie. #mentorship-cohort-july-2023 => #contributor-mentorship => #community-team
+    foreach ( $parent_channels as $parent_channel ) {
+        $parent_channels = array_merge( $parent_channels, get_parent_channels( $parent_channel ) ?: [] );
+    }
+
+    return array_unique( $parent_channels ) ?: false;
 }