Changeset 3215 in josm for trunk/src/org/openstreetmap/josm/gui/widgets/ComboBoxHistory.java

Timestamp:

2010-05-03T10:52:48+02:00 (15 years ago)

Author:

bastiK

Message:

fixed #4979 Exception when opening the download window; fixed #2221 - Unexcepted Exception when typing into "Change values?" box with an input method on Mac OS X

File:

• trunk/src/org/openstreetmap/josm/gui/widgets/ComboBoxHistory.java (modified) (6 diffs)

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

@@ -35 +35 @@
 import javax.swing.DefaultComboBoxModel;
 
-public class ComboBoxHistory extends DefaultComboBoxModel implements Iterable<String> {
+import org.openstreetmap.josm.gui.tagging.ac.*;
+
+public class ComboBoxHistory extends DefaultComboBoxModel implements Iterable<AutoCompletionListItem> {
 
     private int maxSize = 10;
@@ -50 +52 @@
     @Override
     public void addElement(Object o) {
-        String newEntry = (String)o;
+        if (o instanceof String) {
+            o = new AutoCompletionListItem((String) o);
+        }
+
+        String newEntry = ((AutoCompletionListItem)o).getValue();
 
         // if history contains this object already, delete it,
         // so that it looks like a move to the top
         for (int i = 0; i < getSize(); i++) {
-            String oldEntry = (String) getElementAt(i);
+            String oldEntry = ((AutoCompletionListItem) getElementAt(i)).getValue();
             if(oldEntry.equals(newEntry)) {
                 removeElementAt(i);
@@ -75 +81 @@
     }
 
-    public Iterator<String> iterator() {
-        return new Iterator<String>() {
+    public Iterator<AutoCompletionListItem> iterator() {
+        return new Iterator<AutoCompletionListItem>() {
 
             private int position = -1;
@@ -90 +96 @@
             }
 
-            public String next() {
+            public AutoCompletionListItem next() {
                 position++;
-                return getElementAt(position).toString();
+                return (AutoCompletionListItem)getElementAt(position);
             }
 
@@ -98 +104 @@
     }
 
-    public void setItems(List<String> items) {
+    public void setItemsAsString(List<String> items) {
         removeAllElements();
-        Collections.reverse(items);
-        for (String item : items) {
-            addElement(item);
+        for (int i = items.size()-1; i>=0; i--) {
+            addElement(new AutoCompletionListItem(items.get(i)));
         }
-        Collections.reverse(items);
     }
 
-    public List<String> asList() {
+    public List<String> asStringList() {
         List<String> list = new ArrayList<String>(maxSize);
-        for (String item : this) {
-            list.add(item);
+        for (AutoCompletionListItem item : this) {
+            list.add(item.getValue());
         }
         return list;
@@ -125 +129 @@
     private void fireHistoryChanged() {
         for (HistoryChangedListener l : listeners) {
-            l.historyChanged(asList());
+            l.historyChanged(asStringList());
         }
     }