Opened 8 years ago

Closed 8 years ago

Last modified 8 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 8 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 8 years ago.

Download all attachments as: .zip

Change History (11)

comment:1 by Ben Sturmfels, 8 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, 8 years ago

Patch to replace MapQuest tiles with OSM ones

comment:2 by Olivier Mehani, 8 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, 8 years ago

Cc: sturm added

comment:4 by Ben Sturmfels, 8 years ago

Cc: ben@… added; sturm removed

comment:5 by jerome, 8 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, 8 years ago

Good catch! Here.

by Olivier Mehani, 8 years ago

comment:7 by Ben Sturmfels, 8 years ago

Status: acceptedreview

comment:8 by Boris Bobrov, 8 years ago

Resolution: fixed
Status: reviewclosed

Awesome, thanks.

comment:9 by Boris Bobrov, 8 years ago

merged in b8c4c82 and 441fee1

Note: See TracTickets for help on using tickets.