Modify

Opened 7 years ago

Closed 7 years ago

#14288 closed enhancement (fixed)

Verify for improper usage of type=multipolygon in nodes and ways

Reported by: naoliv Owned by: Klumbumbus
Priority: normal Milestone: 17.01
Component: Core validator Version:
Keywords: type multipolygon Cc:

Description

With 254 nodes and 13927 ways with type=multipolygon http://overpass-turbo.eu/s/luN, shouldn't JOSM issue a warning for such cases?

node[type = multipolygon],
way[type = multipolygon] {
        throwWarning: tr("some nice message");
}

type=multipolygon is a property of relations only.

JOSM:

URL:http://josm.openstreetmap.de/svn/trunk
Repository:UUID: 0c6e7542-c601-0410-84e7-c038aed88b3b
Last:Changed Date: 2017-01-25 02:17:36 +0100 (Wed, 25 Jan 2017)
Build-Date:2017-01-25 02:33:55
Revision:11490
Relative:URL: ^/trunk

Identification: JOSM/1.5 (11490 pt_BR) Linux Debian GNU/Linux 9.0 (stretch)
Memory Usage: 274 MB / 10206 MB (95 MB allocated, but free)
Java version: 1.8.0_111-8u111-b14-3-b14, Oracle Corporation, OpenJDK 64-Bit Server VM
Screen: :0.0 1600x900, :0.1 1280x1024
Maximum Screen Size: 1600x1024
Java package: openjdk-8-jre:amd64-8u111-b14-3
Java ATK Wrapper package: libatk-wrapper-java:all-0.33.3-13
VM arguments: [-Dawt.useSystemAAFontSettings=on]
Dataset consistency test: No problems found

Attachments (0)

Change History (6)

comment:1 by Don-vip, 7 years ago

Keywords: type multipolygon added
Milestone: 17.01

comment:2 by Klumbumbus, 7 years ago

13927 ways

This includes the old style multipolygons with the tags on the outer instead of on the relation. JOSM still supports these old style. See also #10529.

in reply to:  2 ; comment:3 by stoecker, 7 years ago

Replying to Klumbumbus:

13927 ways

This includes the old style multipolygons with the tags on the outer instead of on the relation. JOSM still supports these old style. See also #10529.

Yeah, but even for these "type=multipolygon" was never part of the way. That have always been errors.

BTW. I'd still like #10529 to be solved...

in reply to:  3 comment:4 by Klumbumbus, 7 years ago

Replying to stoecker:

Yeah, but even for these "type=multipolygon" was never part of the way.

Ah yes, sorry, I've messed this up.

comment:5 by Klumbumbus, 7 years ago

Owner: changed from team to Klumbumbus
Status: newassigned

comment:6 by Klumbumbus, 7 years ago

Resolution: fixed
Status: assignedclosed

In 11500/josm:

fix #14288 - verify for improper usage of type=multipolygon in nodes and ways

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Klumbumbus.
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.