Delete remote posts on plugin uninstall (#2632)

Author: pfefferle

.github/changelog/2632-from-description

@@ -0,0 +1,4 @@
+Significance: minor
+Type: added
+
+Delete remote posts on plugin uninstall.

includes/class-activitypub.php

@@ -9,6 +9,7 @@
 
 use Activitypub\Collection\Followers;
 use Activitypub\Collection\Following;
+use Activitypub\Collection\Posts;
 
 /**
  * ActivityPub Class.
@@ -85,6 +86,8 @@ public static function uninstall() {
 		\remove_filter( 'pre_wp_update_comment_count_now', array( Comment::class, 'pre_wp_update_comment_count_now' ) );
 		Migration::update_comment_counts( 2000 );
 
+		Posts::delete_all();
+
 		Options::delete();
 	}
 

includes/class-cli.php

@@ -10,6 +10,7 @@
 use Activitypub\Activity\Activity;
 use Activitypub\Collection\Actors;
 use Activitypub\Collection\Outbox;
+use Activitypub\Collection\Posts;
 use Activitypub\Scheduler\Actor;
 
 /**
@@ -107,6 +108,13 @@ private function execute_self_destruct( $assoc_args ) {
 		}
 
 		$processed = $this->process_user_deletions( $user_ids );
+
+		// Delete all remote posts.
+		$deleted_posts = Posts::delete_all();
+		if ( $deleted_posts > 0 ) {
+			\WP_CLI::line( \WP_CLI::colorize( "%G✓%n Deleted {$deleted_posts} remote post(s)." ) );
+		}
+
 		$this->display_completion_message( $processed );
 	}
 
@@ -123,6 +131,7 @@ private function display_self_destruct_warning() {
 		\WP_CLI::line( \WP_CLI::colorize( "%y{$question}%n" ) );
 		\WP_CLI::line( \WP_CLI::colorize( '%y• Send Delete activities to all followers%n' ) );
 		\WP_CLI::line( \WP_CLI::colorize( '%y• Remove your blog from ActivityPub networks%n' ) );
+		\WP_CLI::line( \WP_CLI::colorize( '%y• Delete all cached remote posts%n' ) );
 		\WP_CLI::line( '' );
 	}
 

includes/collection/class-posts.php

@@ -453,6 +453,30 @@ public static function remove_recipient( $post_id, $user_id ) {
 		return \delete_post_meta( $post_id, '_activitypub_user_id', $user_id );
 	}
 
+	/**
+	 * Delete all posts.
+	 *
+	 * Used during plugin uninstall to clean up all remote posts.
+	 *
+	 * @return int The number of posts deleted.
+	 */
+	public static function delete_all() {
+		$post_ids = \get_posts(
+			array(
+				'post_type'   => self::POST_TYPE,
+				'post_status' => array( 'any', 'trash', 'auto-draft' ),
+				'fields'      => 'ids',
+				'numberposts' => -1,
+			)
+		);
+
+		foreach ( $post_ids as $post_id ) {
+			\wp_delete_post( $post_id, true );
+		}
+
+		return count( $post_ids );
+	}
+
 	/**
 	 * Purge old remote posts.
 	 *

tests/phpunit/tests/includes/collection/class-test-posts.php

@@ -1343,6 +1343,85 @@ public function test_remove_hashtags_preserves_structure() {
 		$this->assertStringNotContainsString( '#php', $result );
 	}
 
+	/**
+	 * Test delete_all method deletes all posts.
+	 *
+	 * @covers ::delete_all
+	 */
+	public function test_delete_all() {
+		// Create some posts.
+		self::factory()->post->create_many(
+			5,
+			array(
+				'post_type'   => Posts::POST_TYPE,
+				'post_status' => 'publish',
+			)
+		);
+
+		// Verify posts were created.
+		$count_before = \wp_count_posts( Posts::POST_TYPE )->publish;
+		$this->assertEquals( 5, $count_before );
+
+		// Delete all posts.
+		$deleted = Posts::delete_all();
+
+		// Clear cache to get accurate count.
+		\wp_cache_delete( \_count_posts_cache_key( Posts::POST_TYPE ), 'counts' );
+
+		// Verify all posts were deleted.
+		$count_after = \wp_count_posts( Posts::POST_TYPE )->publish;
+		$this->assertEquals( 0, $count_after );
+
+		// Verify return value.
+		$this->assertEquals( 5, $deleted );
+	}
+
+	/**
+	 * Test delete_all method with mixed post statuses.
+	 *
+	 * @covers ::delete_all
+	 */
+	public function test_delete_all_mixed_statuses() {
+		// Create posts with different statuses.
+		self::factory()->post->create_many(
+			3,
+			array(
+				'post_type'   => Posts::POST_TYPE,
+				'post_status' => 'publish',
+			)
+		);
+		self::factory()->post->create_many(
+			2,
+			array(
+				'post_type'   => Posts::POST_TYPE,
+				'post_status' => 'draft',
+			)
+		);
+		self::factory()->post->create(
+			array(
+				'post_type'   => Posts::POST_TYPE,
+				'post_status' => 'trash',
+			)
+		);
+
+		// Delete all posts.
+		$deleted = Posts::delete_all();
+
+		// Verify all posts were deleted regardless of status.
+		$remaining = \get_posts(
+			array(
+				'post_type'   => Posts::POST_TYPE,
+				'post_status' => 'any',
+				'numberposts' => -1,
+				'fields'      => 'ids',
+			)
+		);
+		$this->assertEmpty( $remaining );
+
+		// Verify return value includes all posts.
+		$this->assertEquals( 6, $deleted );
+	}
+
 	/**
 	 * Test purge method with more than 200 posts.
 	 *