Opened 13 years ago
Closed 13 years ago
#5025 closed defect (bug) (fixed)
BUG? bp_the_body_class missing parameter
| Reported by: | tivnet | Owned by: | boonebgorges |
|---|---|---|---|
| Priority: | lowest | Milestone: | 1.8 |
| Component: | Core | Version: | 1.7 |
| Severity: | trivial | Keywords: | needs-patch |
| Cc: |
Description
This piece of code looks incorrect. $wp_classes does not have a default value, so the call bp_get_the_body_class(); should produce at least a warning.
function bp_the_body_class() {
echo bp_get_the_body_class();
}
function bp_get_the_body_class( $wp_classes, $custom_classes = false ) {
Change History (2)
Note:
See TracTickets
for help on using tickets.
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
I'm not sure why we have a function
bp_the_body_class()in the first place - the_get_version should be all we need, since it filters another value, and is not meant to be called directly. In any case, you're current that we should either have a default value, or inbp_the_body_class()we should be passing a value for the first param.