Make WordPress Core

Opened 7 years ago

Closed 7 years ago

Last modified 7 years ago

#46814 closed defect (bug) (fixed)

Site Health reports "A plugin has prevented updates" when no plugins are activated.

Reported by: axaak Owned by: SergeyBiryukov
Priority: normal Milestone: 5.2
Component: Site Health Version: 5.2
Severity: normal Keywords: site-health has-patch commit
Cc: Focuses:

Description

*Steps to re-create*

Version: WordPress 5.2-beta1-45112
Deactivate all plugins
Run site check get message

A plugin has prevented updates by disabling wp_version_check().

but no plugins are activated. The site cannot be updated but not because of plugins. Better wording might be "Automatic updates are disabled"

Attachments (2)

snapshot3.png (123.1 KB ) - added by axaak 7 years ago.
A plugin has prevented message
46814.patch (844 bytes ) - added by Clorith 7 years ago.

Download all attachments as: .zip

Change History (7)

@axaak
7 years ago

A plugin has prevented message

#1 @knutsp
7 years ago

  • Component GeneralAdministration
  • Resolutionduplicate
  • Status newclosed
  • Versiontrunk

Duplicate of #46616.

Hello @axaak

Welcome back and thanks for the report.

It seems, however, this has already been fixed.

#2 @Clorith
7 years ago

  • Keywords needs-patch added
  • Milestone Awaiting Review5.2
  • Resolution duplicate
  • Status closedreopened

This appears to be a result of the test site running on https without a valid certificate.

We do a call back to the page to test for the wp_version_check filter, and the wp_remote_get call will fail with an invalid certificate, causing it to show an error state.

Sounds like it would be more valid to show a warning state if we can't load the result page.

@Clorith
7 years ago

#3 @Clorith
7 years ago

  • Keywords has-patch commit added; needs-patch removed

46814.patch applies a check for errors in the request, and marks it accordingly if it's a test failure, or an incomplete test.

This means if the only thing we can't do is test for this filter, due to a broken certificate or similar, it will get a recommended status with an appropriate associated text.

#4 @SergeyBiryukov
7 years ago

  • Owner set to SergeyBiryukov
  • Resolutionfixed
  • Status reopenedclosed

In 45118:

Site Health: Display a correct error message for a failing request in wp_version_check() test.

Props Clorith, axaak.
Fixes #46814.

#5 @spacedmonkey
7 years ago

  • Component AdministrationSite Health
Note: See TracTickets for help on using tickets.