Changeset 4536 for trunk/bp-forums/bp-forums-filters.php

Timestamp:

06/19/2011 05:29:51 PM (15 years ago)

Author:

djpaul

Message:

Rework page <title> generation to use WP's wp_title(). Fixes #2598

File:

• trunk/bp-forums/bp-forums-filters.php (modified) (1 diff)

trunk/bp-forums/bp-forums-filters.php

@@ -95 +95 @@
  *
  * @global object $bp
- * @param string $title
+ * @param string $title New page title; see bp_modify_page_title()
+ * @param string $title Original page title
+ * @param string $sep How to separate the various items within the page title.
+ * @param string $seplocation Direction to display title
  * @return string
- */
-function bp_forums_add_forum_topic_to_page_title( $title ) {
+ * @see bp_modify_page_title()
+ */
+function bp_forums_add_forum_topic_to_page_title( $title, $original_title, $sep, $seplocation  ) {
     global $bp;
 
     if ( $bp->current_action == 'forum' && !empty( $bp->action_variables[0] ) && 'topic' == $bp->action_variables[0] )
         if ( bp_has_forum_topic_posts() )
-            $title .= ' | ' . bp_get_the_topic_title();
+            $title .= bp_get_the_topic_title() . " $sep ";
 
     return $title;
 }
-add_filter( 'bp_page_title', 'bp_forums_add_forum_topic_to_page_title' );
+add_filter( 'bp_modify_page_title', 'bp_forums_add_forum_topic_to_page_title', 9, 4 );
 
 /**