WordPress.org
Get WordPress

Make WordPress Core

Opened 13 years ago

Closed 13 years ago

Last modified 13 years ago

#26046 closed defect (bug) (fixed)

jshint shouldn’t throw errors - wp-includes/js/quicktags.js

Reported by: kovshenin Owned by: nacin
Priority: normal Milestone: 3.8
Component: Build/Test Tools Version:
Severity: normal Keywords: has-patch
Cc: Focuses:

Description

Let's fix jshint errors for quicktags.js. The arguments.callee part might be tricky.

Attachments (1)

26046.diff​ (11.4 KB ) - added by kovshenin 13 years ago.

Download all attachments as: .zip

Change History (5)

@kovshenin
13 years ago

#1 @kovshenin
13 years ago

  • Summary JSHint: wp-includes/js/quicktags.js → jshint shouldn’t throw errors - wp-includes/js/quicktags.js

#2 @nacin
13 years ago

  • Owner set to nacin
  • Resolution → fixed
  • Status new → closed

In 26212:

Fix JSHint errors in quicktags.js. Ignore deprecated API rather than marking it as exported globals.

props kovshenin.
fixes #26046.

#3 @SergeyBiryukov
13 years ago

  • Milestone Awaiting Review → 3.8

#4 @azaozz
13 years ago

In 26553:

Quicktags: fix int/string strict comparison, see #26046.

Note: See TracTickets for help on using tickets.