Toolbar does not namespace its cache item with 'toolbar_' making identification difficult and collision more likely.

CommentFileSizeAuthor
#1 cache-id-name-change-2157855-1.patch5.69 KBjessebeach
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jessebeach’s picture

jessebeach CreditAttribution: jessebeach commented
Status: Active » Needs review
Issue tags: +Spark, +sprint
FileSize
cache-id-name-change-2157855-1.patch5.69 KB

A one-line fix and updated tests to reflect the string change.

moshe weitzman’s picture

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

Good and proper.

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
CreditAttribution: webchick commented
Status: Reviewed & tested by the community » Fixed

Committed and pushed to 8.x. Thanks!

Status: Fixed » Closed (fixed)

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

Wim Leers’s picture

Wim Leers
Location Ghent 🇧🇪🇪🇺
CreditAttribution: Wim Leers commented
Issue tags: -sprint