Skip to content

Conversation

@8ctopus
Copy link

@8ctopus 8ctopus commented May 6, 2025

Fix caching takes place in multi-site if plugin is activated on one site and disabled on another one that is a subfolder of the first one

Proposed fix for #343

…ite and disabled on another one that is a subfolder of the first one
@futtta
Copy link
Contributor

futtta commented May 17, 2025

I can confirm this clean & simple change works on my local multisite install. Maybe add an is_multisite() check in there as extra condition?

@8ctopus
Copy link
Author

8ctopus commented May 19, 2025

@futtta Thank you for testing it. What would is_multisite() bring?

@futtta
Copy link
Contributor

futtta commented May 19, 2025

It would make sure to limit the behavior to multisite, where the actual problem occurs?

@8ctopus
Copy link
Author

8ctopus commented May 20, 2025

@futtta correct me if I'm wrong, my understanding is that this code is only executed on single-site if the plugin is enabled, therefore not strictly necessary.

@8ctopus
Copy link
Author

8ctopus commented May 30, 2025

I had to move the code because otherwise the page was never delivered from cache if Apache was not delivering the cache.

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.

2 participants