From 34f26e6711bba490f656000d1042e7ddd91b8f9e Mon Sep 17 00:00:00 2001
From: Henri Sara <henri.sara@itmill.com>
Date: Thu, 11 Mar 2010 15:29:42 +0000
Subject: [PATCH] #4303 BeanItemContainer.addItem(null, Object) did not respect
 contract

svn changeset:11794/svn branch:6.3
---
 src/com/vaadin/data/util/BeanItemContainer.java | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/src/com/vaadin/data/util/BeanItemContainer.java b/src/com/vaadin/data/util/BeanItemContainer.java
index f0d806204f..3e3c4a5538 100644
--- a/src/com/vaadin/data/util/BeanItemContainer.java
+++ b/src/com/vaadin/data/util/BeanItemContainer.java
@@ -258,7 +258,9 @@ public class BeanItemContainer<BT> implements Indexed, Sortable, Filterable,
     public BeanItem<BT> addItemAfter(Object previousItemId, Object newItemId)
             throws UnsupportedOperationException {
         // only add if the previous item is visible
-        if (containsId(previousItemId)) {
+        if (previousItemId == null) {
+            return addItemAtInternalIndex(0, newItemId);
+        } else if (containsId(previousItemId)) {
             return addItemAtInternalIndex(allItems.indexOf(previousItemId) + 1,
                     newItemId);
         } else {
-- 
2.39.5