Changeset 3930

Timestamp:

05/31/2012 06:07:19 PM (14 years ago)

Author:

johnjamesjacoby

Message:

Only filter shortcode queries if query is not already set. Also, use correct filter names.

File:

• branches/plugin/bbp-includes/bbp-core-shortcodes.php (modified) (7 diffs)

branches/plugin/bbp-includes/bbp-core-shortcodes.php

@@ -312 +312 @@
         $this->unset_globals();
 
+        // Filter the query
+        if ( ! bbp_is_topic_archive() ) {
+            add_filter( 'bbp_before_has_topics_parse_args', array( $this, 'display_topic_index_query' ) );
+        }
+
         // Start output buffer
         $this->start( 'bbp_topic_archive' );
-
-        // Filter the query
-        add_filter( 'bbp_pre_has_topics_query', array( $this, 'display_topic_index_query' ) );
 
         // Output template
@@ -369 +371 @@
         }
 
+        // Filter the query
+        if ( ! bbp_is_single_topic() ) {
+            add_filter( 'bbp_before_has_replies_parse_args', array( $this, 'display_topic_query' ) );
+        }
+
         // Start output buffer
         $this->start( 'bbp_single_topic' );
@@ -374 +381 @@
         // Check forum caps
         if ( bbp_user_can_view_forum( array( 'forum_id' => $forum_id ) ) ) {
-
-            // Filter the query
-            add_filter( 'bbp_pre_has_replies_query', array( $this, 'display_topic_query' ) );
-
-            // Output template
             bbp_get_template_part( 'content', 'single-topic' );
 
@@ -542 +544 @@
         $this->unset_globals();
 
-        // Start output buffer
-        $this->start( 'bbp_topics_of_tag' );
+        // Filter the query
+        if ( ! bbp_is_topic_tag() ) {
+            add_filter( 'bbp_before_has_topics_parse_args', array( $this, 'display_topics_of_tag_query' ) );
+        }
+
+        // Start output buffer
+        $this->start( 'bbp_topic_tag' );
 
         // Set passed attribute to $ag_id for clarity
         bbpress()->current_topic_tag_id = $tag_id = $attr['id'];
-
-        // Filter the query
-        add_filter( 'bbp_pre_has_topics_query', array( $this, 'display_topics_of_tag_query' ) );
 
         // Output template
@@ -614 +618 @@
         $this->unset_globals();
 
-        // Load the topic index
+        // Load the view
         bbp_view_query( $view_id );
 
@@ -751 +755 @@
      */
     public function display_topic_query( $args = array() ) {
-
         $args['meta_query'] = array( array(
             'key'     => '_bbp_topic_id',
@@ -770 +773 @@
      */
     public function display_topics_of_tag_query( $args = array() ) {
-
         $args['tax_query'] = array( array(
             'taxonomy' => bbp_get_topic_tag_tax_id(),