Modify

Opened 10 years ago

Closed 10 years ago

Last modified 10 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 by Don-vip, 10 years ago

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 by simon04, 10 years ago

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 by simon04, 10 years ago

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. Next status will be 'reopened'.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.