| |
dink
Account closed
Registered: Mar 2012
Posts: 30 |
Lots of % codes in filenames
Lately it seems like most of the files downloaded have %## translation codes in them, for example, look at the file name in this release: Plumb Crazy +1D (Loadstar)
Could it be that the uploading script isn't properly translating the filename when it gets uploaded? |
|
| |
SIDWAVE
Account closed
Registered: Apr 2002
Posts: 2238 |
yeah, bug. when you dl, the codes become part of filename.
this server is OOOOOOLD
that % doesnt get fixed before entering the db |
| |
Beastifire
Account closed
Registered: Mar 2013
Posts: 40 |
URLs aren't supposed to have arbitrary characters in them. You're supposed to encode weird ones with this method, so this is arguably correct. Don't remember what the RFC document no for URLs is, but that should be possible to look up. |
| |
bepp
Registered: Jun 2010
Posts: 268 |
Agree that it's good practise to encode the url but encoding the text of the link doesn't make any sense. I think it's just a code slip. |
| |
chatGPZ
Registered: Dec 2001
Posts: 11540 |
i think that happens because the text in the link shows the actual filename on the (hidden) ftp. even that is argueable correct (yet ugly, agreed :)) |
| |
Krill
Registered: Apr 2002
Posts: 3104 |
Arguably, this is indeed ugly. URLs can have arbitrary characters in them, but they will get translated to UTF8-percent-encoding and others by your user agent when sent over the wire. So when saving a file, having low-level URL encoding garbage in the filename is... well, ugly. :) |
| |
Burglar
Registered: Dec 2004
Posts: 1139 |
its fixable btw, but it'll be an annoying task, and different browsers will handle stuff differently too.
so, just dont use weird chars in ur filenames ;) |
| |
Fred
Registered: Feb 2003
Posts: 300 |
It's very easy to fix. Just decode the URL for the download display link.
You can do it via Javascript with decodeURIComponent or use the urldecode function in PHP. |
| |
Perff
Administrator
Posts: 1694 |
Added the sugested urldecode for displaying the link.
Also added the urldecode to the script that fetches the file form the fileserver and sends it to the user. Not sure if that will mess up the download in certain cases - if it does please let me know. :) |
| |
chatGPZ
Registered: Dec 2001
Posts: 11540 |
while at it, add a TRIM to all input boxes, its annoying as fuck if you paste something in there and have to remove leadin or trailing spaces to make some search query work :) (now that i say it, i see its fixed in the sidebar, eh =D) |
| |
Perff
Administrator
Posts: 1694 |
Quote:while at it, add a TRIM to all input boxes
Better now? :)
Is there anywhere where it might seem fit NOT to trim input? |
... 6 posts hidden. Click here to view all posts....
|
| Previous - 1 | 2 - Next |
