Add uninstall.php or uninstall routine

[carstingaxion]

Is your enhancement related to a problem? Please describe.

Not that I want anybody to uninstall GatherPress, but people should be able to do that without any worries.

Following WordPress best-practices for uninstallation, GatherPress should have something like this somewhere.

// if uninstall.php is not called by WordPress, die
if ( ! defined( 'WP_UNINSTALL_PLUGIN' ) ) {
    die;
}

$option_name = 'wporg_option';

delete_option( $option_name );

// for site options in Multisite
delete_site_option( $option_name );

// drop a custom database table
global $wpdb;
$wpdb->query( "DROP TABLE IF EXISTS {$wpdb->prefix}mytable" );

In Multisite, looping through all blogs to delete options can be very resource intensive.

Designs

Shouldn't be to much effort, if the existing & related methods can be re-used and we don't forget about the transients in #680

gatherpress/includes/core/classes/class-setup.php

Lines 196 to 198 in 4942ce8

	public function deactivate_gatherpress_plugin(): void {
	flush_rewrite_rules();
	}

and

gatherpress/includes/core/classes/class-setup.php

Lines 307 to 314 in 4942ce8

	public function on_site_delete( array $tables ): array {
	global $wpdb;
	$tables[] = sprintf( Event::TABLE_FORMAT, $wpdb->prefix, Event::POST_TYPE );
	$tables[] = sprintf( Rsvp::TABLE_FORMAT, $wpdb->prefix );
	return $tables;
	}

Describe alternatives you've considered

No response

Code of Conduct

• I agree to follow this project's Code of Conduct

[MervinHernandez]

✅ Viewed 681 - Uninstall workflows - Caleb has it

[MervinHernandez]

Hi @bluecollarcoders where do we stand on drafting an uninstall workflow?

[bluecollarcoders]

Hi @MervinHernandez

I started work on the ticket. I apologize for the late response.

[mauteri]

It would be good to list all the data that needs to be removed, so @bluecollarcoders knows what the scope is. I'll look to get a list of everything some time this week and post it to the ticket. Thx!

[MervinHernandez]

#wcus_2024
@mauteri have we discussed - uninstall should yank settings only or should it also yank data?

[MervinHernandez]

✅ Answer = yank settings + data + custom tables