Opened 9 years ago
Closed 9 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].
Change History (3)
comment:1 by , 9 years ago
| Owner: | changed from to |
|---|---|
| Status: | new → assigned |
comment:2 by , 9 years ago
| Type: | Bug → Cleanup/optimization |
|---|
Note:
See TracTickets
for help on using tickets.
The initial patch reporter said that the Mapnik constructor did not work. In fact, it does work, but requires the
OpenStreetMap.jsfile be present (which is included in OSMWidget). That's the reason why I don't plan to backport the fix.