Modify

Opened 4 weeks ago

Last modified 3 weeks 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 weeks ago.
example file

Download all attachments as: .zip

Change History (4)

comment:1 Changed 4 weeks ago by skyper

Version: latest

Changed 4 weeks ago by skyper

example file

comment:2 Changed 4 weeks ago by skyper

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 Changed 3 weeks ago by GerdP

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

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 skyper
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.