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

Issue/1: Updates compatibility mode to store result in object. #21

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

alexstandiford
Copy link
Contributor

@alexstandiford alexstandiford commented May 27, 2022

This PR updates the use_compatibility_mode method to store the result in an object. This prevents the potentially expensive filter from running every time this method is called.

To Test

  1. Either clone this repository locally, or modify Nicholas to contain the updated version of Nicholas.php included in this PR.
  2. Compatibility mode still behaves as expected when using the site.

@alexstandiford alexstandiford linked an issue May 27, 2022 that may be closed by this pull request
@alexstandiford alexstandiford self-assigned this May 27, 2022
@josh-tt
Copy link

josh-tt commented Jun 1, 2022

! ) Fatal error: Uncaught Error: Class "Underpin\Scripts\Factories\Enqueue_Admin_Script" not found in /vendor/nicholas-wordpress/core/lib/Nicholas.php on line 413 when copying the update version in.

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

Successfully merging this pull request may close these issues.

Compatibility Mode Does Not Cache Result
2 participants