Opened 12 years ago
Closed 12 years ago
#9350 closed enhancement (fixed)
Validate conditional restrictions (e.g. `maxspeed:conditional)
| Reported by: | mkoniecz | Owned by: | team |
|---|---|---|---|
| Priority: | normal | Milestone: | 14.01 |
| Component: | Core validator | Version: | |
| Keywords: | Cc: |
Description (last modified by )
"Value '60 @ (23:00-06:00)' for key 'maxspeed:conditional' not in presets. - Presets do not contain property value" - I received this from validator for osmwww:browse/way/188660834
It should rather check whatever value is valid according to http://wiki.openstreetmap.org/wiki/Conditional_restrictions
Unfortunately I have no idea about any working code that is able to perform this check.
Repository Root: http://josm.openstreetmap.de/svn Build-Date: 2013-11-20 02:34:39 Last Changed Author: Don-vip Revision: 6394 Repository UUID: 0c6e7542-c601-0410-84e7-c038aed88b3b URL: http://josm.openstreetmap.de/svn/trunk Last Changed Date: 2013-11-19 02:29:10 +0100 (Tue, 19 Nov 2013) Last Changed Rev: 6394 Identification: JOSM/1.5 (6394 en_GB) Windows 7 32-Bit Memory Usage: 247 MB / 247 MB (102 MB allocated, but free) Java version: 1.7.0_45, Oracle Corporation, Java HotSpot(TM) Client VM Dataset consistency test: No problems found Plugin: OpeningHoursEditor (29854) Plugin: notes (v0.6)
Attachments (0)
Change History (7)
comment:1 by , 12 years ago
| Description: | modified (diff) |
|---|
comment:2 by , 12 years ago
comment:3 by , 12 years ago
| Summary: | maxspeed:conditional field should be checked whatever it is valid, not whatever value is in presets → Validate conditional restrictions (e.g. `maxspeed:conditional) |
|---|
comment:5 by , 12 years ago
| Milestone: | → 14.01 |
|---|
comment:6 by , 12 years ago
| Resolution: | fixed |
|---|---|
| Status: | closed → reopened |
There is a small problem with the [6605] change. The :conditional tag doesn't apply to one type of restriction, and that is Turn Restrictions. It says so right on the Conditional restrictions wiki page. Thus, the validator shouldn't be throwing up a warning telling people that the day|date|hour_on|off tags on turn restrictions have been deprecated.
Here's a relation (relation 1168738) that you can use to help fix this as it has time restrictions on it, and it's only active during the weekdays, thus has all four tags on it.



In 6404/josm: