Ignore:
Timestamp:
2021-09-08T00:58:22+02:00 (9 months ago)
Author:
Don-vip
Message:

fix #21285 - Some more warnings on relations (patch by skyper):

  • adds warnings for type=destination_sign without destination=* and type=building without building=*
  • adds TRANSPORT_MODE=private to no and use_side as problematic values for ways member of route=bicycle/mtb
  • adds route=hiking/running/walking/fitness_trail and route=horse with inappropriate foot=* respectively horse=* to the warning above.
File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/resources/data/validator/relation.mapcss

    r16255 r18199  
    99/* from http://wiki.openstreetmap.org/wiki/Types_of_relation */
    1010/* see also #9071 */
     11relation[type=boundary][!boundary][!disused:boundary],
     12relation[type=building][!building],
     13relation[type=destination_sign][!destination],
     14relation[type=enforcement][!enforcement],
     15relation[type=public_transport][!public_transport],
    1116relation[type=route][!route],
    1217relation[type=route_master][!route_master],
    13 relation[type=boundary][!boundary][!disused:boundary],
    14 relation[type=public_transport][!public_transport],
    15 relation[type=waterway][!waterway],
    16 relation[type=enforcement][!enforcement] {
     18relation[type=waterway][!waterway] {
    1719  throwWarning: tr("{0} relation without {0} tag", "{1.key}");
    1820  group: tr("missing tag");
     
    4446relation[type=route][route=mtb    ] > way[bicycle=no],
    4547relation[type=route][route=bicycle] > way[bicycle=no],
     48relation[type=route][route=mtb    ] > way[bicycle=private],
     49relation[type=route][route=bicycle] > way[bicycle=private],
    4650relation[type=route][route=mtb    ] > way[bicycle=use_sidepath],
    47 relation[type=route][route=bicycle] > way[bicycle=use_sidepath] {
    48   throwWarning: tr("way with {0} is part of a bicycle route relation", "{0.tag}");
     51relation[type=route][route=bicycle] > way[bicycle=use_sidepath],
     52relation[type=route][route=fitness_trail] > way[foot=no],
     53relation[type=route][route=hiking       ] > way[foot=no],
     54relation[type=route][route=running      ] > way[foot=no],
     55relation[type=route][route=walking      ] > way[foot=no],
     56relation[type=route][route=fitness_trail] > way[foot=private],
     57relation[type=route][route=hiking       ] > way[foot=private],
     58relation[type=route][route=running      ] > way[foot=private],
     59relation[type=route][route=walking      ] > way[foot=private],
     60relation[type=route][route=fitness_trail] > way[foot=use_sidepath],
     61relation[type=route][route=hiking       ] > way[foot=use_sidepath],
     62relation[type=route][route=running      ] > way[foot=use_sidepath],
     63relation[type=route][route=walking      ] > way[foot=use_sidepath],
     64relation[type=route][route=horse] > way[horse=no],
     65relation[type=route][route=horse] > way[horse=private],
     66relation[type=route][route=horse] > way[horse=use_sidepath] {
     67  throwWarning: tr("way with {0} is part of a {1} route relation", "{0.tag}", "{0.key}");
    4968}
Note: See TracChangeset for help on using the changeset viewer.