Ticket #18701: 18701.patch

src/org/openstreetmap/josm/gui/io/DownloadPrimitivesTask.java

@@ -55 +55 @@
     protected void initMultiFetchReader(MultiFetchServerObjectReader reader) {
         getProgressMonitor().indeterminateSubTask(tr("Initializing nodes to download ..."));
         reader.setRecurseDownRelations(fullRelation);
-        reader.appendIds(ids);
+        if (ids != null) {
+            ids.forEach(reader::append);
+        }
     }
 }

src/org/openstreetmap/josm/io/MultiFetchServerObjectReader.java

@@ -131 +131 @@
      *
      * @param id  the id
      */
-    protected void remember(PrimitiveId id) {
+    public void append(PrimitiveId id) {
         if (id.isNew()) return;
         switch(id.getType()) {
         case NODE: nodes.add(id.getUniqueId()); break;
@@ -152 +152 @@
      */
     public MultiFetchServerObjectReader append(DataSet ds, long id, OsmPrimitiveType type) {
         OsmPrimitive p = ds.getPrimitiveById(id, type);
-        switch(type) {
-        case NODE:
-            return appendNode((Node) p);
-        case WAY:
-            return appendWay((Way) p);
-        case RELATION:
-            return appendRelation((Relation) p);
-        default:
-            return this;
-        }
+        return append(p);
     }
 
     /**
@@ -172 +163 @@
      */
     public MultiFetchServerObjectReader appendNode(Node node) {
         if (node == null || node.isNew()) return this;
-        remember(node.getPrimitiveId());
+        append(node.getPrimitiveId());
         return this;
     }
 
@@ -185 +176 @@
     public MultiFetchServerObjectReader appendWay(Way way) {
         if (way == null || way.isNew()) return this;
         if (recurseDownAppended) {
-            for (Node node : way.getNodes()) {
-                if (!node.isNew()) {
-                    remember(node.getPrimitiveId());
-                }
-            }
+            append(way.getNodes());
         }
-        remember(way.getPrimitiveId());
+        append(way.getPrimitiveId());
         return this;
     }
 
@@ -203 +190 @@
      */
     protected MultiFetchServerObjectReader appendRelation(Relation relation) {
         if (relation == null || relation.isNew()) return this;
-        remember(relation.getPrimitiveId());
+        append(relation.getPrimitiveId());
         if (recurseDownAppended) {
             for (RelationMember member : relation.getMembers()) {
                 // avoid infinite recursion in case of cyclic dependencies in relations
@@ -245 +232 @@
      */
     public MultiFetchServerObjectReader append(Collection<? extends OsmPrimitive> primitives) {
         if (primitives == null) return this;
-        for (OsmPrimitive primitive : primitives) {
-            append(primitive);
-        }
-        return this;
-    }
-
-    /**
-     * appends a list of {@link PrimitiveId} to the list of ids which will be fetched from the server.
-     *
-     * @param ids the list of primitive Ids (ignored, if null)
-     * @return this
-     * @since 15811
-     *
-     */
-    public MultiFetchServerObjectReader appendIds(Collection<PrimitiveId> ids) {
-        if (ids == null)
-            return this;
-        for (PrimitiveId id : ids) {
-            if (id.isNew()) continue;
-            switch (id.getType()) {
-            case NODE:
-                nodes.add(id.getUniqueId());
-                break;
-            case WAY:
-            case CLOSEDWAY:
-                ways.add(id.getUniqueId());
-                break;
-            case MULTIPOLYGON:
-            case RELATION:
-                relations.add(id.getUniqueId());
-                break;
-            default:
-                throw new AssertionError();
-            }
-        }
+        primitives.forEach(this::append);
         return this;
     }