Feature 681: Adding uninstall method#716
Feature 681: Adding uninstall method#716bluecollarcoders wants to merge 5 commits intoGatherPress:developfrom
Conversation
| global $wpdb; | ||
|
|
||
| // Delete Options. | ||
| $option_name = 'wporg_option'; |
There was a problem hiding this comment.
This was just an example. We need to remove things we store in the options table (GatherPress settings). In addition to deleting options, should delete user meta, taxonomies, events, venues, etc. Some things like post meta will be deleted automatically with the posts. We might want to delete things in options table by looking for anything that prepends with gatherpress_. What do you think @carstingaxion?
There was a problem hiding this comment.
Take a look at https://github.com/stephenharris/Event-Organiser/blob/65c4e3b0f40a601bebf7d5fc2203da02bbb4d7e9/includes/event-organiser-install.php#L404 as a nice example.
There was a problem hiding this comment.
What I especially liked about the linked implementation was:
- additional check if plugin is network-enabled
- deletion of user_meta
In addition:
For the multiple options I suggest an array function like array_walk calling delete_option on the list of all options and transients as well.
Preview changes with PlaygroundYou can preview the least recent changes for PR#716 by following one of the links below: |
|
Cleaning up old PRs |
3 participants
Description of the Change
This PR adds an uninstall method to clear out any plugin options, settings specific to the plugin, and/or other database entities such as tables.
Where possible, please also include:
Closes #
How to test the Change
Changelog Entry
Credits
Props @bluecollarcoders
Checklist: