Opened 10 years ago

Closed 9 years ago

Last modified 9 years ago

#5466 closed defect (fixed)

MapQuest Developer Tiles no longer available

Reported by: chris Owned by:
Priority: minor Milestone:
Component: programming Keywords:
Cc: ben@… Parent Tickets:

Description

The pages on which I have geotagged images show an empty box with a notice from MapQuest saying:

"As of July 11, 2016, direct tile access has been discontinued."

An example can be seen here: http://m.chrishancockart.com/u/chris/m/westside-local-show-2014-06/

Attachments (2)

0001-Use-OSM-tiles-in-Geolocation-plugin-5466.patch (1.3 KB ) - added by Olivier Mehani 10 years ago.
Patch to replace MapQuest tiles with OSM ones
0002-Update-maps-copyright-notice-to-OSM.patch (1.3 KB ) - added by Olivier Mehani 9 years ago.

Download all attachments as: .zip

Change History (11)

comment:1 by Ben Sturmfels, 10 years ago

Status: newaccepted

Thanks teslas_moustache, I'm able to replicate this issue by uploading the same photo to my local system running against the master branch.

Looks like the appropriate change would be mediagoblin/static/js/geolocation-map.js line 33:

var mqtileUrl = 'http://otile{s}.mqcdn.com/tiles/1.0.0/osm/{z}/{x}/{y}.jpg';

This will likely need to change to a new tile provider. Worth investigating what the default Leaflet tiles are and possibly switching to those.

by Olivier Mehani, 10 years ago

Patch to replace MapQuest tiles with OSM ones

comment:2 by Olivier Mehani, 10 years ago

Here is a patch using OSM tiles instead of MapQuest. You can also pull from this branch [0], which has just the one commit on top of v0.9.0.

[0] https://scm.narf.ssji.net/git/mediagoblin/log/?h=bug5466-replace-mapquest

comment:3 by Ben Sturmfels, 10 years ago

Cc: sturm added

comment:4 by Ben Sturmfels, 10 years ago

Cc: ben@… added; sturm removed

comment:5 by jerome, 10 years ago

The file mediagoblin/plugins/geolocation/templates/mediagoblin/plugins/geolocation/map.html line 46 should also be updated regarding new attribution, MapQuest should be removed.

comment:6 by Olivier Mehani, 9 years ago

Good catch! Here.

by Olivier Mehani, 9 years ago

comment:7 by Ben Sturmfels, 9 years ago

Status: acceptedreview

comment:8 by Boris Bobrov, 9 years ago

Resolution: fixed
Status: reviewclosed

Awesome, thanks.

comment:9 by Boris Bobrov, 9 years ago

merged in b8c4c82 and 441fee1

Note: See TracTickets for help on using tickets.