Changeset 18128 in josm for trunk/src/org/openstreetmap

Timestamp:

2021-08-07T00:40:05+02:00 (3 years ago)

Author:

Don-vip

Message:

fix #21203 - Use Object.toString() to display history combobox items (patch by marcello)

File:

• trunk/src/org/openstreetmap/josm/gui/widgets/HistoryComboBox.java (modified) (2 diffs)

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

@@ -6 +6 @@
 import javax.swing.text.JTextComponent;
 
-import org.openstreetmap.josm.data.tagging.ac.AutoCompletionItem;
 import org.openstreetmap.josm.gui.tagging.ac.AutoCompletingComboBox;
 import org.openstreetmap.josm.spi.preferences.Config;
@@ -56 +55 @@
      */
     public void addCurrentItemToHistory() {
-        Object item = getEditor().getItem();
-        if (item instanceof String) {
-            model.addElement((String) item);
-        } else if (item instanceof AutoCompletionItem) {
-            model.addElement((AutoCompletionItem) item);
-        } else {
-            throw new IllegalArgumentException(item.getClass().getSimpleName() + " is not supported in addCurrentItemToHistory");
-        }
+        model.addElement(getEditor().getItem().toString());
     }