Modify

Opened 8 months ago

Last modified 8 months ago

#24216 new defect

[PATCH] Validator shouldn't raise warning about missing tag for informal crossing

Reported by: mtk2 Owned by: team
Priority: normal Milestone:
Component: Core validator Version: tested
Keywords: tag crossing informal Cc:

Description

What steps will reproduce the problem?

  1. Node representing the crossing of the path and the railway is tagged with only crossing=informal tag.

What is the expected result?

Validator shouldn't raise any warning/error for such node.

What happens instead?

Validator raises a warning "missing tag - pedestrian railway crossing without railway=crossing".

Please provide any additional information below. Attach a screenshot if possible.

According to Wiki (https://wiki.openstreetmap.org/wiki/Tag:crossing%3Dinformal) this tag shouldn't be used with highway=crossing because it shouldn't be expected a real crossing there. Per analogy, this rule should be applied to railway=crossing too.

Relative:URL: ^/trunk
Repository:UUID: 0c6e7542-c601-0410-84e7-c038aed88b3b
Last:Changed Date: 2025-02-27 18:02:18 +0100 (Thu, 27 Feb 2025)
Revision:19342
Build-Date:2025-02-28 02:30:51
URL:https://josm.openstreetmap.de/svn/trunk

Identification: JOSM/1.5 (19342 en) Linux Debian GNU/Linux 12 (bookworm)
Memory Usage: 340 MB / 3976 MB (69 MB allocated, but free)
Java version: 17.0.14+7-Debian-1deb12u1, Debian, OpenJDK 64-Bit Server VM
Look and Feel: javax.swing.plaf.metal.MetalLookAndFeel
Screen: :0.0 1920x1080x[Multi depth]@60Hz (scaling 1.00×1.00)
Maximum Screen Size: 1920×1080
Best cursor sizes: 16×16→16×16, 32×32→32×32
Environment variable LANG: en_US.utf8
System property file.encoding: UTF-8
System property sun.jnu.encoding: UTF-8
Locale info: en_US
Numbers with default locale: 1234567890 -> 1234567890
Desktop environment: GNOME
Java package: openjdk-17-jre:amd64-17.0.14+7-1~deb12u1
Java ATK Wrapper package: libatk-wrapper-java:all-0.40.0-3
libcommons-compress-java: libcommons-compress-java:all-1.22-1
fonts-noto: fonts-noto:all-20201225-1
VM arguments: [--add-exports=java.base/sun.security.action=ALL-UNNAMED, --add-exports=java.desktop/com.sun.imageio.plugins.jpeg=ALL-UNNAMED, --add-exports=java.desktop/com.sun.imageio.spi=ALL-UNNAMED, -Dsun.java2d.opengl=True, -Djosm.home=<josm.pref>]
Dataset consistency test: No problems found

Attachments (1)

24216-Dont-raise-warning-for-node-tagged-as-crossing-informal.diff (946 bytes ) - added by mtk2 8 months ago.
Don't raise warning at the intersection of railway and path if node is tagged as crossing=informal

Download all attachments as: .zip

Change History (3)

by mtk2, 8 months ago

Don't raise warning at the intersection of railway and path if node is tagged as crossing=informal

comment:1 by anonymous, 8 months ago

Summary: Validator shouldn't raise warning about missing tag for informal crossing[PATCH] Validator shouldn't raise warning about missing tag for informal crossing

comment:2 by anonymous, 8 months ago

Version: tested

Modify Ticket

Change Properties
Set your email in Preferences
Action
as new The owner will remain team.
as The resolution will be set. Next status will be 'closed'.
to The owner will be changed from team to the specified user.
Next status will be 'needinfo'. The owner will be changed from team to mtk2.
as duplicate The resolution will be set to duplicate. Next status will be 'closed'. The specified ticket will be cross-referenced with this ticket.
The owner will be changed from team to anonymous. Next status will be 'assigned'.

Add Comment


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