Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

disable merges that would damage relations #1512

Closed
jfirebaugh opened this issue May 22, 2013 · 4 comments · Fixed by #8675
Closed

disable merges that would damage relations #1512

jfirebaugh opened this issue May 22, 2013 · 4 comments · Fixed by #8675
Labels
bug A bug - let's fix this!

Comments

@jfirebaugh
Copy link
Member

Certain merge operations are unambiguously wrong, e.g.:

  • merging ways participating in the 'from' and 'to' roles of a restriction
  • merging ways that are members of different multipolygon relations

We should investigate if it is possible to reliably distinguish such merges from safe merges, and disable the operation in those cases while providing a message in the tooltip explaining why it's disabled.

@bagage
Copy link
Contributor

bagage commented Oct 18, 2015

Would these uses cases be disabled as well?

  • merging ways of different types (highway=secondary and highway=residential for instance)
  • merging items of different names (both names not empty)
  • merging ways of different speed limits

Actually, most merges where values are different should not be done without warning the user first?

@jfirebaugh
Copy link
Member Author

@bagage: you're looking for #2358.

@jfirebaugh
Copy link
Member Author

Note that this is already partially implemented: many cases where merging would damage relations are already prohibited by iD. This issue is still open because the behavior could be extended to cover more cases.

@bhousel bhousel added the bug A bug - let's fix this! label Mar 22, 2017
@AntMadeira
Copy link

AntMadeira commented Jun 11, 2020

Hi there.
I've just read #1461 and #2358 and got here, which is exactly the problem I want to bring about (again). I hope it is ok. If not, please move it to a new issue.

Recently, there were 8 editions in Portugal which damaged major relations. This was done by a mapper that merged ways and changed those relations unwittingly (I presume). Even I have to pay a lot of attention when merging something, because there's no warning about merging elements which belong to different relations. and I know I damaged some relations in this way when I started mapping with iD.
I think this is a huge flaw and should be tackled to prevent further damaging to OSM data all over the world. If iD already warns about deleting relations or part of relations, why not warn when there's a merging going on?

Regards.

jfirebaugh added a commit that referenced this issue Sep 4, 2021
The operation is disabled when attempting to join ways which don't belong to identical sets of relations. Restriction relations are excluded, because they are already handled with slightly different logic.

Fixes #8674
Fixes #8645
Fixes #3825
Fixes #1512
jfirebaugh added a commit that referenced this issue Sep 4, 2021
The operation is disabled when attempting to join ways which don't belong to identical sets of relations. Restriction relations are excluded, because they are already handled with slightly different logic.

Fixes #8674
Fixes #8645
Fixes #3825
Fixes #1512
jfirebaugh added a commit that referenced this issue Sep 4, 2021
The operation is disabled when attempting to join ways which don't belong to identical sets of relations. Restriction relations are excluded, because they are already handled with slightly different logic.

Fixes #8674
Fixes #8645
Fixes #3825
Fixes #1512
jfirebaugh added a commit that referenced this issue Sep 4, 2021
The operation is disabled when attempting to join ways which don't belong to identical sets of relations. Restriction relations are excluded, because they are already handled with slightly different logic.

Fixes #8674
Fixes #8645
Fixes #3825
Fixes #1512
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug A bug - let's fix this!
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants