ngageoint / hootenanny

Hootenanny conflates multiple maps into a single seamless map.
GNU General Public License v3.0
355 stars 74 forks source link

JOSM - Applying a MGCP Preset to Multipolygon removes type=multipolygon tag #4582

Open MikeTho16 opened 3 years ago

MikeTho16 commented 3 years ago

Describe the bug If the mapper creates a multipolygon relation in JOSM, and then applies an MGCP preset, JOSM removes the type=multipolygon tag, which makes the relation invalid, and results in an error when attempting to upload.

Applies to It is believed that this applies generally to any MGCP feature code for which an area is a valid representation. This is a list of feature codes which have been tested and have been found to exhibit this error:

To Reproduce

  1. Open JOSM
  2. Download an area image
  3. Make sure schema is set to "MGCP"
  4. Draw a closed way
  5. Draw another closed way, this one completely within the first.
  6. Select both ways
  7. Tools - > Create Multipolygon
  8. In the tags panel, JOSM displays:
    image
  9. Press F3 on keyboard
  10. Search for "mine"
  11. Click on "MGCP/Culture/Extraction/Extraction Mine"
    image
  12. Click the "Select" button image
  13. In the preset dialog box, click "Apply Preset" image
  14. In the Tag Panel JOSM displays:
    image Notice there is no type=multipolygon tag
  15. Click upload button: image
  16. JOSM displays: image

Expected behavior

Screenshots

Desktop (please complete the following information):

Smartphone (please complete the following information):

Additional context

MikeTho16 commented 2 years ago

I tested this today in JOSM 11.0.0, the behavior is a little different, but I believe it is still an issue.

Here is what the feature looks like with the MGCP schema: image

Here is what the feature looks like with the OSM schema (note no tags identifying it as a mine): image

Here is what happens when an attempt is made to upload that feature: image

brianhatchl commented 2 years ago

Have a fix for this, just need to merge and release.