Modify

Opened 9 years ago

Closed 9 years ago

Last modified 9 years ago

#10120 closed defect (fixed)

landuse=railway is treated as water by validator

Reported by: mkoniecz Owned by: team
Priority: normal Milestone: 14.06
Component: Core validator Version:
Keywords: Cc: simon04

Description

"Overlapping Water Areas" reported combination of https://www.openstreetmap.org/way/219725105 (water) https://www.openstreetmap.org/way/245129136 (not water)

Repository Root: http://josm.openstreetmap.de/svn
Build-Date: 2014-06-09 01:34:44
Last Changed Author: akks
Revision: 7224
Repository UUID: 0c6e7542-c601-0410-84e7-c038aed88b3b
URL: http://josm.openstreetmap.de/svn/trunk
Last Changed Date: 2014-06-08 22:01:41 +0200 (Sun, 08 Jun 2014)
Last Changed Rev: 7224

Identification: JOSM/1.5 (7224 en) Windows 7 32-Bit
Memory Usage: 247 MB / 247 MB (121 MB allocated, but free)
Java version: 1.7.0_55, Oracle Corporation, Java HotSpot(TM) Client VM
Dataset consistency test: No problems found

Plugin: OpeningHoursEditor (30416)
Plugin: RoadSigns (30438)
Plugin: buildings_tools (30485)
Plugin: continuosDownload (28565)
Plugin: lakewalker (30416)
Plugin: measurement (30416)
Plugin: notes (v0.9.2)
Plugin: wikipedia (30449)

Attachments (0)

Change History (3)

comment:1 Changed 9 years ago by Don-vip

Cc: simon04 added

Simon, can you take a look? I don't understand why the spatial test is triggered when a natural=water overlaps any other area (like a landuse=railway or a building=yes):

/* Overlapping areas (spatial test) */
area[natural =~ /^(water|wetland|coastline)$/], area[landuse=reservoir] {
  set water_area;
}

area:closed:areaStyle.water_area  area:closed:areaStyle.water_area {
  throwWarning: tr("Overlapping Water Areas");
}

comment:2 Changed 9 years ago by simon04

Resolution: fixed
Status: newclosed

In 7280/josm:

fix #10120 see #10215 - Validator: do not warn "Overlapping Water Areas" for overlaps of water areas with normal areas

comment:3 Changed 9 years ago by simon04

Milestone: 14.06

Fixed in r7280 by using a workaround. Keep the underlying problem as #10215 for a later fix (since this is rather complicated) …

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain team.
as The resolution will be set.
The resolution will be deleted.

Add Comment


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

 
Note: See TracTickets for help on using tickets.