#6328 closed defect (bug) (fixed)
bp-legacy: Messages loop missing table header action
Reported by: | r-a-y | Owned by: | r-a-y |
---|---|---|---|
Milestone: | 2.3 | Priority: | normal |
Severity: | minor | Version: | 1.7 |
Component: | Templates | Keywords: | has-patch |
Cc: |
Description
In the messages loop, there is an action to add an extra table column - 'bp_messages_inbox_list_item'
:
https://buddypress.trac.wordpress.org/browser/tags/2.2.1/src/bp-templates/bp-legacy/buddypress/members/single/messages/messages-loop.php#L64
But, there isn't a similar hook in the table header:
https://buddypress.trac.wordpress.org/browser/tags/2.2.1/src/bp-templates/bp-legacy/buddypress/members/single/messages/messages-loop.php#L25
This means attempting to add a table column will not work properly.
Attached patch adds a hook called 'bp_messages_inbox_list_header'
in the <thead>
of the template so plugins can properly add a table column.
This bug also exists in bp-default.
Attachments (2)
Change History (8)
#2
@
9 years ago
Anyone got any qualms with the hook name before I commit?
'bp_messages_inbox_list_header'
, 'bp_messages_inbox_list_head'
or something else?
#4
@
9 years ago
Do we have a different spot with a similar hook, or perhaps a hook missing that needs patched as well?
"header" is fine in my books as well.
Updated the patch to include a phpdoc for the hook.