Skip to:
Content

BuddyPress.org

Opened 12 years ago

Closed 12 years ago

#5725 closed enhancement (fixed)

Change default value of bp_disable_profile_sync()

Reported by: DJPaul Owned by: djpaul
Priority: normal Milestone: 2.1
Component: Extended Profile Version:
Severity: normal Keywords:
Cc:

Description

While looking at bp_disable_profile_sync(), the parameter's default value is true. It's used if the bp-disable-profile-sync option is somehow missing from the database. I'd like to suggest we consider changing the default from true to false, because it better fits the expected behaviour given by the name of the option.

Change History (3)

#1 @boonebgorges
12 years ago

  • Milestone Awaiting Review → 2.1

+1 from me

#2 @r-a-y
12 years ago

Do it!

#3 @djpaul
12 years ago

  • Owner set to djpaul
  • Resolution → fixed
  • Status new → closed

In 8553:

Core: change bp_disable_profile_sync()'s default value to false.

The default value is only used if the site is somehow missing the bp-disable-profile-sync option
in the database. The default value for the bp-disable-profile-sync option is false, so having the
function default to disabling profile sync if the option is missing is inconsistent.

Fixes #5725

Note: See TracTickets for help on using tickets.