Skip to:
Content

bbPress.org

Opened 11 years ago

Closed 11 years ago

Last modified 11 years ago

#2388 closed enhancement (fixed)

Remove The Extra Field ID

Reported by: alex-ye's profile alex-ye Owned by: johnjamesjacoby's profile johnjamesjacoby
Milestone: 2.4 Priority: normal
Severity: normal Version: 2.3.2
Component: Back-end Keywords: has-patch
Cc:

Description (last modified by johnjamesjacoby)

Some settings fields have duplicate id attributes. This is likely a copy/paste issue that's echo'ed through the admin settings fields.

Patch attached.

Attachments (1)

settings.php.patch​ (11.7 KB) - added by alex-ye 11 years ago.

Download all attachments as: .zip

Change History (5)

@alex-ye
11 years ago

#1 @johnjamesjacoby
11 years ago

  • Description modified (diff)
  • Milestone changed from Awaiting Review to 2.4

#2 @johnjamesjacoby
11 years ago

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

In 5082:

Remove duplicate id attributes from some admin settings fields. Props alex-ye. Fixes #2388.

#3 @johnjamesjacoby
11 years ago

In 5083:

Clean up admin settings fields attribute order: name, ID, type, value, others... See #2388.

#4 @alex-ye
11 years ago

Thanks for the Description correction :)

Note: See TracTickets for help on using tickets.