#173 closed defect (bug) (no action required)
Namespace issues with filter naming convention
| Reported by: | dfa327 | Owned by: | |
|---|---|---|---|
| Priority: | major | Milestone: | |
| Component: | Version: | 1.0 | |
| Severity: | Keywords: | filter namespace | |
| Cc: |
Description
Hello,
I was trying to run bp-core and ran in to how the add_filter names did not have a namespace associated with them. This caused some other filters with the same name to crash the site. I'd suggest adding at least a bp_ before each filter name.
Example Problem: bp-core-catchuri.php
add_filter( 'template', 'bp_core_force_buddypress_theme' );
Proposed Solution:
add_filter( 'bp_template', 'bp_core_force_buddypress_theme' );
Note:
See TracTickets
for help on using tickets.
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
add_filter() is accessing a filter, not defining. This is a filter already set by WordPress.
If it was doing apply_filter( 'template' ... then it would be a problem.