Modify

Opened 4 years ago

Closed 4 years ago

#2793 closed defect (fixed)

[PATCH] Several errors shown for railway=abandoned

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

Description

No errors should be shown about unconnected ways etc if railway=abandoned since the infrastructure etc has been removed.

Patch attached fixes the problem no idea if it's the best way to do it however.

Attachments (2)

validator.patch.gz (662 bytes) - added by delta_foxtrot2 4 years ago.
validator.patch.gz
crossingways.patch (749 bytes) - added by delta_foxtrot2 4 years ago.
crossingways.patch

Download all attachments as: .zip

Change History (8)

comment:1 Changed 4 years ago by stoecker

  • Summary changed from Several errors shown for railway=abandoned to [PATCH] Several errors shown for railway=abandoned

comment:2 Changed 4 years ago by delta_foxtrot2

  • Cc delta_foxtrot@… added

Changed 4 years ago by delta_foxtrot2

validator.patch.gz

comment:3 Changed 4 years ago by delta_foxtrot2

The new patch also exempts boundary=administrative from being considered a way that nodes end near.

comment:4 Changed 4 years ago by stoecker

  • Resolution set to fixed
  • Status changed from new to closed

In [o16319]. Please no tabs stops and proper indentation.

Changed 4 years ago by delta_foxtrot2

crossingways.patch

comment:5 Changed 4 years ago by delta_foxtrot2

  • Resolution fixed deleted
  • Status changed from closed to reopened

I somehow missed patching the CrossingWays.java file, not sure why I overlooked this, but the latest patch fixes this issue too.

comment:6 Changed 4 years ago by stoecker

  • Resolution set to fixed
  • Status changed from reopened to closed

Add Comment

Modify Ticket

Change Properties
<Author field>
Action
as closed .
as The resolution will be set. Next status will be 'closed'.
The resolution will be deleted. Next status will be 'reopened'.
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.