Make WordPress Core

Opened 23 months ago

Closed 23 months ago

Last modified 23 months ago

#24267 closed defect (bug) (duplicate)

JavaScript Syntax Error

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


If you user HTML-Comments to hide JavaScript for browsers that do not support JavaScript, the closing HTML-Comment-Tag have to be a JavaScript-Comment

So instead of this Code:
<script type="text/javascript">
[some JavaScript]
You should use
<script type="text/javascript">
[some JavaScript]
Otherwise some Browsers will fail because of invalid JavaScript.
This is also descriped on w3schools (http://www.w3schools.com/tags/tag_comment.asp)

This error can be found in:
wp-includes/file.php on line 980
wp-includes/media.php on line 2075

Change History (3)

comment:1 @1994rstefan23 months ago

Sorry, the Bug-Tracker escapes a double slash, it should say (without space):

<script type="text/javascript">
[some JavaScript]
/ / -->

Last edited 23 months ago by 1994rstefan (previous) (diff)

comment:2 @ocean9023 months ago

  • Keywords needs-patch removed
  • Milestone Awaiting Review deleted
  • Status changed from new to closed

No, it's valid.

Duplicate: #23952

Related: #18788

comment:3 @ocean9023 months ago

  • Resolution set to duplicate
Note: See TracTickets for help on using tickets.