Opened 5 years ago
Closed 5 years ago
#701 closed defect (fixed)
[PATCH] Bug in relation handling splitting up ways
| Reported by: | g.gremmen@… | Owned by: | framm |
|---|---|---|---|
| Priority: | major | Component: | Core |
| Version: | latest | Keywords: | relation split road |
| Cc: | ehabkost@… |
Description
When splitting a way, already part of a relation, only one half
of this way will remain part of the relation.
The other half will be unlinked.
This goes unnoticed to the user, so may happen
inadvertently, jeopardizing for example bicycle routes
on this way.
V613
Attachments (1)
Change History (7)
comment:1 Changed 5 years ago by schusch
- Summary changed from Bug in relation handling to Bug in relation handling splitting up ways
comment:2 Changed 5 years ago by schusch
same bug as #780
comment:3 Changed 5 years ago by anonymous
- Cc ehabkost@… added
comment:4 Changed 5 years ago by Dirk Stöcker <openstreetmap@…>
- Summary changed from Bug in relation handling splitting up ways to [PATCH] Bug in relation handling splitting up ways
Attached is a patch, which copies the new way also to the relation, when splitting a way.
Changed 5 years ago by Dirk Stöcker <openstreetmap@…>
comment:5 Changed 5 years ago by framm
- Status changed from new to assigned
Patch committed in r668. Ticket left open because this patch only makes the problem different, it doesn't really fix it - depending on the kind of relation that is being worked on, we don't even want both halves of the way in the relation (think turn restrictions).
comment:6 Changed 5 years ago by anonymous
- Resolution set to fixed
- Status changed from assigned to closed
In r674 a warning has been added for role-based relations. That should do it until automatically handling these relations is a stronger requirement.
i support this bug - splitting ways with relations should always end up in having all part of the previous way having this relations (i disarranged a dual carriage way by splitting it up without even noticing this); (greetings and thanks by the way for josm!)