Author: steve.ebersole@(protected)
Date: 2008-07-01 17:39:28 -0400 (Tue, 01 Jul 2008)
New Revision: 14839

Modified:
 core/trunk/core/src/main/java/org/hibernate/type/CollectionType.java
 core/trunk/testsuite/src/test/java/org/hibernate/test/ops/MergeTest.java
Log:
HHH-3294 : persistent collection dirtiness on merge

Modified: core/trunk/core/src/main/java/org/hibernate/type/CollectionType.java
===================================================================
--- core/trunk/core/src/main/java/org/hibernate/type/CollectionType.java  2008-07-01 16:55:24 UTC (rev 14838)
+++ core/trunk/core/src/main/java/org/hibernate/type/CollectionType.java  2008-07-01 21:39:28 UTC (rev 14839)
@@(protected) @@
   //the array length might not match
   result = replaceElements( original, result, owner, copyCache, session );

-    if (original==target) {
-      //get the elements back into the target
+    if ( original == target ) {
+      // get the elements back into the target making sure to handle dirty flag
+      boolean wasClean = PersistentCollection.class.isInstance( target ) && !( ( PersistentCollection ) target ).isDirty();
     //TODO: this is a little inefficient, don't need to do a whole
     //    deep replaceElements() call
     replaceElements( result, target, owner, copyCache, session );
+      if ( wasClean ) {
+        ( ( PersistentCollection ) target ).clearDirty();
+      }
     result = target;
   }


Modified: core/trunk/testsuite/src/test/java/org/hibernate/test/ops/MergeTest.java
===================================================================
--- core/trunk/testsuite/src/test/java/org/hibernate/test/ops/MergeTest.java  2008-07-01 16:55:24 UTC (rev 14838)
+++ core/trunk/testsuite/src/test/java/org/hibernate/test/ops/MergeTest.java  2008-07-01 21:39:28 UTC (rev 14839)
@@(protected) @@
   cleanup();
  }

+  public void testNoExtraUpdatesOnPersistentMergeVersionedWithCollection() throws Exception {
+    Session s = openSession();
+    s.beginTransaction();
+    VersionedEntity parent = new VersionedEntity( "parent", "parent" );
+    VersionedEntity child = new VersionedEntity( "child", "child" );
+    parent.getChildren().add( child );
+    child.setParent( parent );
+    s.persist( parent );
+    s.getTransaction().commit();
+    s.close();
+
+    clearCounts();
+
+    // parent is now detached, but we have made no changes. so attempt to merge it
+    // into this new session; this should cause no updates...
+    s = openSession();
+    s.beginTransaction();
+    // load parent so that merge will follow entityIsPersistent path
+    VersionedEntity persistentParent = ( VersionedEntity ) s.get( VersionedEntity.class, parent.getId() );
+    // load children
+    VersionedEntity persistentChild = ( VersionedEntity ) persistentParent.getChildren().iterator().next();
+    VersionedEntity mergedParent = ( VersionedEntity ) s.merge( persistentParent ); // <-- This merge leads to failure
+    s.getTransaction().commit();
+    s.close();
+
+    assertUpdateCount( 0 );
+    assertInsertCount( 0 );
+    assertEquals( "unexpected parent version increment", parent.getVersion(), mergedParent.getVersion() );
+    VersionedEntity mergedChild = ( VersionedEntity ) mergedParent.getChildren().iterator().next();
+    assertEquals( "unexpected child version increment", child.getVersion(), mergedChild.getVersion() );
+
+    ///////////////////////////////////////////////////////////////////////
+    // as a control measure, now update the node once it is loaded and
+    // make sure we get an update as a result...
+    s = openSession();
+    s.beginTransaction();
+    persistentParent = ( VersionedEntity ) s.get( VersionedEntity.class, parent.getId() );
+    persistentParent.setName( "new name" );
+    persistentParent.getChildren().add( new VersionedEntity( "child2", "new child" ) );
+    persistentParent = ( VersionedEntity ) s.merge( persistentParent );
+    s.getTransaction().commit();
+    s.close();
+    assertUpdateCount( 1 );
+    assertInsertCount( 1 );
+    ///////////////////////////////////////////////////////////////////////
+
+    // cleanup();
+  }
+
 public void testPersistThenMergeInSameTxnWithVersion() {
   Session s = openSession();
   Transaction tx = s.beginTransaction();

_______________________________________________
hibernate-commits mailing list
hibernate-commits@(protected)
https://lists.jboss.org/mailman/listinfo/hibernate-commits