Make WordPress Core

Opened 13 years ago

Closed 13 years ago

#25870 closed defect (bug) (fixed)

jshint shouldn't throw errors - wp-admin/about.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 (3)

25870.dotnotation.diff (903 bytes ) - added by jorbin 13 years ago.
25870.global.diff (318 bytes ) - added by jorbin 13 years ago.
25870.all.diff (1.6 KB ) - added by kadamwhite 13 years ago.

Download all attachments as: .zip

Change History (8)

@jorbin
13 years ago

#1 @jorbin
13 years ago

  • Component GeneralUnit Tests
  • Keywords has-patch added
  • Milestone Awaiting Review3.8

#2 @kadamwhite
13 years ago

  • Cc kadamwhite@… added

Now, putting aside the fact that this file will probably be going away.

25870.all.diff contains the same changes @jorbin made, plus a minor optimization to cache the reused jQuery object in a $result. I'd like to take this opportunity to make changes like this if we're editing the same lines for JSHint coverage, but I want to run that by jorbin, nacin et al.

#3 @SergeyBiryukov
13 years ago

  • Component Unit TestsBuild Tools

#4 @nacin
13 years ago

25870.all.diff looks good, but in this case, this file will be removed in 3.8 anyway.

#5 @nacin
13 years ago

  • Owner set to nacin
  • Resolutionfixed
  • Status newclosed

In 26065:

Avoid JSHint errors. Use dot notation when able and declare pwsL10n as a global.

props jorbin.
fixes #25870.

Note: See TracTickets for help on using tickets.