Context Navigation

← Previous Ticket
Next Ticket →

#25912 closed defect (bug) (fixed)

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

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

Description

Attachments (5)

common.global.diff (437 bytes ) - added by jorbin 13 years ago.
common.semicolon.diff (718 bytes ) - added by jorbin 13 years ago.
common.vars.diff (2.2 KB ) - added by jorbin 13 years ago.
common.eqeqeq.diff (1.2 KB ) - added by jorbin 13 years ago.
commmon.all.diff (4.2 KB ) - added by jorbin 13 years ago.

Download all attachments as: .zip

Change History (9)

@jorbin
13 years ago

Attachment common.global.diff added

@jorbin
13 years ago

Attachment common.semicolon.diff added

@jorbin
13 years ago

Attachment common.vars.diff added

@jorbin
13 years ago

Attachment common.eqeqeq.diff added

@jorbin
13 years ago

Attachment commmon.all.diff added

#1 @jorbin
13 years ago

the === checks aren't risky here (and are thrown even with eqeqeq being false)

all.diff contains everything in one easy package.

#2 @jorbin
13 years ago

Keywords has-patch added

#3 @nacin
13 years ago

Owner set to nacin
Resolution → fixed
Status new → closed

In 26112:

Free common.js of JSHint errors.

props jorbin.
fixes #25912.

#4 @nacin
13 years ago

In 26208:

Residual JSHint fixes in common.js and edit-comments.js.

see #25912, #25979.

Note: See TracTickets for help on using tickets.

Download in other formats:

Make WordPress Core

Context Navigation

#25912 closed defect (bug) (fixed)

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

Description

Attachments (5)

Change History (9)

@jorbin 13 years ago

@jorbin 13 years ago

@jorbin 13 years ago

@jorbin 13 years ago

@jorbin 13 years ago

#1 @jorbin 13 years ago

#2 @jorbin 13 years ago

#3 @nacin 13 years ago

#4 @nacin 13 years ago

Download in other formats:

@jorbin
13 years ago

@jorbin
13 years ago

@jorbin
13 years ago

@jorbin
13 years ago

@jorbin
13 years ago

#1 @jorbin
13 years ago

#2 @jorbin
13 years ago

#3 @nacin
13 years ago

#4 @nacin
13 years ago