You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@openjpa.apache.org by aw...@apache.org on 2006/08/03 23:48:38 UTC

svn commit: r428538 - /incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/DetachedStateAttachStrategy.java

Author: awhite
Date: Thu Aug  3 14:48:37 2006
New Revision: 428538

URL: http://svn.apache.org/viewvc?rev=428538&view=rev
Log:
Remove incorrect lifecycle callback invocation.


Modified:
    incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/DetachedStateAttachStrategy.java

Modified: incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/DetachedStateAttachStrategy.java
URL: http://svn.apache.org/viewvc/incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/DetachedStateAttachStrategy.java?rev=428538&r1=428537&r2=428538&view=diff
==============================================================================
--- incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/DetachedStateAttachStrategy.java (original)
+++ incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/DetachedStateAttachStrategy.java Thu Aug  3 14:48:37 2006
@@ -128,10 +128,6 @@
         } else
             sm = manager.assertManaged(into);
 
-        // invoke any preAttach on the detached instance
-        broker.fireLifecycleEvent(pc, null, meta,
-            LifecycleEvent.BEFORE_DETACH);
-
         // mark that we attached the instance *before* we
         // fill in values to avoid endless recursion
         manager.setAttachedCopy(pc, into);