Changeset 2533 for trunk/bp-forums/bp-forums-templatetags.php

Timestamp:

02/02/2010 12:40:49 PM (16 years ago)

Author:

apeatling

Message:

Removing database hits inside BP loops to improve overall performance.

File:

• trunk/bp-forums/bp-forums-templatetags.php (modified) (3 diffs)

trunk/bp-forums/bp-forums-templatetags.php

@@ -93 +93 @@
         }
 
+        /* Fetch extra information for topics, so we don't have to query inside the loop */
+        $this->topics = bp_forums_get_topic_extras( &$this->topics );
+
         $this->pag_links = paginate_links( array(
             'base' => add_query_arg( array( 'p' => '%#%', 'n' => $this->pag_num ) ),
@@ -348 +351 @@
         global $forum_template;
 
-        if ( !$name = bp_core_get_userlink( $forum_template->topic->topic_last_poster ) )
+        if ( !$domain = bp_core_get_user_domain( $forum_template->topic->topic_last_poster, $forum_template->topic->topic_last_poster_nicename, $forum_template->topic->topic_last_poster_login ) )
             return __( 'Deleted User', 'buddypress' );
 
-        return apply_filters( 'bp_get_the_topic_last_poster_name', $name );
+        return apply_filters( 'bp_get_the_topic_last_poster_name', '<a href="' . $domain . '">' . $forum_template->topic->topic_last_poster_displayname . '</a>' );
     }
 
@@ -387 +390 @@
         extract( $r, EXTR_SKIP );
 
-        return apply_filters( 'bp_get_the_topic_last_poster_avatar', bp_core_fetch_avatar( array( 'item_id' => $forum_template->topic->topic_last_poster, 'type' => $type, 'width' => $width, 'height' => $height ) ) );
+        return apply_filters( 'bp_get_the_topic_last_poster_avatar', bp_core_fetch_avatar( array( 'email' => $forum_template->topic->topic_last_poster_email, 'item_id' => $forum_template->topic->topic_last_poster, 'type' => $type, 'width' => $width, 'height' => $height ) ) );
     }