Modify

Opened 9 days ago

Last modified 8 days ago

#15648 new defect

Not warning about open islands with coastline

Reported by: naoliv Owned by: team
Priority: normal Milestone:
Component: Core validator Version:
Keywords: Cc:

Description

Validate the attached example.
JOSM properly warns that an "area style way is not closed"

Now select the two ways, add natural=coastline and validate again: we can't see any warning about the open island.

Unless I am missing some coastline feature, open islands/islets should give a warning even if we have natural=coastline

JOSM:

Relative:URL: ^/trunk
Repository:UUID: 0c6e7542-c601-0410-84e7-c038aed88b3b
Last:Changed Date: 2017-12-05 00:58:56 +0100 (Tue, 05 Dec 2017)
Revision:13194
Build-Date:2017-12-05 02:33:31
URL:http://josm.openstreetmap.de/svn/trunk

Identification: JOSM/1.5 (13194 en) Linux Debian GNU/Linux testing (buster)
Memory Usage: 943 MB / 7168 MB (343 MB allocated, but free)
Java version: 9.0.1+11-Debian-1, Oracle Corporation, OpenJDK 64-Bit Server VM
Screen: :0.0 1600x900, :0.1 1280x1024
Maximum Screen Size: 1600x1024
Java package: openjdk-9-jre:amd64-9.0.1+11-1
Java ATK Wrapper package: libatk-wrapper-java:all-0.33.3-13
VM arguments: [--add-modules=java.activation,java.se.ee, -Dawt.useSystemAAFontSettings=on]
Program arguments: [--language=en]
Dataset consistency test: No problems found

Attachments (1)

example.osm (713 bytes) - added by naoliv 9 days ago.

Download all attachments as: .zip

Change History (4)

Changed 9 days ago by naoliv

Attachment: example.osm added

comment:1 Changed 8 days ago by Klumbumbus

place=island should be used either on a single closed way or a multipolygon, not the way like in you example.osm file.

comment:2 Changed 8 days ago by naoliv

That is why I am asking for a validator warning, so we can catch islands that are wrong.

A real example: this island https://www.openstreetmap.org/#map=15/-23.5418/-45.0677 was wrongly split in multiple ways with place=island, like this way https://www.openstreetmap.org/way/525742175/history

Since the ways also had natural=coastline, JOSM didn't give any warning about the wrong ways tagged with place=island

Last edited 8 days ago by naoliv (previous) (diff)

comment:3 Changed 8 days ago by Klumbumbus

Yes, I was a bit confused.

Modify Ticket

Change Properties
Set your email in Preferences
Action
as new The owner will remain team.
as The resolution will be set.
to The owner will be changed from team to the specified user.
The owner will change to naoliv
as duplicate The resolution will be set to duplicate.The specified ticket will be cross-referenced with this ticket
The owner will be changed from team to anonymous.

Add Comment


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

 
Note: See TracTickets for help on using tickets.