Opened 13 years ago

Last modified 8 years ago

#8724 closed defect

Highway preset missing in "properties panel" if it's a closed loop, but not tagged "area=yes" — at Version 9

Reported by: rickmastfan67 Owned by: team
Priority: minor Milestone:
Component: Internal preset Version: latest
Keywords: preset property toogle dialog Cc:

Description (last modified by Don-vip)

Steps to reproduce:

  1. Download the following way: way/11699138
  2. Select the downloaded way.

What happens:
Notice that in the "properties panel" there is no highway preset at the top like when you select a non-loop highway way.

What should happen:
When selecting the "loop" way, the highway preset for type of highway you selected should show up like a normal way that doesn't loop back on itself.

This happens will all types of highways that have presets.

Change History (9)

comment:1 by skyper, 13 years ago

Component: Internal presetCore

comment:2 by Don-vip, 13 years ago

Component: CoreInternal preset

comment:3 by Don-vip, 13 years ago

Resolution: fixed
Status: newclosed

In 5976/josm:

fix #8724 - Add closedway as supported type of more highways

in reply to:  3 comment:4 by skyper, 13 years ago

Replying to Don-vip:

In 5976/josm:

fix #8724 - Add closedway as supported type of more highways

Sorry, I do not understand your change. As #8663 states, closedway is included in way.

Seems to me that code in some other place does not follow this rule.

comment:5 by Don-vip, 13 years ago

Mmmm, it did fix the problem however.

in reply to:  5 comment:6 by skyper, 13 years ago

Resolution: fixed
Status: closedreopened

Replying to Don-vip:

This will not work. Please think about external presets !
Either we change the definition of type:way (#8663) or the code creating the property toogle dialog and this link.

Mmmm, it did fix the problem however.

Only for highways but not for barrier=guard_rail for example.

comment:7 by skyper, 12 years ago

Component: Internal presetCore
Keywords: preset property toogle dialog added

comment:8 by skyper, 11 years ago

I just played around a bit and found another problem with closedway <-> relation.

  • As an area can always be a multipolygon relation we need to always add both (closedway + relation).
  • On the other hand, it might be better to distinguish between relations, as this problem is only true for multipolygons and all other relations would probably be errors.

comment:9 by Don-vip, 8 years ago

Description: modified (diff)
Note: See TracTickets for help on using tickets.