#4896 closed defect (bug) (fixed)
Installed Component Counts
Reported by: |
|
Owned by: | |
---|---|---|---|
Milestone: | 1.7 | Priority: | normal |
Severity: | normal | Version: | 1.7 |
Component: | Core | Keywords: | has-patch commit |
Cc: |
Description
Two parts to this... See attached screenshot...
- I'm told 3 are active, I count 4 active (2 must-use)
- 'All' should be 10 but I get the idea behind this in hiding as much as possible of the retired bbPress 1.x 'Group Forums'. Not sure what the best option to do here is, probably leave it the way it is...
Attachments (2)
Change History (7)
#2
@
12 years ago
- Keywords has-patch dev-feedback added
- Milestone changed from Awaiting Review to 1.7
Members component wasn't being activated by default on a new install. 4896.01.patch addresses this and thus fixes the count, but i'm not sure if there are other consequences of members being active (or not being active) here.
#3
@
12 years ago
- Keywords commit added; dev-feedback removed
From BP's point of view, it doesn't really matter whether 'members' is included in the bp-active-components array - it's a required component, and is force-loaded in the main bootstrap method: https://buddypress.trac.wordpress.org/browser/trunk/bp-core/bp-core-loader.php#L44
So I think that this patch should be fine.
#4
@
12 years ago
- Resolution set to fixed
- Status changed from new to closed
(In [6873]) Fix inaccurate active component count on BP settings screen for new installs. Fixes #4896
The members component is a mandatory component, which we always activate. The active component count only counts active components.
On new installs, the members component was not being set active, leading to the discrepancy. Saving the settings screen (without making any changes) also "fixed" the count as the save routine re-calculates the active components array.
If you save the active components without making any changes, the count corrects itself.