Modify

Opened 4 years ago

Closed 4 years ago

#11389 closed enhancement (fixed)

[Patch] Check for maxspeed in combination with maxspeed:forward/backward

Reported by: Basstoelpel Owned by: Basstoelpel
Priority: normal Milestone: 15.09
Component: Core validator Version: latest
Keywords: validator maxspeed Cc:

Description

Validator should check for the combination of maxspeed and maxspeed:forward or maxspeed:backward. Either the combination is superfluous or inconsistent.

And maybe it should check for equal values in maxspeed:forward and maxspeed:backward, because that can be simplified.

Attachments (1)

validate-maxspeed.patch (7.2 KB) - added by windu.2b 4 years ago.

Download all attachments as: .zip

Change History (7)

Changed 4 years ago by windu.2b

Attachment: validate-maxspeed.patch added

comment:1 Changed 4 years ago by windu.2b

This patch validates the "maxspeed", "maxspeed:backward", "maxspeed:forward", "source:maxspeed" and "zone:maxspeed" values.

If "maxspeed:backward" and "maxspeed:forward" have the same value, a warning message suggests to use the "maxspeed" tag instead.

comment:2 Changed 4 years ago by mdk

Summary: Check for maxspeed in combination with maxspeed:forward/backward[Patch] Check for maxspeed in combination with maxspeed:forward/backward

comment:3 Changed 4 years ago by simon04

Milestone: 15.07

Please see my comment here: ticket:11529#comment:3

comment:4 Changed 4 years ago by Don-vip

Milestone: 15.0715.08

Milestone renamed

comment:5 Changed 4 years ago by Don-vip

Milestone: 15.0815.09
Owner: changed from team to Basstoelpel
Status: newneedinfo

Thanks for the patch. Like for #11529, can you please see if the test can be converted to a MapCSS-based one instead?

comment:6 Changed 4 years ago by Klumbumbus

Resolution: fixed
Status: needinfoclosed

In 8700/josm:

fix #11389 - Add validator tests for combinations of maxspeed, maxspeed:forward and maxspeed:backward; enhance numerical test to forward and backward versions and remove "*=unknown" from whitelist of this numerical test

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Basstoelpel.
as The resolution will be set.
The resolution will be deleted.

Add Comment


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

 
Note: See TracTickets for help on using tickets.