Changeset 4321

Timestamp:

11/03/2012 09:12:24 AM (13 years ago)

Author:

johnjamesjacoby

Message:

Rewrites:

• Introduce unique rewrite tags and rules for user topics and replies.
• Fixes conflict with topic and reply post type rules.
• Fixes bug where user topics and replies pages were not working on some installations.

Location:

trunk

Files:

• bbpress.php (modified) (4 diffs)
• includes/core/functions.php (modified) (1 diff)
• includes/core/template-functions.php (modified) (1 diff)

trunk/bbpress.php

@@ -223 +223 @@
         // Other identifiers
         $this->user_id           = apply_filters( 'bbp_user_id', 'bbp_user' );
+        $this->tops_id           = apply_filters( 'bbp_tops_id', 'bbp_tops' );
+        $this->reps_id           = apply_filters( 'bbp_reps_id', 'bbp_reps' );
         $this->favs_id           = apply_filters( 'bbp_favs_id', 'bbp_favs' );
         $this->subs_id           = apply_filters( 'bbp_subs_id', 'bbp_subs' );
@@ -840 +842 @@
         add_rewrite_tag( '%%' . bbp_get_user_favorites_rewrite_id()     . '%%', '([1]{1,})' ); // User Favorites tag
         add_rewrite_tag( '%%' . bbp_get_user_subscriptions_rewrite_id() . '%%', '([1]{1,})' ); // User Subscriptions tag
+        add_rewrite_tag( '%%' . bbp_get_user_topics_rewrite_id()        . '%%', '([1]{1,})' ); // User Topics Tag
+        add_rewrite_tag( '%%' . bbp_get_user_replies_rewrite_id()       . '%%', '([1]{1,})' ); // User Replies Tag
     }
 
@@ -860 +864 @@
 
         // Unique rewrite ID's
-        $edit_id    = bbp_get_edit_rewrite_id();
-        $view_id    = bbp_get_view_rewrite_id();
-        $user_id    = bbp_get_user_rewrite_id();
-        $favs_id    = bbp_get_user_favorites_rewrite_id();
-        $subs_id    = bbp_get_user_subscriptions_rewrite_id();
-        $topics_id  = bbp_get_topic_post_type();
-        $replies_id = bbp_get_reply_post_type();
+        $edit_id = bbp_get_edit_rewrite_id();
+        $view_id = bbp_get_view_rewrite_id();
+        $user_id = bbp_get_user_rewrite_id();
+        $favs_id = bbp_get_user_favorites_rewrite_id();
+        $subs_id = bbp_get_user_subscriptions_rewrite_id();
+        $tops_id = bbp_get_user_topics_rewrite_id();
+        $reps_id = bbp_get_user_replies_rewrite_id();
 
         // Rewrite rule matches used repeatedly below
@@ -895 +899 @@
 
             // User Pagination|Edit|View
