Modify

Opened 13 years ago

Closed 13 years ago

Last modified 13 years ago

#5844 closed enhancement (wontfix)

[Patch] Validator: Warn when *:{backward|forward}=* appears with oneway=!no

Reported by: AM909 Owned by: team
Priority: normal Milestone:
Component: Core validator Version: latest
Keywords: validator forward backward oneway Cc: AM909

Description

When splitting a way into a dual carriageway, it's possible to overlook the fact that there are *:{backward|forward}=* tags (e.g. source:maxspeed:forward) that need special handling.

Please add a warning to the validator to check for any forward/backward tags when the oneway key is present and its value is not "no".

Attachments (1)

5844.patch (1.4 KB ) - added by simon04 13 years ago.

Download all attachments as: .zip

Change History (6)

comment:1 by AM909, 13 years ago

Type: defectenhancement

by simon04, 13 years ago

Attachment: 5844.patch added

comment:2 by simon04, 13 years ago

Summary: Validator: Warn when *:{backward|forward}=* appears with oneway=!no[Patch] Validator: Warn when *:{backward|forward}=* appears with oneway=!no

The attached patch should provide the desired warning. Additionally, the description of the validator file format is clarified.

comment:3 by stoecker, 13 years ago

Resolution: fixed
Status: newclosed

In [4295/josm]:

fix #5844 - patch by simon04 - Validator: Warn when *:{backward|forward}=* appears with oneway=!no

comment:4 by simon04, 13 years ago

In [4397/josm]:

fix #6760, see #5844 - remove validator warning "oneway combined with *:backward/forward" as it causes false positives

comment:5 by simon04, 13 years ago

Resolution: fixedwontfix

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.