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
DiffFormatter component uses
for indenting/presentation.
Twig auto-escape double encodes this to  
.
Proposed resolution
Marke
safe.
Remaining tasks
Review
User interface changes
None
API changes
None
NB: There is no Diff component - so using config system for now
Comment | File | Size | Author |
---|---|---|---|
#12 | 2314513-10-reroll.patch | 718 bytes | joelpittet |
#10 | 2314513-10.patch | 717 bytes | joelpittet |
diff-formatter-dbl-esc.pass_.patch | 1.44 KB | larowlan | |
diff-formatter-dbl-esc.fail_.patch | 849 bytes | larowlan | |
Comments
Comment #1
larowlanComment #3
jibranThanks.
Comment #4
chx CreditAttribution: chx commented#2280963: Refactor use of SafeMarkup in HWLDFWordAccumulator note: we are not adding more SafeMarkup::set() calls to core. This is not the way. We remove as much as we can instead.
Comment #5
jibranComment #6
jhodgdonAdding issue this is duplicate of as Related.
Comment #7
xjmThis was marked as a duplicate of #2280963: Refactor use of SafeMarkup in HWLDFWordAccumulator, which is now fixed, but did not add test coverage. Let's repurpose this issue to just add the test coverage from this patch?
Comment #10
joelpittetHere's the same patch but test only.
Comment #12
joelpittetReroll
Comment #13
joelpittetComment #14
xjmlol, three minors and counting. Let's get this in. ;)
Comment #15
alexpottCommitted and pushed e439d23 to 8.4.x and ee8f13d to 8.3.x. Thanks!
Backported to 8.3.x cause it is just additional test coverage.