Skip to:

Opened 11 years ago

Closed 11 years ago

#5262 closed defect (bug) (fixed)

Empty user visibility overrides default visibility settings

Reported by: dcavins's profile dcavins Owned by: boonebgorges's profile boonebgorges
Milestone: 1.9 Priority: normal
Severity: normal Version: 1.8.1
Component: Extended Profile Keywords: has-patch


If for some reason the $user_visibililty_levels aren't set in bp_xprofile_get_fields_by_visibility_levels, then the function doesn't enforce the default visibility levels. I simply removed the user_visibility isset check so that the defaults get applied no matter what.

Attachments (1)

default-vis-trump.patch (903 bytes) - added by dcavins 11 years ago.
Patch for change

Download all attachments as: .zip

Change History (3)

11 years ago

Patch for change

#1 @boonebgorges
11 years ago

  • Milestone changed from Awaiting Review to 1.9

Yup, this seems right.

#2 @boonebgorges
11 years ago

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

In 7609:

Allow admin-set field visibility default to override when user hasn't set a value yet

Fixes #5262

Props dcavins

Note: See TracTickets for help on using tickets.