Currently the module checks:

if (FALSE && $cache = cache_get('speedy_min_js_files')) {

I have a feeling this was on purpose since the current cache is permanent and there's no mechanism in the module to invalidate the cache.

But, that means the cache is never used. And that we are generating a new cache on each page load.

CommentFileSizeAuthor
#1 1563792-1-cache.patch1.04 KBjohnalbin

Comments

johnalbin’s picture

johnalbin
they/他
Primary language English
Location Taipei, Taiwan
commented
Status: Active » Needs review
StatusFileSize
new 1563792-1-cache.patch1.04 KB

I made the cache “temporary” so that it is cleared along with other cache when "the cache is cleared". http://api.drupal.org/api/drupal/includes%21cache.inc/function/cache_set/7

mfer’s picture

mfer commented
Status: Needs review » Fixed

Committed. Thanks for catching this. My debug code was still lingering around.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

Anonymous’s picture

(not verified) commented
Issue summary: View changes

Fix broken code comment.