Changeset 13437 for trunk/src/bp-groups/actions/join.php

Timestamp:

03/18/2023 09:42:31 AM (2 years ago)

Author:

imath

Message:

BP Rewrites: start the migration process for the Groups component

• Add rewrite tags & rules for the Groups directory type and the create URLs.
• Introduce bp_get_group_url()/bp_group_url() & to retrieve/output a Groups single item URL using BP Rewrites.
• Introduce bp_get_group_restricted_screens(), bp_get_group_extension_screens() & bp_get_group_screens() to get information about the Groups screens (in particular each screen rewrite ID). These functions will ease slug customizations from the BuddyPress URL settings tab.
• Improve the Group creation process making sure it's using BP Rewrites to build URLs.
• Perform easiest replacements for bp_get_group_permalink()/bp_group_permalink() & bp_get_groups_directory_permalink()/bp_groups_directory_permalink() in favor of bp_get_group_url()/bp_group_url() & bp_get_groups_directory_url()/bp_groups_directory_url()`.
• Improve code formatting & properly escape single group URLs into templates.
• Update impacted Unit Tests.

Props r-a-y, johnjamesjacoby, boonebgorges

Closes ​https://github.com/buddypress/buddypress/pull/75
See #4954

File:

• trunk/src/bp-groups/actions/join.php (modified) (2 diffs)

trunk/src/bp-groups/actions/join.php

@@ -33 +33 @@
             if ( !groups_check_user_has_invite( bp_loggedin_user_id(), $bp->groups->current_group->id ) ) {
                 bp_core_add_message( __( 'There was an error joining the group.', 'buddypress' ), 'error' );
-                bp_core_redirect( bp_get_group_permalink( $bp->groups->current_group ) );
+                bp_core_redirect( bp_get_group_url( $bp->groups->current_group ) );
             }
         }
@@ -43 +43 @@
             bp_core_add_message( __( 'You joined the group!', 'buddypress' ) );
 
-        bp_core_redirect( bp_get_group_permalink( $bp->groups->current_group ) );
+        bp_core_redirect( bp_get_group_url( $bp->groups->current_group ) );
     }