WordPress.org
Get WordPress

Make WordPress Core

Opened 12 years ago

Closed 12 years ago

#30218 closed defect (bug) (fixed)

_.isUndefined( _wpRevisionsSettings ); should be _.isUndefined( window._wpRevisionsSettings );

Reported by: ericlewis Owned by: wonderboymusic
Priority: normal Milestone: 4.1
Component: Revisions Version: 3.6
Severity: normal Keywords: has-patch commit
Cc: Focuses: javascript

Description

This will bail on a JS TypeError if _wpRevisionsSettings is undefined.

Attachments (2)

30218.diff​ (595 bytes ) - added by ericlewis 12 years ago.
30218.2.diff​ (556 bytes ) - added by ericlewis 12 years ago.

Download all attachments as: .zip

Change History (5)

@ericlewis
12 years ago

@ericlewis
12 years ago

#1 @ericlewis
12 years ago

attachment:30218.2.diff​ uses the "is defined else" pattern we use elsewhere in JS.

#2 @ericlewis
12 years ago

  • Keywords commit added

#3 @wonderboymusic
12 years ago

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

In 30130:

Set revision.settings via short-circuit check for window property: window._wpRevisionsSettings || {}

Props ericlewis.
Fixes #30218.

Note: See TracTickets for help on using tickets.