if you look in the history you will see that it was deliberately changed from what you are proposing to what it is now

The PR in question is #9896. Even looking at the PR, I can't understand why this was merged. Regardless of that, the browser in question is EOL and not supported by us anymore. I still think that this PR is the right way to go.

if you look in the history you will see that it was deliberately changed from what you are proposing to what it is now

When reading that old PR‘s code changes I can’t see that it had any trim before. It was comparing the complete unmodified string. So this PR here is not a roll back of the changes from the other PR.

@Hackwar If you want to make it work exactly the same way as the old code regarding other kinds of white space than a single space character, you could use a string with a single space as 2nd parameter for trim, then it would trim only space characters. Not sure if it needs that, but we would play safe and it still has all advantages of your code change.

interesting your change changes the behavior more then I expected:

https://3v4l.org/7LM6I (yours is $z)

Hmm, the zero evaluates to false, too. Indeed a change.

This is why it is always important to look at why code was written the way that it was before changing it

my suggestion is to move it to !empty(trim($value))

Yes, this "changes" behavior. It actually fixes it to be in line with what people expect. The goal is to only show the text when it contains anything. Right now it only removes spaces, but would keep a line break or tab and then show the text. Yes, it also hides it, when the content is only a 0, but I would also argue, that there is no situation where someone would want a single 0 as description. The goal of this PR is to simplify the code and make it readable for people. We want to remove invisible characters of all kinds, so I'm not going to limit the functionality of trim() to just remove spaces. If that is not acceptable, then rather lets close this PR than to waste more time on this.

This pull request has been automatically rebased to 5.1-dev.

Seeing the logic comes back with the summary. However, it is not possible for me to perform a test.
Seeing the documentation of PHP trim function, it cleans up all types of blank spaces in a string.

I have tested this item ✅ successfully on 7ca9182

I have read the code proposed in the PR, and I have checked the PHP documentation (https://www.php.net/manual/en/function.trim.php). The PR certainly brings an advantage: it recognizes and removes not only empty spaces but also other hidden special characters.

" " (ASCII 32 (0x20)), an ordinary space.
"\t" (ASCII 9 (0x09)), a tab.
"\n" (ASCII 10 (0x0A)), a new line (line feed).
"\r" (ASCII 13 (0x0D)), a carriage return.
"\0" (ASCII 0 (0x00)), the NUL-byte.
"\v" (ASCII 11 (0x0B)), a vertical tab.

I have tested this item ✅ successfully on 7ca9182

RTC

This pull request has been automatically rebased to 5.2-dev.

Thanks @Hackwar !