Opened 8 years ago

Closed 8 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 by Claude Paroz, 8 years ago

Owner: changed from nobody to Claude Paroz
Status: newassigned

comment:2 by Claude Paroz, 8 years ago

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 by Claude Paroz <claude@…>, 8 years ago

Resolution: fixed
Status: assignedclosed

In 4e2ee866:

Fixed #26569 -- Updated OSM Mapnik constructor

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