bbPress.org

Is there a way to stop it showing next to the topic title and place it wherever I want? I was thinking of making an extra column to add views in to, instead of having them appear in the title link.

Wittmania, it’s a really easy hack – in fact, if you simply take the code that gets the pageviews and seperate it into a different function, you could call that both from the current function you have to automatically append it and use it as a template tag. If you don’t fancy doing it, is it alright if I quickly hack it for Smurf?

All I did was split display_view_count() into two functions, display_view_count() and get_view_count(). If for example you wanted the views in a seperate column, you’d add the column HTML and then add

<?php $view_count = get_view_count(); echo $view_count; ?>

This is what I replaced display_view_count( $title ) with, here for Smurf’s convenience. You’re about to get mail.

Oh, before I forget – hopefully now that plugins need activation in 1.0-alpha, there’s also an activation hook or the like. That would let you do a lot of preparation first, making the code much easier.

Edit: there is! bb_activate_plugin_XXX. Maybe you can use that – if you want to and figure it out, tell us/make an entry on the wiki.

function display_view_count ($title)

{

global $bbdb, $topic;

$topic_id = $topic->topic_id;



$view_count = get_view_count( $topic_id );



//Builds the text to be appended to the title

$count_display = " <em>($view_count views)</em>"; //This sets the view count var to the existing number of views, if greater than 0

//Makes this plugin play nice with the Page Links plugin by putting the pages (if they exist) on a new line

if (function_exists('page_links_add_links')) {

$count_display .= "n";

}

$title .= $count_display;



return $title;

}



function get_view_count( $topic_id )

{

$view_count = $bbdb->get_var("SELECT meta_value FROM $bbdb->topicmeta WHERE topic_id = $topic_id AND meta_key='views'");



if ( $view_count <= 0 ) {

//If the view count hasn't been initialized...

if ($topic->topic_posts >= 1) {

/* Sets the new record to the number of posts that have been made in a topic */

$view_count = $topic->topic_posts; //sets the view_count number

} else {

$view_count = 0; //can't think of a time when this would be necessary (phantom topics???)

}



//Adds the record to the DB so it isn't zero any longer

$bbdb->query("INSERT INTO $bbdb->topicmeta ( meta_id, topic_id, meta_key, meta_value) VALUES ( NULL , $topic_id, 'views', $view_count )");

}



return $view_count;

}

Change:

function get_view_count( $topic_id )

{

$view_count = $bbdb->get_var("SELECT meta_value FROM $bbdb->topicmeta WHERE topic_id = $topic_id AND meta_key='views'");

To:

function get_view_count( $topic_id )

{

global $bbdb, $topic;



$view_count = $bbdb->get_var("SELECT meta_value FROM $bbdb->topicmeta WHERE topic_id = $topic_id AND meta_key='views'");

????

fel64: I’d think that’s the best way to handle it for normal usage.

Personally, I moved that functionality into a memcached module… not entirely useful for regular installs.

The topic metadata is returned with every set of topics returned to bbpress.

In the front page (or any topic page) for example, in the loop it can be found as $topic->views

There is no need for an extra mysql query. It’s possible to make a function that’s called with $topic if you don’t want to do it as simple as above.

I’ve now come up with an extremely easy/fast way to grab and display the views per forum in the forum list on the front page (or forums with sub-forums.

Here’s the plugin:

function forums_views_append($forums) {

global $bbdb; $sum_meta_value="SUM(meta_value)";

$forums_views = $bbdb->get_results(" SELECT $sum_meta_value,forum_id FROM $bbdb->topicmeta LEFT JOIN $bbdb->topics ON $bbdb->topicmeta.topic_id = $bbdb->topics.topic_id  WHERE $bbdb->topicmeta.meta_key='views'  GROUP BY $bbdb->topics.forum_id");

foreach ($forums_views as $forum_views) {$forums[$forum_views->forum_id]->views=$forum_views->$sum_meta_value; }

return $forums;

}

add_filter('get_forums','forums_views_append');

To display the views, edit your front-page.php template and insert a views header and views column like so:

<th><?php _e('Views'); ?></th>

<th><?php _e('Topics'); ?></th>

<th><?php _e('Posts'); ?></th>

.

<td><?php echo $forum->views; ?></td>

<td><?php forum_topics(); ?></td>

<td><?php forum_posts(); ?></td>

Since there is no clean way to store forum meta data right now in bbpress (apparently an oversight) this will run on every time the forums list is displayed. Maybe not such a good idea for very busy forums, even though it’s a single mysql query (nifty eh?).

Eventually I’ll figure out a nice way to store this and only update upon new posts or new topics to reduce overhead (or a cron job perhaps).

i got this error too

_CK_ i could not udrestand what should i do you mean i found

function get_view_count ( $topic_id )

and

function initialize_view_count( $topic_id )

and put this line above them?

if (!$topic_id) return;