From 39bf9747248aee0d05dc9e9ab5e1ad4cb79ad09b Mon Sep 17 00:00:00 2001
From: Paola De Bartolo <paola.debartolo@gmail.com>
Date: Mon, 30 May 2022 17:46:00 -0300
Subject: [PATCH] feat: upgrade component to Vaadin 23

Close #15
---
 enhanced-grid-flow-demo/pom.xml               |   8 +-
 enhanced-grid-flow/pom.xml                    |   8 +-
 .../enhancedtreegrid/EnhancedTreeGrid.java    | 113 ++++++++++--------
 pom.xml                                       |  11 +-
 4 files changed, 74 insertions(+), 66 deletions(-)

diff --git a/enhanced-grid-flow-demo/pom.xml b/enhanced-grid-flow-demo/pom.xml
index ec53276..97489f2 100644
--- a/enhanced-grid-flow-demo/pom.xml
+++ b/enhanced-grid-flow-demo/pom.xml
@@ -6,7 +6,7 @@
 
     <groupId>com.vaadin.componentfactory</groupId>
     <artifactId>enhanced-grid-flow-demo</artifactId>
-    <version>2.0.2-SNAPSHOT</version>
+    <version>3.0.0</version>
 
     <name>Enhanced Grid Demo</name>
     <packaging>war</packaging>
@@ -18,9 +18,9 @@
     </organization>
 
     <properties>
-        <vaadin.version>22.0.2</vaadin.version>
-        <maven.compiler.source>1.8</maven.compiler.source>
-        <maven.compiler.target>1.8</maven.compiler.target>
+        <vaadin.version>23.0.9</vaadin.version>
+        <maven.compiler.source>11</maven.compiler.source>
+        <maven.compiler.target>11</maven.compiler.target>
         <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
         <project.reporting.outputEncoding>UTF-8</project.reporting.outputEncoding>
         <failOnMissingWebXml>false</failOnMissingWebXml>
diff --git a/enhanced-grid-flow/pom.xml b/enhanced-grid-flow/pom.xml
index 69b7e51..79856b6 100644
--- a/enhanced-grid-flow/pom.xml
+++ b/enhanced-grid-flow/pom.xml
@@ -6,7 +6,7 @@
 
     <groupId>com.vaadin.componentfactory</groupId>
     <artifactId>enhanced-grid-flow</artifactId>
-    <version>2.0.2-SNAPSHOT</version>
+    <version>3.0.0</version>
     <packaging>jar</packaging>
 
     <name>Enhanced Grid</name>
@@ -19,9 +19,9 @@
     </organization>
 
     <properties>
-        <vaadin.version>22.0.2</vaadin.version>
-        <maven.compiler.source>1.8</maven.compiler.source>
-        <maven.compiler.target>1.8</maven.compiler.target>
+        <vaadin.version>23.0.9</vaadin.version>
+        <maven.compiler.source>11</maven.compiler.source>
+        <maven.compiler.target>11</maven.compiler.target>
         <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
         <project.reporting.outputEncoding>UTF-8</project.reporting.outputEncoding>
     </properties>
diff --git a/enhanced-grid-flow/src/main/java/com/vaadin/componentfactory/enhancedtreegrid/EnhancedTreeGrid.java b/enhanced-grid-flow/src/main/java/com/vaadin/componentfactory/enhancedtreegrid/EnhancedTreeGrid.java
index b711588..4bbbc5a 100644
--- a/enhanced-grid-flow/src/main/java/com/vaadin/componentfactory/enhancedtreegrid/EnhancedTreeGrid.java
+++ b/enhanced-grid-flow/src/main/java/com/vaadin/componentfactory/enhancedtreegrid/EnhancedTreeGrid.java
@@ -23,10 +23,13 @@
 import java.util.ArrayList;
 import java.util.Arrays;
 import java.util.Collection;
+import java.util.HashMap;
 import java.util.List;
+import java.util.Map;
 import java.util.NoSuchElementException;
 import java.util.Objects;
 import java.util.Optional;
+import java.util.concurrent.atomic.AtomicLong;
 import java.util.stream.Collectors;
 import java.util.stream.Stream;
 
@@ -40,6 +43,7 @@
 import com.vaadin.flow.component.grid.Filter;
 import com.vaadin.flow.component.grid.Grid;
 import com.vaadin.flow.component.grid.GridArrayUpdater;
+import com.vaadin.flow.component.grid.Grid.Column;
 import com.vaadin.flow.component.grid.GridArrayUpdater.UpdateQueueData;
 import com.vaadin.flow.component.grid.dataview.GridDataView;
 import com.vaadin.flow.component.grid.dataview.GridLazyDataView;
@@ -152,8 +156,13 @@ public UpdateQueueData getUpdateQueueData() {
 		}
 	}
 	
-	private final ValueProvider<T, String> defaultUniqueKeyProvider = item -> String
-	    .valueOf(item.hashCode());
+	private final AtomicLong uniqueKeyCounter = new AtomicLong(0);
+    private final Map<Object, Long> objectUniqueKeyMap = new HashMap<>();
+	
+    ValueProvider<T, String> defaultUniqueKeyProvider = item -> String.valueOf(
+        objectUniqueKeyMap.computeIfAbsent(getDataProvider().getId(item),
+                key -> uniqueKeyCounter.getAndIncrement()));
+
 	
 	private Registration dataProviderRegistration;
 	
@@ -477,57 +486,57 @@ public GridDataView<T> getGenericDataView() {
     }
 
     
