#6328 closed defect (bug) (fixed)
bp-legacy: Messages loop missing table header action
| Reported by: | r-a-y | Owned by: | r-a-y |
|---|---|---|---|
| Priority: | normal | Milestone: | 2.3 |
| Component: | Templates | Version: | 1.7 |
| Severity: | minor | 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
@
11 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?
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
Updated the patch to include a phpdoc for the hook.