Opened 8 years ago
Closed 8 years ago
#7150 closed defect (bug) (fixed)
The blogs component & the bp_core_add_page_mappings() function
Reported by: | imath | Owned by: | imath |
---|---|---|---|
Milestone: | 2.7 | Priority: | normal |
Severity: | normal | Version: | 1.7 |
Component: | Settings | Keywords: | has-patch commit |
Cc: |
Description
So although the Blogs component is a core/built in component, it's the only one that doesn't have his WordPress directory page set on activation when using a multisite configuration.
First, it's really annoying, and i see no reason for such a treatment.
Second, it can make test fails, eg if you do:
$this->go_to( bp_get_blogs_directory_permalink() ); $this->assertTrue( bp_is_blogs_directory() );
It will fail.
Third, and most importantly bp_core_add_page_mappings()
is using "sites" instead of "blogs" which is THE reason why the page is not created when creating the blogs component.
In 2.7, i'd really like to give some love to this component, and the attached patch is the first proof of my love :)
Nice catch, imath!
Looks good.