Modify

Opened 7 years ago

Closed 4 years ago

Last modified 4 years ago

#12556 closed enhancement (fixed)

detect the same way forming a loop by being twice in a multipolygon

Reported by: aceman Owned by: team
Priority: normal Milestone: 16.12
Component: Core validator Version: latest
Keywords: multipolygon Cc:

Description

If a way is a member twice in the same multipolygon and the two occurrences are connected (their index in the relation is n and n+1), JOSM displays the relation fine, with a closed loop in the relation editor. There is also no warning about this. So while this "area" is closed, it is probably useless for a multipolygon.

You can observe this case in the history of way 188885975 .

Attachments (0)

Change History (11)

comment:1 Changed 4 years ago by Don-vip

Keywords: multipolygon added

comment:2 Changed 4 years ago by GerdP

We have the errors

  • "Multipolygon member(s) repeated with same role" and
  • "Multipolygon member(s) repeated with different role"

for this, so what is missing?

comment:3 Changed 4 years ago by GerdP

Resolution: fixed
Status: newclosed

The test that produces these messages was introduced with r11157 (3 years ago)

comment:4 Changed 4 years ago by Don-vip

Milestone: 16.12

comment:5 Changed 4 years ago by GerdP

I tried to set the milestone but Trac only offers those in the future. Do I need special rights for this?

comment:6 Changed 4 years ago by aceman

Yes, looks like those validator checks detect this error these days.
Thanks.

comment:7 in reply to:  5 Changed 4 years ago by Don-vip

Replying to GerdP:

I tried to set the milestone but Trac only offers those in the future. Do I need special rights for this?

I wasn't aware of this. Please try to logout/login from Trac, I've given you full rights on milestones.

comment:8 Changed 4 years ago by GerdP

Milestone: 16.12

I see no change. The drop-down list contains only milestones in the future and I cannot edit the field.

comment:9 Changed 4 years ago by GerdP

Oops, seems I have the right to remove the milestone, but I cannot set it to 16.12 :(

comment:10 Changed 4 years ago by GerdP

@Don-Vip: Please can you set the milestone again?

comment:11 Changed 4 years ago by Klumbumbus

Milestone: 16.12

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.

Add Comment


E-mail address and name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.