Implement $memory_cache in ExternalEntityStorage to avoid this deprecation notice:

The $memory_cache parameter was added in Drupal 8.6.x and will be required in 9.0.0. See https://www.drupal.org/node/2973262

CommentFileSizeAuthor
#3 external_entities-memory_cache-3012827-3.patch2.05 KBrp7
#2 memory_cache-3012827-2.patch1.72 KBpolynya
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

polynya created an issue. See original summary.

polynya’s picture

polynya CreditAttribution: polynya at Nomensa commented
FileSize
memory_cache-3012827-2.patch1.72 KB
rp7’s picture

rp7 CreditAttribution: rp7 commented
FileSize
external_entities-memory_cache-3012827-3.patch2.05 KB

Re-ordered the memory cache argument in the constructor (were still in DEV) + added parameter description.

  • rp7 committed cf31af8 on 8.x-2.x 
    Issue #3012827 by polynya, rp7: Inject the new entity.memory_cache...
rp7’s picture

rp7 CreditAttribution: rp7 commented
Status: Active » Fixed

Thanks for the patch polynya. Commited to 2.x.

Status: Fixed » Closed (fixed)

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

Kristen Pol’s picture

Kristen Pol
she/her
Primary language English
Location Santa Cruz, CA, USA
CreditAttribution: Kristen Pol at Hook 42 commented
Issue tags: +Drupal 9 compatibility

Per a Slack discussion with Gábor Hojtsy regarding usage of D9 tags (Drupal 9, Drupal 9 compatibility, Drupal 9 readiness, etc.), "Drupal 9 compatibility" should be used for contributed projects that need updating and "Drupal 9" was the old tag for D8 issues before the D9 branch was ready. Doing tag cleanup here based on that discussion.