Opened 4 years ago

Closed 4 years ago

Last modified 4 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/

Subtickets

Attachments (2)

0001-Use-OSM-tiles-in-Geolocation-plugin-5466.patch (1.3 KB) - added by Olivier Mehani 4 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 4 years ago.

Download all attachments as: .zip

Change History (11)

comment:1 Changed 4 years ago by Ben Sturmfels

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.

Changed 4 years ago by Olivier Mehani

Patch to replace MapQuest tiles with OSM ones

comment:2 Changed 4 years ago by Olivier Mehani

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 Changed 4 years ago by Ben Sturmfels

Cc: sturm added

comment:4 Changed 4 years ago by Ben Sturmfels

Cc: ben@… added; sturm removed

comment:5 Changed 4 years ago by jerome

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 Changed 4 years ago by Olivier Mehani

Good catch! Here.

Changed 4 years ago by Olivier Mehani

comment:7 Changed 4 years ago by Ben Sturmfels

Status: acceptedreview

comment:8 Changed 4 years ago by Boris Bobrov

Resolution: fixed
Status: reviewclosed

Awesome, thanks.

comment:9 Changed 4 years ago by Boris Bobrov

merged in b8c4c82 and 441fee1

Note: See TracTickets for help on using tickets.