WordPress.org
Get WordPress

Make WordPress Core

Opened 14 years ago

Closed 14 years ago

#15830 closed defect (bug) (fixed)

'editor' script depends on jQuery, but doesn't register it as a dependency

Reported by: jczorkmid's profile jczorkmid Owned by:
Milestone: 3.1 Priority: normal
Severity: normal Version: 3.1
Component: Administration Keywords: has-patch
Focuses: Cc:

Description

source:/trunk/wp-admin/js/editor.js@16394#L1​ (source:/trunk/wp-admin/js/editor.dev.js@16394#L2​) relies on jQuery, but it doesn't register it as a dependency in source:/trunk/wp-includes/script-loader.php@16943#L94​

Attachments (2)

script-loader.diff​ (586 bytes) - added by jczorkmid 14 years ago.
patch
15830.diff​ (594 bytes) - added by JohnPBloch 14 years ago.

Download all attachments as: .zip

Change History (7)

@jczorkmid
14 years ago

patch

#1 @jczorkmid
14 years ago

  • Keywords has-patch added

#2 @nacin
14 years ago

  • Milestone changed from Awaiting Review to 3.1

#3 @JohnPBloch
14 years ago

  • Keywords has-patch removed

Just tested the patch. In Chrome on Mac I got this error when enqueueing editor.js:

Uncaught ReferenceError: wpCookies is not defined

Adding 'utils' as an extra dependency fixes the error. Patch coming in a second.

@JohnPBloch
14 years ago

#4 @JohnPBloch
14 years ago

  • Keywords has-patch added

#5 @ryan
14 years ago

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

(In [17000]) Add jquery and utils dependencies for editor js. Props JohnPBloch, jczorkmid. fixes #15830

Note: See TracTickets for help on using tickets.