Changeset 2484 in josm for trunk/src/org/openstreetmap/josm/gui/dialogs/relation

Timestamp:

2009-11-19T18:09:12+01:00 (14 years ago)

Author:

Gubaer

Message:

fixed #3925: Download relation members takes a long time to return

File:

• trunk/src/org/openstreetmap/josm/gui/dialogs/relation/GenericRelationEditor.java (modified) (6 diffs)

trunk/src/org/openstreetmap/josm/gui/dialogs/relation/GenericRelationEditor.java

@@ -56 +56 @@
 import org.openstreetmap.josm.data.conflict.Conflict;
 import org.openstreetmap.josm.data.osm.DataSet;
-import org.openstreetmap.josm.data.osm.DataSetMerger;
-import org.openstreetmap.josm.data.osm.DataSource;
 import org.openstreetmap.josm.data.osm.OsmPrimitive;
 import org.openstreetmap.josm.data.osm.OsmPrimitiveType;
@@ -1379 +1377 @@
     public static class DownloadTask extends PleaseWaitRunnable {
         private boolean cancelled;
-        private int conflictsCount;
         private Exception lastException;
         private List<Relation> relations;
@@ -1386 +1383 @@
 
         public DownloadTask(List<Relation> relations, OsmDataLayer curLayer, MemberTableModel memberTableModel, Dialog parent) {
-            super(tr("Download relation members"), new PleaseWaitProgressMonitor(parent), false /*
-             * don't
-             * ignore
-             * exception
-             */);
+            super(tr("Download relation members"), new PleaseWaitProgressMonitor(parent), false /* don't ignore exception */);
             this.relations = relations;
             this.curLayer = curLayer;
@@ -1397 +1390 @@
 
         public DownloadTask(List<Relation> relations, OsmDataLayer curLayer, MemberTableModel memberTableModel) {
-            super(tr("Download relation members"), new PleaseWaitProgressMonitor(), false /*
-             * don't
-             * ignore
-             * exception
-             */);
+            super(tr("Download relation members"), new PleaseWaitProgressMonitor(), false /* don't ignore exception */);
             this.relations = relations;
             this.curLayer = curLayer;
@@ -1424 +1413 @@
                 ExceptionDialogUtil.explainException(lastException);
             }
-
-            if (conflictsCount > 0) {
-                JOptionPane.showMessageDialog(
-                        Main.parent,
-                        tr("There were {0} conflicts during import.", conflictsCount),
-                        tr("Warning"),
-                        JOptionPane.WARNING_MESSAGE
-                );
-            }
         }
 
@@ -1438 +1418 @@
         protected void realRun() throws SAXException, IOException, OsmTransferException {
             try {
-                boolean changed = false;
                 for (Relation relation : relations) {
                     progressMonitor.indeterminateSubTask("");
                     OsmServerObjectReader reader = new OsmServerObjectReader(relation.getId(), OsmPrimitiveType.RELATION,
                             true);
-                    DataSet dataSet = reader.parseOsm(progressMonitor
+                    final DataSet dataSet = reader.parseOsm(progressMonitor
                             .createSubTaskMonitor(ProgressMonitor.ALL_TICKS, false));
-                    if (dataSet != null) {
-                        changed = true;
-                        final DataSetMerger visitor = new DataSetMerger(curLayer.data, dataSet);
-                        visitor.merge();
-
-                        // copy the merged layer's data source info
-                        for (DataSource src : dataSet.dataSources) {
-                            curLayer.data.dataSources.add(src);
-                        }
-                        if (!visitor.getConflicts().isEmpty()) {
-                            curLayer.getConflicts().add(visitor.getConflicts());
-                            conflictsCount = visitor.getConflicts().size();
-                        }
-                    }
-                }
-                // FIXME: this is necessary because there are dialogs listening
-                // for DataChangeEvents which manipulate Swing components on this
-                // thread.
-                //
-                if (changed) {
-                    SwingUtilities.invokeLater(new Runnable() {
-                        public void run() {
-                            curLayer.fireDataChange();
-                            curLayer.onPostDownloadFromServer();
-                        }
-                    });
+                    if (dataSet == null)
+                        return;
+                    // has to run on the EDT because mergeFrom may trigger events
+                    // which update the UI
+                    //
+                    SwingUtilities.invokeAndWait(
+                            new Runnable() {
+                                public void run() {
+                                    curLayer.mergeFrom(dataSet);
+                                    curLayer.fireDataChange();
+                                    curLayer.onPostDownloadFromServer();
+                                }
+                            }
+                    );
+
                 }
             } catch (Exception e) {