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

2 participants