Skip to:
Content

BuddyPress.org

Opened 11 years ago

Closed 11 years ago

#6472 closed defect (bug) (fixed)

Duplicate IDs in Profile Fields Admin Panel

Reported by: mercime Owned by: DJPaul
Priority: normal Milestone: 2.4
Component: Extended Profile Version: 1.7
Severity: normal Keywords: has-patch early good-first-bug needs-testing
Cc:

Description

Attached patch removes IDs generated in fieldsets to fix issue.

Attachments (1)

xprofile.diff​ (655 bytes ) - added by mercime 11 years ago.

Download all attachments as: .zip

Change History (7)

@mercime
11 years ago

This ticket was mentioned in ​Slack in #buddypress by boone. ​View the logs.
11 years ago

#2 @johnjamesjacoby
11 years ago

  • Component API → Component - XProfile
  • Keywords early added
  • Milestone Awaiting Review → 2.4
  • Version → 1.7

See also:

  • BP_XProfile_Field_Type::get_edit_field_html_elements();
  • bp_get_form_field_attributes()

Patch looks good, but isn't a regression so moving to 2.4.

Can we double-plus confirm that nothing in the CSS or JS looks for any of these fieldset ID's?

#3 @DJPaul
11 years ago

  • Keywords good-first-bug added

#4 @DJPaul
11 years ago

  • Keywords needs-testing added

Needs a test before we can commit this, then we can keep an eye out for any further issues during 2.4 dev.

#5 @DJPaul
11 years ago

  • Owner set to DJPaul
  • Status new → assigned

Please leave this ticket for me to commit.

I'll get it done later this week.

#6 @mercime
11 years ago

  • Resolution → fixed
  • Status assigned → closed

In 10071:

Remove duplicate IDs from xProfiles admin screen.

Duplicate id values are generated in both the fieldset and in any of the
associated form control elements in the Profile Fields admin screen. The id
value is required for all form controls to be associated with label tags
while the ‘id value for the fieldset` element on the other hand is not
used in either the Profile Fields admin CSS nor JS. Removing the id attribute
from the fieldset element fixes the issue of duplicate ids in the DOM .

Props mercime, boonebgorges, djpaul, johnjamesjacoby.
Fixes #6472.

Note: See TracTickets for help on using tickets.