From d4c360d186a0730235e3ce7478910b90c422ade6 Mon Sep 17 00:00:00 2001 From: Gary Pendergast Date: Fri, 24 Jun 2016 15:55:26 +0000 Subject: [PATCH] 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. git-svn-id: https://develop.svn.wordpress.org/trunk@37859 602fd350-edb4-49c9-b593-d223f7449a82 --- src/wp-admin/js/media.js | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/src/wp-admin/js/media.js b/src/wp-admin/js/media.js index f0c21752c6..e23adebb17 100644 --- a/src/wp-admin/js/media.js +++ b/src/wp-admin/js/media.js @@ -1,4 +1,4 @@ -/* global ajaxurl, attachMediaBoxL10n, _wpMediaGridSettings */ +/* global ajaxurl, attachMediaBoxL10n, _wpMediaGridSettings, showNotice */ var findPosts; ( function( $ ){ @@ -99,9 +99,15 @@ var findPosts; $( '#find-posts-close' ).click( findPosts.close ); $( '#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(); + } } }); });