#4669 closed defect (bug) (fixed)
'bp_head' hook does not exist for theme compat
| Reported by: | r-a-y | Owned by: | |
|---|---|---|---|
| Priority: | normal | Milestone: | 1.7 |
| Component: | Templates | Version: | 1.7 |
| Severity: | normal | Keywords: | has-patch |
| Cc: |
Description
bp-default has a hardcoded do_action( 'bp_head' ) in its header.php.
bp-legacy does not have such a hook. So any code hooked to 'bp_head' will not run when a theme other than bp-default is activated.
In r6529, I added the 'bp_head' hook to 'wp_head', but quickly reverted it due to backwards compatibility as explained in r6177.
In 01.patch, I've added the 'bp_head' hook to the BP_Theme_Compat class. This is so any classes that extend it like BP_Legacy and BP_TurtleShell can inherit it.
Attachments (1)
Note:
See TracTickets
for help on using tickets.
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
Disregard this patch. BP_Theme_Compat class needs to be refactored entirely. New ticket will be created for this.