- Issue created by @andypost
- Status changed to Needs review
almost 2 years ago 8:35pm 7 April 2023 - π«π·France andypost
I'm sure this clean-up could be backported to 9.5
- π«π·France andypost
For role names escaping was added in #2560863: #options for radios and checkboxes uses SafeMarkup::checkPlain() to escape - use Html::escape() instead β other places in #2545972: Remove all code usages SafeMarkup::checkPlain() and rely more on Twig autoescaping β
The last submitted patch, 2: 3352972-2.patch, failed testing. View results β
- Status changed to Needs work
almost 2 years ago 10:32pm 7 April 2023 - π«π·France andypost
Able to reproduce failures
- admin/structure/views/nojs/display/frontpage/page_1/display_title (set name to
<em>Page</em>
and save frontpage view
- admin/structure/views/nojs/display/frontpage/feed_1/displays should display escapedHere's how it looks
- Status changed to Needs review
almost 2 years ago 10:33pm 7 April 2023 - Assigned to Lendude
- π«π·France andypost
@Lendude I'm sure it needs your approval and roles like displays can have names with markup
- last update
over 1 year ago 29,401 pass - Status changed to Needs work
over 1 year ago 11:00pm 30 May 2023 - πΊπΈUnited States smustgrave
Tried testing on 11.x
Able to replicate the issue following #6
But after applying the patch issue still shows.