Changeset 15072 in josm for trunk/src/org/openstreetmap/josm/gui/tagging/presets/TaggingPresetReader.java

Timestamp:

2019-05-11T22:23:33+02:00 (5 years ago)

Author:

Don-vip

Message:

fix #17698 - don't merge single checks into immediate following checkgroup

File:

• trunk/src/org/openstreetmap/josm/gui/tagging/presets/TaggingPresetReader.java (modified) (3 diffs)

trunk/src/org/openstreetmap/josm/gui/tagging/presets/TaggingPresetReader.java

@@ -131 +131 @@
         parser.mapOnStart("roles", Roles.class);
         parser.map("role", Role.class);
-        parser.map("checkgroup", CheckGroup.class);
+        parser.mapBoth("checkgroup", CheckGroup.class);
         parser.map("check", Check.class);
         parser.map("combo", Combo.class);
@@ -167 +167 @@
         XmlObjectParser parser = buildParser();
 
+        /** to detect end of {@code <checkgroup>} */
+        CheckGroup lastcheckgroup = null;
         /** to detect end of {@code <group>} */
         TaggingPresetMenu lastmenu = null;
@@ -281 +283 @@
                         lastrole.roles.add((Role) o);
                     } else if (o instanceof Check) {
-                        checks.add((Check) o);
+                        if (lastcheckgroup != null) {
+                            checks.add((Check) o);
+                        } else {
+                            all.getLast().data.add((TaggingPresetItem) o);
+                        }
                     } else if (o instanceof ComboMultiSelect.PresetListEntry) {
                         listEntries.add((ComboMultiSelect.PresetListEntry) o);
                     } else if (o instanceof CheckGroup) {
-                        all.getLast().data.add((TaggingPresetItem) o);
-                        // Make sure list of checks is empty to avoid adding checks several times
-                        // when used in chunks (fix #10801)
-                        ((CheckGroup) o).checks.clear();
-                        ((CheckGroup) o).checks.addAll(checks);
-                        checks.clear();
+                        CheckGroup cg = (CheckGroup) o;
+                        if (cg == lastcheckgroup) {
+                            lastcheckgroup = null;
+                            all.getLast().data.add(cg);
+                            // Make sure list of checks is empty to avoid adding checks several times
+                            // when used in chunks (fix #10801)
+                            cg.checks.clear();
+                            cg.checks.addAll(checks);
+                            checks.clear();
+                        } else {
+                            lastcheckgroup = cg;
+                        }
                     } else {
                         if (!checks.isEmpty()) {