Changeset 10577

Timestamp:

01/14/2021 04:49:17 AM (6 years ago)

Author:

dd32

Message:

Support: Add a forum handler for the obsolete /profile/$slug URI which should soon be hitting WordPress.

See #5384.

File:

• sites/trunk/wordpress.org/public_html/wp-content/plugins/support-forums/inc/class-hooks.php (modified) (2 diffs)

sites/trunk/wordpress.org/public_html/wp-content/plugins/support-forums/inc/class-hooks.php

@@ -19 +19 @@
                 add_filter( 'old_slug_redirect_post_id',       array( $this, 'disable_wp_old_slug_redirect' ) );
                 add_action( 'template_redirect',               array( $this, 'redirect_update_php_page' ) );
-                add_action( 'template_redirect',               array( $this, 'redirect_legacy_user_structure' ) );
+                add_action( 'template_redirect',               array( $this, 'redirect_legacy_urls' ) );
                 add_action( 'template_redirect',               array( $this, 'redirect_ask_question_plugin_forum' ) );
                 add_filter( 'wp_insert_post_data',             array( $this, 'set_post_date_gmt_for_pending_posts' ) );
@@ -303 +303 @@
 
         /**
-         * Redirect /users/$id to their new /users/$slug permastructure.
-         */
-        public function redirect_legacy_user_structure() {
-                if ( is_404() && get_query_var( 'bbp_user' ) && is_numeric( get_query_var( 'bbp_user' ) ) ) {
+         * Redirect legacy urls to their new permastructure.
+         *  - /users/$id & /profile/$slug to /users/$slug
+         * 
+         */
+        public function redirect_legacy_urls() {
+                global $wp_query;
+
+                if ( ! is_404() ) {
+                        return;
+                }
+
+                // Legacy /profile/$user route
+                if (
+                        ! empty( $wp_query->query['pagename'] ) &&
+                        preg_match( '!^profile/(?P<user>.+)(/|$)!', $wp_query->query['pagename'], $m )
+                ) {
+                        wp_safe_redirect( home_url( '/users/' . urlencode( $m['user'] ) . '/' ), 301 );
+                        exit;
+                }
+
+                // Legacy /users/$user_id route
+                if (
+                        get_query_var( 'bbp_user' ) &&
+                        is_numeric( get_query_var( 'bbp_user' ) )
+                ) {
                         $user = get_user_by( 'id', (int) get_query_var( 'bbp_user' ) );
                         if ( $user ) {
-                                wp_redirect( home_url( '/users/' . $user->user_nicename . '/' ), 301 );
+                                wp_safe_redirect( home_url( '/users/' . $user->user_nicename . '/' ), 301 );
                                 exit;
                         }