Modify

Opened 2 years ago

Closed 22 months ago

#6254 closed defect (fixed)

[Patch] Combine ways should merge multivalue tags more intelligently

Reported by: bilbo Owned by: team
Priority: normal Component: Core
Version: latest Keywords:
Cc:

Description

It is quite common for some tags (like source) to have multiple values separated by semicolon. However, when you combine ways (by pressing C) and the ways have different tags, the dialog to resolve conflicts will appear.

But for multivalued tags the default suggestion is just blind concatenation of all the values.

So if you have three ways, one with source="bing", second with source="bing;survey", third with source="survey;bing", then if you concatenate them and just press enter in the dialog, you'll end up with combined way with source="bing;bing;survey;survey;bing".

Pretty ugly. JOSM should construct the default value more intelligently, so in these common cases (multiple sources) all the sources wil be present just once in the concatenation, so the default value will be usable and not some concatenated nonsense.

Attachments (1)

6254.patch (1.1 KB) - added by simon04 22 months ago.

Download all attachments as: .zip

Change History (3)

Changed 22 months ago by simon04

comment:1 Changed 22 months ago by simon04

  • Summary changed from Combine ways should merge multivalue tags more intelligently to [Patch] Combine ways should merge multivalue tags more intelligently

The attached patch should provide this feature. Please test! :-)

comment:2 Changed 22 months ago by stoecker

  • Resolution set to fixed
  • Status changed from new to closed

In [4291/josm]:

fix #6254- patch by simon04 - Combine ways should merge multivalue tags more intelligently

Add Comment

Modify Ticket

Change Properties
<Author field>
Action
as closed .
as The resolution will be set. Next status will be 'closed'.
The resolution will be deleted. Next status will be 'reopened'.
Author


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

 
Note: See TracTickets for help on using tickets.