Changeset 5129 for trunk/bp-forums/bp-forums-functions.php

Timestamp:

09/09/2011 03:06:13 PM (14 years ago)

Author:

boonebgorges

Message:

Reworks the way that total topic counts are retrieved on user forum pages. See #3557

File:

• trunk/bp-forums/bp-forums-functions.php (modified) (4 diffs)

trunk/bp-forums/bp-forums-functions.php

@@ -291 +291 @@
 }
 
-function bp_forums_total_topic_count_for_user( $user_id = 0 ) {
+/**
+ * Get a total "Topics Started" count for a given user
+ *
+ * @package BuddyPress
+ *
+ * @param int $user_id ID of the user being queried. Falls back on displayed user, then loggedin
+ * @param str $type The current filter/sort type. 'active', 'popular', 'unreplied'
+ * @return int $count The topic count
+ */
+function bp_forums_total_topic_count_for_user( $user_id = 0, $type = 'active' ) {
     global $bp;
 
@@ -300 +309 @@
 
     if ( class_exists( 'BB_Query' ) ) {
-        $query = new BB_Query( 'topic', array( 'topic_author_id' => $user_id, 'page' => 1, 'per_page' => -1, 'count' => true ) );
+        $args = array(
+            'topic_author_id' => $user_id,
+            'page'        => 1,
+            'per_page'    => -1,
+            'count'       => true
+        );
+
+        if ( 'unreplied' == $type )
+            $args['post_count'] = 1;
+
+        $query = new BB_Query( 'topic', $args );
         $count = $query->count;
         $query = null;
@@ -321 +340 @@
  * @return int $count
  */
-function bp_forums_total_replied_count_for_user( $user_id = 0 ) {
+function bp_forums_total_replied_count_for_user( $user_id = 0, $type = 'active' ) {
     global $bp;
 
@@ -341 +360 @@
                 $topics[] = $result->topic_id;
         }
-        $count = count( $topics );
+
+        // Even more unfortunate. If this is filtered by 'unreplied', we have to requery
+        if ( 'unreplied' == $type ) {
+            $topic_ids = implode( ',', $topics );
+            $topics_query = new BB_Query( 'topic', array( 'topic_id' => $topic_ids, 'page' => 1, 'per_page' => -1, 'post_count' => 1 ) );
+            $count = count( $topics_query->results );
+        } else {
+            $count = count( $topics );
+        }
         $query = null;
     } else {