Changeset 27812 for trunk/src/wp-includes/media-template.php

Timestamp:

03/28/2014 12:36:41 PM (10 years ago)

Author:

wonderboymusic

Message:

When creating a new playlist, we don't know from the model what "type" it is right away (there is no longer a video-playlist shortcode) - we need to check the controller's library's prop for type when deciding what settings to display in the modal.

See #27535.

File:

• trunk/src/wp-includes/media-template.php (modified) (2 diffs)

trunk/src/wp-includes/media-template.php

@@ -518 +518 @@
         <h3><?php _e( 'Playlist Settings' ); ?></h3>
 
-        <# var emptyModel = _.isEmpty( data.model );  #>
+        <# var emptyModel = _.isEmpty( data.model ),
+            isVideo = 'video' === data.controller.get('library').props.get('type'); #>
 
         <label class="setting">
@@ -524 +525 @@
                 checked="checked"
             <# } #> />
-            <# if ( 'audio' === data.model.type ) { #>
+            <# if ( isVideo ) { #>
+            <span><?php _e( 'Show Video List' ); ?></span>
+            <# } else { #>
             <span><?php _e( 'Show Tracklist' ); ?></span>
-            <# } else { #>
-            <span><?php _e( 'Show Video List' ); ?></span>
             <# } #>
         </label>
 
-        <# if ( 'audio' === data.model.type ) { #>
+        <# if ( ! isVideo ) { #>
         <label class="setting">
             <input type="checkbox" data-setting="artists" <# if ( emptyModel ) { #>