Changeset 7210 for trunk/src/includes/extend/buddypress/groups.php

Timestamp:

09/21/2021 03:14:26 AM (3 years ago)

Author:

johnjamesjacoby

Message:

BuddyPress Groups: sync forum Title, Description, and Slug to Group.

This change ensures that when editing an existing group with a forum, its stays up-to-date with its parent group.

Props johnjamesjacoby.

In trunk, for 2.7.0.

Fixes #3417.

File:

• trunk/src/includes/extend/buddypress/groups.php (modified) (3 diffs)

trunk/src/includes/extend/buddypress/groups.php

@@ -87 +87 @@
         add_action( 'bbp_after_group_forum_display',  array( $this, 'remove_group_forum_meta_cap_map' ) );
 
-        // Validate group forum IDs when editing topics & replies
+        // Validate group IDs when editing topics & replies
         add_action( 'bbp_edit_topic_pre_extras',      array( $this, 'validate_topic_forum_id' ) );
         add_action( 'bbp_edit_reply_pre_extras',      array( $this, 'validate_reply_to_id'    ) );
 
-        // Check if group-forum status should be changed
-        add_action( 'groups_group_after_save',        array( $this, 'update_group_forum_visibility'   ) );
+        // Check if group-forum attributes should be changed
+        add_action( 'groups_group_after_save',        array( $this, 'update_group_forum' ) );
 
         // bbPress needs to listen to BuddyPress group deletion
@@ -861 +861 @@
 
     /**
-     * Set forums' status to match the privacy status of the associated group
+     * Update forum attributes to match those of the associated group.
      *
      * Fired whenever a group is saved
      *
+     * @since 2.6.7 bbPress (r7208)
+     *
      * @param BP_Groups_Group $group Group object.
      */
-    public static function update_group_forum_visibility( BP_Groups_Group $group ) {
+    public static function update_group_forum( BP_Groups_Group $group ) {
 
         // Get group forum IDs
@@ -903 +905 @@
                         break;
                 }
+            }
+        }
+
+        // Maybe update the first group forum title, content, and slug
+        if ( ! empty( $forum_ids[0] ) ) {
+
+            // Get forum from ID
+            $forum = bbp_get_forum( $forum_ids[0] );
+
+            // Only update the forum if changes are being made
+            if (
+
+                // Title
+                ( $forum->post_title !== $group->name )
+
+                ||
+
+                // Content
+                ( $forum->post_content !== $group->description )
+
+                ||
+
+                // Slug
+                ( $forum->post_name !== $group->slug )
+            ) {
+                wp_update_post(
+                    array(
+                        'ID'           => $forum->ID,
+                        'post_title'   => $group->name,
+                        'post_content' => $group->description,
+                        'post_name'    => $group->slug
+                    )
+                );
             }
         }