WordPress.org
Get WordPress

Make WordPress Core

Opened 14 years ago

Closed 14 years ago

#22931 closed defect (bug) (invalid)

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

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

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)

#1 follow-up: @dd32
14 years 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

#2 in reply to: ↑ 1 @nacin
14 years ago

  • Milestone Awaiting Review
  • Resolution → invalid
  • Status new → 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.