Skip to:
Content

bbPress.org

Changeset 5815


Ignore:
Timestamp:
07/13/2015 01:13:46 PM (11 years ago)
Author:
johnjamesjacoby
Message:

Users: Use bbp_get_user_profile_url() to avoid code duplication.

This commit removes duplicated code used to build user profile URLs & eliminates several superfluous string replacements. It works with pretty & unpretty permalinks, and plays nicely with the BuddyPress extension for linking to a member's forum sections.

Fixes #2831.

File:
1 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/includes/users/template.php

    r5770 r5815  
    354354
    355355                // Allow early overriding of the profile URL to cut down on processing
    356                 $early_profile_url = apply_filters( 'bbp_pre_get_user_profile_url', (int) $user_id );
     356                $early_profile_url = apply_filters( 'bbp_pre_get_user_profile_url', $user_id );
    357357                if ( is_string( $early_profile_url ) ) {
    358358                        return $early_profile_url;
    … …  
    361361                // Pretty permalinks
    362362                if ( $wp_rewrite->using_permalinks() ) {
    363                         $url = $wp_rewrite->root . bbp_get_user_slug() . '/%' . bbp_get_user_rewrite_id() . '%';
    364363
    365364                        // Get username if not passed
    … …  
    368367                        }
    369368
    370                         $url = str_replace( '%' . bbp_get_user_rewrite_id() . '%', $user_nicename, $url );
    371                         $url = home_url( user_trailingslashit( $url ) );
     369                        $url = trailingslashit( $wp_rewrite->root . bbp_get_user_slug() ) . $user_nicename;
     370                        $url = user_trailingslashit( $url );
     371                        $url = home_url( $url );
    372372
    373373                // Unpretty permalinks
    374374                } else {
    375                         $url = add_query_arg( array( bbp_get_user_rewrite_id() => $user_id ), home_url( '/' ) );
     375                        $url = add_query_arg( array(
     376                                bbp_get_user_rewrite_id() => $user_id
     377                        ), home_url( '/' ) );
    376378                }
    377379
    … …  
    436438         * @param string $user_nicename Optional. User nicename
    437439         * @uses bbp_get_user_id() To get user id
     440         * @uses bbp_get_user_profile_url() To get the user profile url
    438441         * @uses WP_Rewrite::using_permalinks() To check if the blog is using
    439442         *                                       permalinks
    … …  
    447450                global $wp_rewrite;
    448451
    449                 $bbp     = bbpress();
    450452                $user_id = bbp_get_user_id( $user_id );
    451453                if ( empty( $user_id ) ) {
    … …  
    453455                }
    454456
     457                // Allow early overriding of the profile edit URL to cut down on processing
     458                $early_profile_url = apply_filters( 'bbp_pre_get_user_profile_edit_url', $user_id );
     459                if ( is_string( $early_profile_url ) ) {
     460                        return $early_profile_url;
     461                }
     462
     463                // Get user profile URL
     464                $profile_url = bbp_get_user_profile_url( $user_id, $user_nicename );
     465
    455466                // Pretty permalinks
    456467                if ( $wp_rewrite->using_permalinks() ) {
    457                         $url = $wp_rewrite->root . bbp_get_user_slug() . '/%' . $bbp->user_id . '%/' . $bbp->edit_id;
    458 
    459                         // Get username if not passed
    460                         if ( empty( $user_nicename ) ) {
    461                                 $user = get_userdata( $user_id );
    462                                 if ( ! empty( $user->user_nicename ) ) {
    463                                         $user_nicename = $user->user_nicename;
    464                                 }
    465                         }
    466 
    467                         $url = str_replace( '%' . $bbp->user_id . '%', $user_nicename, $url );
    468                         $url = home_url( user_trailingslashit( $url ) );
     468                        $url = trailingslashit( $profile_url ) . 'edit';
     469                        $url = user_trailingslashit( $url );
     470                        $url = home_url( $url );
    469471
    470472                // Unpretty permalinks
    471473                } else {
    472                         $url = add_query_arg( array( $bbp->user_id => $user_id, $bbp->edit_id => '1' ), home_url( '/' ) );
     474                        $url = add_query_arg( array(
     475                                bbp_get_edit_rewrite_id() => '1'
     476                        ), $profile_url );
    473477                }
    474478
    … …  
    825829
    826830                // Allow early overriding of the profile URL to cut down on processing
    827                 $early_profile_url = apply_filters( 'bbp_pre_get_favorites_permalink', (int) $user_id );
     831                $early_profile_url = apply_filters( 'bbp_pre_get_favorites_permalink', $user_id );
    828832                if ( is_string( $early_profile_url ) ) {
    829833                        return $early_profile_url;
    830834                }
    831835
     836                // Get user profile URL
     837                $profile_url = bbp_get_user_profile_url( $user_id );
     838
    832839                // Pretty permalinks
    833840                if ( $wp_rewrite->using_permalinks() ) {
    834                         $url = $wp_rewrite->root . bbp_get_user_slug() . '/%' . bbp_get_user_rewrite_id() . '%/%' . bbp_get_user_favorites_rewrite_id() . '%';
    835                         $user = get_userdata( $user_id );
    836                         if ( ! empty( $user->user_nicename ) ) {
    837                                 $user_nicename = $user->user_nicename;
    838                         } else {
    839                                 $user_nicename = $user->user_login;
    840                         }
    841                         $url = str_replace( '%' . bbp_get_user_rewrite_id() . '%', $user_nicename, $url );
    842                         $url = str_replace( '%' . bbp_get_user_favorites_rewrite_id() . '%', bbp_get_user_favorites_slug(), $url );
    843                         $url = home_url( user_trailingslashit( $url ) );
     841                        $url = trailingslashit( $profile_url ) . bbp_get_user_favorites_rewrite_id();
     842                        $url = user_trailingslashit( $url );
     843                        $url = home_url( $url );
    844844
    845845                // Unpretty permalinks
    846846                } else {
    847847                        $url = add_query_arg( array(
    848                                 bbp_get_user_rewrite_id()           => $user_id,
    849848                                bbp_get_user_favorites_rewrite_id() => bbp_get_user_favorites_slug(),
    850                         ), home_url( '/' ) );
     849                        ), $profile_url );
    851850                }
    852851
    … …  
    919918
    920919                // No link if you can't edit yourself
    921                 if ( ! current_user_can( 'edit_user', (int) $user_id ) ) {
     920                if ( ! current_user_can( 'edit_user', $user_id ) ) {
    922921                        return false;
    923922                }
    … …  
    990989
    991990                // Allow early overriding of the profile URL to cut down on processing
    992                 $early_profile_url = apply_filters( 'bbp_pre_get_subscriptions_permalink', (int) $user_id );
     991                $early_profile_url = apply_filters( 'bbp_pre_get_subscriptions_permalink', $user_id );
    993992                if ( is_string( $early_profile_url ) ) {
    994993                        return $early_profile_url;
    995994                }
    996995
     996                // Get user profile URL
     997                $profile_url = bbp_get_user_profile_url( $user_id );
     998
    997999                // Pretty permalinks
    9981000                if ( $wp_rewrite->using_permalinks() ) {
    999                         $url  = $wp_rewrite->root . bbp_get_user_slug() . '/%' . bbp_get_user_rewrite_id() . '%/%' . bbp_get_user_subscriptions_rewrite_id() . '%';
    1000                         $user = get_userdata( $user_id );
    1001                         if ( ! empty( $user->user_nicename ) ) {
    1002                                 $user_nicename = $user->user_nicename;
    1003                         } else {
    1004                                 $user_nicename = $user->user_login;
    1005                         }
    1006                         $url = str_replace( '%' . bbp_get_user_rewrite_id()               . '%', $user_nicename,                    $url );
    1007                         $url = str_replace( '%' . bbp_get_user_subscriptions_rewrite_id() . '%', bbp_get_user_subscriptions_slug(), $url );
    1008                         $url = home_url( user_trailingslashit( $url ) );
     1001                        $url = trailingslashit( $profile_url ) . bbp_get_user_subscriptions_slug();
     1002                        $url = user_trailingslashit( $url );
     1003                        $url = home_url( $url );
    10091004
    10101005                // Unpretty permalinks
    10111006                } else {
    10121007                        $url = add_query_arg( array(
    1013                                 bbp_get_user_rewrite_id()           => $user_id,
    10141008                                bbp_get_user_subscriptions_rewrite_id() => bbp_get_user_subscriptions_slug(),
    1015                         ), home_url( '/' ) );
     1009                        ), $profile_url );
    10161010                }
    10171011
    … …  
    10871081
    10881082                // No link if you can't edit yourself
    1089                 if ( ! current_user_can( 'edit_user', (int) $user_id ) ) {
     1083                if ( ! current_user_can( 'edit_user', $user_id ) ) {
    10901084                        return false;
    10911085                }
    … …  
    14361430
    14371431                // Allow early overriding of the profile URL to cut down on processing
    1438                 $early_url = apply_filters( 'bbp_pre_get_user_topics_created_url', (int) $user_id );
     1432                $early_url = apply_filters( 'bbp_pre_get_user_topics_created_url', $user_id );
    14391433                if ( is_string( $early_url ) ) {
    14401434                        return $early_url;
    14411435                }
    14421436
     1437                // Get user profile URL
     1438                $profile_url = bbp_get_user_profile_url( $user_id );
     1439
    14431440                // Pretty permalinks
    14441441                if ( $wp_rewrite->using_permalinks() ) {
    1445                         $url  = $wp_rewrite->root . bbp_get_user_slug() . '/%' . bbp_get_user_rewrite_id() . '%/' . bbp_get_topic_archive_slug();
    1446                         $user = get_userdata( $user_id );
    1447                         if ( ! empty( $user->user_nicename ) ) {
    1448                                 $user_nicename = $user->user_nicename;
    1449                         } else {
    1450                                 $user_nicename = $user->user_login;
    1451                         }
    1452                         $url = str_replace( '%' . bbp_get_user_rewrite_id() . '%', $user_nicename, $url );
    1453                         $url = home_url( user_trailingslashit( $url ) );
     1442                        $url = trailingslashit( $profile_url ) . bbp_get_topic_archive_slug();
     1443                        $url = user_trailingslashit( $url );
     1444                        $url = home_url( $url );
    14541445
    14551446                // Unpretty permalinks
    14561447                } else {
    14571448                        $url = add_query_arg( array(
    1458                                 bbp_get_user_rewrite_id()        => $user_id,
    14591449                                bbp_get_user_topics_rewrite_id() => '1',
    1460                         ), home_url( '/' ) );
     1450                        ), $profile_url );
    14611451                }
    14621452
    … …  
    14981488
    14991489                // Allow early overriding of the profile URL to cut down on processing
    1500                 $early_url = apply_filters( 'bbp_pre_get_user_replies_created_url', (int) $user_id );
     1490                $early_url = apply_filters( 'bbp_pre_get_user_replies_created_url', $user_id );
    15011491                if ( is_string( $early_url ) ) {
    15021492                        return $early_url;
    15031493                }
    15041494
     1495                // Get user profile URL
     1496                $profile_url = bbp_get_user_profile_url( $user_id );
     1497
    15051498                // Pretty permalinks
    15061499                if ( $wp_rewrite->using_permalinks() ) {
    1507                         $url  = $wp_rewrite->root . bbp_get_user_slug() . '/%' . bbp_get_user_rewrite_id() . '%/' . bbp_get_reply_archive_slug();
    1508                         $user = get_userdata( $user_id );
    1509                         if ( ! empty( $user->user_nicename ) ) {
    1510                                 $user_nicename = $user->user_nicename;
    1511                         } else {
    1512                                 $user_nicename = $user->user_login;
    1513                         }
    1514                         $url = str_replace( '%' . bbp_get_user_rewrite_id() . '%', $user_nicename, $url );
    1515                         $url = home_url( user_trailingslashit( $url ) );
     1500                        $url = trailingslashit( $profile_url ) . bbp_get_reply_archive_slug();
     1501                        $url = user_trailingslashit( $url );
     1502                        $url = home_url( $url );
    15161503
    15171504                // Unpretty permalinks
    15181505                } else {
    15191506                        $url = add_query_arg( array(
    1520                                 bbp_get_user_rewrite_id()         => $user_id,
    15211507                                bbp_get_user_replies_rewrite_id() => '1',
    1522                         ), home_url( '/' ) );
     1508                        ), $profile_url );
    15231509                }
    15241510
Note: See TracChangeset for help on using the changeset viewer.