WordPress.org

Make WordPress Core

Opened 2 years ago

Closed 2 years ago

Last modified 2 years ago

#17364 closed defect (bug) (fixed)

Validate the HTML in the admin area

Reported by: peaceablewhale Owned by: ryan
Priority: normal Milestone: 3.2
Component: Administration Version: 3.2
Severity: normal Keywords: has-patch
Cc: nacin

Description

Well-formness fixes for WordPress Admin interface

Attachments (4)

17364.patch (4.1 KB) - added by peaceablewhale 2 years ago.
17364.html-valid.patch (17.8 KB) - added by ocean90 2 years ago.
17364-wp-users-list-table.diff (807 bytes) - added by xknown 2 years ago.
Use esc_url instead of esc_html in /wp-admin/includes/class-wp-users-list-table.php
17364.2.patch (5.5 KB) - added by ocean90 2 years ago.

Download all attachments as: .zip

Change History (12)

comment:1 dd322 years ago

wp-admin/includes/class-wp-terms-list-table.php

The usage of wp_nonce_url without a nonce isn't ideal. You're probably looking for esc_url() or similar there

peaceablewhale2 years ago

comment:2 peaceablewhale2 years ago

Changed, thanks! :)

ocean902 years ago

comment:3 ocean902 years ago

  • Cc nacin added
  • Milestone changed from Awaiting Review to 3.2
  • Summary changed from Well-formness fixes for WordPress Admin interface to Validate the HTML in the admin area

Added a patch to make the admin area HTML valid on some pages again.

One problem still exists: ID "awaiting-mod" already defined on each page when a comment is in moderation because now the collapsed menu shows also the bubble.

comment:4 ryan2 years ago

In [18010]:

Validation fixes. Props ocean90, peaceablewhale. see #17364

xknown2 years ago

Use esc_url instead of esc_html in /wp-admin/includes/class-wp-users-list-table.php

comment:5 ocean902 years ago

xknown, the problem was, that through esc_url() it becomes something like &#...

Version 0, edited 2 years ago by ocean90 (next)

ocean902 years ago

comment:6 ocean902 years ago

17364.2.patch includes a fix for the ID "awaiting-mod" already defined issue. Use class instead of id.

comment:7 ryan2 years ago

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

In [18114]:

Validation fixes. Props ocean90, xknown. fixes #17364

comment:8 ryan2 years ago

In [18115]:

Validation fixes. Props ocean90, xknown. fixes #17364

Note: See TracTickets for help on using tickets.