#6328 closed defect (bug) (fixed)
bp-legacy: Messages loop missing table header action
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| 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
@
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?
#4
@
11 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.