- ๐ฆ๐บAustralia elgandoz Canberra
This is actually useful within Token and it's not only related with Social Media issue.
I'm using it with TextImage and this patch works as it should.
Only 2 issues:
- the "titleencode" token doesn't appear in the available tokens
- the naming could be improved matching the other patterns, something like "title:encoded" - last update
over 1 year ago 77 pass - last update
over 1 year ago 77 pass - ๐ฎ๐ณIndia prashant.c Dharamshala
Prashant.c โ made their first commit to this issueโs fork.
- last update
over 1 year ago 77 pass - Status changed to Needs review
over 1 year ago 11:48am 12 December 2023 - ๐ฆ๐บAustralia elgandoz Canberra
I swapped
title:encoded
fortitle-encoded
and added the token info.
The resulting token is[current-page:title-encoded]
, but I was wondering if it's any better provide it as utility in order to encode any token, eg[any-string-token:encoded]
.
Any thoughts from the maintainers? - Status changed to Needs work
over 1 year ago 7:55pm 12 December 2023 - ๐จ๐ญSwitzerland berdir Switzerland
To have nested tokens, they need a type, and traversing to children needs to be explicitly supported and implemented for every single token, so any-string-token:encoded is not feasible with the current token system.
I'm not convinced this is needed. It's IMHO up to the logic calling token replace to apply appropriate encoding on any value that's been replaced by the token. You might want to have a look at how metatag for example handles that.
- last update
over 1 year ago 77 pass - last update
over 1 year ago 77 pass