ListArrayType generates an extra UPDATE when persisting an entity vla…

…dmihalcea#180
fafa2008 · Feb 5, 2020 · 878a472 · 878a472
1 parent d96dd81
commit 878a472
Show file tree

Hide file tree

Showing 4 changed files with 72 additions and 0 deletions.
diff --git a/...src/main/java/com/vladmihalcea/hibernate/type/array/internal/ListArrayTypeDescriptor.java b/...src/main/java/com/vladmihalcea/hibernate/type/array/internal/ListArrayTypeDescriptor.java
@@ -75,6 +75,24 @@ public Object wrap(Object value, WrapperOptions options) {
         return list;
     }
 
+    @Override
+    public boolean areEqual(Object one, Object another) {
+        if (one == another) {
+            return true;
+        }
+        if (one == null || another == null) {
+            return false;
+        }
+        if(one instanceof Collection) {
+            one = ((Collection) one).toArray();
+        }
+        if(another instanceof Collection) {
+            another = ((Collection) another).toArray();
+        }
+
+        return ArrayUtil.isEquals(one, another);
+    }
+
     @Override
     public void setParameterValues(Properties parameters) {
         Class entityClass = ReflectionUtils.getClass(parameters.getProperty(DynamicParameterizedType.ENTITY));

diff --git a/...src/main/java/com/vladmihalcea/hibernate/type/array/internal/ListArrayTypeDescriptor.java b/...src/main/java/com/vladmihalcea/hibernate/type/array/internal/ListArrayTypeDescriptor.java
@@ -75,6 +75,24 @@ public Object wrap(Object value, WrapperOptions options) {
         return list;
     }
 
+    @Override
+    public boolean areEqual(Object one, Object another) {
+        if (one == another) {
+            return true;
+        }
+        if (one == null || another == null) {
+            return false;
+        }
+        if(one instanceof Collection) {
+            one = ((Collection) one).toArray();
+        }
+        if(another instanceof Collection) {
+            another = ((Collection) another).toArray();
+        }
+
+        return ArrayUtil.isEquals(one, another);
+    }
+
     @Override
     public void setParameterValues(Properties parameters) {
         Class entityClass = ReflectionUtils.getClass(parameters.getProperty(DynamicParameterizedType.ENTITY));

diff --git a/...src/main/java/com/vladmihalcea/hibernate/type/array/internal/ListArrayTypeDescriptor.java b/...src/main/java/com/vladmihalcea/hibernate/type/array/internal/ListArrayTypeDescriptor.java
@@ -75,6 +75,24 @@ public Object wrap(Object value, WrapperOptions options) {
         return list;
     }
 
+    @Override
+    public boolean areEqual(Object one, Object another) {
+        if (one == another) {
+            return true;
+        }
+        if (one == null || another == null) {
+            return false;
+        }
+        if(one instanceof Collection) {
+            one = ((Collection) one).toArray();
+        }
+        if(another instanceof Collection) {
+            another = ((Collection) another).toArray();
+        }
+
+        return ArrayUtil.isEquals(one, another);
+    }
+
     @Override
     public void setParameterValues(Properties parameters) {
         Class entityClass = ReflectionUtils.getClass(parameters.getProperty(DynamicParameterizedType.ENTITY));

diff --git a/...src/main/java/com/vladmihalcea/hibernate/type/array/internal/ListArrayTypeDescriptor.java b/...src/main/java/com/vladmihalcea/hibernate/type/array/internal/ListArrayTypeDescriptor.java
@@ -75,6 +75,24 @@ public Object wrap(Object value, WrapperOptions options) {
         return list;
     }
 
+    @Override
+    public boolean areEqual(Object one, Object another) {
+        if (one == another) {
+            return true;
+        }
+        if (one == null || another == null) {
+            return false;
+        }
+        if(one instanceof Collection) {
+            one = ((Collection) one).toArray();
+        }
+        if(another instanceof Collection) {
+            another = ((Collection) another).toArray();
+        }
+
+        return ArrayUtil.isEquals(one, another);
+    }
+
     @Override
     public void setParameterValues(Properties parameters) {
         Class entityClass = ReflectionUtils.getClass(parameters.getProperty(DynamicParameterizedType.ENTITY));