Skip to:
Content

BuddyPress.org

Changeset 13437


Ignore:
Timestamp:
03/18/2023 09:42:31 AM (12 months 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

Location:
trunk
Files:
45 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/bp-activity/bp-activity-template.php

    r13436 r13437  
    12281228                if ( bp_is_active( 'groups' ) ) {
    12291229                    $group = groups_get_group( $item_id );
    1230                     $link  = bp_get_group_permalink( $group );
     1230                    $link  = bp_get_group_url( $group );
    12311231                    $name  = $group->name;
    12321232                }
     
    16111611
    16121612                $generated_content->group_url = array(
    1613                     'value'             => bp_get_group_permalink( $group ),
     1613                    'value'             => bp_get_group_url( $group ),
    16141614                    'sanitize_callback' => 'esc_url',
    16151615                );
  • trunk/src/bp-core/deprecated/12.0.php

    r13436 r13437  
    332332    return apply_filters_deprecated( 'bp_get_blogs_directory_permalink', array( $url ), '12.0.0', 'bp_get_blogs_directory_url' );
    333333}
     334
     335/**
     336 * Outputs the group creation numbered steps navbar
     337 *
     338 * @since 3.0.0
     339 * @deprecated 12.0.0
     340 */
     341function bp_nouveau_group_creation_tabs() {
     342    _deprecated_function( __FUNCTION__, '12.0.0', 'bp_group_creation_tabs()' );
     343    bp_group_creation_tabs();
     344}
  • trunk/src/bp-groups/actions/access.php

    r11923 r13437  
    4444        $no_access_args  = array(
    4545            'message'  => __( 'You are not an admin of this group.', 'buddypress' ),
    46             'root'     => bp_get_group_permalink( $current_group ),
     46            'root'     => bp_get_group_url( $current_group ),
    4747            'redirect' => false
    4848        );
  • trunk/src/bp-groups/actions/create.php

    r13177 r13437  
    1818
    1919    // If we're not at domain.org/groups/create/ then return false.
    20     if ( !bp_is_groups_component() || !bp_is_current_action( 'create' ) )
     20    if ( ! bp_is_groups_component() || ! bp_is_current_action( 'create' ) ) {
    2121        return false;
    22 
    23     if ( !is_user_logged_in() )
     22    }
     23
     24    if ( ! is_user_logged_in() ) {
    2425        return false;
    25 
    26     if ( !bp_user_can_create_groups() ) {
     26    }
     27
     28    if ( ! bp_user_can_create_groups() ) {
    2729        bp_core_add_message( __( 'Sorry, you are not allowed to create groups.', 'buddypress' ), 'error' );
    28         bp_core_redirect( bp_get_groups_directory_permalink() );
     30        bp_core_redirect( bp_get_groups_directory_url() );
    2931    }
    3032
     
    199201            do_action( 'groups_group_create_complete', $bp->groups->new_group_id );
    200202
    201             bp_core_redirect( bp_get_group_permalink( $bp->groups->current_group ) );
     203            bp_core_redirect( bp_get_group_url( $bp->groups->current_group ) );
    202204        } else {
    203205            /**
     
    314316function groups_action_sort_creation_steps() {
    315317
    316     if ( !bp_is_groups_component() || !bp_is_current_action( 'create' ) )
     318    if ( ! bp_is_groups_component() || ! bp_is_current_action( 'create' ) ) {
    317319        return false;
     320    }
    318321
    319322    $bp = buddypress();
    320323
    321     if ( !is_array( $bp->groups->group_creation_steps ) )
     324    if ( ! is_array( $bp->groups->group_creation_steps ) ) {
    322325        return false;
     326    }
    323327
    324328    foreach ( (array) $bp->groups->group_creation_steps as $slug => $step ) {
    325         while ( !empty( $temp[$step['position']] ) )
     329        while ( ! empty( $temp[$step['position']] ) ) {
    326330            $step['position']++;
    327 
    328         $temp[$step['position']] = array( 'name' => $step['name'], 'slug' => $slug );
     331        }
     332
     333        $temp[ $step['position'] ] = array(
     334            'rewrite_id' => $step['rewrite_id'],
     335            'name'       => $step['name'],
     336            'slug'       => $slug,
     337        );
    329338    }
    330339
    331340    // Sort the steps by their position key.
    332     ksort($temp);
    333     unset($bp->groups->group_creation_steps);
    334 
    335     foreach( (array) $temp as $position => $step )
    336         $bp->groups->group_creation_steps[$step['slug']] = array( 'name' => $step['name'], 'position' => $position );
     341    ksort( $temp );
     342    unset( $bp->groups->group_creation_steps );
     343
     344    foreach( (array) $temp as $position => $step ) {
     345        $bp->groups->group_creation_steps[ $step['slug'] ] = array(
     346            'rewrite_id'   => $step['rewrite_id'],
     347            'default_slug' => $step['slug'],
     348            'name'         => $step['name'],
     349            'position'     => $position
     350        );
     351    }
    337352
    338353    /**
  • trunk/src/bp-groups/actions/feed.php

    r12590 r13437  
    3636        'title'         => sprintf( _x( '%1$s | %2$s | Activity', 'Group activity RSS title', 'buddypress' ), bp_get_site_name(), bp_get_current_group_name() ),
    3737
    38         'link'          => bp_get_group_permalink( $group ),
     38        'link'          => bp_get_group_url( $group ),
    3939
    4040        /* translators: %s: Group Name. */
  • trunk/src/bp-groups/actions/join.php

    r11971 r13437  
    3333            if ( !groups_check_user_has_invite( bp_loggedin_user_id(), $bp->groups->current_group->id ) ) {
    3434                bp_core_add_message( __( 'There was an error joining the group.', 'buddypress' ), 'error' );
    35                 bp_core_redirect( bp_get_group_permalink( $bp->groups->current_group ) );
     35                bp_core_redirect( bp_get_group_url( $bp->groups->current_group ) );
    3636            }
    3737        }
     
    4343            bp_core_add_message( __( 'You joined the group!', 'buddypress' ) );
    4444
    45         bp_core_redirect( bp_get_group_permalink( $bp->groups->current_group ) );
     45        bp_core_redirect( bp_get_group_url( $bp->groups->current_group ) );
    4646    }
    4747
  • trunk/src/bp-groups/actions/leave-group.php

    r12438 r13437  
    4747
    4848        $group = groups_get_current_group();
    49         $redirect = bp_get_group_permalink( $group );
     49        $redirect = bp_get_group_url( $group );
    5050
    5151        if ( ! $group->is_visible ) {
     
    7878     * Remove invitations where the deleted user is the sender.
    7979     * We'll use groups_uninvite_user() so that notifications will be cleaned up.
    80      */ 
     80     */
    8181    $pending_invites = groups_get_invites( array(
    8282        'inviter_id' => $user_id,
  • trunk/src/bp-groups/actions/random.php

    r11923 r13437  
    1818        $group = BP_Groups_Group::get_random( 1, 1 );
    1919
    20         bp_core_redirect( trailingslashit( bp_get_groups_directory_permalink() . $group['groups'][0]->slug ) );
     20        bp_core_redirect( bp_get_group_url( $group['groups'][0] ) );
    2121    }
    2222}
  • trunk/src/bp-groups/bp-groups-activity.php

    r13367 r13437  
    107107
    108108    $group      = bp_groups_get_activity_group( $activity->item_id );
    109     $group_link = '<a href="' . esc_url( bp_get_group_permalink( $group ) ) . '">' . esc_html( $group->name ) . '</a>';
     109    $group_link = '<a href="' . esc_url( bp_get_group_url( $group ) ) . '">' . esc_html( $group->name ) . '</a>';
    110110
    111111    /* translators: 1: the user link. 2: the group link. */
     
    136136
    137137    $group      = bp_groups_get_activity_group( $activity->item_id );
    138     $group_link = '<a href="' . esc_url( bp_get_group_permalink( $group ) ) . '">' . esc_html( $group->name ) . '</a>';
     138    $group_link = '<a href="' . esc_url( bp_get_group_url( $group ) ) . '">' . esc_html( $group->name ) . '</a>';
    139139
    140140    /* translators: 1: the user link. 2: the group link. */
     
    176176
    177177    $group      = bp_groups_get_activity_group( $activity->item_id );
    178     $group_link = '<a href="' . esc_url( bp_get_group_permalink( $group ) ) . '">' . esc_html( $group->name ) . '</a>';
     178    $group_link = '<a href="' . esc_url( bp_get_group_url( $group ) ) . '">' . esc_html( $group->name ) . '</a>';
    179179
    180180    /*
     
    233233    $group     = bp_groups_get_activity_group( $activity->item_id );
    234234
    235     $group_link = '<a href="' . esc_url( bp_get_group_permalink( $group ) ) . '">' . esc_html( $group->name ) . '</a>';
     235    $group_link = '<a href="' . esc_url( bp_get_group_url( $group ) ) . '">' . esc_html( $group->name ) . '</a>';
    236236
    237237    // Set the Activity update posted in a Group action.
     
    770770     * @param int    $group_id ID of the group. Passed by reference.
    771771     */
    772     $action = apply_filters_ref_array( 'groups_activity_membership_accepted_action', array( sprintf( __( '%1$s joined the group %2$s', 'buddypress' ), bp_core_get_userlink( $user_id ), '<a href="' . bp_get_group_permalink( $group ) . '">' . esc_attr( $group->name ) . '</a>' ), $user_id, &$group ) );
     772    $action = apply_filters_ref_array( 'groups_activity_membership_accepted_action', array( sprintf( __( '%1$s joined the group %2$s', 'buddypress' ), bp_core_get_userlink( $user_id ), '<a href="' . esc_url( bp_get_group_url( $group ) ) . '">' . esc_html( $group->name ) . '</a>' ), $user_id, &$group ) );
    773773
    774774    // Record in activity streams.
  • trunk/src/bp-groups/bp-groups-admin.php

    r13433 r13437  
    639639    $form_url = remove_query_arg( array( 'action', 'deleted', 'no_admins', 'error', 'error_new', 'success_new', 'error_modified', 'success_modified' ), $_SERVER['REQUEST_URI'] );
    640640    $form_url = add_query_arg( 'action', 'save', $form_url );
     641    $create_url = bp_get_groups_directory_url(
     642        array(
     643            'create_single_item' => 1,
     644        )
     645    );
    641646
    642647    /**
     
    655660
    656661        <?php if ( is_user_logged_in() && bp_user_can_create_groups() ) : ?>
    657             <a class="page-title-action" href="<?php echo trailingslashit( bp_get_groups_directory_permalink() . 'create' ); ?>"><?php _e( 'Add New', 'buddypress' ); ?></a>
     662            <a class="page-title-action" href="<?php echo esc_url( $create_url ); ?>"><?php esc_html_e( 'Add New', 'buddypress' ); ?></a>
    658663        <?php endif; ?>
    659664
     
    684689                                            <strong><?php esc_html_e( 'Permalink:', 'buddypress' ) ?></strong>
    685690                                            <span id="bp-groups-permalink">
    686                                                 <?php bp_groups_directory_permalink(); ?> <input type="text" id="bp-groups-slug" name="bp-groups-slug" value="<?php bp_group_slug( $group ); ?>" autocomplete="off"> /
     691                                                <?php bp_groups_directory_url(); ?> <input type="text" id="bp-groups-slug" name="bp-groups-slug" value="<?php bp_group_slug( $group ); ?>" autocomplete="off"> /
    687692                                            </span>
    688                                             <a href="<?php echo bp_group_permalink( $group ) ?>" class="button button-small" id="bp-groups-visit-group"><?php esc_html_e( 'Visit Group', 'buddypress' ) ?></a>
     693                                            <a href="<?php bp_group_url( $group ) ?>" class="button button-small" id="bp-groups-visit-group"><?php esc_html_e( 'Visit Group', 'buddypress' ) ?></a>
    689694                                        </div>
    690695
     
    814819    // Prepare the group items for display.
    815820    $bp_groups_list_table->prepare_items();
     821    $create_url = bp_get_groups_directory_url(
     822        array(
     823            'create_single_item' => 1,
     824        )
     825    );
    816826
    817827    /**
     
    831841
    832842        <?php if ( is_user_logged_in() && bp_user_can_create_groups() ) : ?>
    833             <a class="page-title-action" href="<?php echo trailingslashit( bp_get_groups_directory_permalink() . 'create' ); ?>"><?php _e( 'Add New', 'buddypress' ); ?></a>
     843            <a class="page-title-action" href="<?php echo esc_url( $create_url ); ?>"><?php esc_html_e( 'Add New', 'buddypress' ); ?></a>
    834844        <?php endif; ?>
    835845
  • trunk/src/bp-groups/bp-groups-adminbar.php

    r13395 r13437  
    4646        'id'    => $bp->group_admin_menu_id,
    4747        'title' => __( 'Edit Group', 'buddypress' ),
    48         'href'  => bp_get_group_permalink( $bp->groups->current_group )
     48        'href'  => bp_get_group_url( $bp->groups->current_group )
    4949    ) );
    5050
  • trunk/src/bp-groups/bp-groups-blocks.php

    r13296 r13437  
    6161    // Group name/link/description variables.
    6262    $group_name        = bp_get_group_name( $group );
    63     $group_link        = bp_get_group_permalink( $group );
     63    $group_link        = bp_get_group_url( $group );
    6464    $group_description = '';
    6565    $group_content     = '';
     
    235235
    236236        // Get Member link.
    237         $group_link = bp_get_group_permalink( $group );
     237        $group_link = bp_get_group_url( $group );
    238238
    239239        // Set the Avatar output.
     
    384384    // Make sure the widget ID is unique.
    385385    $widget_id             = uniqid( 'groups-list-' );
    386     $groups_directory_link = bp_get_groups_directory_permalink();
     386    $groups_directory_link = bp_get_groups_directory_url();
    387387
    388388    // Set the Block's title.
     
    465465                    'php',
    466466                    array(
    467                         'data.link'              => bp_get_group_permalink( $group ),
     467                        'data.link'              => bp_get_group_url( $group ),
    468468                        'data.name'              => bp_get_group_name( $group ),
    469469                        'data.avatar_urls.thumb' => bp_core_fetch_avatar(
  • trunk/src/bp-groups/bp-groups-filters.php

    r12906 r13437  
    3434
    3535add_filter( 'bp_get_group_name',                    'wp_filter_kses',        1 );
    36 add_filter( 'bp_get_group_permalink',               'wp_filter_kses',        1 );
     36add_filter( 'bp_get_group_url',                     'wp_filter_kses',        1 );
    3737add_filter( 'bp_get_group_description',             'bp_groups_filter_kses', 1 );
    3838add_filter( 'bp_get_group_description_excerpt',     'wp_filter_kses',        1 );
  • trunk/src/bp-groups/bp-groups-functions.php

    r13395 r13437  
    33343334            array(
    33353335                'name'  => __( 'Group URL', 'buddypress' ),
    3336                 'value' => bp_get_group_permalink( $group ),
     3336                'value' => bp_get_group_url( $group ),
    33373337            ),
    33383338        );
     
    34233423            array(
    34243424                'name'  => __( 'Group URL', 'buddypress' ),
    3425                 'value' => bp_get_group_permalink( $group ),
     3425                'value' => bp_get_group_url( $group ),
    34263426            ),
    34273427            array(
     
    34893489            array(
    34903490                'name'  => __( 'Group URL', 'buddypress' ),
    3491                 'value' => bp_get_group_permalink( $group ),
     3491                'value' => bp_get_group_url( $group ),
    34923492            ),
    34933493            array(
     
    35593559            array(
    35603560                'name'  => __( 'Group URL', 'buddypress' ),
    3561                 'value' => bp_get_group_permalink( $group ),
     3561                'value' => bp_get_group_url( $group ),
    35623562            ),
    35633563            array(
     
    37253725    }
    37263726}
     3727
     3728/**
     3729 * Returns the Group restricted screens.
     3730 *
     3731 * @since 12.0.0
     3732 *
     3733 * @return array The list of the Group restricted screens.
     3734 */
     3735function bp_get_group_restricted_screens() {
     3736    return array(
     3737        'bp_group_create'      => array(
     3738            'rewrite_id' => 'bp_group_create',
     3739            'slug'       => 'create',
     3740            'name'       => _x( 'Create Group root slug', 'Group create restricted rewrite id', 'buddypress' ),
     3741            'context'    => 'create',
     3742        ),
     3743        'bp_group_create_step' => array(
     3744            'rewrite_id' => 'bp_group_create_step',
     3745            'slug'       => 'step',
     3746            'name'       => _x( 'Create step slug', 'Group create restricted rewrite id', 'buddypress' ),
     3747            'context'    => 'create',
     3748        ),
     3749    );
     3750}
     3751
     3752/**
     3753 * Returns all registered Group Extension front-end screens.
     3754 *
     3755 * @since 12.0.0
     3756 *
     3757 * @param string $context The display context. Required. Defaults to `read`.
     3758 * @return array          The list of registered Group Extension screens.
     3759 */
     3760function bp_get_group_extension_screens( $context = 'read' ) {
     3761    $bp = buddypress();
     3762
     3763    $group_extension_screens = array(
     3764        'create' => array(),
     3765        'manage' => array(),
     3766        'read'   => array(),
     3767    );
     3768
     3769    if ( $bp->groups->group_extensions ) {
     3770        foreach ( $bp->groups->group_extensions as $extension_screens ) {
     3771            if ( ! is_array( $extension_screens ) ) {
     3772                continue;
     3773            }
     3774
     3775            foreach ( $extension_screens as $ctext => $extension_screen ) {
     3776                $group_extension_screens[ $ctext ] = array_merge( $group_extension_screens[ $ctext ], $extension_screen );
     3777            }
     3778        }
     3779    }
     3780
     3781    if ( ! array_filter( $group_extension_screens ) || ! isset( $group_extension_screens[ $context ] ) ) {
     3782        return array();
     3783    }
     3784
     3785    return $group_extension_screens[ $context ];
     3786}
     3787
     3788/**
     3789 * Returns all potential Group screens.
     3790 *
     3791 * @since 12.0.0
     3792 *
     3793 * @param string $context The display context. Required. Defaults to `read`.
     3794 * @return array          The list of potential Group screens.
     3795 */
     3796function bp_get_group_screens( $context = 'read' ) {
     3797    $screens = array(
     3798        'create' => array(
     3799            'group-details'     => array(
     3800                'rewrite_id' => 'bp_group_create_group_details',
     3801                'slug'       => 'group-details',
     3802                'name'       => _x( 'Details', 'Group create screen', 'buddypress' ),
     3803                'position'   => 0,
     3804            ),
     3805            'group-settings'    => array(
     3806                'rewrite_id' => 'bp_group_create_group_settings',
     3807                'slug'       => 'group-settings',
     3808                'name'       => _x( 'Settings', 'Group create screen', 'buddypress' ),
     3809                'position'   => 10,
     3810            ),
     3811            'group-avatar'      => array(
     3812                'rewrite_id' => 'bp_group_create_group_avatar',
     3813                'slug'       => 'group-avatar',
     3814                'name'       => _x( 'Photo', 'Group create screen', 'buddypress' ),
     3815                'position'   => 20,
     3816            ),
     3817            'group-cover-image' => array(
     3818                'rewrite_id' => 'bp_group_create_group_cover_image',
     3819                'slug'       => 'group-cover-image',
     3820                'name'       => _x( 'Cover Image', 'Group create screen', 'buddypress' ),
     3821                'position'   => 25,
     3822            ),
     3823            'group-invites'     => array(
     3824                'rewrite_id' => 'bp_group_create_group_invites',
     3825                'slug'       => 'group-invites',
     3826                'name'       => _x( 'Invites', 'Group create screen', 'buddypress' ),
     3827                'position'   => 30,
     3828            ),
     3829        ),
     3830        'read'   => array(
     3831            'home'               => array(
     3832                'rewrite_id'      => 'bp_group_read_home',
     3833                'slug'            => 'home',
     3834                'name'            => _x( 'Home', 'Group read screen', 'buddypress' ),
     3835                'screen_function' => 'groups_screen_group_home',
     3836                'position'        => 10,
     3837                'item_css_id'     => 'home',
     3838            ),
     3839            'request-membership' => array(
     3840                'rewrite_id'      => 'bp_group_read_request_membership',
     3841                'slug'            => 'request-membership',
     3842                'name'            => _x( 'Request Membership', 'Group read screen', 'buddypress' ),
     3843                'screen_function' => 'groups_screen_group_request_membership',
     3844                'position'        => 30,
     3845            ),
     3846            'members'            => array(
     3847                'rewrite_id'      => 'bp_group_read_members',
     3848                'slug'            => 'members',
     3849                /* translators: %s: total member count */
     3850                'name'            => _x( 'Members %s', 'Group read screen', 'buddypress' ),
     3851                'screen_function' => 'groups_screen_group_members',
     3852                'position'        => 60,
     3853                'user_has_access' => false,
     3854                'no_access_url'   => '',
     3855                'item_css_id'     => 'members',
     3856            ),
     3857            'send-invites'       => array(
     3858                'rewrite_id'      => 'bp_group_read_send_invites',
     3859                'slug'            => 'send-invites',
     3860                'name'            => _x( 'Send Invites', 'Group read screen', 'buddypress' ),
     3861                'screen_function' => 'groups_screen_group_invite',
     3862                'position'        => 70,
     3863                'user_has_access' => false,
     3864                'no_access_url'   => '',
     3865                'item_css_id'     => 'invite',
     3866            ),
     3867            'admin'              => array(
     3868                'rewrite_id'      => 'bp_group_read_admin',
     3869                'slug'            => 'admin',
     3870                'name'            => _x( 'Manage', 'Group read screen', 'buddypress' ),
     3871                'screen_function' => 'groups_screen_group_admin',
     3872                'position'        => 1000,
     3873                'user_has_access' => false,
     3874                'no_access_url'   => '',
     3875                'item_css_id'     => 'admin',
     3876            ),
     3877        ),
     3878        'manage' => array(
     3879            'edit-details'        => array(
     3880                'rewrite_id'        => 'bp_group_manage_edit_details',
     3881                'slug'              => 'edit-details',
     3882                'name'              => _x( 'Details', 'Group manage screen', 'buddypress' ),
     3883                'screen_function'   => 'groups_screen_group_admin',
     3884                'position'          => 0,
     3885                'user_has_access'   => false,
     3886                'show_in_admin_bar' => true,
     3887            ),
     3888            'group-settings'      => array(
     3889                'rewrite_id'        => 'bp_group_manage_group_settings',
     3890                'slug'              => 'group-settings',
     3891                'name'              => _x( 'Settings', 'Group manage screen', 'buddypress' ),
     3892                'screen_function'   => 'groups_screen_group_admin',
     3893                'position'          => 10,
     3894                'user_has_access'   => false,
     3895                'show_in_admin_bar' => true,
     3896            ),
     3897            'group-avatar'        => array(
     3898                'rewrite_id'        => 'bp_group_manage_group_avatar',
     3899                'slug'              => 'group-avatar',
     3900                'name'              => _x( 'Photo', 'Group manage screen', 'buddypress' ),
     3901                'screen_function'   => 'groups_screen_group_admin',
     3902                'position'          => 20,
     3903                'user_has_access'   => false,
     3904                'show_in_admin_bar' => true,
     3905            ),
     3906            'group-cover-image'   => array(
     3907                'rewrite_id'        => 'bp_group_manage_group_cover_image',
     3908                'slug'              => 'group-cover-image',
     3909                'name'              => _x( 'Cover Image', 'Group manage screen', 'buddypress' ),
     3910                'screen_function'   => 'groups_screen_group_admin',
     3911                'position'          => 25,
     3912                'user_has_access'   => false,
     3913                'show_in_admin_bar' => true,
     3914            ),
     3915            'manage-members'      => array(
     3916                'rewrite_id'        => 'bp_group_manage_manage_members',
     3917                'slug'              => 'manage-members',
     3918                'name'              => _x( 'Members', 'Group manage screen', 'buddypress' ),
     3919                'screen_function'   => 'groups_screen_group_admin',
     3920                'position'          => 30,
     3921                'user_has_access'   => false,
     3922                'show_in_admin_bar' => true,
     3923            ),
     3924            'membership-requests' => array(
     3925                'rewrite_id'        => 'bp_group_manage_membership_requests',
     3926                'slug'              => 'membership-requests',
     3927                'name'              => _x( 'Requests', 'Group manage screen', 'buddypress' ),
     3928                'screen_function'   => 'groups_screen_group_admin',
     3929                'position'          => 40,
     3930                'user_has_access'   => false,
     3931                'show_in_admin_bar' => true,
     3932            ),
     3933            'delete-group'        => array(
     3934                'rewrite_id'        => 'bp_group_manage_delete_group',
     3935                'slug'              => 'delete-group',
     3936                'name'              => _x( 'Delete', 'Group manage screen', 'buddypress' ),
     3937                'screen_function'   => 'groups_screen_group_admin',
     3938                'position'          => 1000,
     3939                'user_has_access'   => false,
     3940                'show_in_admin_bar' => true,
     3941            ),
     3942        ),
     3943    );
     3944
     3945    if ( ! isset( $screens[ $context ] ) ) {
     3946        return array();
     3947    }
     3948
     3949    $context_screens         = array();
     3950    $custom_screens          = apply_filters( 'bp_get_group_custom_' . $context . '_screens', $context_screens );
     3951    $group_extension_screens = bp_get_group_extension_screens( $context );
     3952
     3953    if ( $group_extension_screens ) {
     3954        $custom_screens = array_merge( $custom_screens, $group_extension_screens );
     3955    }
     3956
     3957    if ( $custom_screens && ! wp_is_numeric_array( $custom_screens ) ) {
     3958        // The screen key (used as default slug) and `rewrite_id` prop need to be unique.
     3959        $valid_custom_screens   = array_diff_key( $custom_screens, $screens[ $context ] );
     3960        $existing_rewrite_ids = array_column( $screens[ $context ], 'rewrite_id' );
     3961        $existing_rewrite_ids = array_merge(
     3962            $existing_rewrite_ids,
     3963            // BP Group Reserved rewrite IDs.
     3964            array_keys( bp_get_group_restricted_screens() )
     3965        );
     3966
     3967        foreach ( $valid_custom_screens as $key_screen => $screen ) {
     3968            if ( ! isset( $screen['rewrite_id'] ) || ! in_array( $screen['rewrite_id'], $existing_rewrite_ids, true ) ) {
     3969                continue;
     3970            }
     3971
     3972            unset( $valid_custom_screens[ $key_screen ] );
     3973        }
     3974
     3975        $context_screens = array_merge( $screens[ $context ], $valid_custom_screens );
     3976    } else {
     3977        $context_screens = $screens[ $context ];
     3978    }
     3979
     3980    return $context_screens;
     3981}
  • trunk/src/bp-groups/bp-groups-notifications.php

    r13436 r13437  
    5252                /* translators: 1: the old group permalink. 2: the new group permalink. */
    5353                _x( '* Permalink changed from "%1$s" to "%2$s".', 'Group update email text', 'buddypress' ),
    54                 esc_url( bp_get_group_permalink( $old_group ) ),
    55                 esc_url( bp_get_group_permalink( $group ) )
     54                esc_url( bp_get_group_url( $old_group ) ),
     55                esc_url( bp_get_group_url( $group ) )
    5656            );
    5757        }
     
    9090                'group'        => $group,
    9191                'group.id'     => $group_id,
    92                 'group.url'    => esc_url( bp_get_group_permalink( $group ) ),
     92                'group.url'    => esc_url( bp_get_group_url( $group ) ),
    9393                'group.name'   => $group->name,
    9494                'unsubscribe'  => esc_url( bp_email_get_unsubscribe_link( $unsubscribe_args ) ),
     
    216216            'group.id'           => $group_id,
    217217            'group.name'         => $group->name,
    218             'group.url'          => esc_url( bp_get_group_permalink( $group ) ),
     218            'group.url'          => esc_url( bp_get_group_url( $group ) ),
    219219            'requesting-user.id' => $requesting_user_id,
    220220        ),
     
    291291            'group'       => $group,
    292292            'group.id'    => $group_id,
    293             'group.url'   => esc_url( bp_get_group_permalink( $group ) ),
     293            'group.url'   => esc_url( bp_get_group_url( $group ) ),
    294294            'group.name'  => $group->name,
    295295            'promoted_to' => $promoted_to,
     
    363363        'tokens' => array(
    364364            'group'          => $group,
    365             'group.url'      => bp_get_group_permalink( $group ),
     365            'group.url'      => esc_url( bp_get_group_url( $group ) ),
    366366            'group.name'     => $group->name,
    367367            'inviter.name'   => bp_core_get_userlink( $inviter_user_id, true, false ),
     
    397397    switch ( $action ) {
    398398        case 'new_membership_request':
    399             $group_id = $item_id;
     399            $group_id           = $item_id;
    400400            $requesting_user_id = $secondary_item_id;
    401401
    402             $group = groups_get_group( $group_id );
    403             $group_link = bp_get_group_permalink( $group );
    404             $amount = 'single';
    405 
    406             // Set up the string and the filter
    407             // because different values are passed to the filters,
    408             // we'll return values inline.
     402            $group      = groups_get_group( $group_id );
     403            $group_link = bp_get_group_url( $group );
     404            $amount     = 'single';
     405
     406            /*
     407             * Set up the string and the filter
     408             * because different values are passed to the filters,
     409             * we'll return values inline.
     410             */
    409411            if ( (int) $total_items > 1 ) {
    410412                /* translators: 1: number of group membership requests. 2: group name. */
    411                 $text = sprintf( __( '%1$d new membership requests for the group "%2$s"', 'buddypress' ), (int) $total_items, $group->name );
    412                 $amount = 'multiple';
    413                 $notification_link = $group_link . 'admin/membership-requests/?n=1';
     413                $text              = sprintf( __( '%1$d new membership requests for the group "%2$s"', 'buddypress' ), (int) $total_items, $group->name );
     414                $amount            = 'multiple';
     415                $notification_link = add_query_arg(
     416                    array(
     417                        'n' => 1,
     418                    ),
     419                    bp_get_group_url(
     420                        $group,
     421                        array(
     422                            'single_item_action'           => bp_rewrites_get_slug( 'groups', 'bp_group_read_admin', 'admin' ),
     423                            'single_item_action_variables' => array( bp_rewrites_get_slug( 'groups', 'bp_group_manage_membership_requests', 'membership-requests' ) ),
     424                        )
     425                    )
     426                );
    414427
    415428                if ( 'string' == $format ) {
     
    430443                     * @param string $notification_link The permalink for notification.
    431444                     */
    432                     return apply_filters( 'bp_groups_' . $amount . '_' . $action . 's_notification', '<a href="' . $notification_link . '">' . $text . '</a>', $group_link, $total_items, $group->name, $text, $notification_link );
     445                    return apply_filters( 'bp_groups_' . $amount . '_' . $action . 's_notification', '<a href="' . esc_url( $notification_link ) . '">' . esc_html( $text ) . '</a>', $group_link, $total_items, $group->name, $text, $notification_link );
    433446                } else {
    434447
     
    455468            } else {
    456469                $user_fullname = bp_core_get_user_displayname( $requesting_user_id );
     470
    457471                /* translators: %s: member name */
    458                 $text = sprintf( __( '%s requests group membership', 'buddypress' ), $user_fullname );
    459                 $notification_link = $group_link . 'admin/membership-requests/?n=1';
     472                $text              = sprintf( __( '%s requests group membership', 'buddypress' ), $user_fullname );
     473                $notification_link = $notification_link = add_query_arg(
     474                    array(
     475                        'n' => 1,
     476                    ),
     477                    bp_get_group_url(
     478                        $group,
     479                        array(
     480                            'single_item_action'           => bp_rewrites_get_slug( 'groups', 'bp_group_read_admin', 'admin' ),
     481                            'single_item_action_variables' => array( bp_rewrites_get_slug( 'groups', 'bp_group_manage_membership_requests', 'membership-requests' ) ),
     482                        )
     483                    )
     484                );
    460485
    461486                if ( 'string' == $format ) {
     
    476501                     * @param string $notification_link The permalink for notification.
    477502                     */
    478                     return apply_filters( 'bp_groups_' . $amount . '_' . $action . '_notification', '<a href="' . $notification_link . '">' . $text . '</a>', $group_link, $user_fullname, $group->name, $text, $notification_link );
     503                    return apply_filters( 'bp_groups_' . $amount . '_' . $action . '_notification', '<a href="' . esc_url( $notification_link ) . '">' . esc_html( $text ) . '</a>', $group_link, $user_fullname, $group->name, $text, $notification_link );
    479504                } else {
    480505
     
    506531            $group_id = $item_id;
    507532
    508             $group = groups_get_group( $group_id );
    509             $group_link = bp_get_group_permalink( $group );
    510             $amount = 'single';
     533            $group      = groups_get_group( $group_id );
     534            $group_link = bp_get_group_url( $group );
     535            $amount     = 'single';
    511536
    512537            if ( (int) $total_items > 1 ) {
    513538                /* translators: 1: number of accepted group membership requests. 2: group name. */
    514                 $text = sprintf( __( '%1$d accepted group membership requests for the group "%2$s"', 'buddypress' ), (int) $total_items, $group->name );
    515                 $amount = 'multiple';
     539                $text              = sprintf( __( '%1$d accepted group membership requests for the group "%2$s"', 'buddypress' ), (int) $total_items, $group->name );
     540                $amount            = 'multiple';
    516541                $notification_link = trailingslashit( bp_loggedin_user_domain() . bp_get_groups_slug() ) . '?n=1';
    517542
     
    530555                     * @param string $notification_link The permalink for notification.
    531556                     */
    532                     return apply_filters( 'bp_groups_' . $amount . '_' . $action . '_notification', '<a href="' . $notification_link . '">' . $text . '</a>', $total_items, $group->name, $text, $notification_link );
     557                    return apply_filters( 'bp_groups_' . $amount . '_' . $action . '_notification', '<a href="' . esc_url( $notification_link ) . '">' . esc_html( $text ) . '</a>', $total_items, $group->name, $text, $notification_link );
    533558                } else {
    534559
     
    552577            } else {
    553578                /* translators: %s: group name. */
    554                 $text = sprintf( __( 'Membership for group "%s" accepted', 'buddypress' ), $group->name );
    555                 $filter = 'bp_groups_single_membership_request_accepted_notification';
    556                 $notification_link = $group_link . '?n=1';
     579                $text              = sprintf( __( 'Membership for group "%s" accepted', 'buddypress' ), $group->name );
     580                $filter            = 'bp_groups_single_membership_request_accepted_notification';
     581                $notification_link = add_query_arg( 'n', 1, $group_link );
    557582
    558583                if ( 'string' == $format ) {
     
    570595                     * @param string $notification_link The permalink for notification.
    571596                     */
    572                     return apply_filters( 'bp_groups_' . $amount . '_' . $action . '_notification', '<a href="' . $notification_link . '">' . $text . '</a>', $group_link, $group->name, $text, $notification_link );
     597                    return apply_filters( 'bp_groups_' . $amount . '_' . $action . '_notification', '<a href="' . esc_url( $notification_link ) . '">' . esc_html( $text ) . '</a>', $group_link, $group->name, $text, $notification_link );
    573598                } else {
    574599
     
    597622            $group_id = $item_id;
    598623
    599             $group = groups_get_group( $group_id );
    600             $group_link = bp_get_group_permalink( $group );
    601             $amount = 'single';
     624            $group      = groups_get_group( $group_id );
     625            $group_link = bp_get_group_url( $group );
     626            $amount     = 'single';
    602627
    603628            if ( (int) $total_items > 1 ) {
    604629                /* translators: 1: number of accepted group membership requests. 2: group name. */
    605                 $text = sprintf( __( '%1$d rejected group membership requests for the group "%2$s"', 'buddypress' ), (int) $total_items, $group->name );
    606                 $amount = 'multiple';
     630                $text              = sprintf( __( '%1$d rejected group membership requests for the group "%2$s"', 'buddypress' ), (int) $total_items, $group->name );
     631                $amount            = 'multiple';
    607632                $notification_link = trailingslashit( bp_loggedin_user_domain() . bp_get_groups_slug() ) . '?n=1';
    608633
     
    621646                     * @param string $notification_link The permalink for notification.
    622647                     */
    623                     return apply_filters( 'bp_groups_' . $amount . '_' . $action . '_notification', '<a href="' . $notification_link . '">' . $text . '</a>', $total_items, $group->name, $text, $notification_link );
     648                    return apply_filters( 'bp_groups_' . $amount . '_' . $action . '_notification', '<a href="' . esc_url( $notification_link ) . '">' . esc_html( $text ) . '</a>', $total_items, $group->name, $text, $notification_link );
    624649                } else {
    625650
     
    643668            } else {
    644669                /* translators: %s: group name. */
    645                 $text = sprintf( __( 'Membership for group "%s" rejected', 'buddypress' ), $group->name );
    646                 $notification_link = $group_link . '?n=1';
     670                $text              = sprintf( __( 'Membership for group "%s" rejected', 'buddypress' ), $group->name );
     671                $notification_link = add_query_arg( 'n', 1, $group_link );
    647672
    648673                if ( 'string' == $format ) {
     
    660685                     * @param string $notification_link The permalink for notification.
    661686                     */
    662                     return apply_filters( 'bp_groups_' . $amount . '_' . $action . '_notification', '<a href="' . $notification_link . '">' . $text . '</a>', $group_link, $group->name, $text, $notification_link );
     687                    return apply_filters( 'bp_groups_' . $amount . '_' . $action . '_notification', '<a href="' . esc_url( $notification_link ) . '">' . esc_html( $text ) . '</a>', $group_link, $group->name, $text, $notification_link );
    663688                } else {
    664689
     
    687712            $group_id = $item_id;
    688713
    689             $group = groups_get_group( $group_id );
    690             $group_link = bp_get_group_permalink( $group );
    691             $amount = 'single';
     714            $group      = groups_get_group( $group_id );
     715            $group_link = bp_get_group_url( $group );
     716            $amount     = 'single';
    692717
    693718            if ( (int) $total_items > 1 ) {
    694719                /* translators: %d: number of groups the user has been promoted admin for */
    695                 $text = sprintf( __( 'You were promoted to an admin in %d groups', 'buddypress' ), (int) $total_items );
    696                 $amount = 'multiple';
     720                $text              = sprintf( __( 'You were promoted to an admin in %d groups', 'buddypress' ), (int) $total_items );
     721                $amount            = 'multiple';
    697722                $notification_link = trailingslashit( bp_loggedin_user_domain() . bp_get_groups_slug() ) . '?n=1';
    698723
     
    709734                     * @param string $notification_link The permalink for notification.
    710735                     */
    711                     return apply_filters( 'bp_groups_' . $amount . '_' . $action . '_notification', '<a href="' . $notification_link . '">' . $text . '</a>', $total_items, $text, $notification_link );
     736                    return apply_filters( 'bp_groups_' . $amount . '_' . $action . '_notification', '<a href="' . esc_url( $notification_link ) . '">' . esc_html( $text ) . '</a>', $total_items, $text, $notification_link );
    712737                } else {
    713738                    /**
     
    729754            } else {
    730755                /* translators: %s: group name. */
    731                 $text = sprintf( __( 'You were promoted to an admin in the group "%s"', 'buddypress' ), $group->name );
    732                 $notification_link = $group_link . '?n=1';
     756                $text              = sprintf( __( 'You were promoted to an admin in the group "%s"', 'buddypress' ), $group->name );
     757                $notification_link = add_query_arg( 'n', 1, $group_link );
    733758
    734759                if ( 'string' == $format ) {
     
    745770                     * @param string $notification_link The permalink for notification.
    746771                     */
    747                     return apply_filters( 'bp_groups_' . $amount . '_' . $action . '_notification', '<a href="' . $notification_link . '">' . $text . '</a>', $group_link, $group->name, $text, $notification_link );
     772                    return apply_filters( 'bp_groups_' . $amount . '_' . $action . '_notification', '<a href="' . esc_url( $notification_link ) . '">' . esc_html( $text ) . '</a>', $group_link, $group->name, $text, $notification_link );
    748773                } else {
    749774                    /**
     
    771796            $group_id = $item_id;
    772797
    773             $group = groups_get_group( $group_id );
    774             $group_link = bp_get_group_permalink( $group );
    775             $amount = 'single';
     798            $group      = groups_get_group( $group_id );
     799            $group_link = bp_get_group_url( $group );
     800            $amount     = 'single';
    776801
    777802            if ( (int) $total_items > 1 ) {
    778803                /* translators: %d: number of groups the user has been promoted mod for */
    779                 $text = sprintf( __( 'You were promoted to a mod in %d groups', 'buddypress' ), (int) $total_items );
    780                 $amount = 'multiple';
     804                $text              = sprintf( __( 'You were promoted to a mod in %d groups', 'buddypress' ), (int) $total_items );
     805                $amount            = 'multiple';
    781806                $notification_link = trailingslashit( bp_loggedin_user_domain() . bp_get_groups_slug() ) . '?n=1';
    782807
     
    793818                     * @param string $notification_link The permalink for notification.
    794819                     */
    795                     return apply_filters( 'bp_groups_' . $amount . '_' . $action . '_notification', '<a href="' . $notification_link . '">' . $text . '</a>', $total_items, $text, $notification_link );
     820                    return apply_filters( 'bp_groups_' . $amount . '_' . $action . '_notification', '<a href="' . esc_url( $notification_link ) . '">' . esc_html( $text ) . '</a>', $total_items, $text, $notification_link );
    796821                } else {
    797822                    /**
     
    813838            } else {
    814839                /* translators: %s: group name. */
    815                 $text = sprintf( __( 'You were promoted to a mod in the group "%s"', 'buddypress' ), $group->name );
    816                 $notification_link = $group_link . '?n=1';
     840                $text              = sprintf( __( 'You were promoted to a mod in the group "%s"', 'buddypress' ), $group->name );
     841                $notification_link = add_query_arg( 'n', 1, $group_link );
    817842
    818843                if ( 'string' == $format ) {
     
    829854                     * @param string $notification_link The permalink for notification.
    830855                     */
    831                     return apply_filters( 'bp_groups_' . $amount . '_' . $action . '_notification', '<a href="' . $notification_link . '">' . $text . '</a>', $group_link, $group->name, $text, $notification_link );
     856                    return apply_filters( 'bp_groups_' . $amount . '_' . $action . '_notification', '<a href="' . esc_url( $notification_link ) . '">' . esc_html( $text ) . '</a>', $group_link, $group->name, $text, $notification_link );
    832857                } else {
    833858                    /**
     
    853878
    854879        case 'group_invite':
    855             $group_id = $item_id;
    856             $group = groups_get_group( $group_id );
    857             $group_link = bp_get_group_permalink( $group );
    858             $amount = 'single';
     880            $group_id   = $item_id;
     881            $group      = groups_get_group( $group_id );
     882            $group_link = bp_get_group_url( $group );
     883            $amount     = 'single';
    859884
    860885            $notification_link = bp_loggedin_user_domain() . bp_get_groups_slug() . '/invites/?n=1';
     
    897922            } else {
    898923                /* translators: %s: group name. */
    899                 $text = sprintf( __( 'You have an invitation to the group: %s', 'buddypress' ), $group->name );
     924                $text   = sprintf( __( 'You have an invitation to the group: %s', 'buddypress' ), $group->name );
    900925                $filter = 'bp_groups_single_group_invite_notification';
    901926
     
    913938                     * @param string $notification_link The permalink for notification.
    914939                     */
    915                     return apply_filters( 'bp_groups_' . $amount . '_' . $action . '_notification', '<a href="' . $notification_link . '">' . $text . '</a>', $group_link, $group->name, $text, $notification_link );
     940                    return apply_filters( 'bp_groups_' . $amount . '_' . $action . '_notification', '<a href="' . esc_url( $notification_link ) . '">' . esc_html( $text ) . '</a>', $group_link, $group->name, $text, $notification_link );
    916941                } else {
    917942                    /**
     
    12571282            'group.id'        => $group_id,
    12581283            'group.name'      => $group->name,
    1259             'group.url'       => esc_url( bp_get_group_permalink( $group ) ),
     1284            'group.url'       => esc_url( bp_get_group_url( $group ) ),
    12601285            'leave-group.url' => esc_url(
    12611286                bp_members_get_user_url(
  • trunk/src/bp-groups/bp-groups-template.php

    r13436 r13437  
    118118 */
    119119function bp_get_groups_directory_url( $path_chunks = array() ) {
    120     $supported_chunks = array_fill_keys( array( 'create_single_item', 'directory_type' ), true );
     120    $supported_chunks = array_fill_keys( array( 'create_single_item', 'create_single_item_variables', 'directory_type' ), true );
    121121
    122122    $path_chunks = bp_parse_args(
     
    12371237
    12381238/**
    1239  * Output the permalink for the group.
    1240  *
    1241  * @since 1.0.0
     1239 * Output the URL for the group.
     1240 *
     1241 * @since 12.0.0
    12421242 *
    12431243 * @param false|int|string|BP_Groups_Group $group (Optional) The Group ID, the Group Slug or the Group object.
    12441244 *                                                Default: false.
    12451245 */
     1246function bp_group_url( $group = false ) {
     1247    echo esc_url( bp_get_group_url( $group ) );
     1248}
     1249
     1250/**
     1251 * Returns the Groups single item's URL.
     1252 *
     1253 * @since 12.0.0
     1254 *
     1255 * @param integer|BP_Groups_Group $group The group ID or the Group object.
     1256 * @param array                   $path_chunks {
     1257 *     An array of arguments. Optional.
     1258 *
     1259 *     @type string $single_item_component        The component slug the action is relative to.
     1260 *     @type string $single_item_action           The slug of the action to perform.
     1261 *     @type array  $single_item_action_variables An array of additional informations about the action to perform.
     1262 * }
     1263 * @return string The URL built for the BP Rewrites URL parser.
     1264 */
     1265function bp_get_group_url( $group = 0, $path_chunks = array() ) {
     1266    $url  = '';
     1267    $slug = groups_get_slug( $group );
     1268
     1269    if ( $group instanceof BP_Groups_Group ) {
     1270        $group_id = (int) $group->id;
     1271    } else {
     1272        $group_id = (int) $group;
     1273    }
     1274
     1275    if ( $slug ) {
     1276        $supported_chunks = array_fill_keys( array( 'single_item_component', 'single_item_action', 'single_item_action_variables' ), true );
     1277        $path_chunks      = bp_parse_args(
     1278            array_intersect_key( $path_chunks, $supported_chunks ),
     1279            array(
     1280                'component_id' => 'groups',
     1281                'single_item'  => $slug,
     1282            )
     1283        );
     1284
     1285        $url = bp_rewrites_get_url( $path_chunks );
     1286    }
     1287
     1288    /**
     1289     * Filters the URL for the passed group.
     1290     *
     1291     * @since 12.0.0
     1292     *
     1293     * @param string  $url      The group url.
     1294     * @param integer $group_id The group ID.
     1295     * @param string  $slug     The group slug.
     1296     * @param array   $path_chunks {
     1297     *     An array of arguments. Optional.
     1298     *
     1299     *     @type string $single_item_component        The component slug the action is relative to.
     1300     *     @type string $single_item_action           The slug of the action to perform.
     1301     *     @type array  $single_item_action_variables An array of additional informations about the action to perform.
     1302     * }
     1303     */
     1304    return apply_filters( 'bp_get_group_url', $url, $group_id, $slug, $path_chunks );
     1305}
     1306
     1307/**
     1308 * Output the permalink for the group.
     1309 *
     1310 * @since 1.0.0
     1311 * @deprecated 12.0.0
     1312 *
     1313 * @param false|int|string|BP_Groups_Group $group (Optional) The Group ID, the Group Slug or the Group object.
     1314 *                                                Default: false.
     1315 */
    12461316function bp_group_permalink( $group = false ) {
    1247     echo bp_get_group_permalink( $group );
     1317    _deprecated_function( __FUNCTION__, '12.0.0', 'bp_group_url' );
     1318    bp_group_url( $group );
    12481319}
    12491320    /**
     
    12581329     */
    12591330    function bp_get_group_permalink( $group = false ) {
    1260         $group = bp_get_group( $group );
    1261 
    1262         if ( empty( $group->id ) ) {
    1263             return '';
    1264         }
     1331        /*
     1332         * This function is used at many places and we need to review all this
     1333         * places during the 12.0 development cycle. Using BP Rewrites means we
     1334         * cannot concatenate URL chunks to build our URL anymore. We now need
     1335         * to use `bp_get_group_url( $group, $array )` and make sure to use
     1336         * the right arguments inside this `$array`.
     1337         *
     1338         * @todo Once every link reviewed, we'll be able to remove this check
     1339         *       and let PHPUnit tell us the one we forgot, eventually!
     1340         */
     1341        if ( ! buddypress()->is_phpunit_running ) {
     1342            _deprecated_function( __FUNCTION__, '12.0.0', 'bp_get_group_url' );
     1343        }
     1344
     1345        $url = bp_get_group_url( $group );
    12651346
    12661347        /**
     
    12691350         * @since 1.0.0
    12701351         * @since 2.5.0 Added the `$group` parameter.
    1271          *
    1272          * @param string          $permalink Permalink for the group.
    1273          * @param BP_Groups_Group $group     The group object.
    1274          */
    1275         return apply_filters( 'bp_get_group_permalink', trailingslashit( bp_get_groups_directory_permalink() . bp_get_group_slug( $group ) . '/' ), $group );
     1352         * @deprecated 12.0.0
     1353         *
     1354         * @param string          $url   Permalink for the group.
     1355         * @param BP_Groups_Group $group The group object.
     1356         */
     1357        return apply_filters_deprecated( 'bp_get_group_permalink', array( $url, $group ), '12.0.0', 'bp_get_group_url' );
    12761358    }
    12771359
     
    13061388        $link = sprintf(
    13071389            '<a href="%s" class="bp-group-home-link %s-home-link">%s</a>',
    1308             esc_url( bp_get_group_permalink( $group ) ),
     1390            esc_url( bp_get_group_url( $group ) ),
    13091391            esc_attr( bp_get_group_slug( $group ) ),
    13101392            esc_html( bp_get_group_name( $group ) )
     
    31993281        }
    32003282
     3283        $views = bp_get_group_screens( 'read' );
     3284        if ( isset( $views[ $page ]['rewrite_id'] ) ) {
     3285            $page = bp_rewrites_get_slug( 'groups', $views[ $page ]['rewrite_id'], $page );
     3286        }
     3287
     3288        $url = bp_get_group_url(
     3289            $group,
     3290            array(
     3291                'single_item_action' => $page,
     3292            )
     3293        );
     3294
    32013295        /**
    32023296         * Filters the 'action' attribute for a group form.
     
    32053299         * @since 2.5.0 Added the `$group` parameter.
    32063300         *
    3207          * @param string          $value Action attribute for a group form.
     3301         * @param string          $url  Action attribute for a group form.
    32083302         * @param BP_Groups_Group $group The group object.
    32093303         * @param int|string|bool $page  Page slug.
    32103304         */
    3211         return apply_filters( 'bp_group_form_action', trailingslashit( bp_get_group_permalink( $group ) . $page ), $group, $page );
     3305        return apply_filters( 'bp_group_form_action', $url, $group, $page );
    32123306    }
    32133307
     
    35593653         * @param object $group Group object.
    35603654         */
    3561         return apply_filters( 'bp_get_group_leave_reject_link', bp_get_group_permalink( $group ), $group );
     3655        return apply_filters( 'bp_get_group_leave_reject_link', bp_get_group_url( $group ), $group );
    35623656    }
    35633657
     
    37193813                            'wrapper_class'     => 'group-button ' . $group->status,
    37203814                            'wrapper_id'        => 'groupbutton-' . $group->id,
    3721                             'link_href'         => add_query_arg( 'redirect_to', bp_get_group_permalink( $group ), bp_get_group_accept_invite_link( $group ) ),
     3815                            'link_href'         => add_query_arg( 'redirect_to', bp_get_group_url( $group ), bp_get_group_accept_invite_link( $group ) ),
    37223816                            'link_text'         => __( 'Accept Invitation', 'buddypress' ),
    37233817                            'link_title'        => __( 'Accept Invitation', 'buddypress' ),
     
    37353829                            'wrapper_class'     => 'group-button pending ' . $group->status,
    37363830                            'wrapper_id'        => 'groupbutton-' . $group->id,
    3737                             'link_href'         => bp_get_group_permalink( $group ),
     3831                            'link_href'         => bp_get_group_url( $group ),
    37383832                            'link_text'         => __( 'Request Sent', 'buddypress' ),
    37393833                            'link_title'        => __( 'Request Sent', 'buddypress' ),
     
    38283922            'link_text'  => __( 'Create a Group', 'buddypress' ),
    38293923            'link_class' => 'group-create no-ajax',
    3830             'link_href'  => trailingslashit( bp_get_groups_directory_permalink() . 'create' ),
     3924            'link_href'  => bp_get_groups_directory_url(
     3925                array(
     3926                    'create_single_item' => 1,
     3927                )
     3928            ),
    38313929            'wrapper'    => false,
    38323930            'block_self' => false,
     
    48904988
    48914989/**
    4892  * @since 1.0.0
    4893  *
    4894  * @return bool
     4990 * Outputs the Group creation tabs.
     4991 *
     4992 * @since 1.0.0
    48954993 */
    48964994function bp_group_creation_tabs() {
    4897     $bp = buddypress();
    4898 
    4899     if ( !is_array( $bp->groups->group_creation_steps ) ) {
     4995    $bp           = buddypress();
     4996    $create_steps = $bp->groups->group_creation_steps;
     4997
     4998    if ( ! is_array( $create_steps ) ) {
    49004999        return false;
    49015000    }
    49025001
    4903     if ( !bp_get_groups_current_create_step() ) {
    4904         $keys = array_keys( $bp->groups->group_creation_steps );
     5002    if ( ! bp_get_groups_current_create_step() ) {
     5003        $keys                            = array_keys( $create_steps );
    49055004        $bp->groups->current_create_step = array_shift( $keys );
    49065005    }
     
    49085007    $counter = 1;
    49095008
    4910     foreach ( (array) $bp->groups->group_creation_steps as $slug => $step ) {
    4911         $is_enabled = bp_are_previous_group_creation_steps_complete( $slug ); ?>
    4912 
    4913         <li<?php if ( bp_get_groups_current_create_step() == $slug ) : ?> class="current"<?php endif; ?>><?php if ( $is_enabled ) : ?><a href="<?php bp_groups_directory_permalink(); ?>create/step/<?php echo $slug ?>/"><?php else: ?><span><?php endif; ?><?php echo $counter ?>. <?php echo $step['name'] ?><?php if ( $is_enabled ) : ?></a><?php else: ?></span><?php endif ?></li><?php
     5009    foreach ( (array) $create_steps as $create_step => $step ) {
     5010        $is_enabled    = bp_are_previous_group_creation_steps_complete( $create_step );
     5011        $current_class = '';
     5012        $step_name     = $step['name'];
     5013
     5014        if ( bp_get_groups_current_create_step() === $create_step ) {
     5015            $current_class = ' class="current"';
     5016        }
     5017
     5018        if ( $is_enabled && isset( $create_steps[ $create_step ]['rewrite_id'], $create_steps[ $create_step ]['default_slug'] ) ) {
     5019            $create_step_slug = bp_rewrites_get_slug( 'groups', 'bp_group_create_step', 'step' );
     5020            $step_slug        = bp_rewrites_get_slug( 'groups', $create_steps[ $create_step ]['rewrite_id'], $create_steps[ $create_step ]['default_slug'] );
     5021            $url              = bp_get_groups_directory_url(
     5022                array(
     5023                    'create_single_item'           => 1,
     5024                    'create_single_item_variables' => array( $create_step_slug, $step_slug ),
     5025                )
     5026            );
     5027
     5028            $step_name = sprintf( '<a href="%1$s">%2$s. %3$s</a>', esc_url( $url ), absint( $counter ), esc_html( $step_name ) );
     5029        } else {
     5030            $step_name = sprintf( '<span>%1$s. %2$s</span>', absint( $counter ), esc_html( $step_name ) );
     5031        }
     5032
     5033
     5034        printf( '<li%1$s>%2$s</li>', $current_class, $step_name );
    49145035        $counter++;
    4915     }
    4916 
    4917     unset( $is_enabled );
     5036        unset( $is_enabled );
     5037    }
    49185038
    49195039    /**
     
    49545074 */
    49555075    function bp_get_group_creation_form_action() {
    4956         $bp = buddypress();
    4957 
    4958         if ( !bp_action_variable( 1 ) ) {
    4959             $keys = array_keys( $bp->groups->group_creation_steps );
     5076        $bp           = buddypress();
     5077        $create_steps = $bp->groups->group_creation_steps;
     5078        $url          = '';
     5079
     5080        if ( ! bp_action_variable( 1 ) ) {
     5081            $keys = array_keys( $create_steps );
    49605082            $bp->action_variables[1] = array_shift( $keys );
    49615083        }
    49625084
     5085        $create_step  = bp_action_variable( 1 );
     5086        if ( $create_step && isset( $create_steps[ $create_step ]['rewrite_id'], $create_steps[ $create_step ]['default_slug'] ) ) {
     5087            $create_step_slug = bp_rewrites_get_slug( 'groups', 'bp_group_create_step', 'step' );
     5088            $step_slug        = bp_rewrites_get_slug( 'groups', $create_steps[ $create_step ]['rewrite_id'], $create_steps[ $create_step ]['default_slug'] );
     5089
     5090            $url = bp_get_groups_directory_url(
     5091                array(
     5092                    'create_single_item'           => 1,
     5093                    'create_single_item_variables' => array( $create_step_slug, $step_slug ),
     5094                )
     5095            );
     5096        }
     5097
    49635098        /**
    49645099         * Filters the group creation form action.
     
    49665101         * @since 1.1.0
    49675102         *
    4968          * @param string $value Action to be used with group creation form.
    4969          */
    4970         return apply_filters( 'bp_get_group_creation_form_action', trailingslashit( bp_get_groups_directory_permalink() . 'create/step/' . bp_action_variable( 1 ) ) );
     5103         * @param string $url Action to be used with group creation form.
     5104         */
     5105        return apply_filters( 'bp_get_group_creation_form_action', $url );
    49715106    }
    49725107
     
    52985433     */
    52995434    function bp_get_group_creation_previous_link() {
    5300         $bp    = buddypress();
    5301         $steps = array_keys( $bp->groups->group_creation_steps );
     5435        $create_steps = buddypress()->groups->group_creation_steps;
     5436        $steps        = array_keys( $create_steps );
     5437        $url          = '';
    53025438
    53035439        // Loop through steps.
     
    53145450
    53155451        // Generate the URL for the previous step.
    5316         $group_directory = bp_get_groups_directory_permalink();
    5317         $create_step     = 'create/step/';
    5318         $previous_step   = array_pop( $previous_steps );
    5319         $url             = trailingslashit( $group_directory . $create_step . $previous_step );
     5452        $previous_step = array_pop( $previous_steps );
     5453
     5454        if ( isset( $create_steps[ $previous_step ]['rewrite_id'], $create_steps[ $previous_step ]['default_slug'] ) ) {
     5455            $create_step_slug = bp_rewrites_get_slug( 'groups', 'bp_group_create_step', 'step' );
     5456            $previous_step    = bp_rewrites_get_slug( 'groups', $create_steps[ $previous_step ]['rewrite_id'], $create_steps[ $previous_step ]['default_slug'] );
     5457
     5458            $url = bp_get_groups_directory_url(
     5459                array(
     5460                    'create_single_item'           => 1,
     5461                    'create_single_item_variables' => array( $create_step_slug, $previous_step ),
     5462                )
     5463            );
     5464        }
    53205465
    53215466        /**
     
    65826727                $url = bp_get_group_permalink( $current_group ) . $action;
    65836728            } else {
    6584                 $url = bp_get_group_permalink( $current_group );
     6729                $url = bp_get_group_url( $current_group );
    65856730            }
    65866731
  • trunk/src/bp-groups/bp-groups-widgets.php

    r13153 r13437  
    6868            <li <?php bp_group_class(); ?>>
    6969                <div class="item-avatar">
    70                     <a href="<?php bp_group_permalink() ?>"><?php bp_group_avatar_thumb() ?></a>
     70                    <a href="<?php bp_group_url() ?>"><?php bp_group_avatar_thumb() ?></a>
    7171                </div>
    7272
  • trunk/src/bp-groups/classes/class-bp-group-extension.php

    r13399 r13437  
    731731
    732732        if ( $user_can_see_nav_item ) {
    733             $group_permalink = bp_get_group_permalink( groups_get_current_group() );
     733            $group_permalink = bp_get_group_url( groups_get_current_group() );
    734734
    735735            bp_core_create_subnav_link( array(
     
    750750
    751751        if ( $user_can_visit ) {
    752             $group_permalink = bp_get_group_permalink( groups_get_current_group() );
     752            $group_permalink = bp_get_group_url( groups_get_current_group() );
    753753
    754754            bp_core_register_subnav_screen_function( array(
  • trunk/src/bp-groups/classes/class-bp-groups-component.php

    r13432 r13437  
    375375
    376376        // Set default Group creation steps.
    377         $group_creation_steps = array(
    378             'group-details'  => array(
    379                 'name'       => _x( 'Details', 'Group screen nav', 'buddypress' ),
    380                 'position'   => 0
    381             ),
    382             'group-settings' => array(
    383                 'name'       => _x( 'Settings', 'Group screen nav', 'buddypress' ),
    384                 'position'   => 10
    385             )
    386         );
    387 
    388         // If avatar uploads are not disabled, add avatar option.
     377        $group_creation_steps = bp_get_group_screens( 'create' );
     378
     379        // If avatar uploads are disabled, remove avatar view.
    389380        $disabled_avatar_uploads = (int) bp_disable_group_avatar_uploads();
    390         if ( ! $disabled_avatar_uploads && $bp->avatar->show_avatars ) {
    391             $group_creation_steps['group-avatar'] = array(
    392                 'name'     => _x( 'Photo', 'Group screen nav', 'buddypress' ),
    393                 'position' => 20
    394             );
    395         }
    396 
    397         if ( bp_group_use_cover_image_header() ) {
    398             $group_creation_steps['group-cover-image'] = array(
    399                 'name'     => _x( 'Cover Image', 'Group screen nav', 'buddypress' ),
    400                 'position' => 25
    401             );
    402         }
    403 
    404         // If invitations are enabled, add invitations.
    405         if ( bp_is_active( 'groups', 'invitations' ) ) {
    406             $group_creation_steps['group-invites'] = array(
    407                 'name'     => _x( 'Invites',  'Group screen nav', 'buddypress' ),
    408                 'position' => 30
    409             );
     381        if ( $disabled_avatar_uploads || empty( $bp->avatar->show_avatars ) ) {
     382            unset( $group_creation_steps['group-avatar'] );
     383        }
     384
     385        // If cover images are disabled, remove its view.
     386        if ( ! bp_group_use_cover_image_header() ) {
     387            unset( $group_creation_steps['group-cover-image'] );
     388        }
     389
     390        // If the invitations feature is not active, remove the corresponding view.
     391        if ( ! bp_is_active( 'groups', 'invitations' ) ) {
     392            unset( $group_creation_steps['group-invites'] );
    410393        }
    411394
     
    415398         * @since 1.1.0
    416399         *
    417          * @param array $value Array of preconfigured group creation steps.
     400         * @param array $group_creation_steps Array of preconfigured group creation steps.
    418401         */
    419402        $this->group_creation_steps = apply_filters( 'groups_create_group_steps', $group_creation_steps );
     
    542525
    543526        // Prepare for a redirect to the canonical URL.
    544         $bp->canonical_stack['base_url'] = bp_get_group_permalink( $this->current_group );
     527        $bp->canonical_stack['base_url'] = bp_get_group_url( $this->current_group );
    545528
    546529        if ( bp_current_action() ) {
     
    670653            ), 'groups' );
    671654
    672             $group_link = bp_get_group_permalink( $this->current_group );
     655            $group_link = bp_get_group_url( $this->current_group );
    673656
    674657            // Add the "Home" subnav item, as this will always be present.
     
    919902                    'id'       => 'my-account-' . $this->id . '-create',
    920903                    'title'    => _x( 'Create a Group', 'My Account Groups sub nav', 'buddypress' ),
    921                     'href'     => trailingslashit( bp_get_groups_directory_permalink() . 'create' ),
     904                    'href'     => bp_get_groups_directory_url(
     905                        array(
     906                            'create_single_item' => 1,
     907                        )
     908                    ),
    922909                    'position' => 90
    923910                );
     
    1009996        // Just let BP Component fire 'bp_groups_register_taxonomies'.
    1010997        return parent::register_taxonomies();
     998    }
     999
     1000    /**
     1001     * Adds the Groups directory type & Group create rewrite tags.
     1002     *
     1003     * @since 12.0.0
     1004     *
     1005     * @param array $rewrite_tags Optional. See BP_Component::add_rewrite_tags() for
     1006     *                            description.
     1007     */
     1008    public function add_rewrite_tags( $rewrite_tags = array() ) {
     1009        $rewrite_tags = array(
     1010            'directory_type'               => '([^/]+)',
     1011            'create_single_item'           => '([1]{1,})',
     1012            'create_single_item_variables' => '(.+?)',
     1013        );
     1014
     1015        parent::add_rewrite_tags( $rewrite_tags );
     1016    }
     1017
     1018    /**
     1019     * Adds the Groups directory type & Group create rewrite rules.
     1020     *
     1021     * @since 12.0.0
     1022     *
     1023     * @param array $rewrite_rules Optional. See BP_Component::add_rewrite_rules() for
     1024     *                             description.
     1025     */
     1026    public function add_rewrite_rules( $rewrite_rules = array() ) {
     1027        $create_slug = bp_rewrites_get_slug( 'groups', 'group_create', 'create' );
     1028
     1029        $rewrite_rules = array(
     1030            'directory_type'      => array(
     1031                'regex' => $this->root_slug . '/' . bp_get_groups_group_type_base() . '/([^/]+)/?$',
     1032                'order' => 50,
     1033                'query' => 'index.php?' . $this->rewrite_ids['directory'] . '=1&' . $this->rewrite_ids['directory_type'] . '=$matches[1]',
     1034            ),
     1035            'create_single_item' => array(
     1036                'regex' => $this->root_slug . '/' . $create_slug . '/?$',
     1037                'order' => 40,
     1038                'query' => 'index.php?' . $this->rewrite_ids['directory'] . '=1&' . $this->rewrite_ids['create_single_item'] . '=1',
     1039            ),
     1040            'create_single_item_variables' => array(
     1041                'regex' => $this->root_slug . '/' . $create_slug . '/(.+?)/?$',
     1042                'order' =>30,
     1043                'query' => 'index.php?' . $this->rewrite_ids['directory'] . '=1&' . $this->rewrite_ids['create_single_item'] . '=1&' . $this->rewrite_ids['create_single_item_variables'] . '=$matches[1]',
     1044            ),
     1045        );
     1046
     1047        parent::add_rewrite_rules( $rewrite_rules );
    10111048    }
    10121049
  • trunk/src/bp-groups/classes/class-bp-groups-list-table.php

    r13147 r13437  
    589589        $delete_url = wp_nonce_url( $base_url . "&amp;action=delete", 'bp-groups-delete' );
    590590        $edit_url   = $base_url . '&amp;action=edit';
    591         $view_url   = bp_get_group_permalink( $item_obj );
     591        $view_url   = bp_get_group_url( $item_obj );
    592592
    593593        /**
  • trunk/src/bp-groups/classes/class-bp-groups-widget.php

    r13108 r13437  
    101101        echo $before_widget;
    102102
    103         $title = ! empty( $instance['link_title'] ) ? '<a href="' . bp_get_groups_directory_permalink() . '">' . $title . '</a>' : $title;
     103        $title = ! empty( $instance['link_title'] ) ? '<a href="' . esc_url( bp_get_groups_directory_url() ) . '">' . $title . '</a>' : $title;
    104104
    105105        echo $before_title . $title . $after_title;
     
    126126        <?php if ( bp_has_groups( $group_args ) ) : ?>
    127127            <div class="item-options" id="groups-list-options">
    128                 <a href="<?php bp_groups_directory_permalink(); ?>" id="newest-groups"<?php if ( $instance['group_default'] == 'newest' ) : ?> class="selected"<?php endif; ?>><?php _e("Newest", 'buddypress') ?></a>
     128                <a href="<?php bp_groups_directory_url(); ?>" id="newest-groups"<?php if ( $instance['group_default'] == 'newest' ) : ?> class="selected"<?php endif; ?>><?php _e("Newest", 'buddypress') ?></a>
    129129                <span class="bp-separator" role="separator"><?php echo esc_html( $separator ); ?></span>
    130                 <a href="<?php bp_groups_directory_permalink(); ?>" id="recently-active-groups"<?php if ( $instance['group_default'] == 'active' ) : ?> class="selected"<?php endif; ?>><?php _e("Active", 'buddypress') ?></a>
     130                <a href="<?php bp_groups_directory_url(); ?>" id="recently-active-groups"<?php if ( $instance['group_default'] == 'active' ) : ?> class="selected"<?php endif; ?>><?php _e("Active", 'buddypress') ?></a>
    131131                <span class="bp-separator" role="separator"><?php echo esc_html( $separator ); ?></span>
    132                 <a href="<?php bp_groups_directory_permalink(); ?>" id="popular-groups" <?php if ( $instance['group_default'] == 'popular' ) : ?> class="selected"<?php endif; ?>><?php _e("Popular", 'buddypress') ?></a>
     132                <a href="<?php bp_groups_directory_url(); ?>" id="popular-groups" <?php if ( $instance['group_default'] == 'popular' ) : ?> class="selected"<?php endif; ?>><?php _e("Popular", 'buddypress') ?></a>
    133133                <span class="bp-separator" role="separator"><?php echo esc_html( $separator ); ?></span>
    134                 <a href="<?php bp_groups_directory_permalink(); ?>" id="alphabetical-groups" <?php if ( $instance['group_default'] == 'alphabetical' ) : ?> class="selected"<?php endif; ?>><?php _e("Alphabetical", 'buddypress') ?></a>
     134                <a href="<?php bp_groups_directory_url(); ?>" id="alphabetical-groups" <?php if ( $instance['group_default'] == 'alphabetical' ) : ?> class="selected"<?php endif; ?>><?php _e("Alphabetical", 'buddypress') ?></a>
    135135            </div>
    136136
     
    139139                    <li <?php bp_group_class(); ?>>
    140140                        <div class="item-avatar">
    141                             <a href="<?php bp_group_permalink() ?>" class="bp-tooltip" data-bp-tooltip="<?php bp_group_name() ?>"><?php bp_group_avatar_thumb() ?></a>
     141                            <a href="<?php bp_group_url() ?>" class="bp-tooltip" data-bp-tooltip="<?php bp_group_name() ?>"><?php bp_group_avatar_thumb() ?></a>
    142142                        </div>
    143143
  • trunk/src/bp-groups/screens/single/request-membership.php

    r12719 r13437  
    3333        }
    3434
    35         bp_core_redirect( bp_get_group_permalink( $bp->groups->current_group ) );
     35        bp_core_redirect( bp_get_group_url( $bp->groups->current_group ) );
    3636    }
    3737
     
    6060            bp_core_add_message( __( 'Your membership request was sent to the group administrator successfully. You will be notified when the group administrator responds to your request.', 'buddypress' ) );
    6161        }
    62         bp_core_redirect( bp_get_group_permalink( $bp->groups->current_group ) );
     62        bp_core_redirect( bp_get_group_url( $bp->groups->current_group ) );
    6363    }
    6464
  • trunk/src/bp-groups/screens/single/send-invites.php

    r12433 r13437  
    4343         */
    4444        do_action( 'groups_screen_group_invite', $bp->groups->current_group->id );
    45         bp_core_redirect( bp_get_group_permalink( $bp->groups->current_group ) );
     45        bp_core_redirect( bp_get_group_url( $bp->groups->current_group ) );
    4646
    4747    } elseif ( !bp_action_variable( 0 ) ) {
  • trunk/src/bp-members/classes/class-bp-members-component.php

    r13436 r13437  
    788788
    789789    /**
    790      * Add the Registration and Activation rewrite tags.
     790     * Adds the Members directory type, Registration and Activation rewrite tags.
    791791     *
    792792     * @since 12.0.0
     
    797797    public function add_rewrite_tags( $rewrite_tags = array() ) {
    798798        $rewrite_tags = array(
     799            'directory_type'      => '([^/]+)',
    799800            'member_register'     => '([1]{1,})',
    800801            'member_activate'     => '([1]{1,})',
     
    806807
    807808    /**
    808      * Add the Registration and Activation rewrite rules.
     809     * Adds the Registration and Activation rewrite rules.
    809810     *
    810811     * @since 12.0.0
     
    841842
    842843    /**
    843      * Add the Registration and Activation permastructs.
     844     * Adds the Registration and Activation permastructs.
    844845     *
    845846     * @since 12.0.0
  • trunk/src/bp-templates/bp-legacy/buddypress-functions.php

    r13395 r13437  
    16081608                _e( 'Error requesting membership', 'buddypress' );
    16091609            } else {
    1610                 echo '<a id="group-' . esc_attr( $group->id ) . '" class="group-button disabled pending membership-requested" rel="membership-requested" href="' . bp_get_group_permalink( $group ) . '">' . __( 'Request Sent', 'buddypress' ) . '</a>';
     1610                echo '<a id="group-' . esc_attr( $group->id ) . '" class="group-button disabled pending membership-requested" rel="membership-requested" href="' . esc_url( bp_get_group_url( $group ) ) . '">' . __( 'Request Sent', 'buddypress' ) . '</a>';
    16111611            }
    16121612        break;
  • trunk/src/bp-templates/bp-legacy/buddypress/groups/groups-loop.php

    r12791 r13437  
    77 * @package BuddyPress
    88 * @subpackage bp-legacy
    9  * @version 3.0.0
     9 * @version 12.0.0
    1010 */
    1111
     
    5959            <?php if ( ! bp_disable_group_avatar_uploads() ) : ?>
    6060                <div class="item-avatar">
    61                     <a href="<?php bp_group_permalink(); ?>"><?php bp_group_avatar( 'type=thumb&width=50&height=50' ); ?></a>
     61                    <a href="<?php bp_group_url(); ?>"><?php bp_group_avatar( 'type=thumb&width=50&height=50' ); ?></a>
    6262                </div>
    6363            <?php endif; ?>
  • trunk/src/bp-templates/bp-legacy/buddypress/groups/index.php

    r12595 r13437  
    55 * @package BuddyPress
    66 * @subpackage bp-legacy
    7  * @version 3.0.0
     7 * @version 12.0.0
    88 */
    99
     
    6161            <ul>
    6262                <li class="selected" id="groups-all">
    63                     <a href="<?php bp_groups_directory_permalink(); ?>">
     63                    <a href="<?php bp_groups_directory_url(); ?>">
    6464                        <?php
    6565                        /* translators: %s: all groups count */
  • trunk/src/bp-templates/bp-legacy/buddypress/groups/single/cover-image-header.php

    r12791 r13437  
    55 * @package BuddyPress
    66 * @subpackage bp-legacy
    7  * @version 3.0.0
     7 * @version 12.0.0
    88 */
    99
     
    1616
    1717<div id="cover-image-container">
    18     <a id="header-cover-image" href="<?php echo esc_url( bp_get_group_permalink() ); ?>"></a>
     18    <a id="header-cover-image" href="<?php bp_group_url(); ?>"></a>
    1919
    2020    <div id="item-header-cover-image">
    2121        <?php if ( ! bp_disable_group_avatar_uploads() ) : ?>
    2222            <div id="item-header-avatar">
    23                 <a href="<?php echo esc_url( bp_get_group_permalink() ); ?>">
     23                <a href="<?php bp_group_url(); ?>">
    2424
    2525                    <?php bp_group_avatar(); ?>
  • trunk/src/bp-templates/bp-legacy/buddypress/groups/single/group-header.php

    r12791 r13437  
    55 * @package BuddyPress
    66 * @subpackage bp-legacy
    7  * @version 3.0.0
     7 * @version 12.0.0
    88 */
    99
     
    6060<?php if ( ! bp_disable_group_avatar_uploads() ) : ?>
    6161    <div id="item-header-avatar">
    62         <a href="<?php echo esc_url( bp_get_group_permalink() ); ?>" class="bp-tooltip" data-bp-tooltip="<?php echo esc_attr( bp_get_group_name() ); ?>">
     62        <a href="<?php bp_group_url(); ?>" class="bp-tooltip" data-bp-tooltip="<?php echo esc_attr( bp_get_group_name() ); ?>">
    6363
    6464            <?php bp_group_avatar(); ?>
  • trunk/src/bp-templates/bp-legacy/buddypress/members/single/groups/invites.php

    r13273 r13437  
    55 * @package BuddyPress
    66 * @subpackage bp-legacy
    7  * @version 11.0.0
     7 * @version 12.0.0
    88 */
    99
     
    2929                <?php if ( ! bp_disable_group_avatar_uploads() ) : ?>
    3030                    <div class="item-avatar">
    31                         <a href="<?php bp_group_permalink(); ?>"><?php bp_group_avatar( 'type=thumb&width=50&height=50' ); ?></a>
     31                        <a href="<?php bp_group_url(); ?>"><?php bp_group_avatar( 'type=thumb&width=50&height=50' ); ?></a>
    3232                    </div>
    3333                <?php endif; ?>
  • trunk/src/bp-templates/bp-nouveau/buddypress/groups/groups-loop.php

    r12791 r13437  
    44 *
    55 * @since 3.0.0
    6  * @version 7.0.0
     6 * @version 12.0.0
    77 */
    88
     
    2626                    <?php if ( ! bp_disable_group_avatar_uploads() ) : ?>
    2727                        <div class="item-avatar">
    28                             <a href="<?php bp_group_permalink(); ?>"><?php bp_group_avatar( bp_nouveau_avatar_args() ); ?></a>
     28                            <a href="<?php bp_group_url(); ?>"><?php bp_group_avatar( bp_nouveau_avatar_args() ); ?></a>
    2929                        </div>
    3030                    <?php endif; ?>
  • trunk/src/bp-templates/bp-nouveau/buddypress/groups/single/cover-image-header.php

    r12801 r13437  
    44 *
    55 * @since 3.0.0
    6  * @version 7.0.0
     6 * @version 12.0.0
    77 */
    88?>
     
    1414        <?php if ( ! bp_disable_group_avatar_uploads() ) : ?>
    1515            <div id="item-header-avatar">
    16                 <a href="<?php echo esc_url( bp_get_group_permalink() ); ?>" title="<?php echo esc_attr( bp_get_group_name() ); ?>">
     16                <a href="<?php bp_group_url(); ?>" title="<?php echo esc_attr( bp_get_group_name() ); ?>">
    1717
    1818                    <?php bp_group_avatar(); ?>
  • trunk/src/bp-templates/bp-nouveau/buddypress/groups/single/group-header.php

    r12801 r13437  
    44 *
    55 * @since 3.0.0
    6  * @version 7.0.0
     6 * @version 12.0.0
    77 */
    88?>
     
    1212<?php if ( ! bp_disable_group_avatar_uploads() ) : ?>
    1313    <div id="item-header-avatar">
    14         <a href="<?php echo esc_url( bp_get_group_permalink() ); ?>" class="bp-tooltip" data-bp-tooltip="<?php echo esc_attr( bp_get_group_name() ); ?>">
     14        <a href="<?php bp_group_url(); ?>" class="bp-tooltip" data-bp-tooltip="<?php echo esc_attr( bp_get_group_name() ); ?>">
    1515
    1616            <?php bp_group_avatar(); ?>
  • trunk/src/bp-templates/bp-nouveau/buddypress/members/single/groups/invites.php

    r13273 r13437  
    44 *
    55 * @since 3.0.0
    6  * @version 11.0.0
     6 * @version 12.0.0
    77 */
    88?>
     
    2727                <?php if ( ! bp_disable_group_avatar_uploads() ) : ?>
    2828                    <div class="item-avatar">
    29                         <a href="<?php bp_group_permalink(); ?>"><?php bp_group_avatar(); ?></a>
     29                        <a href="<?php bp_group_url(); ?>"><?php bp_group_avatar(); ?></a>
    3030                    </div>
    3131                <?php endif; ?>
  • trunk/src/bp-templates/bp-nouveau/includes/groups/functions.php

    r13433 r13437  
    562562        'slug'      => 'all', // slug is used because BP_Core_Nav requires it, but it's the scope
    563563        'li_class'  => array( 'selected' ),
    564         'link'      => bp_get_groups_directory_permalink(),
     564        'link'      => bp_get_groups_directory_url(),
    565565        'text'      => __( 'All Groups', 'buddypress' ),
    566566        'count'     => bp_get_total_group_count(),
     
    591591                'slug'      => 'create', // slug is used because BP_Core_Nav requires it, but it's the scope
    592592                'li_class'  => array( 'no-ajax', 'group-create', 'create-button' ),
    593                 'link'      => trailingslashit( bp_get_groups_directory_permalink() . 'create' ),
     593                'link'      => bp_get_groups_directory_url(
     594                    array(
     595                        'create_single_item' => 1,
     596                    )
     597                ),
    594598                'text'      => __( 'Create a Group', 'buddypress' ),
    595599                'count'     => false,
  • trunk/src/bp-templates/bp-nouveau/includes/groups/template-tags.php

    r13310 r13437  
    251251
    252252    return (int) bp_get_user_meta( $user_id, '_bp_nouveau_restrict_invites_to_friends' );
    253 }
    254 
    255 /**
    256  * Outputs the group creation numbered steps navbar
    257  *
    258  * @since 3.0.0
    259  *
    260  * @todo This output isn't localised correctly.
    261  */
    262 function bp_nouveau_group_creation_tabs() {
    263     $bp = buddypress();
    264 
    265     if ( ! is_array( $bp->groups->group_creation_steps ) ) {
    266         return;
    267     }
    268 
    269     if ( ! bp_get_groups_current_create_step() ) {
    270         $keys                            = array_keys( $bp->groups->group_creation_steps );
    271         $bp->groups->current_create_step = array_shift( $keys );
    272     }
    273 
    274     $counter = 1;
    275 
    276     foreach ( (array) $bp->groups->group_creation_steps as $slug => $step ) {
    277         $is_enabled = bp_are_previous_group_creation_steps_complete( $slug ); ?>
    278 
    279         <li<?php if ( bp_get_groups_current_create_step() === $slug ) : ?> class="current"<?php endif; ?>>
    280             <?php if ( $is_enabled ) : ?>
    281                 <a href="<?php echo esc_url( bp_groups_directory_permalink() . 'create/step/' . $slug . '/' ); ?>">
    282                     <?php echo (int) $counter; ?> <?php echo esc_html( $step['name'] ); ?>
    283                 </a>
    284             <?php else : ?>
    285                 <?php echo (int) $counter; ?>. <?php echo esc_html( $step['name'] ); ?>
    286             <?php endif ?>
    287         </li>
    288             <?php
    289         $counter++;
    290     }
    291 
    292     unset( $is_enabled );
    293 
    294     /**
    295      * Fires at the end of the creation of the group tabs.
    296      *
    297      * @since 1.0.0
    298      */
    299     do_action( 'groups_creation_tabs' );
    300253}
    301254
  • trunk/src/bp-templates/bp-nouveau/includes/template-tags.php

    r13433 r13437  
    22642264
    22652265    } elseif ( bp_is_group() ) {
    2266         $url = rawurlencode( bp_get_group_permalink( groups_get_current_group() ) );
     2266        $url = rawurlencode( bp_get_group_url( groups_get_current_group() ) );
    22672267
    22682268    } elseif ( ! empty( $r['object'] ) && ! empty( $r['item_id'] ) ) {
     
    22742274
    22752275            if ( ! empty( $group->id ) ) {
    2276                 $url = rawurlencode( bp_get_group_permalink( $group ) );
     2276                $url = rawurlencode( bp_get_group_url( $group ) );
    22772277            }
    22782278        }
  • trunk/tests/phpunit/testcases/core/nav/backCompat.php

    r13436 r13437  
    5555
    5656        $group = groups_get_group( $g );
    57         $group_permalink = bp_get_group_permalink( $group );
     57        $group_permalink = bp_get_group_url( $group );
    5858
    5959        $this->go_to( $group_permalink );
     
    303303            'slug' => 'foo-subnav',
    304304            'parent_slug' => bp_get_current_group_slug(),
    305             'parent_url' => bp_get_group_permalink( groups_get_current_group() ),
     305            'parent_url' => bp_get_group_url( groups_get_current_group() ),
    306306            'screen_function' => 'foo_subnav',
    307307        ) );
  • trunk/tests/phpunit/testcases/core/nav/bpCoreMaybeHookNewSubnavScreenFunction.php

    r13436 r13437  
    171171        $old_current_user = get_current_user_id();
    172172        $this->set_current_user( $u );
     173        $this->set_permalink_structure( '/%postname%/' );
    173174
    174175        $group = groups_get_group( $g );
    175176
    176         $this->go_to( bp_get_group_permalink( $group ) );
    177 
    178         $subnav_item = array(
    179             'user_has_access' => false,
    180             'no_access_url' => bp_get_group_permalink( $group ),
    181         );
    182 
    183         // Just test relevant info
    184         $found = bp_core_maybe_hook_new_subnav_screen_function( $subnav_item );
    185         $this->assertSame( 'failure', $found['status'] );
    186         $this->assertSame( bp_get_group_permalink( $group ), $found['redirect_args']['root'] );
     177        $this->go_to( bp_get_group_url( $group ) );
     178
     179        $subnav_item = array(
     180            'user_has_access' => false,
     181            'no_access_url' => bp_get_group_url( $group ),
     182        );
     183
     184        // Just test relevant info
     185        $found = bp_core_maybe_hook_new_subnav_screen_function( $subnav_item );
     186        $this->assertSame( 'failure', $found['status'] );
     187        $this->assertSame( bp_get_group_url( $group ), $found['redirect_args']['root'] );
    187188
    188189        // Clean up
     
    198199        $group = groups_get_group( $g );
    199200
    200         $this->go_to( bp_get_group_permalink( $group ) );
     201        $this->go_to( bp_get_group_url( $group ) );
    201202
    202203        $subnav_item = array(
  • trunk/tests/phpunit/testcases/core/nav/bpCoreNewNavItem.php

    r13436 r13437  
    7878        $group = groups_get_group( $g );
    7979
    80         $this->go_to( bp_get_group_permalink( $group ) );
     80        $this->go_to( bp_get_group_url( $group ) );
    8181
    8282        $this->assertTrue( buddypress()->bp_nav[ $group->slug ]['position'] === -1 );
  • trunk/tests/phpunit/testcases/core/nav/bpCoreRemoveSubnavItem.php

    r11737 r13437  
    7777        // In group context
    7878        $g_obj = groups_get_group( $g1 );
    79         $this->go_to( bp_get_group_permalink( $g_obj ) );
     79        $this->go_to( bp_get_group_url( $g_obj ) );
    8080
    8181        bp_core_new_subnav_item( array(
     
    8383            'slug' => 'clam',
    8484            'parent_slug' => bp_get_current_group_slug(),
    85             'parent_url' => bp_get_group_permalink( $g_obj ),
     85            'parent_url' => bp_get_group_url( $g_obj ),
    8686            'screen_function' => 'clam_subnav',
    8787        ) );
  • trunk/tests/phpunit/testcases/groups/activity.php

    r12395 r13437  
    2525        $g_obj = groups_get_group( $g );
    2626
    27         $expected = sprintf( __( '%s created the group %s', 'buddypress' ), bp_core_get_userlink( $u ),  '<a href="' . bp_get_group_permalink( $g_obj ) . '">' . $g_obj->name . '</a>' );
     27        $expected = sprintf( __( '%s created the group %s', 'buddypress' ), bp_core_get_userlink( $u ),  '<a href="' . esc_url( bp_get_group_url( $g_obj ) ) . '">' . $g_obj->name . '</a>' );
    2828
    2929        $this->assertSame( $expected, $a_obj->action );
     
    4747        $g_obj = groups_get_group( $g );
    4848
    49         $expected = sprintf( __( '%s joined the group %s', 'buddypress' ), bp_core_get_userlink( $u ),  '<a href="' . bp_get_group_permalink( $g_obj ) . '">' . $g_obj->name . '</a>' );
     49        $expected = sprintf( __( '%s joined the group %s', 'buddypress' ), bp_core_get_userlink( $u ),  '<a href="' . esc_url( bp_get_group_url( $g_obj ) ) . '">' . $g_obj->name . '</a>' );
    5050
    5151        $this->assertSame( $expected, $a_obj->action );
     
    124124        $this->assertNotEmpty( $a['activities'] );
    125125
    126         $expected = sprintf( esc_html__( '%s changed the name of the group %s from "%s" to "%s"', 'buddypress' ), bp_core_get_userlink( $u ),  '<a href="' . bp_get_group_permalink( $group ) . '">Foo</a>', $group->name, 'Foo' );
     126        $expected = sprintf( esc_html__( '%s changed the name of the group %s from "%s" to "%s"', 'buddypress' ), bp_core_get_userlink( $u ),  '<a href="' . esc_url( bp_get_group_url( $group ) ) . '">Foo</a>', $group->name, 'Foo' );
    127127        $this->assertSame( $expected, $a['activities'][0]->action );
    128128
     
    156156        $this->assertNotEmpty( $a['activities'] );
    157157
    158         $expected = sprintf( esc_html__( '%s changed the description of the group %s from "%s" to "%s"', 'buddypress' ), bp_core_get_userlink( $u ),  '<a href="' . bp_get_group_permalink( $group ) . '">' . $group->name . '</a>', $group->description, 'Bar' );
     158        $expected = sprintf( esc_html__( '%s changed the description of the group %s from "%s" to "%s"', 'buddypress' ), bp_core_get_userlink( $u ),  '<a href="' . esc_url( bp_get_group_url( $group ) ) . '">' . $group->name . '</a>', $group->description, 'Bar' );
    159159        $this->assertSame( $expected, $a['activities'][0]->action );
    160160
     
    189189        $this->assertNotEmpty( $a['activities'] );
    190190
    191         $expected = sprintf( __( '%s changed the permalink of the group %s.', 'buddypress' ), bp_core_get_userlink( $u ),  '<a href="' . bp_get_group_permalink( $new_group_details ) . '">' . $group->name . '</a>' );
     191        $expected = sprintf( __( '%s changed the permalink of the group %s.', 'buddypress' ), bp_core_get_userlink( $u ),  '<a href="' . esc_url( bp_get_group_url( $new_group_details ) ) . '">' . $group->name . '</a>' );
    192192        $this->assertSame( $expected, $a['activities'][0]->action );
    193193
     
    221221        $this->assertNotEmpty( $a['activities'] );
    222222
    223         $expected = sprintf( __( '%s changed the name and description of the group %s', 'buddypress' ), bp_core_get_userlink( $u ),  '<a href="' . bp_get_group_permalink( $group ) . '">Foo</a>' );
     223        $expected = sprintf( __( '%s changed the name and description of the group %s', 'buddypress' ), bp_core_get_userlink( $u ),  '<a href="' . esc_url( bp_get_group_url( $group ) ) . '">Foo</a>' );
    224224        $this->assertSame( $expected, $a['activities'][0]->action );
    225225
     
    244244        $g_obj = groups_get_group( $g );
    245245
    246         $expected = sprintf( esc_html__( '%1$s posted an update in the group %2$s', 'buddypress' ), bp_core_get_userlink( $u ),  '<a href="' . esc_url( bp_get_group_permalink( $g_obj ) ) . '">' . esc_html( $g_obj->name ) . '</a>' );
     246        $expected = sprintf( esc_html__( '%1$s posted an update in the group %2$s', 'buddypress' ), bp_core_get_userlink( $u ),  '<a href="' . esc_url( bp_get_group_url( $g_obj ) ) . '">' . esc_html( $g_obj->name ) . '</a>' );
    247247
    248248        $this->assertSame( $expected, $a_obj->action );
  • trunk/tests/phpunit/testcases/groups/class-bp-group-extension.php

    r13436 r13437  
    242242        $e = new $class_name();
    243243
    244         $this->go_to( bp_get_group_permalink( $g_obj ) );
     244        $this->go_to( bp_get_group_url( $g_obj ) );
    245245
    246246        $e->_register();
     
    266266        $e = new $class_name();
    267267
    268         $this->go_to( bp_get_group_permalink( $g_obj ) );
     268        $this->go_to( bp_get_group_url( $g_obj ) );
    269269
    270270        $e->_register();
     
    295295        // Test as non-logged-in user
    296296        $this->set_current_user( 0 );
    297         $this->go_to( bp_get_group_permalink( $g_obj ) );
     297        $this->go_to( bp_get_group_url( $g_obj ) );
    298298        $e->_register();
    299299        $this->assertFalse( isset( buddypress()->bp_options_nav[ $g_obj->slug ][ $e->slug ] ) );
     
    306306        $this->set_current_user( $u );
    307307        $this->add_user_to_group( $u, $g );
    308         $this->go_to( bp_get_group_permalink( $g_obj ) );
     308        $this->go_to( bp_get_group_url( $g_obj ) );
    309309        $e->_register();
    310310        $this->assertTrue( isset( buddypress()->bp_options_nav[ $g_obj->slug ][ $e->slug ] ) );
     
    340340        // Test as non-logged-in user
    341341        $this->set_current_user( 0 );
    342         $this->go_to( bp_get_group_permalink( $g_obj ) );
     342        $this->go_to( bp_get_group_url( $g_obj ) );
    343343        $e->_register();
    344344        $this->assertTrue( isset( buddypress()->bp_options_nav[ $g_obj->slug ][ $e->slug ] ) );
     
    351351        $this->set_current_user( $u );
    352352        $this->add_user_to_group( $u, $g );
    353         $this->go_to( bp_get_group_permalink( $g_obj ) );
     353        $this->go_to( bp_get_group_url( $g_obj ) );
    354354        $e->_register();
    355355        $this->assertTrue( isset( buddypress()->bp_options_nav[ $g_obj->slug ][ $e->slug ] ) );
     
    371371        ) );
    372372        $g_obj = groups_get_group( $g );
    373         $this->go_to( bp_get_group_permalink( $g_obj ) );
     373        $this->go_to( bp_get_group_url( $g_obj ) );
    374374
    375375        $this->set_current_user( 0 );
     
    398398        ) );
    399399        $g_obj = groups_get_group( $g );
    400         $this->go_to( bp_get_group_permalink( $g_obj ) );
     400        $this->go_to( bp_get_group_url( $g_obj ) );
    401401
    402402        $e1 = new BPTest_Group_Extension_Access_Anyone();
     
    441441        $this->set_current_user( $u );
    442442
    443         $this->go_to( bp_get_group_permalink( $g_obj ) );
     443        $this->go_to( bp_get_group_url( $g_obj ) );
    444444
    445445        $e1 = new BPTest_Group_Extension_Access_Anyone();
     
    486486        $this->add_user_to_group( $u, $g );
    487487
    488         $this->go_to( bp_get_group_permalink( $g_obj ) );
     488        $this->go_to( bp_get_group_url( $g_obj ) );
    489489
    490490        $e1 = new BPTest_Group_Extension_Access_Anyone();
     
    533533        $gm->promote( 'mod' );
    534534
    535         $this->go_to( bp_get_group_permalink( $g_obj ) );
     535        $this->go_to( bp_get_group_url( $g_obj ) );
    536536
    537537        $e1 = new BPTest_Group_Extension_Access_Anyone();
     
    580580        $gm->promote( 'admin' );
    581581
    582         $this->go_to( bp_get_group_permalink( $g_obj ) );
     582        $this->go_to( bp_get_group_url( $g_obj ) );
    583583
    584584        $e1 = new BPTest_Group_Extension_Access_Anyone();
     
    622622        $this->set_current_user( 0 );
    623623
    624         $this->go_to( bp_get_group_permalink( $g_obj ) );
     624        $this->go_to( bp_get_group_url( $g_obj ) );
    625625
    626626        $e1 = new BPTest_Group_Extension_Access_Anyone();
     
    664664        $this->set_current_user( 0 );
    665665
    666         $this->go_to( bp_get_group_permalink( $g_obj ) );
     666        $this->go_to( bp_get_group_url( $g_obj ) );
    667667
    668668        $e1 = new BPTest_Group_Extension_ShowTab_Anyone();
     
    707707        $this->set_current_user( $u );
    708708
    709         $this->go_to( bp_get_group_permalink( $g_obj ) );
     709        $this->go_to( bp_get_group_url( $g_obj ) );
    710710
    711711        $e1 = new BPTest_Group_Extension_ShowTab_Anyone();
     
    752752        $this->add_user_to_group( $u, $g );
    753753
    754         $this->go_to( bp_get_group_permalink( $g_obj ) );
     754        $this->go_to( bp_get_group_url( $g_obj ) );
    755755
    756756        $e1 = new BPTest_Group_Extension_ShowTab_Anyone();
     
    799799        $gm->promote( 'mod' );
    800800
    801         $this->go_to( bp_get_group_permalink( $g_obj ) );
     801        $this->go_to( bp_get_group_url( $g_obj ) );
    802802
    803803        $e1 = new BPTest_Group_Extension_ShowTab_Anyone();
     
    846846        $gm->promote( 'admin' );
    847847
    848         $this->go_to( bp_get_group_permalink( $g_obj ) );
     848        $this->go_to( bp_get_group_url( $g_obj ) );
    849849
    850850        $e1 = new BPTest_Group_Extension_ShowTab_Anyone();
     
    885885        $g_obj = groups_get_group( $g );
    886886
    887         $this->go_to( bp_get_group_permalink( $g_obj ) );
     887        $this->go_to( bp_get_group_url( $g_obj ) );
    888888
    889889        $e1 = new BPTest_Group_Extension_Widget_Method();
     
    907907        $g_obj = groups_get_group( $g );
    908908
    909         $this->go_to( trailingslashit( bp_get_group_permalink( $g_obj ) ) . 'members/' );
     909        $this->go_to(
     910            bp_get_group_url(
     911                $g_obj,
     912                array(
     913                    'single_item_component' => 'members',
     914                )
     915            )
     916        );
    910917
    911918        $e1 = new BPTest_Group_Extension_Widget_Method();
     
    939946        bp_register_group_extension( $f_class_name );
    940947
    941         $this->go_to( bp_get_group_permalink( $g_obj ) );
     948        $this->go_to( bp_get_group_url( $g_obj ) );
    942949        $nav = buddypress()->groups->nav->get_secondary( array(
    943950            'parent_slug' => $g_obj->slug ,
  • trunk/tests/phpunit/testcases/groups/class-bp-groups-member.php

    r13436 r13437  
    259259        // In group context
    260260        $g_obj = groups_get_group( $g );
    261         $this->go_to( bp_get_group_permalink( $g_obj ) );
     261        $this->go_to( bp_get_group_url( $g_obj ) );
    262262        groups_update_groupmeta( $g, 'invite_status', 'mods' );
    263263        $this->assertFalse( bp_groups_user_can_send_invites( null, $u_nonmembers ) );
  • trunk/tests/phpunit/testcases/groups/template.php

    r13414 r13437  
    66#[AllowDynamicProperties]
    77class BP_Tests_Groups_Template extends BP_UnitTestCase {
     8    protected $groups_template = null;
    89
    910    public function set_up() {
     
    10061007        $g   = $this->factory->group->create();
    10071008        $p   = 2;
    1008         $url = trailingslashit( bp_get_group_permalink( $g ) . $p );
     1009        $url = bp_get_group_url(
     1010            $g,
     1011            array(
     1012                'single_item_action' => $p,
     1013            )
     1014        );
    10091015
    10101016        $this->assertSame( bp_get_group_form_action( $p, $g ), $url );
Note: See TracChangeset for help on using the changeset viewer.