-
Notifications
You must be signed in to change notification settings - Fork 101
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
Disable Divi caching #5597
base: develop
Are you sure you want to change the base?
Disable Divi caching #5597
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #5597 +/- ##
=============================================
+ Coverage 29.52% 29.58% +0.06%
Complexity 4780 4780
=============================================
Files 282 282
Lines 20577 20582 +5
=============================================
+ Hits 6076 6090 +14
+ Misses 14501 14492 -9 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- We should prevent a slew of php warnings for customer's currently defining these. So like:
if ( ! defined( 'ET_DISABLE_FILE_BASED_CACHE' ) ) {
define( 'ET_DISABLE_FILE_BASED_CACHE', true );
}
-
Should we also be including the rest of the logic mentioned here (with additional safeguards to not register and constant that is already set): https://docs.wpvip.com/plugins/incompatibilities/#divi ?
-
Lastly, let's try to limit the scope of the code to only running when we're sure it is needed - i.e. Divi is present on the site. So wrapping all of the logic inside of something like
if defined( 'ET_CORE_VERSION' ) { ... }
. Load order will be important though, might have to hook it all ontoafter_setup_theme
for example.
0ebb89d
to
b2f7446
Compare
144cd84
to
aebaaad
Compare
Thanks for the detailed feedback @WPprodigy! I've applied changes in e7d167b |
Quality Gate passedIssues Measures |
@kat3samsin thanks! I'm not entirely sure how Divi caching works, is it appropriate for it to use VIP filesystem at all? |
@rinatkhaziev thanks for looking.
From what I understand, the dynamic loading (dynamic css) feature of Divi is trying to write in |
Description
Added disabling of Divi Caching by default. VIP already supports page and object caching Divi Caching can cause bloated media directory.
Docs
Changelog Description
Disables Divi plugin caching
Added disabling of Divi Caching by default.
Pre-review checklist
Please make sure the items below have been covered before requesting a review:
Pre-deploy checklist
Steps to Test