#3677 closed defect (bug) (fixed)
bb-config-location should default to something intelligent
| Reported by: | boonebgorges | Owned by: | |
|---|---|---|---|
| Priority: | normal | Milestone: | 1.5.1 |
| Component: | Forums | Version: | 1.5 |
| Severity: | minor | Keywords: | dev-feedback |
| Cc: |
Description
I recently worked with someone who, for some unexplained reason, kept having her 'bb-config-location' option deleted, which resulted in bp_forums_is_installed_correctly() returning false. I'm not sure why the bb-config-location option keeps getting deleted - probably a plugin conflict (I couldn't reproduce it on a clean install). But, in any case, it would have continued to work even in the absence of bb-config-location if the default value for that option, as set in bp_core_get_root_options(), were something sensible: ABSPATH . 'bb-config.php' instead of ABSPATH. And since we do an is_file() check before actually loading that value into the $bp->forums object, there's no chance of false positives.
Objections?
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
Good idea