Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Multiple crossing nodes on crossing way not tagged correctly #1292

Closed
atiannicelli opened this issue Jan 11, 2024 · 1 comment
Closed

Multiple crossing nodes on crossing way not tagged correctly #1292

atiannicelli opened this issue Jan 11, 2024 · 1 comment
Labels
bug Something isn't working feature-validator Issue or request related to the validator
Milestone

Comments

@atiannicelli
Copy link

Description

If I try to use the "Tag this crossing as 'lines'" suggestion to fix a way with two crossings on it then it does not make the change that I asked it to make. (See steps to reproduce for details)

Screenshots

Before fixing the intersection:
Screenshot 2024-01-11 at 10 03 01 AM

After clicking on "Tag this crossing as 'lines'"
Screenshot 2024-01-11 at 10 03 25 AM

Version

2.2.3

What browser are you seeing the problem on? What version are you running?

Chrome v120.0

The OS you're using

mac

Steps to reproduce

I have a way with the following tags

crossing=marked
crossing:markings=yes
footway=crossing
highway=footway

and TWO nodes on that way that have the following tags:

crossing=marked
crossing:markings=lines
highway=crossing

When I click on the way it gives me this warning:

Crossing line and point tags conflict

Tag this crossing as 'marked'
Tag this crossing as 'lines'
Remove all crossing tags
Ignore this issue

I then click on the "Tag this crossing as 'lines'" and instead of the way being changed to crossing:marking=lines it changes the nodes to crossing:marking=yes.

If I remove one of the crossing nodes (and the way that it is on) then it behaves correctly so I think this is specific to having two crossing nodes on the crossing way.

The browser URL at the time you encountered the bug

https://rapideditor.org/edit#background=Bing&datasets=fbRoads,msBuildings&disable_features=boundaries&id=w706113826&map=20.00/42.34726/-88.13017

The auto-detected useragent string for your browser (leave blank if you're manually filling this form out)

Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/120.0.0.0 Safari/537.36

@bhousel
Copy link
Contributor

bhousel commented Jan 11, 2024

Whoa good catch @atiannicelli - this was a caused by an unfortunate typo that only occured if that part of code was visited more than once. Easy fix though, I'll get a patch out soon.

@bhousel bhousel added this to the v2.2.4 milestone Jan 11, 2024
@bhousel bhousel added bug Something isn't working feature-validator Issue or request related to the validator labels Jan 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working feature-validator Issue or request related to the validator
Projects
None yet
Development

No branches or pull requests

2 participants