Changeset 10791

Timestamp:

05/19/2016 06:04:47 PM (9 years ago)

Author:

r-a-y

Message:

URI: Support members shortlink redirection.

This commit redirects /members/me/* to /members/LOGGEDIN_USER_SLUG/*.
If root profiles is enabled, then this redirects /me/* to
/LOGGEDIN_USER_SLUG/*. If a user is logged out, /me/ redirects to the
login page and when properly authenticated, will redirect back to the
logged-in user's page.

This will allow devs to create dynamic links to a user's profile, which is
useful for all sorts of things -- nav menus, support links, etc.

Props r-a-y, dcavins.

Fixes #6325.

Location:

trunk

Files:

• src/bp-core/bp-core-catchuri.php (modified) (1 diff)
• tests/phpunit/testcases/routing/members.php (modified) (1 diff)

trunk/src/bp-core/bp-core-catchuri.php

@@ -534 +534 @@
     }
 }
+
+/**
+ * Members user shortlink redirector.
+ *
+ * Redirects x.com/members/me/* to x.com/members/{LOGGED_IN_USER_SLUG}/*
+ *
+ * @since 2.6.0
+ *
+ * @param string $member_slug The current member slug.
+ */
+function bp_core_members_shortlink_redirector( $member_slug ) {
+    /**
+     * Shortlink slug to redirect to logged-in user.
+     *
+     * x.com/members/me/* will redirect to x.com/members/{LOGGED_IN_USER_SLUG}/*
+     *
+     * @since 2.6.0
+     *
+     * @var string $slug Defaults to 'me'.
+     */
+    $me_slug = apply_filters( 'bp_core_members_shortlink_slug', 'me' );
+
+    // Check if we're on our special shortlink slug. If not, bail.
+    if ( $me_slug !== $member_slug ) {
+        return $member_slug;
+    }
+
+    // If logged out, redirect user to login.
+    if ( false === is_user_logged_in() ) {
+        // Add our login redirector hook.
+        add_action( 'template_redirect', 'bp_core_no_access', 0 );
+
+        return $member_slug;
+    }
+
+    $user = wp_get_current_user();
+
+    return bp_core_get_username( $user->ID, $user->user_nicename, $user->user_login );
+}
+add_filter( 'bp_core_set_uri_globals_member_slug', 'bp_core_members_shortlink_redirector' );
 
 /**

trunk/tests/phpunit/testcases/routing/members.php

@@ -109 +109 @@
         $this->assertTrue( is_404() );
     }
+
+    /**
+     * @ticket BP6325
+     */
+    function test_members_shortlink_redirector() {
+        $shortlink_member_slug = 'me';
+
+        $this->go_to( bp_get_members_directory_permalink() . $shortlink_member_slug );
+
+        $this->assertSame( get_current_user_id(), bp_displayed_user_id() );
+    }
 }