You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@jackrabbit.apache.org by cl...@apache.org on 2007/05/21 21:06:57 UTC

svn commit: r540242 - /jackrabbit/trunk/contrib/jackrabbit-jcr-mapping/jcr-mapping/src/main/java/org/apache/jackrabbit/ocm/persistence/impl/PersistenceManagerImpl.java

Author: clombart
Date: Mon May 21 12:06:56 2007
New Revision: 540242

URL: http://svn.apache.org/viewvc?view=rev&rev=540242
Log:
Review the implementation of the move method. 
If the workspace.move is used, the source node is not deleted.
If the session.move, it works fine (the source node is deleted). 

I don't know why and start a new discussion on the mailing list. 

Modified:
    jackrabbit/trunk/contrib/jackrabbit-jcr-mapping/jcr-mapping/src/main/java/org/apache/jackrabbit/ocm/persistence/impl/PersistenceManagerImpl.java

Modified: jackrabbit/trunk/contrib/jackrabbit-jcr-mapping/jcr-mapping/src/main/java/org/apache/jackrabbit/ocm/persistence/impl/PersistenceManagerImpl.java
URL: http://svn.apache.org/viewvc/jackrabbit/trunk/contrib/jackrabbit-jcr-mapping/jcr-mapping/src/main/java/org/apache/jackrabbit/ocm/persistence/impl/PersistenceManagerImpl.java?view=diff&rev=540242&r1=540241&r2=540242
==============================================================================
--- jackrabbit/trunk/contrib/jackrabbit-jcr-mapping/jcr-mapping/src/main/java/org/apache/jackrabbit/ocm/persistence/impl/PersistenceManagerImpl.java (original)
+++ jackrabbit/trunk/contrib/jackrabbit-jcr-mapping/jcr-mapping/src/main/java/org/apache/jackrabbit/ocm/persistence/impl/PersistenceManagerImpl.java Mon May 21 12:06:56 2007
@@ -1118,10 +1118,10 @@
 	 * @see org.apache.jackrabbit.ocm.persistence.PersistenceManager#move(java.lang.String, java.lang.String)
 	 */
     public void move(String srcPath, String destPath){
-        Workspace workspace = session.getWorkspace();
+        //Workspace workspace = session.getWorkspace();
         try {
             
-        	workspace.move(srcPath,destPath);
+        	session.move(srcPath,destPath);
             
         }catch(javax.jcr.nodetype.ConstraintViolationException cve){
             throw new PersistenceException(