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

This is a child of #1775842: [meta] Convert all variables to state and/or config systems

Files: 
CommentFileSizeAuthor
#3 2102503-3.patch1.02 KBswentel
PASSED: [[SimpleTest]]: [MySQL] 58,447 pass(es).
[ View ]

Comments

Category:task» bug

This 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.

Priority:Normal» Critical

Status:Active» Needs review
StatusFileSize
new1.02 KB
PASSED: [[SimpleTest]]: [MySQL] 58,447 pass(es).
[ View ]

Status:Needs review» Needs work
Issue tags:-Configuration system, -Config novice

The last submitted patch, 2102503-3.patch, failed testing.

Status:Needs work» Needs review

#3: 2102503-3.patch queued for re-testing.

Status:Needs review» Needs work

The last submitted patch, 2102503-3.patch, failed testing.

Status:Needs work» Needs review
Issue tags:+Configuration system, +Config novice

#3: 2102503-3.patch queued for re-testing.

Was broken because of #2057401: Make the node entity database schema sensible - should be good now.

Status:Needs review» Reviewed & tested by the community

We can think about injection and so on later on I think, let's just get rid of that variable_get().

Status:Reviewed & tested by the community» Fixed

Committed/pushed to 8.x, thanks!

Issue tags:+D8MI, +language-content

Add D8MI tags.

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

Issue summary:View changes

Updated issue summary.