WordPress.org

Make WordPress Core

Opened 5 months ago

Closed 5 months ago

Last modified 5 months ago

#25917 closed defect (bug) (fixed)

jshint shouldn't throw errors - wp-admin/js/customize-control.js

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

Description (last modified by SergeyBiryukov)

[L98:C17] W081: Too many var statements.
[L133:C1] W099: Mixed spaces and tabs.
[L134:C1] W099: Mixed spaces and tabs.
[L135:C1] W099: Mixed spaces and tabs.
[L136:C1] W099: Mixed spaces and tabs.
[L781:C1] W099: Mixed spaces and tabs.
[L782:C1] W099: Mixed spaces and tabs.
[L783:C1] W099: Mixed spaces and tabs.
[L923:C56] W033: Missing semicolon.
[L927:C64] W033: Missing semicolon.
[L931:C65] W033: Missing semicolon.
[L1001:C13] W081: Too many var statements.
[L11:C24] W098: 'element' is defined but never used.
[L131:C49] W098: 'options' is defined but never used.
[L131:C40] W098: 'event' is defined but never used.
[L200:C35] W098: 'up' is defined but never used.
[L323:C21] W098: 'self' is defined but never used.
[L492:C20] W098: 'url' is defined but never used.
[L711:C18] W098: 'query' is defined but never used.
[L976:C62] W098: 'event' is defined but never used.

Attachments (1)

customize-controls.diff (8.9 KB) - added by jorbin 5 months ago.

Download all attachments as: .zip

Change History (4)

jorbin5 months ago

comment:1 nacin5 months ago

  • Component changed from General to Build Tools
  • Milestone changed from Awaiting Review to 3.8

comment:2 nacin5 months ago

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

In 26206:

Fix JSHint errors in customize-controls.js.

props jorbin.
fixes #25917.

comment:3 SergeyBiryukov5 months ago

  • Description modified (diff)
Note: See TracTickets for help on using tickets.