-	/**
-	* Adds a new Hierarchy column to this {@link Grid} with a value provider.
-	* The value is converted to String when sent to the client by using
-	* {@link String#valueOf(Object)}.
-	* <p>
-	* Hierarchy column is rendered by using 'vaadin-grid-tree-toggle' web
-	* component.
-	*
-	* @param valueProvider
-	*            the value provider
-	* @return the created hierarchy column
-	*/
-	public EnhancedColumn<T> addHierarchyColumn(ValueProvider<T, ?> valueProvider) {
-		EnhancedColumn<T> column = addColumn(TemplateRenderer
-		        .<T> of("<vaadin-grid-tree-toggle "
-		                + "leaf='[[item.leaf]]' expanded='{{expanded}}' level='[[level]]'>[[item.name]]"
-		                + "</vaadin-grid-tree-toggle>")
-		        .withProperty("leaf",
-		                item -> !getDataCommunicator().hasChildren(item))
-		        .withProperty("name",
-		                value -> String.valueOf(valueProvider.apply(value))));
-		final SerializableComparator<T> comparator = 
-		        (a, b) -> compareMaybeComparables(valueProvider.apply(a),
-		                valueProvider.apply(b));
-		column.setComparator(comparator);
-	
-	return column;
-	}
+    /**
+     * Adds a new Hierarchy column to this {@link Grid} with a value provider.
+     * The value is converted to String when sent to the client by using
+     * {@link String#valueOf(Object)}.
+     * <p>
+     * Hierarchy column is rendered by using 'vaadin-grid-tree-toggle' web
+     * component.
+     *
+     * @param valueProvider
+     *            the value provider
+     * @return the created hierarchy column
+     */
+    public EnhancedColumn<T> addHierarchyColumn(ValueProvider<T, ?> valueProvider) {
+        EnhancedColumn<T> column = addColumn(TemplateRenderer
+                .<T> of("<vaadin-grid-tree-toggle "
+                        + "leaf='[[!item.children]]' expanded='{{expanded}}' level='[[level]]'>[[item.name]]"
+                        + "</vaadin-grid-tree-toggle>")
+                .withProperty("children",
+                        item -> getDataCommunicator().hasChildren(item))
+                .withProperty("name",
+                        value -> String.valueOf(valueProvider.apply(value))));
+        final SerializableComparator<T> comparator = (a,
+                b) -> compareMaybeComparables(valueProvider.apply(a),
+                        valueProvider.apply(b));
+        column.setComparator(comparator);
+
+        return column;
+    }
 	
-	/**
-	* Adds a new Hierarchy column that shows components.
-	* <p>
-	* <em>NOTE:</em> Using {@link ComponentRenderer} is not as efficient as the
-	* built in renderers.
-	* </p>
-	*
-	* @param componentProvider
-	*            a value provider that will return a component for the given
-	*            item
-	* @param <V>
-	*            the component type
-	* @return the new column
-	* @see #addColumn(Renderer)
-	* @see #removeColumn(Column)
-	*/
-	public <V extends Component> EnhancedColumn<T> addComponentHierarchyColumn(
-	    ValueProvider<T, V> componentProvider) {
-		return addColumn(new HierarchyColumnComponentRenderer<V, T>(
-	        componentProvider).withProperty("leaf",
-	                item -> !getDataCommunicator().hasChildren(item)));
-	}
+    /**
+     * Adds a new Hierarchy column that shows components.
+     * <p>
+     * <em>NOTE:</em> Using {@link ComponentRenderer} is not as efficient as the
+     * built in renderers.
+     * </p>
+     *
+     * @param componentProvider
+     *            a value provider that will return a component for the given
+     *            item
+     * @param <V>
+     *            the component type
+     * @return the new column
+     * @see #addColumn(Renderer)
+     * @see #removeColumn(Column)
+     */
+    public <V extends Component> EnhancedColumn<T> addComponentHierarchyColumn(
+            ValueProvider<T, V> componentProvider) {
+        return addColumn(new HierarchyColumnComponentRenderer<V, T>(
+                componentProvider).withProperty("children",
+                        item -> getDataCommunicator().hasChildren(item)));
+    }
 	
 	
 	/**
diff --git a/pom.xml b/pom.xml
index 6f5afa8..0c790d4 100755
--- a/pom.xml
+++ b/pom.xml
@@ -3,7 +3,7 @@
 
     <groupId>com.vaadin.componentfactory</groupId>
     <artifactId>enhanced-grid-flow-root</artifactId>
-    <version>2.0.2-SNAPSHOT</version>
+    <version>3.0.0</version>
     <packaging>pom</packaging>
     <modules>
         <module>enhanced-grid-flow</module>
@@ -13,12 +13,11 @@
     <description>enhanced-grid-flow</description>
 
     <properties>
-        <vaadin.version>22.0.2</vaadin.version>
-        <maven.compiler.source>1.8</maven.compiler.source>
-        <maven.compiler.target>1.8</maven.compiler.target>
+        <vaadin.version>23.0.9</vaadin.version>
+        <maven.compiler.source>11</maven.compiler.source>
+        <maven.compiler.target>11</maven.compiler.target>
         <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
         <project.reporting.outputEncoding>UTF-8</project.reporting.outputEncoding>
-        <enhanced-grid-flow.version>2.0.0</enhanced-grid-flow.version>
     </properties>
     <inceptionYear>2020</inceptionYear>
     <organization>
@@ -31,7 +30,7 @@
             <dependency>
                 <groupId>${project.groupId}</groupId>
                 <artifactId>enhanced-grid-flow</artifactId>
-                <version>${enhanced-grid-flow.version}</version>
+                <version>${project.version}</version>
             </dependency>
             <dependency>
                 <groupId>com.vaadin</groupId>