[buddypress-trac] [BuddyPress] #2063: new action hooks in forums-loop.php and groups/single/forum/topic.php

[buddypress-trac at lists.automattic.com]

#2063: new action hooks in forums-loop.php and groups/single/forum/topic.php
------------------------+---------------------------------------------------
Reporter:  dwenaus      |       Owner:       
    Type:  enhancement  |      Status:  new  
Priority:  trivial      |   Milestone:  1.2.2
Keywords:               |  
------------------------+---------------------------------------------------
Changes (by dwenaus):

  * keywords:  has-patch =>


Comment:

 @andrew_s1 , I think this could be done differently to conform more to the
 BP coding standards. I would suggest a much simpler solution, that doesn't
 require passing the post id into the class - it's already in the id, which
 is enough:

 change `<li id="post-<?php bp_the_topic_post_id() ?>">`
 to `<li id="post-<?php bp_the_topic_post_id() ?>" <?php
 do_action('bp_the_topic_post_class'); ?> >`
 simple.

 adding an action in the admin links is not needed in the topic.php code.
 it would be best done in the bp_the_topic_post_admin_links function in bp-
 forums-templatetags.php. but for the plugin purposes, it is not needed at
 all.

-- 
Ticket URL: <http://trac.buddypress.org/ticket/2063#comment:2>
BuddyPress <http://buddypress.org/>
BuddyPress