Opened 17 months ago
Closed 7 months ago
#7259 closed enhancement (fixed)
[patch] Validation Enhancements for water features
| Reported by: | oldtopos | Owned by: | team |
|---|---|---|---|
| Priority: | normal | Component: | Core validator |
| Version: | latest | Keywords: | Validation |
| Cc: |
Description
I've modified OverlappingAreas and CrossingWays slightly, such that OverlappingAreas reports water area features separately as Warnings rather than informational.
CrossingWays is modified to report crossing water ways separately.
These changes are a great help when validating the addition of NHD features.
Attachments (1)
Change History (4)
Changed 17 months ago by oldtopos
comment:1 Changed 17 months ago by stoecker
- Summary changed from Validation Enhancements for water features to [patch] Validation Enhancements for water features
comment:2 Changed 17 months ago by simon04
- Component changed from Core to Core validator
The patch looks mostly fine, besides:
- remove debugging output (System.out.println)
- use consistent indentation and spacing around parenthesis (e.g., if (!isWaterArea2) {})
Unfortunately, all of this validation tests get very complex when adding special cases (like water areas overlaps, railway overlaps). IMHO, the way to go is to fix the cooperation between filters and validator tests (see #6195). Then, one can specifically filter for cases of interest. Thus, I'd refrain from committing this patch now.
comment:3 Changed 7 months ago by bastiK
- Resolution set to fixed
- Status changed from new to closed
In 5550/josm:



diff file for validation test changes