add action hook to forum topic theme li

[dwenaus]

for the rate forum posts plugin i've created it would be helpful to alter forum topic classes, for this i'm requesting a new action hook in the file /bp-default/groups/single/forum/topic.php

on line 29 change this:

<li id="post-<?php bp_the_topic_post_id() ?>">

to something like

<li id="post-<?php bp_the_topic_post_id() ?>" <?php do_action( 'groups_forum_topic_post_class' ) ?> >

or maybe topic_post_li

right now i'm doing it with ajax, which necessitates another useless server call, as pointed out by andrew_1. (thanks :)

[dwenaus]

this diff works and is tested.

[dwenaus]

tested and it works

[dwenaus]

please implement this in 1.2.4

[dwenaus]

while you are at this one, it would make a lot of sense to add action hooks at the top and bottom of this file. They are sorely missing.

[dwenaus]

I've attached a new diff file that incorporates other very needed hooks for this file.
PLEASE include in 1.2.4

[dwenaus]

ignore 2129.diff

[dwenaus]

Replying to dwenaus:

ignore 2129.diff

use 2129_forum_topic.diff instead

[johnjamesjacoby]

Going to move this to 1.3.

Both topics.php and edit.php will need to be normalized, and that's a little too much to put into a point release.

[dwenaus]

PLEASE PLEASE PLEASE do not move this to 1.3. I've been waiting sooooo long for these simple hooks to be added into core.

Soon the email subscription/digest plugin will be very popular when we release it in about a week, and having the hook on line 19 will mean that people don't have to go in and manually add it every upgrade.

likewise the hook on line 33 will mean the rate forum posts plugin can change the look of posts without a useless extra jquery call on every page load.

is there any way you could at least add just the changes on line 19 and 33? and normalize the page later?

and active buddypress developer is begging you, PLEASE :)

[dwenaus]

if adding line 33 is too much, can you at least add line 19?

[dwenaus]

I'm just going to switch this back to 1.2.4 so that someone sees my plea :) I hope that's OK.

[johnjamesjacoby]

(In [2972]) Fixes #2129.