Similar to D7 request #2231163: Tokens add token support for D8

CommentFileSizeAuthor
#5 field_formatter_class-token_support-2990781.patch1.68 KBpookmish

Comments

pookmish created an issue. See original summary.

Mile3’s picture

Mile3 commented

I agree, this would be very helpful.

pifagor’s picture

pifagor
Location 🇺🇦 Rivne
commented
Issue tags: +epam-contrib-2018
romixua’s picture

romixua
Location 🇺🇦 lviv
commented
Assigned: Unassigned » romixua
pookmish’s picture

pookmish commented
Status: Active » Needs review
StatusFileSize
new field_formatter_class-token_support-2990781.patch1.68 KB
pifagor’s picture

pifagor
Location 🇺🇦 Rivne
commented

.

alex_optim’s picture

alex_optim
Location Tallinn
commented
Status: Needs review » Reviewed & tested by the community

Looks good.

pifagor’s picture

pifagor
Location 🇺🇦 Rivne
commented

  • pifagor committed 9080241 on 8.x-1.x authored by pookmish 
    Issue #2990781 by pookmish, pifagor, Romixua, Mile3, alex_optim: Token...
pifagor’s picture

pifagor
Location 🇺🇦 Rivne
commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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

andrewmacpherson’s picture

andrewmacpherson commented

The change from using Html::escape() to using Html::cleanCssIdentifier() was out of scope for this issue. It wasn't necessary for supporting tokens, and caused a couple of bugs: