Opened 6 years ago

Last modified 6 years ago

#19094 closed enhancement

[Patch] Warning about "placement:forward/backward=transition" — at Version 3

Reported by: skyper Owned by: team
Priority: normal Milestone: 20.07
Component: Core validator Version:
Keywords: lanes-tagging placement transition Cc:

Description (last modified by skyper)

What steps will reproduce the problem?

  1. Have a way with placement:forward/backward=transition
  2. Run validator

What is the expected result?

A warning suggesting to use placement=transition

What happens instead?

No informational warning

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

There is no need to use forward/backward with transition but it does not really harm. Still tagging scheme would be more consistent if only placement=transition would be used.

Relative:URL: ^/trunk
Repository:UUID: 0c6e7542-c601-0410-84e7-c038aed88b3b
Last:Changed Date: 2020-04-13 22:05:56 +0200 (Mon, 13 Apr 2020)
Revision:16294
Build-Date:2020-04-14 01:30:47
URL:https://josm.openstreetmap.de/svn/trunk

Change History (4)

comment:1 by skyper, 6 years ago

Oh, I see, there is no warning about multiple occurrences of placement[:forward/backward/both_ways]. This needs to be an error.

by skyper, 6 years ago

Attachment: josm_19094.patch​ added

patch including warnings about multiple use and transition with placement:*

comment:2 by skyper, 6 years ago

Find attached patch josm_19094.patch​ which adds warnings for both issues

  • Warning about transition as values of placement:*
  • Error for multiple use of placement[:*]

comment:3 by skyper, 6 years ago

Description: modified (diff)
Priority: minor → normal
Summary: Informational warning about "placement:forward/backward=transition" → [Patch] Warning about "placement:forward/backward=transition"
Note: See TracTickets for help on using tickets.