Problem/Motivation
The ContentTranslationController uses a variable called 'anonymous', but the variable system will not be available in Drupal 8.
core/modules/content_translation/lib/Drupal/content_translation/ContentTranslationController.php
280 '#description' => t('Leave blank for %anonymous.', array('%anonymous' => variable_get('anonymous', t('Anonymous')))),
Proposed resolution
Convert this variable to CMI, or remove it in favour of standard translation features supported by t()
Remaining tasks
User interface changes
API changes
Related Issues
This is a child of #1775842: [meta] Convert all variables to state and/or config systems
Comment | File | Size | Author |
---|---|---|---|
#3 | 2102503-3.patch | 1.02 KB | swentel |
Comments
Comment #1
BerdirThis has been moved to \Drupal::config('user.settings')->get('anonymous'). Should be an easy fix.
As this is just a left-over and was already converted, I'd say this is a bug, not a task.
Comment #2
catchComment #3
swentel CreditAttribution: swentel commentedComment #5
Berdir#3: 2102503-3.patch queued for re-testing.
Comment #7
swentel CreditAttribution: swentel commented#3: 2102503-3.patch queued for re-testing.
Comment #8
swentel CreditAttribution: swentel commentedWas broken because of #2057401: Make the node entity database schema sensible - should be good now.
Comment #9
BerdirWe can think about injection and so on later on I think, let's just get rid of that variable_get().
Comment #10
catchCommitted/pushed to 8.x, thanks!
Comment #11
Gábor HojtsyAdd D8MI tags.
Comment #12.0
(not verified) CreditAttribution: commentedUpdated issue summary.