Make WordPress Core

Opened 17 years ago

Closed 16 years ago

#11937 closed defect (bug) (fixed)

ID clash in 'Link Categories/Post Tags' Page

Reported by: Utkarsh Owned by: dd32
Priority: low Milestone: 3.0
Component: Administration Version: 2.9
Severity: minor Keywords: has-patch
Cc: Focuses:

Description

There are 2 elements each with the ID 'name', 'slug' and 'description' in wp-admin/edit-link-categories.php For this reason, the <label> for these items in the Add Link Category part do not point to the respective input boxes.

Attachments (2)

11937.3.patch (3.0 KB ) - added by ShaneF 17 years ago.
11937.4.diff (4.8 KB ) - added by ShaneF 17 years ago.

Download all attachments as: .zip

Change History (15)

#1 @Utkarsh
17 years ago

  • Cc admin@… added

#2 @scribu
17 years ago

  • Component GeneralAdministration
  • Priority normallow
  • Severity normalminor

#3 @ShaneF
17 years ago

  • Keywords has-patch commit added; need-patch removed
  • Owner set to ShaneF
  • Status newaccepted

@ShaneF
17 years ago

#4 @ShaneF
17 years ago

Leaving this at Milestone 3.0. Right now I don't want to change any compatibility with 2.9.x series.

#5 @ryan
17 years ago

wp_insert_term() expects description and slug. I don't see any mapping from link_description and link_slug back to description and slug.

#6 follow-up: @dd32
17 years ago

  • Keywords needs-patch added; has-patch commit removed

The Edit tags page suffers from this same issue i think.

Its the Table column headers which are conflicting i think, Probably best to give them the id of 'column-$column' instead of modifying the form?

#7 @ShaneF
17 years ago

Ok. :-) Working on it.

@ShaneF
17 years ago

#8 @ShaneF
17 years ago

  • Cc ShaneF added
  • Keywords has-patch commit added; needs-patch removed
  • Summary ID clash in Link Categories PageID clash in 'Link Categories/Post Tags' Page

#9 in reply to: ↑ 6 @ShaneF
17 years ago

Replying to dd32:

The Edit tags page suffers from this same issue i think.

Its the Table column headers which are conflicting i think, Probably best to give them the id of 'column-$column' instead of modifying the form?

I looked into this before. All the javascript code would have to be changed to reflect that and it's more than just these two pages that use the same javascript.

#10 @nacin
16 years ago

  • Keywords commit removed
  • Owner changed from ShaneF to dd32
  • Status acceptedassigned

#11 @dd32
16 years ago

Is there a need to change the Fields Names? Its the ID's which are conflicting.

I'm not 100%, But from what i can see, its only the field Labels which are using the ID's.

I'm going to commit a change to the ID's, if people can test and report if they notice anything odd, that'd be appreciated.

#12 @dd32
16 years ago

(In [13852]) Change form field ID's to prevent ID clashes with listing table column headers. See #11937

#13 @dd32
16 years ago

  • Resolutionfixed
  • Status assignedclosed

the HTML Validator doesnt mention any issues on the affected pages, and functionality seems to be working still.

Closing as fixed, Re-open if this has caused a bug.

Note: See TracTickets for help on using tickets.