Fixing bug in usage of ltrim

[viable-hartman]

Pretty sure the intention of this was to remove the '/', not lose the path, which this would do. So fixing it. Looks like this has been in the code going back quite a while, so might want to port it back.

[ruflin]

This indeed looks odd :-( I wonder why this didn't break other things.

@viable-hartman

• Could you add a changelog entry?
• Do you see a way we could test this change?

[viable-hartman]

@ruflin I will add a changelog entry once I get a tiny bit of spare time. I did not trace through the code to see what or if this actually breaks anything, but believe it's probably rarely run into because the default path usually is just '/' which of course wouldn't affect much. Further it looks like other areas of the code do ltrim like this to deal with path, so perhaps those were there because this has been broken. So short answer is, I'd have to trace the code to determine where to test, but just noticed this was wrong due to another piece of older code I was looking at.

[ruflin]

@viable-hartman Thank you! Hoping you get some spare time soon ;-)

[deguif]

@viable-hartman would you be able to add a changelog?

[deguif]

@ruflin I just added a changelog here.
If you could have a look as I planned it for 7.1.1 too ;)