Opened 2 years ago

Closed 2 years ago

#26569 closed Cleanup/optimization (fixed)

OSM.Mapnik is an obsolete constructor

Reported by: Claude Paroz Owned by: Claude Paroz
Component: GIS Version: 1.9
Severity: Normal Keywords:
Cc: Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

OpenLayers.Layer.OSM.Mapnik can simply be replaced by OpenLayers.Layer.OSM(), as was done in [1ec25c4fb7df0fdb].

PR

Change History (3)

comment:1 Changed 2 years ago by Claude Paroz

Owner: changed from nobody to Claude Paroz
Status: newassigned

comment:2 Changed 2 years ago by Claude Paroz

Type: BugCleanup/optimization

The initial patch reporter said that the Mapnik constructor did not work. In fact, it does work, but requires the OpenStreetMap.js file be present (which is included in OSMWidget). That's the reason why I don't plan to backport the fix.

comment:3 Changed 2 years ago by Claude Paroz <claude@…>

Resolution: fixed
Status: assignedclosed

In 4e2ee866:

Fixed #26569 -- Updated OSM Mapnik constructor

Note: See TracTickets for help on using tickets.
Back to Top