WordPress.org
Get WordPress

Make WordPress Core

Opened 11 years ago

Closed 11 years ago

Last modified 11 years ago

#25945 closed defect (bug) (fixed)

jshint shouldn't throw errors - wp-includes/js/shortcode.js

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

Description

The included patch for this ticket will resolve JSHint errors for the Shortcode JS file.

Additional description of the changes will be coming once the patch is complete.

Attachments (1)

shortcode.diff​ (4.0 KB) - added by tommcfarlin 11 years ago.
Resolving the JSHint errors regarding conditional braces and unused function argumnets.

Download all attachments as: .zip

Change History (5)

#1 follow-up: @nacin
11 years ago

  • Summary changed from jshint shouldn't throw errors - wp-admin/js/shortcode.js to jshint shouldn't throw errors - wp-includes/js/shortcode.js

This file is actually wp-includes/js/shortcode.js.

#2 in reply to: ↑ 1 @tommcfarlin
11 years ago

Replying to nacin:

This file is actually wp-includes/js/shortcode.js.

Yeah - was in the habit of typing the wp-admin/js path when creating tickets. Thanks for the catch!

The proper file will be patched considering it doesn't exist elsewhere :).

@tommcfarlin
11 years ago

Resolving the JSHint errors regarding conditional braces and unused function argumnets.

#3 @nacin
11 years ago

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

In 26228:

Fix JSHint errors in shortcode.js.

props tommcfarlin.
fixes #25945.

#4 @SergeyBiryukov
11 years ago

  • Milestone changed from Awaiting Review to 3.8
Note: See TracTickets for help on using tickets.