Skip to:
Content

BuddyPress.org

Opened 13 years ago

Closed 13 years ago

Last modified 13 years ago

#3677 closed defect (bug) (fixed)

bb-config-location should default to something intelligent

Reported by: boonebgorges's profile boonebgorges Owned by:
Milestone: 1.5.1 Priority: normal
Severity: minor Version: 1.5
Component: Forums 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?

Change History (3)

#1 @DJPaul
13 years ago

Good idea

#2 @boonebgorges
13 years ago

  • Resolution set to fixed
  • Status changed from new to closed

(In [5239]) Makes bb-config-location default to a meaningful path, to better cover edge cases where the bb-config-location option is missing. Fixes #3677

#3 @boonebgorges
13 years ago

(In [5240]) Makes bb-config-location default to a meaningful path, to better cover edge cases where the bb-config-location option is missing. Fixes #3677

Note: See TracTickets for help on using tickets.