Follow-up to #2571953: Remove !placeholder in syslog module
Problem/Motivation
See #2566503: [meta] Replace remaining !placeholder for Non-URL HTML outputs only
modules/system/src/Form/ModulesListForm.php: $reasons[] = $this->t('This version is not compatible with Drupal !core_version and should be replaced.', array(
modules/system/src/Form/ModulesListForm.php: $reasons[] = $this->t('This module requires PHP version @php_required and is incompatible with PHP version !php_version.', array(
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
Reference: https://www.drupal.org/core/beta-changes
| 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. |
Comments
Comment #2
nlisgo commentedGoing to see if I can help on this one.
Comment #3
nlisgo commentedComment #4
dawehnerPerfect!
Comment #5
alexpottCommitted 31905f4 and pushed to 8.0.x. Thanks!
Comment #8
nevergoneAlready committed.
Fixed.