Changed AbstractInMemoryContainer.getItemIds() to return List (#8829)

diff --git a/src/com/vaadin/data/util/AbstractBeanContainer.java b/src/com/vaadin/data/util/AbstractBeanContainer.java
index bed3ca04508d22d56b76dc9a192c9601e99ffbb7..300f92ea3a217c7eb66cc88647aa4a1ae81f6eae 100644 (file)
--- a/src/com/vaadin/data/util/AbstractBeanContainer.java
+++ b/src/com/vaadin/data/util/AbstractBeanContainer.java
@@ -8,6 +8,7 @@ import java.util.Collection;
 import java.util.HashMap;
 import java.util.Iterator;
 import java.util.LinkedHashMap;
+import java.util.List;
 import java.util.Map;
 
 import com.vaadin.data.Container;
@@ -247,8 +248,8 @@ public abstract class AbstractBeanContainer<IDTYPE, BEANTYPE> extends
      */
     @Override
     @SuppressWarnings("unchecked")
-    public Collection<IDTYPE> getItemIds() {
-        return (Collection<IDTYPE>) super.getItemIds();
+    public List<IDTYPE> getItemIds() {
+        return (List<IDTYPE>) super.getItemIds();
     }
 
     /*

diff --git a/src/com/vaadin/data/util/AbstractInMemoryContainer.java b/src/com/vaadin/data/util/AbstractInMemoryContainer.java
index cc7f469c0cba7cab828f4e582047816aeae6c838..795aabb41953c6a46d27620e22adecceead4c062 100644 (file)
--- a/src/com/vaadin/data/util/AbstractInMemoryContainer.java
+++ b/src/com/vaadin/data/util/AbstractInMemoryContainer.java
@@ -165,8 +165,8 @@ public abstract class AbstractInMemoryContainer<ITEMIDTYPE, PROPERTYIDCLASS, ITE
         }
     }
 
-    public Collection<?> getItemIds() {
-        return Collections.unmodifiableCollection(getVisibleItemIds());
+    public List<?> getItemIds() {
+        return Collections.unmodifiableList(getVisibleItemIds());
     }
 
     // Container.Ordered