As a result of #2990781: Token Support I'm suggesting to remove duplicate class addition and remove clear token value if no values present.

CommentFileSizeAuthor
#2 field_formatter_class-clear_token-3017781.patch1.03 KBpookmish

Comments

pookmish created an issue. See original summary.

pookmish’s picture

pookmish commented
Status: Active » Needs review
StatusFileSize
new field_formatter_class-clear_token-3017781.patch1.03 KB
pookmish’s picture

pookmish commented

I'm not sure if theres a desire but maybe also a callback/alter on the token replacement?

alex_optim’s picture

alex_optim
Location Tallinn
commented

Looks good.

pifagor’s picture

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

  • pifagor committed 7d58d19 on 8.x-1.x authored by pookmish 
    Issue #3017781 by pookmish, alex_optim: Remove token class
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.