Skip to content
This repository was archived by the owner on Nov 27, 2018. It is now read-only.

Conversation

@72quadrat
Copy link
Contributor

...like german umlauts and such...

new: $path = str_replace('%2F', '/', urlencode($path));

…rs like german umlauts and such...

new: $path = str_replace('%2F', '/', urlencode($path));
@jimdoescode
Copy link
Owner

Have you confirmed that having spaces encoded with '+' as urlencode does works?

@72quadrat
Copy link
Contributor Author

OOps,... nope they don't :-(

@72quadrat
Copy link
Contributor Author

Should be rawurlencode no urlencode

@jimdoescode
Copy link
Owner

OK I'll close this one. Just open a new one with rawurlencode.

@72quadrat
Copy link
Contributor Author

did it sorry

@jimdoescode
Copy link
Owner

I haven't gotten the new pull request yet. Did you create a new one?

@72quadrat
Copy link
Contributor Author

Yes, i have it, and on github i've Seen it also. I'm now out for the Summer, Best, Timo

.
.
.
.
.
Timo Höner
.
mail@timohoener.de
72quadrat.de
.
Mobil 01796786881
Festnetz 03067
Gabriel Max Straße 4
10245 Berlin

Am 12.05.2012 um 18:39 schrieb Jimreply@reply.github.com:

I haven't gotten the new pull request yet. Did you create a new one?


Reply to this email directly or view it on GitHub:
#3 (comment)

@jimdoescode jimdoescode reopened this May 12, 2012
@jimdoescode
Copy link
Owner

Hmm the changes don't seem to be reflected in this pull request and I haven't received notice of a new one. I'll just update it myself and close this request. Thanks for the fix.

@jimdoescode jimdoescode reopened this May 12, 2012
@jimdoescode
Copy link
Owner

Oh never mind it just showed up. Github is acting weird. OK Thanks.

jimdoescode pushed a commit that referenced this pull request May 12, 2012
Changed the path url encoding so that is should work with special chars ...
@jimdoescode jimdoescode merged commit 4f24c71 into jimdoescode:master May 12, 2012
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants