Make WordPress Core

Opened 5 years ago

Closed 5 years ago

#13902 closed defect (bug) (fixed)

Invalid markup in import.php

Reported by: wahgnube Owned by:
Milestone: 3.0 Priority: normal
Severity: trivial Version: 3.0
Component: Validation Keywords: has-patch dev-feedback
Focuses: Cc:


I have attached a tiny patch for wp-admin/import.php. This replaces
some & characters with &, making the page valid XHTML.

Attachments (4)

import.patch (685 bytes) - added by wahgnube 5 years ago.
import-2.patch (688 bytes) - added by wahgnube 5 years ago.
import-3.patch (688 bytes) - added by wahgnube 5 years ago.
13902.patch (1.9 KB) - added by ocean90 5 years ago.

Download all attachments as: .zip

Change History (11)

@wahgnube5 years ago

comment:1 @wahgnube5 years ago

  • Milestone changed from Unassigned to 3.0

comment:2 @ocean905 years ago

  • Keywords needs-patch added; has-patch removed
  • Version set to 3.0

You shoud use esc_url().

comment:3 @wahgnube5 years ago

  • Keywords has-patch added; needs-patch removed

I have done so.

@wahgnube5 years ago

comment:4 @ocean905 years ago

Better is:

esc_url( admin_url( 'xxxx' ) )


comment:5 @wahgnube5 years ago

I meant to do that, really! :)

@wahgnube5 years ago

@ocean905 years ago

comment:6 @ocean905 years ago

  • Keywords dev-feedback added

13902.patch is only a cosmetic patch,it includes the patch from wahgnube too.

comment:7 @nacin5 years ago

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

(In [15264]) Two birds with one stone. fixes #13902, props ocean90, wahgnube. fixes #13555, props hakre.

Note: See TracTickets for help on using tickets.