Problem/Motivation

API calls are currently being made in savings_united_preprocess_block(), which are uncached.

Proposed resolution

Implement global caching by default that can be altered by the site (e.g. cache per term, etc).

This could be achieved using cache tags to generate the cache keys.

CommentFileSizeAuthor
#2 3021272-2-caching.patch2.02 KBdavidgrayston
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

davidgrayston created an issue. See original summary.

davidgrayston’s picture

davidgrayston CreditAttribution: davidgrayston at Dennis commented
FileSize
3021272-2-caching.patch2.02 KB

Patch attached - opted for the simpler approach of caching per API request URL

davidgrayston’s picture

davidgrayston CreditAttribution: davidgrayston at Dennis commented
Status: Needs work » Needs review
davidgrayston’s picture

davidgrayston CreditAttribution: davidgrayston at Dennis commented
Issue summary: View changes
guardiola86’s picture

guardiola86 CreditAttribution: guardiola86 commented
Status: Needs review » Reviewed & tested by the community

Thanks, your patch works well. I'll merge it.

guardiola86’s picture

guardiola86 CreditAttribution: guardiola86 commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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