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

Timestamp:

01/14/2021 04:49:17 AM (4 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;
             }