Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.Problem/Motivation
* @param string $cache_key
* Cache key prefix to use, the language code will be appended
* automatically.
I can't find anything that would append the language code. I can find, however, three plugin managers doing
$this->setCacheBackend($cache_backend, 'contextual_links_plugins:' . $language_manager->getCurrentLanguage()->getId(), array('contextual_links_plugins'));
$this->setCacheBackend($cache_backend, 'local_action_plugins:' . $language_manager->getCurrentLanguage()->getId(), array('local_action'));
$this->setCacheBackend($cache, 'local_task_plugins:' . $language_manager->getCurrentLanguage()->getId(), array('local_task'));
so either these three are wrong or the doxygen is wrong.
Proposed resolution
Remaining tasks
User interface changes
API changes
Data model changes
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | 2668752-3.patch | 1.06 KB | mr.baileys |
Comments
Comment #2
tim.plunkett#2281905: Stop caching plugin discovery/info hooks by language changed this, looks like the doxygen was left unchanged
Comment #3
mr.baileysComment #4
chx CreditAttribution: chx at Smartsheet commentedThanks for the patch! I haven't rolled one because I am unsure: what do we say about what those three are doing? If anyone else stumbled on them like I did , why? Do we leave a comment on them? Do we leave a comment here ? Just leave it alone?
Comment #8
BerdirI think just removing the wrong information is enough.
Comment #9
xjmWorks for me. Thanks @Berdir! Committed to 8.4.x and cherry-picked to 8.3.x.