Modify

Opened 10 years ago

Closed 9 years ago

Last modified 5 years 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 (7.4 KB ) - added by Klumbumbus 7 years ago.

Download all attachments as: .zip

Change History (13)

comment:1 by Aun Johnsen <lists@…>, 10 years ago

Cc: lists@… added

comment:2 by naoliv, 10 years ago

It seems that the beers scared people here

comment:3 by Klumbumbus, 10 years ago

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

comment:4 by naoliv, 10 years ago

Description: modified (diff)

comment:5 by Klumbumbus, 9 years ago

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 by Klumbumbus, 9 years ago

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

comment:7 by Klumbumbus, 9 years ago

Milestone: 16.02

comment:8 by Klumbumbus, 9 years ago

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

comment:9 by Klumbumbus, 7 years ago

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.

comment:11 by Klumbumbus, 7 years ago

In 12895/josm:

see #11270, see #9297 - avoid false positive missing railway crossing warnig for combined railway highway ways.

by Klumbumbus, 7 years ago

Attachment: ticket11270.osm added

comment:12 by Klumbumbus, 5 years ago

In 15314/josm:

fix #18070, see #15483, fix #17022, see #11270 - restore display of tram when tagged together or overlapping with highway, handle light_rail the same as tram, i.e. draw on top of highway and don't warn about missing railway=crossing|level_crossing

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.