From f7abbea6bef3eb46fa4f3580a7da2a2440463266 Mon Sep 17 00:00:00 2001
From: Pekka Hyvönen <pekka@vaadin.com>
Date: Mon, 6 Nov 2017 20:20:58 +0200
Subject: HasItems.setItems(T... items) should allow edits (#10290)

* HasItems.setItems(T... items) should allow edits

Arrays.asList() creates a immutable Arrays.ArrayList, preventing users from doing dataProvider.getItems() and updating the returned collection. This makes it impossible to keep the same data provider, update it and keep the filters & sorting, and then just call dataProvider.refreshAll() to get changes visible. We should not require users to create a new data provider in this case.
This is the same for DataProvider.ofItems(T... items)

* fix missing whitespace from test error message
---
 server/pom.xml | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

(limited to 'server/pom.xml')

diff --git a/server/pom.xml b/server/pom.xml
index 91df364a5e..c38f528e70 100644
--- a/server/pom.xml
+++ b/server/pom.xml
@@ -127,7 +127,12 @@
             <artifactId>slf4j-log4j12</artifactId>
             <scope>test</scope>
         </dependency>
-
+        <dependency>
+            <groupId>org.reflections</groupId>
+            <artifactId>reflections</artifactId>
+            <version>0.9.11</version>
+            <scope>test</scope>
+        </dependency>
         <!-- For manual testing with PostgreSQL (see SQLTestConstants) -->
         <!-- <dependency><groupId>postgresql</groupId><artifactId>postgresql</artifactId><version>9.1-901.jdbc3</version></dependency> -->
     </dependencies>
-- 
cgit v1.2.3