Details
-
Bug
-
Status: Closed
-
Major
-
Resolution: Fixed
-
3.8.3, 3.9
-
MOODLE_38_STABLE, MOODLE_39_STABLE
-
MOODLE_38_STABLE
-
MDL-68677-master -
Description
MDL-66367 introduced a M.cfg.templaterev to separate out the cachejs and cachetemplates better.
Right now that value works inconsistently and will allow some things to be cached, and others not.
We have a weird distinction between templatePromises and cachePartialPromises and the templaterev is allowing templatePromises to be cache don page, but not cachePartialPromises.
This variable was intended to only prevent use of the browser Local Storage API for persistent caching. Right now it prevents even use of some in-page caching.
We need to adjust the core/templates and make the templaterev check a few lines later so that it allows in-page caching.
This makes developer experience much more pleasing.