Opened 18 years ago
Closed 18 years ago
#606 closed enhancement (fixed)
Many places where $bp->root_domain is more appropriate than site_url()
| Reported by: | johnbillion | Owned by: | |
|---|---|---|---|
| Priority: | major | Milestone: | |
| Component: | Version: | 1.0 | |
| Severity: | Keywords: | has-patch | |
| Cc: | johnbillion |
Description
Many template functions such as bp_search_form() use site_url() to print the site URL. The problem is that site_url() returns the URL of the current blog, not the main site URL.
For example, if you use bp_search_form() to print the BP search form in a member blog theme, the form action will point to the current blog and not the main site, and will produce an error when submitted.
If $bp->root_domain was used instead this would allow these template functions to be used within member blog themes.
Note that some template functions such as bp_signup_page() already use $bp->root_domain where others use site_url().
Patch coming in a lil' while.
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
Patch which also includes replacing some occurances of
get_option('home')with$bp->root_domainto allow these functions to be used within member blog themes.