-            $user_slug . $tops_page_rule => 'index.php?' . $user_id  . '=' . $wp_rewrite->preg_index( 1 ) . '&' . $topics_id  . '=1&paged=' . $wp_rewrite->preg_index( 2 ),
-            $user_slug . $reps_page_rule => 'index.php?' . $user_id  . '=' . $wp_rewrite->preg_index( 1 ) . '&' . $replies_id . '=1&paged=' . $wp_rewrite->preg_index( 2 ),
-            $user_slug . $favs_page_rule => 'index.php?' . $user_id  . '=' . $wp_rewrite->preg_index( 1 ) . '&' . $favs_id    . '=1&paged=' . $wp_rewrite->preg_index( 2 ),
-            $user_slug . $subs_page_rule => 'index.php?' . $user_id  . '=' . $wp_rewrite->preg_index( 1 ) . '&' . $subs_id    . '=1&paged=' . $wp_rewrite->preg_index( 2 ),
-            $user_slug . $tops_rule      => 'index.php?' . $user_id  . '=' . $wp_rewrite->preg_index( 1 ) . '&' . $topics_id  . '=1',
-            $user_slug . $reps_rule      => 'index.php?' . $user_id  . '=' . $wp_rewrite->preg_index( 1 ) . '&' . $replies_id . '=1',
-            $user_slug . $favs_rule      => 'index.php?' . $user_id  . '=' . $wp_rewrite->preg_index( 1 ) . '&' . $favs_id    . '=1',
-            $user_slug . $subs_rule      => 'index.php?' . $user_id  . '=' . $wp_rewrite->preg_index( 1 ) . '&' . $subs_id    . '=1',
-            $user_slug . $edit_rule      => 'index.php?' . $user_id  . '=' . $wp_rewrite->preg_index( 1 ) . '&' . $edit_id    . '=1',
+            $user_slug . $tops_page_rule => 'index.php?' . $user_id  . '=' . $wp_rewrite->preg_index( 1 ) . '&' . $tops_id . '=1&paged=' . $wp_rewrite->preg_index( 2 ),
+            $user_slug . $reps_page_rule => 'index.php?' . $user_id  . '=' . $wp_rewrite->preg_index( 1 ) . '&' . $reps_id . '=1&paged=' . $wp_rewrite->preg_index( 2 ),
+            $user_slug . $favs_page_rule => 'index.php?' . $user_id  . '=' . $wp_rewrite->preg_index( 1 ) . '&' . $favs_id . '=1&paged=' . $wp_rewrite->preg_index( 2 ),
+            $user_slug . $subs_page_rule => 'index.php?' . $user_id  . '=' . $wp_rewrite->preg_index( 1 ) . '&' . $subs_id . '=1&paged=' . $wp_rewrite->preg_index( 2 ),
+            $user_slug . $tops_rule      => 'index.php?' . $user_id  . '=' . $wp_rewrite->preg_index( 1 ) . '&' . $tops_id . '=1',
+            $user_slug . $reps_rule      => 'index.php?' . $user_id  . '=' . $wp_rewrite->preg_index( 1 ) . '&' . $reps_id . '=1',
+            $user_slug . $favs_rule      => 'index.php?' . $user_id  . '=' . $wp_rewrite->preg_index( 1 ) . '&' . $favs_id . '=1',
+            $user_slug . $subs_rule      => 'index.php?' . $user_id  . '=' . $wp_rewrite->preg_index( 1 ) . '&' . $subs_id . '=1',
+            $user_slug . $edit_rule      => 'index.php?' . $user_id  . '=' . $wp_rewrite->preg_index( 1 ) . '&' . $edit_id . '=1',
             $user_slug . $root_rule      => 'index.php?' . $user_id  . '=' . $wp_rewrite->preg_index( 1 ),
 

trunk/includes/core/functions.php

@@ -395 +395 @@
 
 /**
+ * Return the unique ID for user topics rewrite rules
+ *
+ * @since bbPress (r4321)
+ * @return string
+ */
+function bbp_get_user_topics_rewrite_id() {
+    return bbpress()->tops_id;
+}
+
+/**
+ * Return the unique ID for user replies rewrite rules
+ *
+ * @since bbPress (r4321)
+ * @return string
+ */
+function bbp_get_user_replies_rewrite_id() {
+    return bbpress()->reps_id;
+}
+
+/**
  * Return the unique ID for user caps rewrite rules
  *

trunk/includes/core/template-functions.php

@@ -262 +262 @@
         $is_favs    = $posts_query->get( bbp_get_user_favorites_rewrite_id()     );
         $is_subs    = $posts_query->get( bbp_get_user_subscriptions_rewrite_id() );
-        $is_topics  = $posts_query->get( bbp_get_topic_post_type()               );
-        $is_replies = $posts_query->get( bbp_get_reply_post_type()               );
+        $is_topics  = $posts_query->get( bbp_get_user_topics_rewrite_id()        );
+        $is_replies = $posts_query->get( bbp_get_user_replies_rewrite_id()       );
 
         // View or edit?