WordPress.org
Get WordPress

Make WordPress Core

Opened 11 years ago

Closed 11 years ago

Last modified 11 years ago

#26011 closed defect (bug) (fixed)

jshint shouldn't throw errors: wp-includes/js/wp-auth-check.js

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

Description

Cleanup of wp-includes/js/wp-auth-check

W002 : Value of 'e' may be overwritten in IE8 or earlier
W099 : Mixed spaces and tabs
W014 : Bad line breaking before '&&'
W117 : 'adminpage' is not defined
W098 : 'check' is defined but never used
W098 : 'e' is defined but never used

Attachments (1)

26011.diff​ (1.7 KB) - added by mdbitz 11 years ago.
JSHint Cleanup of wp-auth-check.js

Download all attachments as: .zip

Change History (4)

@mdbitz
11 years ago

JSHint Cleanup of wp-auth-check.js

#1 @mdbitz
11 years ago

  • Cc matt@… added
  • Keywords has-patch added

#2 @nacin
11 years ago

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

In 26195:

Fix JSHint errors in six files.

props mdbitz.
fixes ##26011, #26012, #26013, #26014, #26038, #26039.

#3 @nacin
11 years ago

  • Milestone changed from Awaiting Review to 3.8

Moving fixed JSHint error tickets to 3.8.

Note: See TracTickets for help on using tickets.