Changeset 35669 for trunk/src/wp-admin/includes/plugin.php

Timestamp:

11/18/2015 05:57:48 PM (9 years ago)

Author:

DrewAPicture

Message:

Plugins: Rename the delete_plugin action hook (introduced in [35094]) to deleted_plugin as it fires following a plugin deletion attempt.

Further, introduce a new delete_plugin action hook, to be fired before a plugin deletion attempt. Both changes bring parity with other such transactional hooks in core that fire before and after certain actions, including on plugin activation/deactivation and install/uninstall, among others.

Props johnjamesjacoby.
Fixes #26904.

File:

• trunk/src/wp-admin/includes/plugin.php (modified) (2 diffs)

trunk/src/wp-admin/includes/plugin.php

@@ -817 +817 @@
         }
 
+        /**
+         * Fires immediately before a plugin deletion attempt.
+         *
+         * @since 4.4.0
+         *
+         * @param string $plugin_file Plugin file name.
+         */
+        do_action( 'delete_plugin', $plugin_file );
+
         $this_plugin_dir = trailingslashit( dirname( $plugins_dir . $plugin_file ) );
+
         // If plugin is in its own directory, recursively delete the directory.
         if ( strpos( $plugin_file, '/' ) && $this_plugin_dir != $plugins_dir ) { //base check on if plugin includes directory separator AND that it's not the root plugin folder
@@ -833 +843 @@
          * @param bool   $deleted     Whether the plugin deletion was successful.
          */
-        do_action( 'delete_plugin', $plugin_file, $deleted );
+        do_action( 'deleted_plugin', $plugin_file, $deleted );
 
         if ( ! $deleted ) {