Skip to:
Content

BuddyPress.org

Opened 12 years ago

Closed 12 years ago

#3798 closed defect (bug) (duplicate)

bbPress Group Forums fails installation when defines are missing

Reported by: r-a-y's profile r-a-y Owned by:
Milestone: 1.6 Priority: normal
Severity: normal Version: 1.5.1
Component: Forums Keywords: dev-feedback
Cc:

Description

Just encountered this issue where the "DB_CHARSET" define was not defined on a certain WP install's wp-config.php.

So during group forum installation, the generated bb-config.php came prefilled with the following:

define( 'BBDB_CHARSET', 'DB_CHARSET' );

And this generated SQL errors preventing the bbPress tables from being installed.

However, the BuddyPress Forums Setup page says everything is hunky-dory even when the tables were not added to the DB.

Could be an edge-case, but might be good to add proper fallback variables during the creation of bb-config.php.

Change History (2)

#1 @boonebgorges
12 years ago

  • Milestone changed from Awaiting Review to 1.6

Looks like bbPress doesn't really *need* to have a charset; I see a couple lines that look like
'charset' => defined( 'BBDB_CHARSET' ) ? BBDB_CHARSET : false

So I think we can probably just get away with wrapping our own define() line in if ( defined( 'DB_CHARSET' ) ). Needs testing, though; we may need to fall back on utf8.

#2 @johnjamesjacoby
12 years ago

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

Will be a non-issue going forward with bbPress 2. See #3598.

Note: See TracTickets for help on using tickets.