Opened 10 years ago

Closed 10 years ago

Last modified 10 years ago

#9643 closed defect (fixed)

[patch] cleanup SlippyMapBBoxChooser from a TileSource implementation

Reported by: glebius Owned by: team
Priority: normal Milestone: 14.02
Component: Core Version:
Keywords: Cc:



I'm working on fixing #7017 and decided to fix it not in a quick & dirty manner, but give some refresh to the tile sources code.

The first patch removes implementation of TileSource from JOSM code. This is a layering violation, since JOSM should only utilize TileSources, all implementations should live in jmapviewer. This one is created merely to change name of a TileSource, the rest of code is ugly cut'n'paste.

I have already changed default name of cycle map layer from "OSM Cycle Map" to "Cyclemap", so the attached patch will not introduce any user visible changes. All names would remain the same.

In a case if we really need to have different layer names in BBox chooser and in the main JOSM windows, I will just extend TileSource with a new constructor that takes String name argument. However, I doubt that would be needed.

In any case, the patch is needed to allow me freely work on TileSource class inside jmapviewer, without breaking JOSM build on every change.

Attachments (1) (3.5 KB ) - added by glebius 10 years ago.

Download all attachments as: .zip

Change History (6)

by glebius, 10 years ago

comment:1 by stoecker, 10 years ago

Was added in [3602]. I see no reason for this wrapper.

comment:2 by stoecker, 10 years ago

Resolution: fixed
Status: newclosed

In 6785/josm:

fix #9643 - remove unneeded wrapper class - patch by glebius

comment:3 by stoecker, 10 years ago


fix it not in a quick & dirty manner, but give some refresh to the tile sources code.

I like that. Quick and Dirty usually causes trouble later. A proper solution is better.

comment:4 by Don-vip, 10 years ago

Milestone: 14.02

in reply to:  3 comment:5 by Don-vip, 10 years ago

Replying to stoecker:


fix it not in a quick & dirty manner.

I like that.

+1. Thanks a lot ! :)

Modify Ticket

Change Properties
Set your email in Preferences
as closed The owner will remain team.
as The resolution will be set.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment

E-mail address and name can be saved in the Preferences .
Note: See TracTickets for help on using tickets.