Add an action link to allow for plugins to be deleted individually. Also switch to storing the deletion information and redirecting before displaying. Fixes #9835 props DD32.
git-svn-id: http://svn.automattic.com/wordpress/trunk@11371 1a063a9b-81f0-0310-95a4-ce76da25c4cd
This commit is contained in:
parent
f4331cacbd
commit
afcaa4b48d
|
@ -11,10 +11,8 @@ require_once('admin.php');
|
||||||
|
|
||||||
if ( isset($_POST['clear-recent-list']) )
|
if ( isset($_POST['clear-recent-list']) )
|
||||||
$action = 'clear-recent-list';
|
$action = 'clear-recent-list';
|
||||||
elseif ( isset($_GET['action']) )
|
elseif ( isset($_REQUEST['action']) )
|
||||||
$action = $_GET['action'];
|
$action = $_REQUEST['action'];
|
||||||
elseif ( isset($_POST['action']) )
|
|
||||||
$action = $_POST['action'];
|
|
||||||
else
|
else
|
||||||
$action = false;
|
$action = false;
|
||||||
|
|
||||||
|
@ -31,6 +29,9 @@ if ( $status != $default_status && 'search' != $status )
|
||||||
|
|
||||||
$page = isset($_REQUEST['paged']) ? $_REQUEST['paged'] : 1;
|
$page = isset($_REQUEST['paged']) ? $_REQUEST['paged'] : 1;
|
||||||
|
|
||||||
|
//Clean up request URI from temporary args for screen options/paging uri's to work as expected.
|
||||||
|
$_SERVER['REQUEST_URI'] = remove_query_arg(array('error', 'deleted', 'activate', 'activate-multi', 'deactivate', 'deactivate-multi', '_error_nonce'), $_SERVER['REQUEST_URI']);
|
||||||
|
|
||||||
if ( !empty($action) ) {
|
if ( !empty($action) ) {
|
||||||
switch ( $action ) {
|
switch ( $action ) {
|
||||||
case 'activate':
|
case 'activate':
|
||||||
|
@ -163,7 +164,9 @@ if ( !empty($action) ) {
|
||||||
} //Endif verify-delete
|
} //Endif verify-delete
|
||||||
$delete_result = delete_plugins($plugins);
|
$delete_result = delete_plugins($plugins);
|
||||||
|
|
||||||
wp_cache_delete('plugins', 'plugins');
|
set_transient('plugins_delete_result_'.$user_ID, $delete_result); //Store the result in a cache rather than a URL param due to object type & length
|
||||||
|
wp_redirect("plugins.php?deleted=true&plugin_status=$status&paged=$page");
|
||||||
|
exit;
|
||||||
break;
|
break;
|
||||||
case 'clear-recent-list':
|
case 'clear-recent-list':
|
||||||
update_option('recently_activated', array());
|
update_option('recently_activated', array());
|
||||||
|
@ -198,7 +201,10 @@ if ( !empty($invalid) )
|
||||||
}
|
}
|
||||||
?>
|
?>
|
||||||
</div>
|
</div>
|
||||||
<?php elseif ( 'delete-selected' == $action ) :
|
<?php elseif ( isset($_GET['deleted']) ) :
|
||||||
|
$delete_result = get_transient('plugins_delete_result_'.$user_ID);
|
||||||
|
delete_transient('plugins_delete_result'); //Delete it once we're done.
|
||||||
|
|
||||||
if ( is_wp_error($delete_result) ) : ?>
|
if ( is_wp_error($delete_result) ) : ?>
|
||||||
<div id="message" class="updated fade"><p><?php printf( __('Plugin could not be deleted due to an error: %s'), $delete_result->get_error_message() ); ?></p></div>
|
<div id="message" class="updated fade"><p><?php printf( __('Plugin could not be deleted due to an error: %s'), $delete_result->get_error_message() ); ?></p></div>
|
||||||
<?php else : ?>
|
<?php else : ?>
|
||||||
|
@ -285,6 +291,11 @@ if ( isset($_GET['s']) ) {
|
||||||
}
|
}
|
||||||
|
|
||||||
$plugin_array_name = "${status}_plugins";
|
$plugin_array_name = "${status}_plugins";
|
||||||
|
if ( empty($$plugin_array_name) && $status != 'all' ) {
|
||||||
|
$status = 'all';
|
||||||
|
$plugin_array_name = "${status}_plugins";
|
||||||
|
}
|
||||||
|
|
||||||
$plugins = &$$plugin_array_name;
|
$plugins = &$$plugin_array_name;
|
||||||
|
|
||||||
//Paging.
|
//Paging.
|
||||||
|
@ -362,6 +373,9 @@ function print_plugins_table($plugins, $context = '') {
|
||||||
$actions[] = '<a href="' . $plugin_data['PluginURI'] . '" title="' . __( 'Visit plugin homepage' ) . '">' . __('View Site') . '</a>';
|
$actions[] = '<a href="' . $plugin_data['PluginURI'] . '" title="' . __( 'Visit plugin homepage' ) . '">' . __('View Site') . '</a>';
|
||||||
}
|
}
|
||||||
|
|
||||||
|
if ( ! $is_active && current_user_can('delete_plugins') )
|
||||||
|
$actions[] = '<a href="' . wp_nonce_url('plugins.php?action=delete-selected&checked[]=' . $plugin_file . '&plugin_status=' . $context . '&paged=' . $page, 'bulk-manage-plugins') . '" title="' . __('Delete this plugin') . '" class="delete">' . __('Delete') . '</a>';
|
||||||
|
|
||||||
$actions = apply_filters( 'plugin_action_links', $actions, $plugin_file, $plugin_data, $context );
|
$actions = apply_filters( 'plugin_action_links', $actions, $plugin_file, $plugin_data, $context );
|
||||||
$actions = apply_filters( "plugin_action_links_$plugin_file", $actions, $plugin_file, $plugin_data, $context );
|
$actions = apply_filters( "plugin_action_links_$plugin_file", $actions, $plugin_file, $plugin_data, $context );
|
||||||
$action_count = count($actions);
|
$action_count = count($actions);
|
||||||
|
|
Loading…
Reference in New Issue