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
See #2566503: [meta] Replace remaining !placeholder for Non-URL HTML outputs only
modules/language/src/Element/LanguageConfiguration.php: LanguageInterface::LANGCODE_SITE_DEFAULT => t("Site's default language (!language)", array('!language' => static::languageManager()->getDefaultLanguage()->getName())),
modules/language/src/Plugin/Derivative/LanguageBlock.php: $this->derivatives[$type]['admin_label'] = t('Language switcher (!type)', array('!type' => $info[$type]['name']));
Proposed resolution
Remaining tasks
Agree that removing HTML support makes sense.
User interface changes
None
API changes
Date format strings no longer support adding HTML using the \ escape character.
Data model changes
None
Beta phase evaluation
| Issue category | Bug because at the moment date formats support HTML but it is escaped |
|---|---|
| Issue priority | Major because part of the critical to remove !placeholder |
| Disruption | Disruptive for existing sites that are adding HTML to date formats. If HTML is required in a formatted date then the site should implement a custom field formatter to do this. |
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | 2571947-remove-placeholder-language-module.patch | 984 bytes | nevergone |
Comments
Comment #2
nevergone CreditAttribution: nevergone commentedComment #3
dawehnerIt is perfect!
Comment #4
alexpottWhen reviewing this I debated whether or not we should be getting the translated language name but that would be a change and therefore out of scope.
Committed ce3f378 and pushed to 8.0.x. Thanks!
Comment #7
nevergone CreditAttribution: nevergone commentedAlready committed.
Fixed.