From 57ff1fc5b5597df4bf7a631fbf6bb507bc85d1f3 Mon Sep 17 00:00:00 2001
From: Artur Signell <artur@vaadin.com>
Date: Mon, 25 Jun 2012 19:02:37 +0300
Subject: [PATCH] Changed AbstractInMemoryContainer.getItemIds() to return List
 (#8829)

---
 src/com/vaadin/data/util/AbstractBeanContainer.java     | 5 +++--
 src/com/vaadin/data/util/AbstractInMemoryContainer.java | 4 ++--
 2 files changed, 5 insertions(+), 4 deletions(-)

diff --git a/src/com/vaadin/data/util/AbstractBeanContainer.java b/src/com/vaadin/data/util/AbstractBeanContainer.java
index bed3ca0450..300f92ea3a 100644
--- 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 cc7f469c0c..795aabb419 100644
--- 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
-- 
2.39.5