Remove theme_forum_topic_navigation()

Shouldn't the css be removed too?

There is also an example in system.api.php, I think it should be removed too.

6.x/HEAD/advanced_forum/advanced_forum.module:    foreach ($theme_registry['forum_topic_navigation']['preprocess functions'] as $key => $value) {
6.x/HEAD/advanced_forum/advanced_forum.module:      if ($value = 'template_preprocess_forum_topic_navigation') {
6.x/HEAD/advanced_forum/advanced_forum.module:        unset($theme_registry['forum_topic_navigation']['preprocess functions'][$key]);

is also in system.api.php as example, I'm not sure, whether it should be left in.

I support removing this function.

So here is the reupload.

Interestingly enough, I can't apply this patch so asking for a re-test.

Committed to CVS HEAD. Thanks!

Actually, some forum software does have this, they just don't glom it on to the end of the text like Drupal does. AF 2.x moves it to a more sensible place at the end of the thread but does leave the warning that the query isn't good on large forums. I'm curious, is the query unfixable? I'll likely still offer it as an option in AF in D7 and it would be lovely if the query was better but I understand if you'd rather simply take it out of core rather than fix it.

Michelle

@catch: Ok, will do. Thanks. Would be nice to still have this as an option in AF for folks who like the feature. Less complaints about it being taken out of core, then. :)

Michelle

Why not integrate the source code of this module into core:
http://drupal.org/project/prev_next

The code is available.
Why not use it.

Okay, I am glad this feature was remove.
It was a great overhead for forums like ours.

Bye and thanks.