tracktype=grade1 with surface=unpaved is always wrong

[mkoniecz]

What steps will reproduce the problem?

1. Tag highway=track tracktype=grade1 surface=dirt
2. Run validator

What is the expected result?

Validator complains that unpaved surface with tractype=grade1 makes no sense

What happens instead?

Nothing.

Please provide any additional information below. Attach a screenshot if possible.

It may happen as result someone inverting order of tracktypes or if tracktype and surface tags were added separately, maybe years apart.

It makes sense to check it everywhere where both surface and tracktype is tagged, not only on tracks.

Other unpaved values mismatching with tracktype=grade1

"unpaved","compacted","gravel","fine_gravel","pebblestone","ground","earth","dirt","grass","sand","mud","ice","salt","snow","woodchips"

grass_paver is unique and using it with tracktype=grade1 may make sense.

URL:https://josm.openstreetmap.de/svn/trunk
Repository:UUID: 0c6e7542-c601-0410-84e7-c038aed88b3b
Last:Changed Date: 2019-04-03 00:33:43 +0200 (Wed, 03 Apr 2019)
Build-Date:2019-04-03 01:30:50
Revision:14954
Relative:URL: ^/trunk

Identification: JOSM/1.5 (14954 en) Linux Ubuntu 16.04.6 LTS
Memory Usage: 435 MB / 869 MB (78 MB allocated, but free)
Java version: 1.8.0_201-b09, Oracle Corporation, Java HotSpot(TM) 64-Bit Server VM
Screen: :0.0 1366x768
Maximum Screen Size: 1366x768
Dataset consistency test: No problems found

Plugins:
+ OpeningHoursEditor (34867)
+ buildings_tools (34904)
+ continuosDownload (82)
+ imagery_offset_db (34867)
+ measurement (34867)
+ reverter (34961)
+ todo (30306)

Validator rules:
+ ${HOME}/Documents/install_moje/OSM software/josm/data/validator/deprecated.mapcss
+ ${HOME}/Documents/install_moje/OSM software/josm/data/validator/unnecessary.mapcss
+ ${HOME}/Documents/install_moje/OSM software/josm/data/validator/combinations.mapcss

Last errors/warnings:
- W: java.net.SocketTimeoutException: connect timed out
- W: java.net.SocketTimeoutException: connect timed out
- W: java.net.SocketTimeoutException: connect timed out
- W: java.net.SocketTimeoutException: connect timed out
- W: java.net.SocketTimeoutException: connect timed out
- W: java.net.SocketTimeoutException: connect timed out
- W: java.net.SocketTimeoutException: connect timed out
- W: java.net.SocketTimeoutException: connect timed out
- W: java.net.SocketTimeoutException: connect timed out
- W: java.net.SocketTimeoutException: connect timed out

[Klumbumbus]

In 14986/josm:

fix #17592 - warn about tracktype=grade1 with surface which does not fit with grade1

[Klumbumbus]

In 14987/josm:

• see #2760 see #13804 - fix validator rule syntax
• see #17592 - fix validator rule syntax
• fix #17594 - warn about tracktype=grade2 with surface=sand|mud