Static cache for File entities in D8 needs backport to D7
From issue https://www.drupal.org/node/1292374
TODO:
- add $reset parameter to file_load() and file_load_multiple(),
- use 'static cache' => TRUE by default in system_entity_info,
- improve tests with $reset parameter.

CommentFileSizeAuthor
#8 backport_file_static_caching-2678618-8-D7.patch12.24 KBfortis
#5 backport_file_static_caching-2678618-5-D7.patch7.65 KBfortis
#2 backport_file_static_caching-2678618-2-D7.patch6.18 KBfortis
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

fortis created an issue. See original summary.

fortis’s picture

fortis CreditAttribution: fortis as a volunteer and at Redditor commented
FileSize
backport_file_static_caching-2678618-2-D7.patch6.18 KB

if you're able to review and test the patch, please do)

fortis’s picture

fortis CreditAttribution: fortis as a volunteer and at Redditor commented
Status: Active » Needs review

Status: Needs review » Needs work

The last submitted patch, 2: backport_file_static_caching-2678618-2-D7.patch, failed testing.

fortis’s picture

fortis CreditAttribution: fortis as a volunteer and at Redditor commented
FileSize
backport_file_static_caching-2678618-5-D7.patch7.65 KB

small fixes, now it should pass

fortis’s picture

fortis CreditAttribution: fortis as a volunteer and at Redditor commented
Status: Needs work » Needs review

Status: Needs review » Needs work

The last submitted patch, 5: backport_file_static_caching-2678618-5-D7.patch, failed testing.

fortis’s picture

fortis CreditAttribution: fortis as a volunteer and at Redditor commented
Status: Needs work » Needs review
FileSize
backport_file_static_caching-2678618-8-D7.patch12.24 KB

added $reset parameter in tests