Modify

Opened 2 years ago

Closed 23 months ago

Last modified 22 months ago

#5844 closed enhancement (wontfix)

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

Reported by: Am909 Owned by: team
Priority: normal 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 23 months ago.

Download all attachments as: .zip

Change History (6)

comment:1 Changed 2 years ago by Am909

  • Type changed from defect to enhancement

Changed 23 months ago by simon04

comment:2 Changed 23 months ago by simon04

  • Summary changed from Validator: Warn when *:{backward|forward}=* appears with oneway=!no to [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 Changed 23 months ago by stoecker

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

In [4295/josm]:

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

comment:4 Changed 22 months ago by simon04

In [4397/josm]:

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

comment:5 Changed 22 months ago by simon04

  • Resolution changed from fixed to wontfix

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.