Modify

Opened 6 years ago

Last modified 6 years ago

#10813 new enhancement

Use ID for imagery sources in toolbar, image offsets and source-entry for uploads instead of name

Reported by: rickmastfan67 Owned by: team
Priority: normal Milestone:
Component: Core imagery Version: latest
Keywords: Cc:

Description

Just today, somebody updated the 'Tiger 2013' entry for the imagery map to the 'Tiger 2014'. ( http://josm.openstreetmap.de/wiki/Maps/USA?action=diff&version=12 ) I already had the 'Tiger 2013' stored in my toolbar so I would have one click access to it if I needed it while mapping. So, when JOSM automatically updated the imagery list at startup, it broke the toolbar entry for the 'Tiger 2013' imagery instead of updating it to the 'Tiger 2014 automatically.

Since the info change was on the same line, I would have expected JOSM to update my toolbar entry for it. Would there be anyway in the future to allow this to happen without me having to re-add the 'updated' year version?

Repository Root: http://josm.openstreetmap.de/svn
Build-Date: 2014-12-05 02:34:20
Last Changed Author: Don-vip
Revision: 7764
Repository UUID: 0c6e7542-c601-0410-84e7-c038aed88b3b
Relative URL: ^/trunk
URL: http://josm.openstreetmap.de/svn/trunk
Last Changed Date: 2014-12-05 01:10:47 +0100 (Fri, 05 Dec 2014)
Last Changed Rev: 7764

Identification: JOSM/1.5 (7764 en) Windows 7 64-Bit
Memory Usage: 222 MB / 1820 MB (110 MB allocated, but free)
Java version: 1.7.0_71, Oracle Corporation, Java HotSpot(TM) 64-Bit Server VM

Plugins:
- ImageryCache (30738)
- OpeningHoursEditor (30737)
- buildings_tools (30762)
- measurement (30737)
- mirrored_download (30762)
- notes (v0.9.5)
- osmarender (30738)
- reverter (30737)
- turnrestrictions (30762)
- undelete (30762)
- utilsplugin2 (30762)

Attachments (0)

Change History (23)

comment:1 Changed 6 years ago by stoecker

So, when JOSM automatically updated the imagery list at startup, it broke the toolbar entry for the 'Tiger 2013' imagery instead of updating it to the 'Tiger 2014 automatically.

How? Was it unaccesible (based on the old name)?

comment:2 Changed 6 years ago by rickmastfan67

There was a 'toolbar' error and the icon in the toolbar for it disappeared. Had to manually add back the new '2014' version of it. Wish I would have saved a copy of the error in the CL, but didn't.

comment:3 Changed 6 years ago by stoecker

Milestone: 14.12

Hmm, maybe we should switch to the entry id for toolbar name when existing. But this needs an fallback for old entries.

comment:4 Changed 6 years ago by Don-vip

Milestone: 14.1215.01

comment:5 Changed 6 years ago by Don-vip

Milestone: 15.0115.02

move tickets that have not been treated this month to next milestone

comment:6 Changed 6 years ago by rickmastfan67

Just happened again with the Tiger 2014 layer because somebody tweaked the name for it. :(

And yes, it was un-accessible because of the old name. It threw up errors in the CL about there being no definition for the tool bar entry. Kinda like INFO: Could not load tool definition *** and had the old imagery name for that layer. Also, the icon for it was gone in the toolbar.

Anyways, this just seems to be a problem for entries that are added from the wiki database to the toolbar. Doesn't seem to effect entries you've personally manually added to the 'local' list.

Last edited 6 years ago by rickmastfan67 (previous) (diff)

comment:7 Changed 6 years ago by stoecker

Ticket #11041 has been marked as a duplicate of this ticket.

comment:8 Changed 6 years ago by stoecker

Summary: Way to update imagery sources in toolbarUse ID for imagery sources in toolbar and image offsets instead of name

Id should not only be used for toolbar, but also for imagery offsets.

comment:9 Changed 6 years ago by aceman

Where can I get the ID so I can edit the config file to make offsets match again and visible?

comment:10 Changed 6 years ago by stoecker

That's nothing you can do. We need to adapt source. In the past our maps data had no specific ID, so we used the name instead. For these where we now have an ID, we need to use it instead of the name, so name changes do no harm.

comment:11 Changed 6 years ago by Don-vip

Milestone: 15.0215.03

comment:12 Changed 6 years ago by stoecker

Summary: Use ID for imagery sources in toolbar and image offsets instead of nameUse ID for imagery sources in toolbar, image offsets and source-entry for uploads instead of name

comment:13 Changed 6 years ago by StefanB

Also consider using the id in URL parameter: https://josm.openstreetmap.de/mapsview?entry=ID
instead of name, as it is now: https://josm.openstreetmap.de/mapsview?entry=NAME
(linked from wiki page Maps)

comment:14 Changed 6 years ago by stoecker

Milestone: 15.0315.04

Move one month.

comment:15 Changed 6 years ago by simon04

Using the id for the toolbar is as simple as changing org.openstreetmap.josm.data.imagery.ImageryInfo#getToolbarName. Then however the old toolbar icons will no longer work. Implementing a migration process is more complex …

comment:16 Changed 6 years ago by stoecker

Me need to support both methods anyway, as not all maps have an ID.

comment:17 Changed 6 years ago by aceman

For those use the (untranslated) name as the ID?

comment:18 Changed 6 years ago by stoecker

Milestone: 15.0415.05

comment:19 Changed 6 years ago by Don-vip

Milestone: 15.0515.06

comment:20 Changed 6 years ago by Don-vip

Milestone: 15.0615.07

skip milestone 15.06

comment:21 Changed 6 years ago by Don-vip

Milestone: 15.0715.08

Milestone renamed

comment:22 Changed 6 years ago by Don-vip

Milestone: 15.08

removing from milestone as it seems nobody's working on it right now.

comment:23 Changed 6 years ago by rickmastfan67

And it happened once again when the entry for the Tiger data had the date changed from 2014 to 2015.

https://josm.openstreetmap.de/wiki/Maps/USA?version=17

Did capture the exact error that showed up this time unlike in comment:6.
INFO: Could not load tool definition imagery_New & Misaligned TIGER Roads \(2014\)

Using r8692.

Modify Ticket

Change Properties
Set your email in Preferences
Action
as new The owner will remain team.
as The resolution will be set.
to The owner will be changed from team to the specified user.
The owner will change to rickmastfan67
as duplicate The resolution will be set to duplicate.The specified ticket will be cross-referenced with this ticket
The owner will be changed from team to anonymous.

Add Comment


E-mail address and name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.