[OSM-dev] trace file extension in links to data

Stefan Baebler stefan.baebler at gmail.com
Fri Feb 6 21:26:15 GMT 2009


Hi!

It is a nice practice to indicate the file type in the url of the
download link to let users know what they are getting before actually
pushing it down their throat.

Filename on the trace detail page often says "somecity.gpx" (as at the
time of upload), download link is simply "data" and the resulting file
is named "123456.gpx.gz" (gzipped later on server?), which makes it a
bit confusing.

Example:
http://www.openstreetmap.org/user/Rigless/traces/308275
has a download link to
http://www.openstreetmap.org/trace/308275/data
my aim is to change that to
http://www.openstreetmap.org/trace/308275/data.gpx

and similarly at
http://www.openstreetmap.org/user/StefanB/traces/137465
has the link:
http://www.openstreetmap.org/trace/137465/data
that should be:
http://www.openstreetmap.org/trace/137465/data.gpx.gz

I'm attaching an *UNTESTED* patch for above changes in the rails port.

Going a step further would be to match also the base filename, not just
extensions:
Eg instead:
http://www.openstreetmap.org/trace/308275/data
link would be
http://www.openstreetmap.org/trace/308275/308275.gpx
or even just
http://www.openstreetmap.org/trace/308275.gpx


I couldn't find traces with other extensions (.tar.gz, .zip, .gpx.bz2
...) for additional test cases, but it should be easy by peeking into
database or into folder full of traces. Or having some faith that if it
works for two types it should work for all of them.

Related bug: http://trac.openstreetmap.org/ticket/498 - there the
extension ".gpx" was probably hardcoded into the link.

Can someone please apply the patch into their test rails setup and test
it before committing to svn and pushing it to the main site please?

thank you!
Stefan


-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: extension-in-trace-download-links.patch
URL: <http://lists.openstreetmap.org/pipermail/dev/attachments/20090206/503bbdfa/attachment.ksh>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 250 bytes
Desc: OpenPGP digital signature
URL: <http://lists.openstreetmap.org/pipermail/dev/attachments/20090206/503bbdfa/attachment.pgp>


More information about the dev mailing list