Throw exception early on null itemId (#16541)

Change-Id: Iff42a6e9e5e00006c96fa82e102ff145449c6f88

diff --git a/server/src/main/java/com/vaadin/ui/Table.java b/server/src/main/java/com/vaadin/ui/Table.java
index b303fddd81294efcd5b5929483567e027dd00a86..f518f4f161eb1406416e242ffa864ab8b121e620 100644 (file)
--- a/server/src/main/java/com/vaadin/ui/Table.java
+++ b/server/src/main/java/com/vaadin/ui/Table.java
@@ -2221,6 +2221,9 @@ public class Table extends AbstractSelect implements Action.Container,
             List<?> itemIds = getItemIds(firstIndex, rows);
             for (int i = 0; i < rows && i < itemIds.size(); i++) {
                 Object id = itemIds.get(i);
+                if (id == null) {
+                    throw new IllegalStateException("Null itemId returned from container");
+                }
                 // Start by parsing the values, id should already be set
                 parseItemIdToCells(cells, id, i, firstIndex, headmode, cols,
                         colids, firstIndexNotInCache, iscomponent,