Changeset 37859

Timestamp:

06/24/2016 03:55:26 PM (8 years ago)

Author:

pento

Message:

Media: Prompt when deleting multiple items from the Media Library

We prompt when deleting a single item from the Media Library, but not when deleting multiple items. This inconsistency is now rectified.

Props southp.
Fixes #28235.

File:

• trunk/src/wp-admin/js/media.js (modified) (2 diffs)

trunk/src/wp-admin/js/media.js

@@ -1 @@
-/* global ajaxurl, attachMediaBoxL10n, _wpMediaGridSettings */
+/* global ajaxurl, attachMediaBoxL10n, _wpMediaGridSettings, showNotice */
 
 var findPosts;
@@ -100 +100 @@
         $( '#doaction, #doaction2' ).click( function( event ) {
             $( 'select[name^="action"]' ).each( function() {
-                if ( $(this).val() === 'attach' ) {
+                var optionValue = $( this ).val();
+
+                if ( 'attach' === optionValue ) {
                     event.preventDefault();
                     findPosts.open();
+                } else if ( 'delete' === optionValue ) {
+                    if ( ! showNotice.warn() ) {
+                        event.preventDefault();
+                    }
                 }
             });