Modify

Opened 3 years ago

Last modified 23 months ago

#5120 reopened defect

moving tags from outer way to MP-relation

Reported by: brogo Owned by: bilbo
Priority: major Component: Plugin multipoly
Version: Keywords:
Cc:

Description

When you create an multipolygon (e.g. a lake with an island), the tags (e.g. natural=water) belong to the relation, not to the outer way. If the outer way has tags, these should be moved to the new MP-relation.

Attachments (0)

Change History (18)

comment:1 Changed 3 years ago by bilbo

Not necessarily - the tags can be both on the outer way(s) or in the relation.

Copying tags to relation is good only if there are multiple outer ways (as the information should be present only once in database), it is not very good for single way, as it is then harder to edit the tags in JOSM (you have to click on relation editor to see and edit the tags) - though it is technically correct :)

I guess I'll add this feature with setting to copy the tags: "never"/"always"/"only if multiple outer ways" (which would be the default)

comment:2 Changed 3 years ago by bastiK

We could set "always" as default. This is what the wiki says for some time now. (Since Frederik Ramm put it there...)

Basically it is the task of the editor to make area (multipolygon) handling easier.

comment:3 Changed 3 years ago by bilbo

Well, the feature is not yet complete, but until #5592 is fixed, I don't think having "always" as default is a good idea.

comment:4 Changed 2 years ago by Zverikk

Ticket #5835 has been marked as a duplicate of this ticket.

comment:5 Changed 2 years ago by Zverikk

Sooooooo?.. Can we have at least an option somewhere deep in settings list? :)

comment:6 Changed 2 years ago by malenki

bastik, could you please link the place of the wiki you refer to?

comment:7 Changed 2 years ago by stoecker

Link: http://wiki.openstreetmap.org/wiki/Multipolygon#Tagging

Note: You can edit tags of a relation in JOSM like for ways. You only need to select the relation (right click relation and select in popup in several places in josm).

comment:8 Changed 2 years ago by malenki

Thanks for the link. Frederik Ramm put the mentioned suggestion on the page. Later he moved it to the discussion from where later Stoecker did move it back to the main page.
I do not agree (and don't see the sense) in tagging no outer way but put the tags into the relation as long as the tagged outer way is used in one multipolygon only.
This makes things IMO only more complicated and less obvious.

comment:9 Changed 2 years ago by stoecker

Well the relation is the object, so it should be tagged instead of the way. Nevertheless tagging outer ways will be supported also in the future.

comment:10 follow-up: Changed 2 years ago by malenki

So if there is a multipolygon contained in a multipolygon following you suggestions I should add the inner multipolygon but not the outer way of the inner multipolygon to the outer multipolygon as inner?

comment:11 in reply to: ↑ 10 Changed 2 years ago by bastiK

Replying to malenki:

So if there is a multipolygon contained in a multipolygon following you suggestions I should add the inner multipolygon but not the outer way of the inner multipolygon to the outer multipolygon as inner?

This is nonsense. We define areas by the boundary lines and not by a mixture of boundary line and adjacent areas. Also, it has nothing to do with putting the tags on the relation as the actual object that represents the area. In practice, it may be more convenient to tag the single outer way, but we shouldn't stop people who like to do it right.

comment:12 Changed 2 years ago by malenki

Thanks for the "nonsense". I only continued stoecker's thought of using the object (the multipolygon) instead of some helping way.

comment:13 follow-up: Changed 2 years ago by stoecker

It is plainly wrong. Not the area is the inner part, but the outer ring of this area. Which you can easily verify when you have a wood containing of 10 small parts scattered around. Then not this multipolygon-wood is the inner element of the whatevers laying around (e.g. multiple landuses like farms), but the individual outer rings.

You try to construct silly examples and mangle different topics to prove your point. As bastiK correctly says this is nonsense.

comment:14 in reply to: ↑ 13 Changed 2 years ago by malenki

Replying to stoecker:

You try to construct silly examples and mangle different topics to prove your point. As bastiK correctly says this is nonsense.

Neither is my example "silly" (I have seen and created quite a lot multipolys contaning multipolys) nor did I want to prove something nor mangle different topics. I asked a question based on continuing your thoughts, so what? Maybe I am just too stupid to follow the hi-flying thoughts in here?
Anyway I thank you very much for the sensible, rational and friendly answers.

comment:15 Changed 2 years ago by Zverikk

BTW, this functionality is now implemented in the reltoolbox plugin: http://wiki.openstreetmap.org/wiki/JOSM/Plugins/Relation_Toolbox

comment:16 follow-up: Changed 2 years ago by skyper

  • Resolution set to fixed
  • Status changed from new to closed

This has been fixed.

r4114

comment:17 in reply to: ↑ 16 Changed 2 years ago by skyper

  • Resolution fixed deleted
  • Status changed from closed to reopened
  • Type changed from enhancement to defect

Replying to skyper:

This has been fixed.

r4114

Was to fast !

Now it does not copy the tags to the way if using undo.

r4115

comment:18 Changed 23 months ago by skyper

Still problem.

r4228, o26254

Add Comment

Modify Ticket

Change Properties
<Author field>
Action
as reopened .
as The resolution will be set. Next status will be 'closed'.
to The owner will be changed from bilbo. Next status will be 'new'.
Next status will be 'needinfo'.The owner will change to brogo
as duplicate The resolution will be set to duplicate. Next status will be 'closed'.The specified ticket will be cross-referenced with this ticket
Author


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

 
Note: See TracTickets for help on using tickets.