WordPress.org

Make WordPress Core

#22931 closed defect (bug) (invalid)

Use double quotes for attribute values when loading JS in script-loader.php

Reported by: gish Owned by:
Milestone: Priority: normal
Severity: normal Version:
Component: General Keywords:
Focuses: Cc:

Description

In /wp-includes/script-loader.php:685 script tags are built. The tags however have single quotes instead of double quotes, which doesn't conform to HTML5 standards.

Change History (2)

comment:1 follow-up: dd3216 months ago

The HTML5 standard accept 3 different formats:

id=example
id='example'
id="example"

single quotes are OK as per http://www.w3.org/TR/html-markup/syntax.html#attr-value-single-quoted

comment:2 in reply to: ↑ 1 nacin16 months ago

  • Milestone Awaiting Review deleted
  • Resolution set to invalid
  • Status changed from new to closed

Replying to dd32:

The HTML5 standard accept 3 different formats:

Just adding that our own standards require either single or double quotes, and that no quotes is a big no-no for security reasons. (http://www.wonko.com/post/html-escaping)

Note: See TracTickets for help on using tickets.