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

Apply check_package() on bulk_upgrade #371

Closed
wants to merge 2 commits into from

Conversation

salcode
Copy link

@salcode salcode commented Aug 25, 2023

I'm opening this as a Draft PR for reference, as these changes should resolve #357

However, I believe this is a WordPress core issue and have opened Trac Ticket 59198 in the hopes it can be resolved there.


Add the check_package() function to the
"upgrader_source_selection" filter hook.

This applies the check for PHP minimum version number
before "bulk" updating a plugin.

Note: A "bulk" upgrade occurs even when a single plugin is updated,
e.g. the following uses the "bulk" upgrade code

wp plugin update gutenberg

Resolves #357

Add the check_package() function to the
"upgrader_source_selection" filter hook.

This applies the check for PHP minimum version number
before "bulk" updating a plugin.

Note: A "bulk" upgrade occurs even when a single plugin is updated,
e.g. the following uses the "bulk" upgrade code

wp plugin update gutenberg

Resolves wp-cli#357
@danielbachhuber
Copy link
Member

Thanks for diving into this, @salcode !

It looks like the issue is fixed in trunk now, and may be backported to earlier versions of WordPress. The upstream fix is most appropriate, I think.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
2 participants