Skip to content

Avoid deleting theme if the specified version cannot be found#349

Merged
schlessera merged 3 commits intomainfrom
144-avoid-deleting-theme
Feb 7, 2023
Merged

Avoid deleting theme if the specified version cannot be found#349
schlessera merged 3 commits intomainfrom
144-avoid-deleting-theme

Conversation

@danielbachhuber
Copy link
Member

@danielbachhuber danielbachhuber commented Jan 27, 2023

Fixes #144

The "delete before install" behavior was originally added in wp-cli/wp-cli#1510

Interestingly, it doesn't appear necessary and may have been copy paste from the wp plugin update code.

Seems like it should be fine to remove if all of the existing tests pass. I moved a few other relevant tests into my new feature file.

@danielbachhuber danielbachhuber requested a review from a team January 27, 2023 18:39
@danielbachhuber danielbachhuber marked this pull request as ready for review January 27, 2023 18:40
@danielbachhuber danielbachhuber self-assigned this Jan 27, 2023
@danielbachhuber danielbachhuber added bug command:theme-update Related to 'theme update' command command:theme Related to 'theme' command labels Jan 27, 2023
@danielbachhuber danielbachhuber added this to the 2.1.10 milestone Jan 27, 2023
@schlessera
Copy link
Member

The "delete before install" behavior was originally added in wp-cli/wp-cli#1510

It was already added prior to that, with wp-cli/wp-cli@d4fa0c6

@schlessera schlessera merged commit d89798a into main Feb 7, 2023
@schlessera schlessera deleted the 144-avoid-deleting-theme branch February 7, 2023 05:53
msaladna added a commit to apisnetworks/apiscp-webapp-wordpress that referenced this pull request Jul 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug command:theme Related to 'theme' command command:theme-update Related to 'theme update' command

Projects

None yet

Development

Successfully merging this pull request may close these issues.

wp theme update testtheme --version=xxx deletes theme if not in WP gallery

2 participants