james-server-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Norman Maurer (JIRA)" <server-...@james.apache.org>
Subject [jira] Commented: (IMAP-151) Replace MailboxMapper.existsMailboxStartingWith() with hasChildren()
Date Mon, 07 Jun 2010 16:13:38 GMT

    [ https://issues.apache.org/jira/browse/IMAP-151?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12876295#action_12876295

Norman Maurer commented on IMAP-151:

Hi Tim,

using Node.hasNodes() will not work with the current jcr layout. I'm not sure we should adjust
it. The problem is this:

If you have two mailboxes with the names:
* INBOX.test

If you now remove INBOX you need to be sure that INBOX.test still exists, thats an IMAP RFC

So I dedicited to just create a folder with the fullname of the Mailbox. But I agree from
the idea of having Folders it would more sence to put them in Hierachical order..

> Replace MailboxMapper.existsMailboxStartingWith() with hasChildren()
> --------------------------------------------------------------------
>                 Key: IMAP-151
>                 URL: https://issues.apache.org/jira/browse/IMAP-151
>             Project: JAMES Imap
>          Issue Type: Improvement
>            Reporter: Tim-Christian Mundt
>         Attachments: IMAP-151.patch
> existsMailboxStartingWith() is only used to find out whether a mailbox has children.
For some implementations it's more efficient to use another mechanism (e.g. JCR nodes have
a hasNodes() method). Thus existsMailboxStartingWith() should be removed, hasChildren() should
be added and the implementations should make use of that.

This message is automatically generated by JIRA.
You can reply to this email to add a comment to the issue online.

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

View raw message