From aa0816ce6e573e91a8bbe5e5df4aca5a800db54c Mon Sep 17 00:00:00 2001 From: vilo Date: Fri, 30 Jan 2015 12:05:31 +0100 Subject: [PATCH] Throw exception early on null itemId (#16541) Change-Id: Iff42a6e9e5e00006c96fa82e102ff145449c6f88 --- server/src/main/java/com/vaadin/ui/Table.java | 3 +++ 1 file changed, 3 insertions(+) diff --git a/server/src/main/java/com/vaadin/ui/Table.java b/server/src/main/java/com/vaadin/ui/Table.java index b303fddd81..f518f4f161 100644 --- 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, -- 2.39.5