Changeset 34827 in osm for applications/editors/josm/plugins

Timestamp:

2019-01-16T09:18:48+01:00 (6 years ago)

Author:

gerdp

Message:

see #14545 - Open o5m files with negative IDs fails
Same solution for o5m: use AbstractReader.buildPrimitive()

File:

• applications/editors/josm/plugins/o5m/src/org/openstreetmap/josm/plugins/o5m/io/O5mReader.java (modified) (6 diffs)

applications/editors/josm/plugins/o5m/src/org/openstreetmap/josm/plugins/o5m/io/O5mReader.java

@@ -18 +18 @@
 import org.openstreetmap.josm.data.coor.LatLon;
 import org.openstreetmap.josm.data.osm.DataSet;
-import org.openstreetmap.josm.data.osm.Node;
+import org.openstreetmap.josm.data.osm.NodeData;
 import org.openstreetmap.josm.data.osm.OsmPrimitiveType;
-import org.openstreetmap.josm.data.osm.Relation;
+import org.openstreetmap.josm.data.osm.RelationData;
 import org.openstreetmap.josm.data.osm.RelationMemberData;
 import org.openstreetmap.josm.data.osm.User;
-import org.openstreetmap.josm.data.osm.Way;
+import org.openstreetmap.josm.data.osm.WayData;
 import org.openstreetmap.josm.data.osm.UploadPolicy;
 import org.openstreetmap.josm.gui.progress.NullProgressMonitor;
@@ -257 +257 @@
             if (version == 0)
                 discourageUpload = true;
-            Node node = new Node(lastNodeId, version == 0 ? 1 : version);
-            node.setCoor(new LatLon(flat, flon).getRoundedToOsmPrecision());
-
-
-            checkCoordinates(node.getCoor());
+            NodeData nd = new NodeData(lastNodeId);
+            nd.setVersion(version == 0 ? 1 : version);
+            nd.setCoor(new LatLon(flat, flon).getRoundedToOsmPrecision());
+
+            checkCoordinates(nd.getCoor());
             checkChangesetId(lastChangeSet);
-            node.setChangesetId((int) lastChangeSet);
+            nd.setChangesetId((int) lastChangeSet);
             // User id
             if (lastTs != 0) {
                 checkTimestamp(lastTs);
-                node.setTimestamp(new Date(lastTs * 1000));
+                nd.setTimestamp(new Date(lastTs * 1000));
                 if (osmUser != null)
-                    node.setUser(osmUser);
+                    nd.setUser(osmUser);
             }
             if (bytesToRead > 0) {
                 Map<String, String> keys = readTags();
-                node.setKeys(keys);
-            }
-            externalIdMap.put(node.getPrimitiveId(), node);
+                nd.setKeys(keys);
+            }
+            buildPrimitive(nd);
+            
         } catch (IllegalDataException e) {
             exception = e;
@@ -299 +300 @@
             if (version == 0)
                 discourageUpload = true;
-            final Way way = new Way(lastWayId, version == 0 ? 1 : version);
+            final WayData wd = new WayData(lastWayId);
+            wd.setVersion(version == 0 ? 1 : version);
             checkChangesetId(lastChangeSet);
-            way.setChangesetId((int) lastChangeSet);
+            wd.setChangesetId((int) lastChangeSet);
             // User id
             if (lastTs != 0) {
                 checkTimestamp(lastTs);
-                way.setTimestamp(new Date(lastTs * 1000));
+                wd.setTimestamp(new Date(lastTs * 1000));
                 if (osmUser != null)
-                    way.setUser(osmUser);
+                    wd.setUser(osmUser);
             }
 
@@ -320 +322 @@
 
             Map<String, String> keys = readTags();
-            way.setKeys(keys);
-            ways.put(way.getUniqueId(), nodeIds);
-            externalIdMap.put(way.getPrimitiveId(), way);
+            wd.setKeys(keys);
+            ways.put(wd.getUniqueId(), nodeIds);
+            buildPrimitive(wd);
         } catch (IllegalDataException e) {
             exception = e;
@@ -345 +347 @@
             if (version == 0)
                 discourageUpload = true;
-            final Relation rel = new Relation(lastRelId, version == 0 ? 1 : version);
+            final RelationData rel = new RelationData(lastRelId);
+            rel.setVersion(version == 0 ? 1 : version);
             checkChangesetId(lastChangeSet);
             rel.setChangesetId((int) lastChangeSet);
@@ -378 +381 @@
             rel.setKeys(keys);
             relations.put(rel.getUniqueId(), members);
-            externalIdMap.put(rel.getPrimitiveId(), rel);
+            buildPrimitive(rel);
         } catch (IllegalDataException e) {
             exception = e;