From 1d1d95f4d5f038a9c4c663d90db7cb90b571ff4d Mon Sep 17 00:00:00 2001 From: Scott Taylor Date: Wed, 6 Aug 2014 17:23:49 +0000 Subject: [PATCH] In `media.view.BulkSelectionActionButton.click()`, do not assume that `delete` is the only possible bulk action. Fixes #29101. git-svn-id: https://develop.svn.wordpress.org/trunk@29403 602fd350-edb4-49c9-b593-d223f7449a82 --- src/wp-includes/js/media-grid.js | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/src/wp-includes/js/media-grid.js b/src/wp-includes/js/media-grid.js index 0c912fb5c4..cb8447df7f 100644 --- a/src/wp-includes/js/media-grid.js +++ b/src/wp-includes/js/media-grid.js @@ -647,10 +647,12 @@ var selection = this.controller.controller.state().get('selection'); media.view.Button.prototype.click.apply( this, arguments ); - // Currently assumes delete is the only action - if ( confirm( l10n.warnBulkDelete ) ) { - while ( selection.length > 0) { - selection.at(0).destroy(); + if ( 'delete' === this.controller.model.get( 'currentAction' ) ) { + // Currently assumes delete is the only action + if ( confirm( l10n.warnBulkDelete ) ) { + while ( selection.length > 0 ) { + selection.at(0).destroy(); + } } }