Modify ↓

Opened 3 years ago

Closed 3 years ago

Last modified 3 years ago

#21192 closed enhancement (invalid)

[Patch] Warn about multiple access values with *:lanes and/or :forward/backward

Reported by: skyper Owned by: team
Priority: normal Milestone:
Component: Core validator Version:
Keywords: access lanes_tagging multiple value Cc:

Description (last modified by skyper)

This is a followup of #19419.

Access tags per direction are not yet checked: *(:both_ways)?:[backward|forward] und *:both_ways as all *:lanes* see my comment 11 on #19419.

bus:forward=private;designated
bus:both_ways:backward=no;designated
access:lanes=no|yes;no|yes
access:lanes:both_ways:forward=agricultural;forestry;no|yes

I would go with a general warning as mentioning the exact keys would need many lines of code.

Attachments (1)

josm_21192.patch​ (5.5 KB ) - added by skyper 3 years ago.
patch file

Download all attachments as: .zip

Change History (12)

comment:1 by skyper, 3 years ago

Owner: changed from team to skyper

by skyper, 3 years ago

Attachment: josm_21192.patch​ added

patch file

comment:2 by skyper, 3 years ago

Milestone: → 21.08
Owner: changed from skyper to team
Summary: Warn about multiple access values with *:lanes and/or :forward/backward → [Patch] Warn about multiple access values with *:lanes and/or :forward/backward

Please, find attached patch file​ which adds the checks for access tags with direction and/or lanes.

comment:3 by Don-vip, 3 years ago

Resolution: → fixed
Status: new → closed

In 18132/josm:

fix #21192 - Warn about multiple access values with *:lanes and/or :forward/backward (patch by skyper)

comment:4 by skyper, 3 years ago

Resolution: fixed
Status: closed → reopened

Please, revert, see my comment 17 on #19419.

comment:5 by skyper, 3 years ago

Description: modified (diff)
Summary: [Patch] Warn about multiple access values with *:lanes and/or :forward/backward → Warn about conflicting, multiple access values with *:lanes and/or :forward/backward

comment:6 by Don-vip, 3 years ago

In 18182/josm:

see #19419 - see #21192 - revert "Multiple access values" checks

comment:7 by Don-vip, 3 years ago

Resolution: → fixed
Status: reopened → closed

comment:8 by skyper, 3 years ago

Summary: Warn about conflicting, multiple access values with *:lanes and/or :forward/backward → [Patch] Warn about multiple access values with *:lanes and/or :forward/backward

comment:9 by skyper, 3 years ago

See #21249 for a new try.

comment:10 by anonymous, 3 years ago

Milestone: 21.08

comment:11 by Don-vip, 3 years ago

Resolution: fixed → invalid

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain team.
as The resolution will be set.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment
E-mail address and name can be saved in the Preferences .
Attachments ↑ Description ↑
 
Note: See TracTickets for help on using tickets.