Skip to content

/ db-command

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

Make wp db clean get all the tables with prefix #152

Merged
merged 2 commits into from Nov 3, 2019
Merged

Make wp db clean get all the tables with prefix #152

merged 2 commits into from Nov 3, 2019
+1 −1

Conversation

@marksabbath
Copy link
Collaborator

marksabbath commented Nov 3, 2019

This PR fixes the issue where wp db clean was not dropping tables that use the same database prefix, but weren't in the WordPress schema.

Fixes #150
@marksabbath
Make wp db clean get all the tables with prefix
Loading status checks…
7caec75
@marksabbath marksabbath requested a review from wp-cli/committers as a code owner Nov 3, 2019
@marksabbath
Remove extra line
Loading status checks…
0c992e7
@schlessera
schlessera approved these changes Nov 3, 2019
View changes
@schlessera schlessera added this to the 2.0.5 milestone Nov 3, 2019
@schlessera schlessera merged commit a5bcd3e into master Nov 3, 2019
1 check passed
1 check passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@schlessera schlessera deleted the fix/db-clean branch Nov 3, 2019
@schlessera
Copy link
Member

schlessera commented Nov 3, 2019

Thanks for the PR, @marksabbath !
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@schlessera schlessera

Assignees
No one assigned
Labels
bug command:db-clean
Milestone
2.0.5
Linked issues

Successfully merging this pull request may close these issues.

Db clean not removing all tables with $table_prefix
2 participants
@marksabbath @schlessera
You can’t perform that action at this time.