Child ticket of #1775842: [meta] Convert all variables to state and/or config systems.

There is a call to variable_get('cache') in language.inc that was missed when this variable was moved to the config system:

core/includes/language.inc
475 $cache = !isset($method['cache']) || $user->isAuthenticated() || $method['cache'] == variable_get('cache', 0);

For an example that was converted see:
7.x: http://drupalcode.org/project/drupal.git/blame/ace62d67ea77fdfe0e72a2eb3...
8.x: http://drupalcode.org/project/drupal.git/blame/7e1a0ca0bba94a66a60094b0d...

CommentFileSizeAuthor
#3 2101661-3.patch1.33 KBswentel
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

ianthomas_uk’s picture

ianthomas_uk
he/him
Primary language English
Location Brighton, UK
CreditAttribution: ianthomas_uk commented
Issue tags: +Configuration system

tag

catch’s picture

catch
he/him
Primary language English
CreditAttribution: catch commented
Priority: Normal » Critical
swentel’s picture

swentel CreditAttribution: swentel commented
Status: Active » Needs review
FileSize
2101661-3.patch1.33 KB
Berdir’s picture

Berdir
Primary language German
Location Switzerland
CreditAttribution: Berdir commented
Status: Needs review » Reviewed & tested by the community

Same code as in _drupal_bootstrap_page_cache().

catch’s picture

catch
he/him
Primary language English
CreditAttribution: catch commented
Status: Reviewed & tested by the community » Fixed

Committed/pushed to 8.x, thanks!

Gábor Hojtsy’s picture

Gábor Hojtsy
he/him
Primary language Hungarian
Location Hungary
CreditAttribution: Gábor Hojtsy commented
Issue tags: +D8MI, +language-base

D8MI tags.

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