beautytips_manager_get_custom_tips caches disabled tips, wasting time and memory on our system. the function could change to cache only enabled tips and the management page could be changed to select all direct. The management page would then get everything while the more frequently used pages would get only the enabled tips.

CommentFileSizeAuthor
#1 beautytips_cache.patch1.85 KBpeterx
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

peterx’s picture

peterx CreditAttribution: peterx commented
FileSize
beautytips_cache.patch1.85 KB

Thought I would work on a patch. First change the code to have the cache name defined in one place.

Then move the cache clear all to one place. Cache clear all is used in at least two places. Also my add-on module uses the cache clear all function, the main reason why I moved it out to a separate function.

pifagor’s picture

pifagor
Location 🇺🇦 Rivne
CreditAttribution: pifagor as a volunteer and at Drupal Ukraine Community for Drupal Ukraine Community commented
Status: Active » Closed (outdated)