Modify

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)

validations.diff (9.1 KB) - added by oldtopos 17 months ago.
diff file for validation test changes

Download all attachments as: .zip

Change History (4)

Changed 17 months ago by oldtopos

diff file for validation test changes

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:

applied #7259 - Validation Enhancements for water features (patch by oldtopos, slightly modified)

Add Comment

Modify Ticket

Change Properties
<Author field>
Action
as closed .
as The resolution will be set. Next status will be 'closed'.
The resolution will be deleted. Next status will be 'reopened'.
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.