Modify

Opened 4 years ago

Last modified 4 years ago

#19217 new defect

Split way: Wrong position of new member under special conditions of route relation

Reported by: skyper Owned by: team
Priority: normal Milestone:
Component: Core Version: latest
Keywords: template_report split way route relation order Cc:

Description

This is a followup of #18596 and might be related to #18018.

Please, use attached example file: josm_split_way_relation_order.osm

What steps will reproduce the problem?

  1. Open road relation "A 1" in relation editor and look at the order
  2. Select node "split" and split way
  3. Reload data in relation editor and look at the order

What is the expected result?

The new member should be added in between the last "forward" member and the last member with empty role

What happens instead?

The new member is added behind the last member with empty role

Please provide any additional information below. Attach a screenshot if possible.

link is used in route relations and should only be considered to connect with other members with role link. For empty role members they should be the last option.

The bus relation "1" shows the same issue but without link.

In both cases, it is not needed to look at the relation list neighbor below as the correct position can already be determined by the neighbor above which is connected.

Relative:URL: ^/trunk
Repository:UUID: 0c6e7542-c601-0410-84e7-c038aed88b3b
Last:Changed Date: 2020-05-10 15:58:34 +0200 (Sun, 10 May 2020)
Revision:16402
Build-Date:2020-05-11 01:30:50
URL:https://josm.openstreetmap.de/svn/trunk

Attachments (1)

josm_split_way_relation_order.osm (4.8 KB ) - added by skyper 4 years ago.
example file

Download all attachments as: .zip

Change History (5)

comment:1 by skyper, 4 years ago

Version: latest

by skyper, 4 years ago

example file

comment:2 by skyper, 4 years ago

Found another problem with role forward and did update the example file.

  1. Look at bus relation "2" and split at "split again"

The new member is added at the wrong position.

  • Does not happen with backward see relation "2B".
  • Does not happen at node "A".

comment:3 by GerdP, 4 years ago

I see two problems in the current code:
1) It first looks at the member above, than at the member below. It doesn't care when both give conflicting information.
2) The role link is not considered special

comment:4 by skyper, 4 years ago

This is similar to #18018.

Modify Ticket

Change Properties
Set your email in Preferences
Action
as new The owner will remain team.
as The resolution will be set. Next status will be 'closed'.
to The owner will be changed from team to the specified user.
Next status will be 'needinfo'. The owner will be changed from team to skyper.
as duplicate The resolution will be set to duplicate. Next status will be 'closed'. The specified ticket will be cross-referenced with this ticket.
The owner will be changed from team to anonymous. Next status will be 'assigned'.

Add Comment


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