Textpattern CMS support forum
You are not logged in. Register | Login | Help
- Topics: Active | Unanswered
Re: Poll: useful 'escape' transforms
Bloke wrote #308388:
For example, if an Italian editor wrote an article and inadvertently put into a custom field the value
1.234.567,89(meaning: one million, two hundred and thirty four thousand, five-hundered and sixty-seven point eight nine)
That would be a bad idea, even if we enable localized (here Italian) numbers input parsing. Suppose that an American author on the same site has put 1,234. Does it mean 1234 or 1.234 now, how to tell? An extra parameter is needed, and this is more <txp:evaluate /> or plugins territory.
Offline
Re: Poll: useful 'escape' transforms
I googled and found LC_MONETARY mentioned in this comment.
Offline
Re: Poll: useful 'escape' transforms
michaelkpate wrote #308390:
I googled and found LC_MONETARY mentioned in this comment.
Yes, but it requires an extra parameter '%.2n' (impossible in escape atm), unless we decide that $666 is the only valid format in US and get blamed for Russian interference :-)
Offline
Offline
Re: Poll: useful 'escape' transforms
etc wrote #308389:
That would be a bad idea, even if we enable localized (here Italian) numbers input parsing.
Yes. bad idea. But we’ve just opened up this Bad Place because we’ve allowed people to use the admin-side UI in their own language, which may be different from the front-end language! Caveat utilitor…
Suppose that an American author on the same site has put
1,234. Does it mean1234or1.234now, how to tell?
Exactly. That’s why I was curious about your str_replace() usage. If you replace commas with dots when formatting number, we’re essentially stripping off the “decimal” part of any value expressed in a European format. Are we? Or have I misinterpreted what it does?
Should we leave it alone and let the risk lie with the site admins that their authors might use a variety of number formats (and it’s therefore up to the admin to enforce a common input number format so that the output of such transforms is guaranteed).
As it stands, with us stripping out commas from the strings, could that inadvertently alter the output? Or is the str_replace() there to try to ensure consistent input to the NumberFormatter?
The smd plugin menagerie — for when you need one more gribble of power from Textpattern. Bleeding-edge code available on GitHub.
Txp Builders – finely-crafted code, design and Txp
Offline
Re: Poll: useful 'escape' transforms
etc wrote #308392:
It is applied to
floatval()which does not output thousand separators (afaik), but localizes the decimal separator.
Ah, right. That explains it.
The smd plugin menagerie — for when you need one more gribble of power from Textpattern. Bleeding-edge code available on GitHub.
Txp Builders – finely-crafted code, design and Txp
Offline
Re: Poll: useful 'escape' transforms
Bloke wrote #308393:
Should we leave it alone and let the risk lie with the site admins that their authors might use a variety of number formats (and it’s therefore up to the admin to enforce a common input number format so that the output of such transforms is guaranteed).
I think we should strongly advise the site admins to use the decimal-point-only-no-thousand-separator format if they intend doing some maths (arithmetic, comparison) with numbers. They will still be able to output them as they wish.
Offline
Re: Poll: useful 'escape' transforms
Putting this here as an example of “weird things can happen”. Be careful.
escape="textile, p" can sometimes have crazy consequences. Take this string, input in an image caption field: Bells for the South Side CD cover (ECM)
In a TXP form (for an image gallery, code trimmed to the minimum):
<a href="<txp:image_url />" class="image-lightbox" data-caption="<txp:image_info escape='textile, p' />">output:
<a href="http://txptest.dev/images/12.jpg" class=" image-lightbox" data-caption="Bells for the South Side cover (<span class="caps">ECM</span>)">Pay attention to the sequence of " in the code above [1]…
Two work arounds:
- use single
'quotes for the data-attribute:data-caption='<txp:image_info escape='textile, p' />' - I think for my use cases, this will suffice:
escape="textile, p, span".
PS – I never liked that habit of Textile to wrap sequences of 3 or more capital letters in a <span />.
1 the browser closes the data-caption attribute after the opening " of class="caps", then threats the word caps as an attribute. The rest is rendered as content of the @<a />.
Where is that emoji for a solar powered submarine when you need it ?
Sand space – admin theme for Textpattern
Offline
Re: Poll: useful 'escape' transforms
Earlier in this thread, there was mention of escape="strip_tags" but couldn’t find it in the docs, only the reference to “some tag”. After looking at the code, it seems it is there but is now escape="tags". Is that correct?
If so, it needs adding to the docs on tag escaping. I can submit a PR for that if you like.
This neatly consigns etz_striptags to the list of no longer needed plugins.
TXP Builders – finely-crafted code, design and txp
Offline
Re: Poll: useful 'escape' transforms
jakob wrote #313065:
… it seems it is there but is now
escape="tags". Is that correct?If so, it needs adding to the docs on tag escaping. I can submit a PR for that if you like.
Yes, please. Thanks!
And the poll is still open for the next round of plugins deprecation.
Offline
Re: Poll: useful 'escape' transforms
Thanks for the clarification.
PR submitted and already online (thanks Phil!): tag escaping.
TXP Builders – finely-crafted code, design and txp
Offline