We are unnecessarily escaping the values in preprocess functions. Twig auto-escape will do this for us.

Calling SafeMarkup::checkPlain() unnecessarily pollutes the safe string list, therefore a bug.

CommentFileSizeAuthor
#2 2545990.2.patch1.54 KBalexpott

Comments

alexpott created an issue. See original summary.

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Status: Active » Needs review
StatusFileSize
new 2545990.2.patch1.54 KB
dawehner’s picture

dawehner
Primary language German
commented
Status: Needs review » Reviewed & tested by the community

Perfect

catch’s picture

catch
he/him
Primary language English
commented
Status: Reviewed & tested by the community » Fixed

Committed/pushed to 8.0.x, thanks!

  • catch committed 0ed610a on 8.0.x 
    Issue #2545990 by alexpott: Remove unnecessary SafeMarkup::checkPlain()...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.