Modify

Opened 15 years ago

Closed 15 years ago

Last modified 15 years ago

#2794 closed defect (fixed)

[PATCH] Invalid "Way end node near other way" error

Reported by: delta_foxtrot2 Owned by: team
Priority: major Milestone:
Component: Core validator Version:
Keywords: Cc: delta_foxtrot@…

Description

A way that ends with a node being highway=turning_circle close to another way shows an error about a "Way end node near other way" and this isn't correct as the turning circle should indicate the way being final and that it doesn't have to connect to anything.

I'm trying to code a fix for this myself, but I don't fully understand the framework yet and I'm struggling to figure out how this would be solved.

Attachments (0)

Change History (6)

comment:1 by delta_foxtrot2, 15 years ago

Cc: delta_foxtrot@… added

comment:2 by delta_foxtrot2, 15 years ago

Summary: Invalid "Way end node near other way" error[PATCH] Invalid "Way end node near other way" error

Line 67 of UnconnectedWays.java just needs to following 2 lines of code to fix this issue:

            if(en.get("highway") != null && en.get("highway").equals("turning_circle"))
                continue;

comment:3 by delta_foxtrot2, 15 years ago

In the case that there isn't a turning circle, noexit=yes should also be considered terminal

            if(en.get("noexit") != null && en.get("noexit").equals("yes"))
                continue;

comment:4 by delta_foxtrot2, 15 years ago

Testing further there is another corner case, some ways end with a barrier, which in some cases turn into access=private, otherwise they are terminal nodes and shouldn't be connected and shouldn't show a warning about not being connected.

Adding the following code below the above items solves this.

            if(en.get("barrier") != null)
                continue;

comment:5 by stoecker, 15 years ago

Resolution: fixed
Status: newclosed

In [o16319]. Boolean checks aren't that easy.

in reply to:  5 comment:6 by delta_foxtrot2, 15 years ago

Replying to stoecker:

In [o16319]. Boolean checks aren't that easy.

This is a case of corner cases that haven't been caught previously, this isn't probably not all of them but as I come across others I'll try and work out fixes for them.

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.