WordPress.org
Get WordPress

Make WordPress Core

Opened 13 years ago

Closed 13 years ago

#26000 closed defect (bug) (fixed)

jshint shouldn't throw errors: wp-admin/js/image-edit.js

Reported by: dougwollison Owned by: SergeyBiryukov
Priority: high Milestone: 3.8
Component: Build/Test Tools Version:
Severity: normal Keywords:
Cc: Focuses:

Description

Patch to come

Attachments (5)

26000.diff​ (30.6 KB ) - added by dougwollison 13 years ago.
Initial changes: spacing, braces, equals, key fixes, globals. Note: typof callback !== 'undefined' or 'unknown'?
26000.2.diff​ (30.5 KB ) - added by dougwollison 13 years ago.
Revised declaration of globals.
26000.3.diff​ (11.0 KB ) - added by dougwollison 13 years ago.
Just the jshint fixes.
26000.4.diff​ (11.0 KB ) - added by jorbin 13 years ago.
refresh from root
26000.patch​ (628 bytes ) - added by markoheijnen 13 years ago.

Download all attachments as: .zip

Change History (12)

@dougwollison
13 years ago

Initial changes: spacing, braces, equals, key fixes, globals. Note: typof callback !== 'undefined' or 'unknown'?

#1 @dougwollison
13 years ago

  • Keywords has-patch added

@dougwollison
13 years ago

Revised declaration of globals.

@dougwollison
13 years ago

Just the jshint fixes.

#2 @jorbin
13 years ago

  • Milestone Awaiting Review → 3.8

@jorbin
13 years ago

refresh from root

#3 @jorbin
13 years ago

  • Keywords commit added

Many of the eqeqeq here are necessary and since those are potentially breaking changes, I tested this patch and everything looks good. What I tested:

  • Cropping via selecting
  • Changing crop size via input boxes
  • Rotating left
  • Rotating RIght
  • Flipping vertical
  • Flipping horizontal
  • Going "back" for all of these
  • going "forward" with all of these
  • Saving
  • Restoring original

#4 @SergeyBiryukov
13 years ago

26000.4.diff​ has a typo in line 33 (coming from the first patch). Will fix on commit.

#5 @SergeyBiryukov
13 years ago

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

In 26292:

Fix JSHint errors in image-edit.js.

props dougwollison.
fixes #26000.

#6 @markoheijnen
13 years ago

  • Keywords has-patch commit removed
  • Priority normal → high
  • Resolution fixed
  • Status closed → reopened

This commit broke the ability to save the changed made to an image

@markoheijnen
13 years ago

#7 @SergeyBiryukov
13 years ago

  • Resolution → fixed
  • Status reopened → closed

In 26425:

Fix the ability to save changes made to an image.

props markoheijnen.
fixes #26000.

Note: See TracTickets for help on using tickets.