Changeset 11198 in josm for trunk/src/org/openstreetmap/josm/gui/widgets/EditableList.java

Timestamp:

2016-10-30T22:32:05+01:00 (9 years ago)

Author:

Don-vip

Message:

fix #13881 - handle cases where JOptionPane.showInputDialog returns an empty string

File:

• trunk/src/org/openstreetmap/josm/gui/widgets/EditableList.java (modified) (3 diffs)

trunk/src/org/openstreetmap/josm/gui/widgets/EditableList.java

@@ -50 +50 @@
                     title,
                     JOptionPane.QUESTION_MESSAGE);
-            if (source != null) {
+            if (source != null && !source.isEmpty()) {
                 ((DefaultListModel<String>) sourcesList.getModel()).addElement(source);
             }
@@ -65 +65 @@
                 if (sourcesList.getModel().getSize() == 0) {
                     String source1 = JOptionPane.showInputDialog(Main.parent, title, title, JOptionPane.QUESTION_MESSAGE);
-                    if (source1 != null) {
+                    if (source1 != null && !source1.isEmpty()) {
                         ((DefaultListModel<String>) sourcesList.getModel()).addElement(source1);
                     }
@@ -82 +82 @@
                         JOptionPane.QUESTION_MESSAGE, null, null,
                         sourcesList.getSelectedValue());
-                if (source2 != null) {
+                if (source2 != null && !source2.isEmpty()) {
                     ((DefaultListModel<String>) sourcesList.getModel()).setElementAt(source2, row);
                 }