From: Artur Signell <artur.signell@itmill.com>
Date: Mon, 1 Mar 2010 15:42:21 +0000 (+0000)
Subject: #4249 - IndexedContainer.addItemAfter(Object) should return item id, not item
X-Git-Tag: 6.7.0.beta1~2013
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=b9d426e2d06f8dbc78b74e1e76a1ee03b2a218a7;p=vaadin-framework.git

#4249 - IndexedContainer.addItemAfter(Object) should return item id, not item

svn changeset:11566/svn branch:6.3
---

diff --git a/src/com/vaadin/data/util/IndexedContainer.java b/src/com/vaadin/data/util/IndexedContainer.java
index 3d4a98fb97..048d868ee0 100644
--- a/src/com/vaadin/data/util/IndexedContainer.java
+++ b/src/com/vaadin/data/util/IndexedContainer.java
@@ -576,7 +576,11 @@ public class IndexedContainer implements Container.Indexed,
         // Creates a new id
         final Object id = generateId();
 
-        return addItemAfter(previousItemId, id);
+        if (addItemAfter(previousItemId, id) != null) {
+            return id;
+        } else {
+            return null;
+        }
     }
 
     /*