Changeset 23268

Timestamp:

01/04/2013 08:01:04 PM (12 years ago)

Author:

nacin

Message:

Media: Check that a router region exists before attempting to access its child views. Add relevant documentation.

Merges [23260] to the 3.5 branch.

props koopersmith.
fixes #22930.

File:

• branches/3.5/wp-includes/js/media-views.js (modified) (1 diff)

branches/3.5/wp-includes/js/media-views.js

@@ -523 +523 @@
                 mode = frame.content.mode();
 
-            if ( this.active && ! selection.length && ! router.get( mode ) )
+            // If the state is active, no items are selected, and the current
+            // content mode is not an option in the state's router (provided
+            // the state has a router), reset the content mode to the default.
+            if ( this.active && ! selection.length && router && ! router.get( mode ) )
                 this.frame.content.render( this.get('content') );
         },