Merge pull request #84 from BhargavBhandari90/GH-75

Add --all flag for plugin uninstall command

Author: schlessera

features/plugin-uninstall.feature

@@ -27,12 +27,47 @@ Feature: Uninstall a WordPress plugin
 
   Scenario: Attempting to uninstall a plugin that doesn't exist
     When I try `wp plugin uninstall edit-flow`
-    Then STDOUT should be:
+    Then STDERR should be:
       """
-      Success: Plugin already uninstalled.
+      Warning: The 'edit-flow' plugin could not be found.
+      Error: No plugins uninstalled.
       """
-    And STDERR should be:
+    And the return code should be 1
+
+  Scenario: Uninstall all installed plugins
+    When I run `wp plugin uninstall --all`
+    Then STDOUT should be:
       """
-      Warning: The 'edit-flow' plugin could not be found.
+      Uninstalled and deleted 'akismet' plugin.
+      Uninstalled and deleted 'hello' plugin.
+      Success: Uninstalled 2 of 2 plugins.
       """
     And the return code should be 0
+
+    When I run the previous command again
+    Then STDOUT should be:
+      """
+      Success: No plugins uninstalled.
+      """
+
+  Scenario:  Uninstall all installed plugins when one or more activated
+    When I run `wp plugin activate --all`
+    Then STDOUT should contain:
+      """
+      Success: Activated 2 of 2 plugins.
+      """
+
+    When I try `wp plugin uninstall --all`
+    Then STDERR should be:
+      """
+      Warning: The 'akismet' plugin is active.
+      Warning: The 'hello' plugin is active.
+      Error: No plugins uninstalled.
+      """
+    And the return code should be 1
+
+    When I run `wp plugin uninstall --deactivate --all`
+    Then STDOUT should contain:
+      """
+      Success: Uninstalled 2 of 2 plugins.
+      """

features/plugin.feature

@@ -67,15 +67,11 @@ Feature: Manage WordPress plugins
       | Zombieland | active | none   | 0.1.0   |
 
     When I try `wp plugin uninstall Zombieland`
-    Then STDERR should contain:
-      """
-      The 'Zombieland' plugin is active.
-      """
-    And STDERR should contain:
+    Then STDERR should be:
       """
+      Warning: The 'Zombieland' plugin is active.
       Error: No plugins uninstalled.
       """
-    And STDOUT should be empty
     And the return code should be 1
 
     When I run `wp plugin deactivate Zombieland`
@@ -96,15 +92,20 @@ Feature: Manage WordPress plugins
     And the {PLUGIN_DIR}/zombieland file should not exist
 
     When I try the previous command again
-    Then STDERR should be:
+    Then STDERR should contain:
       """
-      Warning: The 'Zombieland' plugin could not be found.
+      Warning:
       """
-    And STDOUT should be:
+    And STDERR should contain:
       """
-      Success: Plugin already uninstalled.
+      Zombieland
       """
-    And the return code should be 0
+    And STDERR should contain:
+      """
+      Error: No plugins uninstalled.
+      """
+    And STDOUT should be empty
+    And the return code should be 1
 
   Scenario: Install a plugin, activate, then force install an older version of the plugin
     Given a WP install

src/Plugin_Command.php

@@ -810,7 +810,7 @@ public function get( $args, $assoc_args ) {
 	 *
 	 * ## OPTIONS
 	 *
-	 * <plugin>...
+	 * [<plugin>...]
 	 * : One or more plugins to uninstall.
 	 *
 	 * [--deactivate]
@@ -820,15 +820,30 @@ public function get( $args, $assoc_args ) {
 	 * : If set, the plugin files will not be deleted. Only the uninstall procedure
 	 * will be run.
 	 *
+	 * [--all]
+	 * : If set, all plugins will be uninstalled.
+	 *
 	 * ## EXAMPLES
 	 *
 	 *     $ wp plugin uninstall hello
 	 *     Uninstalled and deleted 'hello' plugin.
 	 *     Success: Installed 1 of 1 plugins.
 	 */
 	public function uninstall( $args, $assoc_args = array() ) {
+
+		$all = Utils\get_flag_value( $assoc_args, 'all', false );
+
+		// Check if plugin names of --all is passed.
+		if ( ! ( $args = $this->check_optional_args_and_all( $args, $all, 'uninstall' ) ) ) {
+			return;
+		}
+
 		$successes = $errors = 0;
 		$plugins = $this->fetcher->get_many( $args );
+		if ( count( $plugins ) < count( $args ) ) {
+			$errors = count( $args ) - count( $plugins );
+		}
+
 		foreach ( $plugins as $plugin ) {
 			if ( is_plugin_active( $plugin->file ) && ! WP_CLI\Utils\get_flag_value( $assoc_args, 'deactivate' ) ) {
 				WP_CLI::warning( "The '{$plugin->name}' plugin is active." );
@@ -1082,7 +1097,7 @@ private function get_all_plugins() {
 	 * @param bool $all All flag.
 	 * @return array Same as $args if not all, otherwise all slugs.
 	 */
-	private function check_optional_args_and_all( $args, $all ) {
+	private function check_optional_args_and_all( $args, $all, $verb = 'install' ) {
 		if ( $all ) {
 			$args = array_map( function( $file ){
 				return Utils\get_plugin_name( $file );
@@ -1093,7 +1108,9 @@ private function check_optional_args_and_all( $args, $all ) {
 			if ( ! $all ) {
 				WP_CLI::error( 'Please specify one or more plugins, or use --all.' );
 			}
-			WP_CLI::success( 'No plugins installed.' ); // Don't error if --all given for BC.
+
+			$past_tense_verb = Utils\past_tense_verb( $verb );
+			WP_CLI::success( "No plugins {$past_tense_verb}." ); // Don't error if --all given for BC.
 		}
 
 		return $args;