Strings improvement: trim ZWSP char

nette / utils

🛠 Lightweight utilities for string & array manipulation, image handling, safe JSON encoding/decoding, validation, slug or strong password generating etc.

https://doc.nette.org/utils

Other

1.98k stars 147 forks source link

Strings improvement: trim ZWSP char #304

Closed CrazyMuffin closed 11 months ago

CrazyMuffin commented 11 months ago

new feature
BC break? yes - if someone relies on having ZWSP non-trimmed while using default parameter.
doc PR: Not needed, function is covered and does not mention the default characters

Sneaky Zero-Width Space char \u{200B} is currently untrimmed by trim function, while I'd expect it to do so.

Workaround:

Fork or extend the class and overwrite default param
Manually set updated charlist as second argument to every trim() call

dg commented 11 months ago

Thanks