Skip to:
Content

BuddyPress.org

Opened 14 years ago

Closed 14 years ago

#2694 closed defect (bug) (fixed)

Default xprofile radio element interferes with setting value in profile editor

Reported by: jond's profile jond Owned by:
Milestone: 1.5 Priority: minor
Severity: Version:
Component: Extended Profile Keywords: needs-patch
Cc: jond

Description

Steps to repeat:

  1. Create a new radio field
  2. Add two different options (example: Yes, No)
  3. Set the second option as the default
  4. Save the changes and load up the member profile editor
  5. Set the first option and try to save

The second (default) option is still selected as both input elements are getting the checked attribute.

Attachments (1)

bp-xprofile-templatetags.diff (685 bytes) - added by jond 14 years ago.
Patch file (diff) of fixes for the default radio setting conflict

Download all attachments as: .zip

Change History (8)

@jond
14 years ago

Patch file (diff) of fixes for the default radio setting conflict

#1 @jond
14 years ago

  • Keywords has-patch added

#2 @jond
14 years ago

  • Cc jond added

#3 @djpaul
14 years ago

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

(In [3377]) Correct handling of default xprofile radio button selection. Fixes #2694, props jond (trunk)

#4 @djpaul
14 years ago

(In [3378]) Correct handling of default xprofile radio button selection. Fixes #2694, props jond (branch)

#5 @DJPaul
14 years ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

Reopening as this fix needs to be applied to other multiple options, such as drop-down boxes, not just radio buttons.

#6 @DJPaul
14 years ago

  • Keywords needs-patch added; has-patch removed
  • Milestone changed from 1.2.7 to 1.3

#7 @DJPaul
14 years ago

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

I've just tested this and xprofile fields with multiple options work ok.

Note: See TracTickets for help on using tickets.