diff options
author | vilo <viliam.durina@gmail.com> | 2015-01-30 12:05:31 +0100 |
---|---|---|
committer | Vaadin Code Review <review@vaadin.com> | 2016-05-20 08:40:15 +0000 |
commit | aa0816ce6e573e91a8bbe5e5df4aca5a800db54c (patch) | |
tree | 3e778918f4d22dc36f73af8c3b5d465839840c6f /server | |
parent | 8858aa624d121629539591127b569cdb33968192 (diff) | |
download | vaadin-framework-aa0816ce6e573e91a8bbe5e5df4aca5a800db54c.tar.gz vaadin-framework-aa0816ce6e573e91a8bbe5e5df4aca5a800db54c.zip |
Throw exception early on null itemId (#16541)
Change-Id: Iff42a6e9e5e00006c96fa82e102ff145449c6f88
Diffstat (limited to 'server')
-rw-r--r-- | server/src/main/java/com/vaadin/ui/Table.java | 3 |
1 files changed, 3 insertions, 0 deletions
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, |