Skip to content

Broken “Re-enable transient caching” button for “Transient caching of parsed stylesheets is disabled” site health test #6549

Closed
@westonruter

Description

@westonruter

Bug Description

The Site Health test for “Transient caching of parsed stylesheets is disabled” has a “Re-enable transient caching” button that is supposed to dismiss the site health test, but as can be seen here there is something wrong with the button. The styling seems to be missing:

image

Also, clicking the button has no effect. So the JS is also not being enqueued on this page.

It seems the problem is that I currently have the “Health Check & Troubleshooting” plugin active and so I'm accessing the page via /wp-admin/tools.php?page=health-check.

But when that plugin is deactivated, then the Site Health page is located at /wp-admin/site-health.php and here I can see the Site Health test as expected:

image

Nevertheless, even though the styling looks right, the button still doesn't do anything. So a required JS file is missing.

Expected Behaviour

The stylesheet transient caching site health test should work properly when the “Health Check & Troubleshooting” plugin is active or deactivated.

Steps to reproduce

  1. Manually disable stylesheet transient caching by running WP-CLI wp shell and execute AMP_Options_Manager::update_option('amp_css_transient_monitor_disable_caching', true)
  2. Go to the Site Health screen (with and without the “Health Check & Troubleshooting” plugin active)
  3. Try to re-enable transient caching by clicking the button.

Do not alter or remove anything below. The following sections will be managed by moderators only.

Acceptance criteria

Implementation brief

QA testing instructions

Demo

Changelog entry

Metadata

Metadata

Assignees

Labels

BugSomething isn't working

Type

No type

Projects

No projects

Milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions