Opened 13 years ago
Closed 12 years ago
#7259 closed enhancement (fixed)
[patch] Validation Enhancements for water features
Reported by: | oldtopos | Owned by: | team |
---|---|---|---|
Priority: | normal | Milestone: | |
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)
by , 13 years ago
Attachment: | validations.diff added |
---|
comment:1 by , 13 years ago
Summary: | Validation Enhancements for water features → [patch] Validation Enhancements for water features |
---|
comment:2 by , 13 years ago
Component: | Core → 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.
diff file for validation test changes