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/12/21 13:46:16 UTC

svn commit: r1051476 - in /james/mailbox/trunk/jcr/src: main/java/org/apache/james/mailbox/jcr/mail/JCRMailboxMapper.java test/java/org/apache/james/mailbox/jcr/JCRMailboxManagerTest.java

Author: norman
Date: Tue Dec 21 12:46:16 2010
New Revision: 1051476

URL: http://svn.apache.org/viewvc?rev=1051476&view=rev
Log:
Fix JCRMailboxMapper.list()

Modified:
    james/mailbox/trunk/jcr/src/main/java/org/apache/james/mailbox/jcr/mail/JCRMailboxMapper.java
    james/mailbox/trunk/jcr/src/test/java/org/apache/james/mailbox/jcr/JCRMailboxManagerTest.java

Modified: james/mailbox/trunk/jcr/src/main/java/org/apache/james/mailbox/jcr/mail/JCRMailboxMapper.java
URL: http://svn.apache.org/viewvc/james/mailbox/trunk/jcr/src/main/java/org/apache/james/mailbox/jcr/mail/JCRMailboxMapper.java?rev=1051476&r1=1051475&r2=1051476&view=diff
==============================================================================
--- james/mailbox/trunk/jcr/src/main/java/org/apache/james/mailbox/jcr/mail/JCRMailboxMapper.java (original)
+++ james/mailbox/trunk/jcr/src/main/java/org/apache/james/mailbox/jcr/mail/JCRMailboxMapper.java Tue Dec 21 12:46:16 2010
@@ -226,7 +226,7 @@ public class JCRMailboxMapper extends Ab
             String queryString = "/jcr:root/" + MAILBOXES_PATH + "//element(*,jamesMailbox:mailbox)";
             QueryResult result = manager.createQuery(queryString, Query.XPATH).execute();
             NodeIterator it = result.getNodes();
-            if (it.hasNext()) {
+            while (it.hasNext()) {
                 mList.add(new JCRMailbox(it.nextNode(), getLogger()));
             }
             return mList;

Modified: james/mailbox/trunk/jcr/src/test/java/org/apache/james/mailbox/jcr/JCRMailboxManagerTest.java
URL: http://svn.apache.org/viewvc/james/mailbox/trunk/jcr/src/test/java/org/apache/james/mailbox/jcr/JCRMailboxManagerTest.java?rev=1051476&r1=1051475&r2=1051476&view=diff
==============================================================================
--- james/mailbox/trunk/jcr/src/test/java/org/apache/james/mailbox/jcr/JCRMailboxManagerTest.java (original)
+++ james/mailbox/trunk/jcr/src/test/java/org/apache/james/mailbox/jcr/JCRMailboxManagerTest.java Tue Dec 21 12:46:16 2010
@@ -71,15 +71,7 @@ public class JCRMailboxManagerTest exten
         setMailboxManager(new JCRMailboxManager(mf, null, uidProvider));
 
     }
-    
-    /* TODO list() does not seem to work for JCR atm
-     * 
-     * @see org.apache.james.mailbox.MailboxManagerTest#testList()
-     */
-    @Test
-    public void testList() throws MailboxException, UnsupportedEncodingException {
-        
-    }
+
     
     /**
      * @throws BadCredentialsException



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