Opened 16 years ago
Closed 16 years ago
#3556 closed defect (fixed)
False positive conflicts when merging nodes
| Reported by: | pieren | Owned by: | Gubaer |
|---|---|---|---|
| Priority: | critical | Milestone: | |
| Component: | Core | Version: | latest |
| Keywords: | Cc: |
Description
False conflicts appears on latest josm tests:
- create two new empty nodes A and B
- add on node A a dummy tag (e.g. "bla"="bla")
- merge with empty node B
It creates a conflict where it shouldn't. Tag "bla" doesn't exists in node B.
Attachments (0)
Change History (10)
comment:1 by , 16 years ago
| Resolution: | → wontfix |
|---|---|
| Status: | new → closed |
follow-up: 3 comment:2 by , 16 years ago
| Resolution: | wontfix |
|---|---|
| Status: | closed → reopened |
Wow, this will be a big change from the JOSM behaviour for merging. I belong to the group who expect a conflict when there is a real conflict (two keys, different values) and clean-up tag combinations only when it is necessary, otherwise I don't merge.
May I suggest to make this new behaviour configurable ? My guess is that you will have a lot of complains about this additional dialog.
comment:3 by , 16 years ago
May I suggest to make this new behaviour configurable ? My guess is that you will have a lot of complains about this additional dialog.
Not yet, yours is the the first one :-)
Except perhaps #3460, but this one complains about a slightly different aspect of the new conflict resolution dialog.
comment:4 by , 16 years ago
I think I understand now why this is really annoying. Even when you combine a complete new and empty way (or an empty node), we have now this dialog. When you merge or combine something existing with some new element, we don't expect such dialog (which could be understandable if both elements carry some information).
follow-up: 6 comment:5 by , 16 years ago
Maybe make omit the dialog when only one joined element has tags at all? That should solve most of the annoyance without losing the better information.
comment:6 by , 16 years ago
| Owner: | changed from to |
|---|---|
| Status: | reopened → new |
Replying to stoecker:
Maybe make omit the dialog when only one joined element has tags at all? That should solve most of the annoyance without losing the better information.
Yes, I think so too. I'll see what I can do.
comment:10 by , 16 years ago
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
This was introduced due to another ticket (don't remember the id any more).
The very fact that "bla" doesn't exist on B *is* in conflict that it exists on B. How should the merged node look like? Should it have a tag "bla"="bla" or not? Some users expect it to have the tag, other don't and a third group would like to decide from case to case.
JOSM now displays a conflict dialog. The default decision is to keep "bla"="bla" but you can override it by selecting the value "None" in the list of possible values for the tag "bla".