Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add data deletion for WPJM transients #1428

Merged
merged 1 commit into from
Apr 11, 2018

Conversation

alexsanford
Copy link
Contributor

Contributes to #1362

This PR, on plugin deletion, deletes the transients associated with WPJM from the database. The following transients (as regular expressions) should be deleted:

  • _job_manager_activation_redirect
  • get_job_listings-transient-version
  • jm_.*

Testing

  • First, you may want to back up your data, or copy it to a fresh WordPress installation.

  • Ensure that the tests pass.

  • Add some of the above transients to the database.

  • Delete the WPJM plugin.

  • Inspect the database to ensure that those transients have been deleted. You can use this query to see all the transients in the DB: SELECT option_id, option_name from wp_options WHERE option_name LIKE "_transient_%". You can also use a plugin like "Transients Manager" to check them.

  • Ensure that the transient itself and it's timeout value have been deleted (e.g. both _transient__job_manager_activation_redirect and _transient_timeout__job_manager_activation_redirect).

  • Please also test on a multisite installation. When the plugin is deleted from the Network Admin, the transients should be deleted across all sites.

@alexsanford alexsanford self-assigned this Apr 9, 2018
@alexsanford alexsanford requested review from jom and donnapep April 9, 2018 18:06
Copy link
Member

@jom jom left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, this does. 🌟

@alexsanford alexsanford force-pushed the add/data-deletion-transients branch from f40d547 to c862e32 Compare April 11, 2018 12:54
@alexsanford alexsanford merged commit 7faf9f6 into master Apr 11, 2018
@alexsanford alexsanford deleted the add/data-deletion-transients branch April 11, 2018 13:00
@jom jom added this to the 1.31.0 milestone Apr 11, 2018
@jom jom mentioned this pull request Apr 18, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants