You are viewing a plain text version of this content. The canonical link for it is here.
Posted to server-dev@james.apache.org by no...@apache.org on 2010/06/24 18:59:40 UTC

svn commit: r957627 - /james/imap/trunk/jcr/src/main/java/org/apache/james/imap/jcr/mail/model/JCRMailbox.java

Author: norman
Date: Thu Jun 24 16:59:40 2010
New Revision: 957627

URL: http://svn.apache.org/viewvc?rev=957627&view=rev
Log:
Rename node when mailbox name changed (IMAP-162)

Modified:
    james/imap/trunk/jcr/src/main/java/org/apache/james/imap/jcr/mail/model/JCRMailbox.java

Modified: james/imap/trunk/jcr/src/main/java/org/apache/james/imap/jcr/mail/model/JCRMailbox.java
URL: http://svn.apache.org/viewvc/james/imap/trunk/jcr/src/main/java/org/apache/james/imap/jcr/mail/model/JCRMailbox.java?rev=957627&r1=957626&r2=957627&view=diff
==============================================================================
--- james/imap/trunk/jcr/src/main/java/org/apache/james/imap/jcr/mail/model/JCRMailbox.java (original)
+++ james/imap/trunk/jcr/src/main/java/org/apache/james/imap/jcr/mail/model/JCRMailbox.java Thu Jun 24 16:59:40 2010
@@ -23,6 +23,7 @@ import javax.jcr.RepositoryException;
 
 import org.apache.commons.logging.Log;
 import org.apache.jackrabbit.JcrConstants;
+import org.apache.jackrabbit.util.Text;
 import org.apache.james.imap.jcr.JCRImapConstants;
 import org.apache.james.imap.jcr.Persistent;
 import org.apache.james.imap.store.mail.model.Mailbox;
@@ -140,6 +141,9 @@ public class JCRMailbox implements Mailb
         if (isPersistent()) {
             try {
                 node.setProperty(NAME_PROPERTY, name);
+                // move the node 
+                // See https://issues.apache.org/jira/browse/IMAP-162
+                node.getSession().move(node.getPath(), node.getParent().getPath() + NODE_DELIMITER + Text.escapePath(name));
             } catch (RepositoryException e) {
                 logger.error("Unable to access property " + NAME_PROPERTY, e);
             }
@@ -175,12 +179,6 @@ public class JCRMailbox implements Mailb
         node.setProperty(LASTUID_PROPERTY, getLastUid());   
         
         this.node = node;
-        /*
-        id = 0;
-        lastUid = 0;
-        name = null;
-        uidValidity = 0;
-        */
     }
     
     @Override



---------------------------------------------------------------------
To unsubscribe, e-mail: server-dev-unsubscribe@james.apache.org
For additional commands, e-mail: server-dev-help@james.apache.org