search

getkirby-v2 / toolkit

This is the deprecated toolkit for Kirby v2.
http://getkirby.com
81 stars 50 forks source link

url::unIdn adds slash in front of query #249

Closed covovker closed 7 years ago

covovker commented 7 years ago

Passing url like this http://site.com/something?param=value adds slash after "something". Therefore this code die(var_dump(url::unIdn('http://site.com/something?param=value'))); produces the following output: string(38) "http://site.com/something/?param=value"

That affects using go() to redirect to servers where that slash matters.

lukasbestle commented 7 years ago

You are right, another edge case for url::build(). I have fixed this on the develop branch and added unit tests for a few other cases. :)