When mapping a forest inside a forest and make that inner, the inner landuse=forest tag is removed.

[SekeRob]

Reproducible: First encountered with a named forest area with specific tree species inside a large forest MP. Regardless if directly touched or adding another inner area of any kind to the forest MP, the landuse=forest tag is removed from any 'landuse=forest' inner area. All other tags are left alone such as leaf_cycle/type, name, etc. The inner area renders blank area within the forest. Tested in ID Editor added the forest key back, then on next edit in JOSM the tag gets removed again from the inner area.

[GerdP]

I assume you mean the Update multipolygon action (Ctrl+Shift+B)?
At least I can reproduce it with this action with r19039.

[SekeRob]

Yes, the shortcut combo. Actually not tested what happens if added via the relation toolbox... removed the forest area as member, opened RT and added. Fine. Added another non-forest inner element via the shortcut and white space on the inner forest. Any unsuspecting mapper given the note there's a dozen more inner areas to map will cause it to go blank again. When arriving first time to the named area it was already blank on Carto which is why I looked and went thru a few cycles before realizing what was going on. (I religiously visit Carto with a few Ctrl+F5 to see the map effect of the last CS).

[GerdP]

It works like this since 10 years or so, I think since r5225. Both the Create and Update multipolygon action will remove the tags from the members (all of them). I wonder why nobody complained so far.
I think the solution is to not remove any tags beside an area=yes from any inner way. No unit test seems to check this so far.

[GerdP]

patch and unit test

[SekeRob]

I can see when doing a create it removes the tags from the polygon outline and moves them to the type=multipolygon relation. Think there's a validation rule that says there's a 'style' tags on an outline which happens when merging an MP with another polygon. We have this pop up panel now asking which tags should be proliferated to the expanded relation. In my case it's removing tags from an polygon made inner (with the shortcut). I'd not be surprised if there's a wood MP and an inner is added also of wood cause of area specificity, it makes the assumption of 'wood+wood is too much'.

[skyper]

Why are the tags of a member with role inner removed?

[GerdP]

That's the question. It was implemented like this in r5225 and never changed since.

[GerdP]

Of course only those tags which are moved to the multipolygon relation are removed, and there are some exceptions, but that's what it does. I guess at that time few objects had detailed sub tags, and a hole in something with a tag X typically doesn't have tag X.

[GerdP]

In 19066/josm:

fix #23641: When mapping a forest inside a forest and make that inner, the inner landuse=forest tag is removed.

• change code so the tags of inner ways which are common to the multipolygon are not removed.
• add unit test