$disabled_components['bp-blogs.php'] default to FALSE

[kayue]

Should be true instead.

The bug I found is I installed BP in a non-multi-site Wordpress, the "Tracks blogs" option isn't showing in admin panel's BuddyPress Component Setup.

At the mean time, $disabled_componentsbp-blogs.php? default to FALSE . Since the option isn't showing in Component Setup page, there is no way I can disable the "Tracks blogs" option.

[r-a-y]

There's a check to see if multisite is enabled, if so, then the "Blog Tracking" option is displayed in the admin area.

Might make sense to remove the check in bp-core-admin.php so standard WP admins can choose to disable blog tracking.

Attached is a patch against BP v1.2.6. Anyone see any consequences with doing this?

---

If the patch isn't acceptable, you can manually add bp-blogs.php to the bp-deactivated-components option with the "bp_deactivated_components" filter (located in bp-loader.php).

Or you can use the Block Activity Stream Types plugin:
​http://wordpress.org/extend/plugins/buddypress-block-activity-stream-types/

[boonebgorges]

I like the idea of removing the multisite check and letting single-WP blog admins decide whether they would like to track blog posts. This patch will need to be refreshed for the trunk. Also, the helper text ('Tracks blogs, blog posts...') will need to be reworked to account for (a) the fact that there is no more WPMU, and (b) the possibility that single-WP admins might see the option.

[boonebgorges]

Also, this patch will need to be revisited when #2943 is fixed.

[r-a-y]

Refreshed patch; also trims some trailing whitespace

[DJPaul]

What does JJJ think? This would be a big change.

[DJPaul]

Punting, because this change could potentially impact a lot of existing code around the blogs component which assumes it always runs on multisite. It would need to be carefully reviewed and the code audited.

[DJPaul]

Do we still want to change this, or has it already been addressed sometime in the last two years? Moving to 1.8 for review.

[boonebgorges]

I'm pretty sure it's no longer relevant as of 1.7, when our configuration of default components was changed. If I'm wrong about this, please reopen ticket with details.