Changeset 15358 in josm for trunk/src/org/openstreetmap/josm/actions/downloadtasks/PostDownloadHandler.java

Timestamp:

2019-09-19T12:26:55+02:00 (5 years ago)

Author:

Don-vip

Message:

fix #18105 - Show a notification if no notes can be downloaded

File:

• trunk/src/org/openstreetmap/josm/actions/downloadtasks/PostDownloadHandler.java (modified) (4 diffs)

trunk/src/org/openstreetmap/josm/actions/downloadtasks/PostDownloadHandler.java

@@ -7 +7 @@
 import java.util.ArrayList;
 import java.util.Collection;
+import java.util.HashSet;
 import java.util.LinkedHashSet;
 import java.util.Set;
@@ -34 +35 @@
     private Consumer<Collection<Object>> errorReporter;
 
+    private static final Set<String> NO_DATA_ERROR_MESSAGES = new HashSet<>();
+
     /**
      * Creates a new {@link PostDownloadHandler}
@@ -54 +57 @@
         this(task, future);
         this.errorReporter = errorReporter;
+    }
+
+    /**
+     * Adds a new translated error message indicating that no data has been downloaded.
+     * @param message new translated error message indicating that no data has been downloaded.
+     * @return {@code true} if the message was not already known
+     * @since 15358
+     */
+    public static boolean addNoDataErrorMessage(String message) {
+        return NO_DATA_ERROR_MESSAGES.add(message);
+    }
+
+    /**
+     * Determines if a translated error message indicates that no data has been downloaded.
+     * @param message translated error message to check
+     * @return {@code true} if the message indicates that no data has been downloaded
+     * @since 15358
+     */
+    public static boolean isNoDataErrorMessage(Object message) {
+        return NO_DATA_ERROR_MESSAGES.contains(message);
     }
 
@@ -87 +110 @@
                     if (error instanceof Exception) {
                         ExceptionDialogUtil.explainException((Exception) error);
-                    } else if (tr("No data found in this area.").equals(error)) {
+                    } else if (isNoDataErrorMessage(error)) {
                         new Notification(error.toString()).setIcon(JOptionPane.WARNING_MESSAGE).show();
                     } else {