Opened 4 years ago
Closed 4 years ago
#2755 closed defect (fixed)
[PATCH] data error caused by duplicate node fix applied outside of the downloaded area
| Reported by: | Harry Wood | Owned by: | dmuecke |
|---|---|---|---|
| Priority: | major | Component: | Core validator |
| Version: | latest | Keywords: | |
| Cc: | delta_foxtrot@… |
Description
When you validate data which is outside of the downloaded area the plugin allows you to "fix" this data, which can cause nasty data errors ("dataset violates a precodition" when you try to upload)
You can reproduce this within the TIGER data along county boundary roads where there are a lot of duplicate nodes. Try to fix a node which has other (not downloaded) roads connected to it.
How to fix this though?
The plugin defaults to validating all downloaded data including data outside the downloaded area. Maybe this is wrong. Maybe it should only validate nodes within the downloaded area. Or maybe it should only allow fixing within the downloaded area. Or maybe for the duplicate nodes fix, it should give a warning similar to the message which appears if you try to delete a node outside the downloaded area (try it)
Attachments (1)
Change History (7)
comment:1 Changed 4 years ago by dmuecke
- Owner changed from team to dmuecke
comment:2 Changed 4 years ago by delta_foxtrot2
- Cc delta_foxtrot@… added
Changed 4 years ago by dmuecke
comment:3 Changed 4 years ago by dmuecke
- Summary changed from data error caused by duplicate node fix applied outside of the downloaded area to [PATCH] data error caused by duplicate node fix applied outside of the downloaded area
comment:4 Changed 4 years ago by dmuecke
- Status changed from new to assigned
Any reason why this patch isn't merged into JOSM?
comment:5 Changed 4 years ago by Gubaer
- Status changed from assigned to new
Send dmuecke an email with instructions on how to apply the patches to the plugins, build the new plugins, and deploy them.
comment:6 Changed 4 years ago by stoecker
- Resolution set to fixed
- Status changed from new to closed
In [o16970] (modified a bit).



I've added a dialog box to warn users before merging nodes outside downloaded area as recommended.