Changeset 9486 in josm

Timestamp:

2016-01-16T14:16:03+01:00 (9 years ago)

Author:

simon04

Message:

Note upload: explain first exception to the user

File:

• trunk/src/org/openstreetmap/josm/actions/upload/UploadNotesTask.java (modified) (4 diffs)

trunk/src/org/openstreetmap/josm/actions/upload/UploadNotesTask.java

@@ -14 +14 @@
 import org.openstreetmap.josm.data.notes.NoteComment;
 import org.openstreetmap.josm.data.osm.NoteData;
+import org.openstreetmap.josm.gui.ExceptionDialogUtil;
 import org.openstreetmap.josm.gui.PleaseWaitRunnable;
 import org.openstreetmap.josm.gui.progress.ProgressMonitor;
@@ -118 +119 @@
             } catch (Exception e) {
                 Main.error("Failed to upload note to server: " + note.getId());
+                Main.error(e);
                 failedNotes.put(note, e);
             }
@@ -130 +132 @@
             noteData.updateNotes(updatedNotes);
             if (!failedNotes.isEmpty()) {
-                Main.error("Some notes failed to upload");
                 StringBuilder sb = new StringBuilder();
                 for (Map.Entry<Note, Exception> entry : failedNotes.entrySet()) {
@@ -138 +139 @@
                 Main.error("Notes failed to upload: " + sb.toString());
                 JOptionPane.showMessageDialog(Main.map, sb.toString(), tr("Notes failed to upload"), JOptionPane.ERROR_MESSAGE);
+                ExceptionDialogUtil.explainException(failedNotes.values().iterator().next());
             }
         }