Jetpack: check for other plugins before uninstall cleanup.

projects/packages/connection/changelog/update-jetpack-no-disconnect-on-uninstall

@@ -0,0 +1,4 @@
+Significance: minor
+Type: added
+
+Add a method to check if Jetpack is ready for uninstall cleanup.

projects/packages/connection/composer.json

@@ -65,7 +65,7 @@
 			"link-template": "https://github.com/Automattic/jetpack-connection/compare/v${old}...v${new}"
 		},
 		"branch-alias": {
-			"dev-trunk": "1.58.x-dev"
+			"dev-trunk": "1.59.x-dev"
 		}
 	},
 	"config": {

projects/packages/connection/src/class-manager.php

@@ -2558,4 +2558,18 @@ public static function get_site_id() {
 		}
 		return (int) $site_id;
 	}
+
+	/**
+	 * Check if Jetpack is ready for uninstall cleanup.
+	 *
+	 * @param string $current_plugin_slug The current plugin's slug.
+	 *
+	 * @return bool
+	 */
+	public static function is_ready_for_cleanup( $current_plugin_slug ) {
+		$active_plugins = get_option( Plugin_Storage::ACTIVE_PLUGINS_OPTION_NAME );
+
+		return empty( $active_plugins ) || ! is_array( $active_plugins )
+			|| ( count( $active_plugins ) === 1 && array_key_exists( $current_plugin_slug, $active_plugins ) );
+	}
 }

projects/packages/connection/src/class-package-version.php

@@ -12,7 +12,7 @@
  */
 class Package_Version {
 
-	const PACKAGE_VERSION = '1.58.3';
+	const PACKAGE_VERSION = '1.59.0-alpha';
 
 	const PACKAGE_SLUG = 'connection';
 

projects/packages/connection/tests/php/test_Manager_unit.php

@@ -643,4 +643,46 @@ public function test_disconnect_site_will_remove_tracked_package_versions() {
 	public function filter_api_constant( $value, $name ) {
 		return constant( __NAMESPACE__ . "\Utils::DEFAULT_$name" );
 	}
+
+	/**
+	 * Test the `is_ready_for_cleanup()` method with the negative result (has other connected plugins).
+	 *
+	 * @return void
+	 */
+	public function test_is_ready_for_cleanup_no() {
+		$option_filter = function () {
+			return array(
+				'jetpack-backup' => array(
+					'name' => 'Jetpack Backup',
+				),
+			);
+		};
+
+		add_filter( 'pre_option_jetpack_connection_active_plugins', $option_filter );
+		$is_ready = Manager::is_ready_for_cleanup( 'jetpack' );
+		remove_filter( 'pre_option_jetpack_connection_active_plugins', $option_filter );
+
+		$this->assertFalse( $is_ready );
+	}
+
+	/**
+	 * Test the `is_ready_for_cleanup()` method with the positive result (no other connected plugins).
+	 *
+	 * @return void
+	 */
+	public function test_is_ready_for_cleanup_yes() {
+		$option_filter = function () {
+			return array(
+				'jetpack' => array(
+					'name' => 'Jetpack',
+				),
+			);
+		};
+
+		add_filter( 'pre_option_jetpack_connection_active_plugins', $option_filter );
+		$is_ready = Manager::is_ready_for_cleanup( 'jetpack' );
+		remove_filter( 'pre_option_jetpack_connection_active_plugins', $option_filter );
+
+		$this->assertTrue( $is_ready );
+	}
 }

projects/plugins/backup/changelog/update-jetpack-no-disconnect-on-uninstall

@@ -0,0 +1,5 @@
+Significance: patch
+Type: changed
+Comment: Updated composer.lock.
+
+

projects/plugins/boost/changelog/update-jetpack-no-disconnect-on-uninstall

@@ -0,0 +1,5 @@
+Significance: patch
+Type: changed
+Comment: Updated composer.lock.
+
+

projects/plugins/inspect/changelog/update-jetpack-no-disconnect-on-uninstall

@@ -0,0 +1,5 @@
+Significance: patch
+Type: changed
+Comment: Updated composer.lock.
+
+

projects/plugins/jetpack/changelog/update-jetpack-no-disconnect-on-uninstall

@@ -0,0 +1,4 @@
+Significance: patch
+Type: compat
+
+Do not cleanup options on plugin uninstall if there are other connected plugins to let them stay connected.

projects/plugins/jetpack/uninstall.php

@@ -6,6 +6,7 @@
  */
 
 use Automattic\Jetpack\Backup\Helper_Script_Manager;
+use Automattic\Jetpack\Connection\Manager as Connection_Manager;
 use Automattic\Jetpack\Sync\Sender;
 
 /**
@@ -27,6 +28,11 @@ function jetpack_uninstall() {
 
 	require JETPACK__PLUGIN_DIR . 'vendor/autoload_packages.php';
 
+	if ( method_exists( Connection_Manager::class, 'is_ready_for_cleanup' ) && ! Connection_Manager::is_ready_for_cleanup( dirname( plugin_basename( __FILE__ ) ) ) ) {
+		// There are other active Jetpack plugins, no need for cleanup.
+		return;
+	}
+
 	Jetpack_Options::delete_all_known_options();
 
 	// Delete all legacy options.

projects/plugins/migration/changelog/update-jetpack-no-disconnect-on-uninstall

@@ -0,0 +1,5 @@
+Significance: patch
+Type: changed
+Comment: Updated composer.lock.
+
+

projects/plugins/migration/composer.json

@@ -72,6 +72,6 @@
 			"automattic/jetpack-autoloader": true,
 			"automattic/jetpack-composer-plugin": true
 		},
-		"autoloader-suffix": "c4802e05bbcf59fd3b6350e8d3e5482c_migrationⓥ1_0_1"
+		"autoloader-suffix": "c4802e05bbcf59fd3b6350e8d3e5482c_migrationⓥ1_0_2_alpha"
 	}
 }

projects/plugins/migration/wpcom-migration.php

@@ -4,7 +4,7 @@
  * Plugin Name: Move to WordPress.com
  * Plugin URI: https://wordpress.org/plugins/wpcom-migration
  * Description: A WordPress plugin that helps users to migrate their sites to WordPress.com.
- * Version: 1.0.1
+ * Version: 1.0.2-alpha
  * Author: Automattic
  * Author URI: https://wordpress.com/
  * License: GPLv2 or later

projects/plugins/protect/changelog/update-jetpack-no-disconnect-on-uninstall

@@ -0,0 +1,5 @@
+Significance: patch
+Type: changed
+Comment: Updated composer.lock.
+
+

projects/plugins/search/changelog/update-jetpack-no-disconnect-on-uninstall

@@ -0,0 +1,5 @@
+Significance: patch
+Type: changed
+Comment: Updated composer.lock.
+
+

projects/plugins/social/changelog/update-jetpack-no-disconnect-on-uninstall

@@ -0,0 +1,5 @@
+Significance: patch
+Type: changed
+Comment: Updated composer.lock.
+
+

projects/plugins/starter-plugin/changelog/update-jetpack-no-disconnect-on-uninstall

@@ -0,0 +1,5 @@
+Significance: patch
+Type: changed
+Comment: Updated composer.lock.
+
+

projects/plugins/videopress/changelog/update-jetpack-no-disconnect-on-uninstall

@@ -0,0 +1,5 @@
+Significance: patch
+Type: changed
+Comment: Updated composer.lock.
+
+