Follow-up to #2571923: Remove HTML support from date formats and replace remaining !placeholder for aggregator
See #2566503: [meta] Replace remaining !placeholder for Non-URL HTML outputs only
modules/book/src/BookManager.php: '#description' => $this->t('The parent page in the book. The maximum depth for a book and all child pages is !maxdepth. Some pages in the selected book may not be available as parents if selecting them would exceed this limit.', a$ray('!maxdepth' => static::BOOK_MAX_DEPTH)),
Agree that removing HTML support makes sense.
None
Date format strings no longer support adding HTML using the \ escape character.
None
| 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 | 2571937_1.patch | 1.22 KB | chx |
Comments
Comment #2
chx commentedThis must have been some micro optimization approximately one billion years ago.
Comment #3
dawehnerThis is just perfect! I'm really curious indeed how this ever went in.
Comment #4
chx commentedI can tell you: if it didn't need escaping why escape it? That was the logic, perfectly valid at the time. It's not even microoptimization if avoiding a performance hit, no matter how miniscule, does not take effort and does not add complexity.
Comment #5
alexpottCommitted db10fff and pushed to 8.0.x. Thanks!
Comment #8
chx commentedBot timing is awful.