Modify

Opened 3 years ago

Closed 3 years ago

#20089 closed defect (fixed)

Don't flag railway=rail crossing railway=yard

Reported by: GerdP Owned by: GerdP
Priority: normal Milestone: 20.12
Component: Core validator Version:
Keywords: template_report Cc:

Description

What steps will reproduce the problem?

  1. Download area around https://www.openstreetmap.org/way/266945129
  2. run validator

What is the expected result?

no warnings about Crossing railways

What happens instead?

Crossing railways (8)

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

Build-Date:2020-11-15 10:37:11
Revision:17310
Is-Local-Build:true

Identification: JOSM/1.5 (17310 SVN en) Windows 10 64-Bit
OS Build number: Windows 10 Home 2004 (19041)
Memory Usage: 758 MB / 1753 MB (389 MB allocated, but free)
Java version: 1.8.0_272-b10, AdoptOpenJDK, OpenJDK 64-Bit Server VM
Look and Feel: com.sun.java.swing.plaf.windows.WindowsLookAndFeel
Screen: \Display0 1920x1080 (scaling 1.0x1.0)
Maximum Screen Size: 1920x1080
Best cursor sizes: 16x16 -> 32x32, 32x32 -> 32x32
VM arguments: [-agentlib:jdwp=transport=dt_socket,suspend=y,address=localhost:52777, -ea, -javaagent:D:\eclipse-java-2020-09\eclipse\configuration\org.eclipse.osgi\215\0\.cp\lib\javaagent-shaded.jar, -Dfile.encoding=UTF-8]
Program arguments: [--debug]
Dataset consistency test: No problems found

Plugins:
+ buildings_tools (35640)
+ o5m (35640)
+ pbf (35640)
+ poly (35640)
+ reverter (35640)
+ undelete (35640)
+ utilsplugin2 (35640)

Validator rules:
+ c:\josm\core\resources\data\validator\geometry.mapcss

Attachments (2)

20089.patch (651 bytes ) - added by GerdP 3 years ago.
20089.2.patch (712 bytes ) - added by GerdP 3 years ago.

Download all attachments as: .zip

Change History (7)

comment:1 by GerdP, 3 years ago

Owner: changed from team to GerdP
Status: assignednew

by GerdP, 3 years ago

Attachment: 20089.patch added

comment:2 by GerdP, 3 years ago

with patch the test ignores crossings of ways if one is railway=yard and the other is not

by GerdP, 3 years ago

Attachment: 20089.2.patch added

comment:3 by GerdP, 3 years ago

Make sure both ways have railway=* tag. First patch suppressed also warnings about highway=* crossing railway=yard,

comment:4 by Don-vip, 3 years ago

Milestone: 20.1120.12

Milestone renamed

comment:5 by GerdP, 3 years ago

Resolution: fixed
Status: newclosed

In 17348/josm:

fix #20089: Don't flag railway=rail crossing railway=yard

  • ignore crossings of ways if one is railway=yard and the other is not and both have the railway key

Modify Ticket

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