WordPress.org
Get WordPress

Make WordPress Core

Opened 13 years ago

Closed 13 years ago

#26040 closed defect (bug) (fixed)

jshint shouldn't throw errors: wp-includes/js/customize-loader.js

Reported by: mdbitz Owned by: nacin
Priority: normal Milestone: 3.8
Component: Build/Test Tools Version:
Severity: normal Keywords:
Cc: Focuses:

Description

jshint cleanup of wp-includes/js/customize-loader.js

W117 : '_wpCustomizeLoaderSettings' is not defined
W098 : 'e' is defined but never used

Attachments (1)

26040.patch​ (584 bytes ) - added by mdbitz 13 years ago.
jshint cleanup of customize-loader (global and unused named function param)

Download all attachments as: .zip

Change History (3)

@mdbitz
13 years ago

jshint cleanup of customize-loader (global and unused named function param)

#1 @nacin
13 years ago

  • Milestone Awaiting Review → 3.8

Thanks!

We've been omitting :false for read-only globals. If false makes more sense, we can change it project-wide, but it's more verbose (and some of these files currently have a lot of globals).

#2 @nacin
13 years ago

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

In 26194:

Fix JSHint errors for customize-loader.

props mdbitz.
fixes #26040.

Note: See TracTickets for help on using tickets.