Modify

Opened 2 years ago

Closed 18 months ago

Last modified 7 days ago

#11270 closed enhancement (fixed)

Test for missing/wrong railway=crossing|level_crossing

Reported by: naoliv Owned by: team
Priority: normal Milestone: 16.02
Component: Core validator Version:
Keywords: Cc: lists@…

Description (last modified by naoliv)

We exchange this for 3 beers if accepted or for 1 beer if something needs to be adjusted (1 beer too if not accepted):

way[railway =~ /^(light_rail|miniature|narrow_gauge|preserved|rail|tram)$/] > node { set .is_in_railway }
way[highway =~ /^(bridleway|cycleway|footway|path|steps)$/] > node[railway != crossing].is_in_railway {
        throwWarning: tr("should have railway=crossing where pedestrians cross");
        fixAdd: "railway=crossing";
}

way[highway =~ /^(bus_guideway|living_street|motorway|pedestrian|primary|raceway|residential|road|secondary|service|tertiary|track|trunk|unclassified)/] > node[railway != level_crossing].is_in_railway {
        throwWarning: tr("should have railway=level_crossing where road cross");
        fixAdd: "railway=level_crossing";
}

Attachments (1)

ticket11270.osm (6.9 KB) - added by Klumbumbus 7 days ago.

Download all attachments as: .zip

Change History (11)

comment:1 Changed 2 years ago by Aun Johnsen <lists@…>

Cc: lists@… added

comment:2 Changed 2 years ago by naoliv

It seems that the beers scared people here

comment:3 Changed 2 years ago by Klumbumbus

Ticket #9300 has been marked as a duplicate of this ticket.

comment:4 Changed 2 years ago by naoliv

Description: modified (diff)

comment:5 Changed 18 months ago by Klumbumbus

Resolution: fixed
Status: newclosed

In 9839/josm:

fix #11270 - add validator warning for missing/wrong railway=crossing|level_crossing (modified patch by naoliv)

comment:6 Changed 18 months ago by Klumbumbus

Ticket #12519 has been marked as a duplicate of this ticket.

comment:7 Changed 18 months ago by Klumbumbus

Milestone: 16.02

comment:8 Changed 17 months ago by Klumbumbus

Ticket #9298 has been marked as a duplicate of this ticket.

comment:9 Changed 7 days ago by Klumbumbus

In 12595/josm:

see #11270 - don't warn about missing railway=crossing|level_crossing for questionable cases e.g. when the highway changes from residential to footway at the railway crossing node. This avoids an infinite loop of the validator fix button.

Changed 7 days ago by Klumbumbus

Attachment: ticket11270.osm added

comment:10 Changed 7 days ago by Klumbumbus

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain team.
as The resolution will be set. Next status will be 'closed'.
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.