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 Robert Burrell Donkin <ro...@gmail.com> on 2007/11/09 13:55:27 UTC

[MailboxAPI] MailboxManager.list

in order to properly support LIST and LSUB for the default torque
implementation, i've committed a expression parser
(http://svn.apache.org/repos/asf/james/server/trunk/torque-mailboxmanager-function/src/main/java/org/apache/james/mailboxmanager/torque/MailboxExpression.java).

a couple of questions:

1. would this be useful enough for independent implementation to
justify repackaging out of torque and into mailbox?
2. would it be better to change the API

ListResult[] list(String base, String expression, boolean subscribed)
throws MailboxManagerException;

to accept expression objects capable (similar to MailboxExpression)
rather than strings?

- robert

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


Re: [MailboxAPI] MailboxManager.list

Posted by Stefano Bagnara <ap...@bago.org>.
Robert Burrell Donkin ha scritto:
> in order to properly support LIST and LSUB for the default torque
> implementation, i've committed a expression parser
> (http://svn.apache.org/repos/asf/james/server/trunk/torque-mailboxmanager-function/src/main/java/org/apache/james/mailboxmanager/torque/MailboxExpression.java).
> 
> a couple of questions:
> 
> 1. would this be useful enough for independent implementation to
> justify repackaging out of torque and into mailbox?

+1
If the code is not torque specific it makes sense to move up to mailbox.

> 2. would it be better to change the API
> 
> ListResult[] list(String base, String expression, boolean subscribed)
> throws MailboxManagerException;
> 
> to accept expression objects capable (similar to MailboxExpression)
> rather than strings?

+1

Stefano


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


Re: [MailboxAPI] MailboxManager.list

Posted by Robert Burrell Donkin <ro...@gmail.com>.
On Nov 9, 2007 2:02 PM, Zsombor <gz...@gmail.com> wrote:
>
> On 11/9/07, Robert Burrell Donkin <ro...@gmail.com> wrote:
> >
> > in order to properly support LIST and LSUB for the default torque
> > implementation, i've committed a expression parser
> > (
> > http://svn.apache.org/repos/asf/james/server/trunk/torque-mailboxmanager-function/src/main/java/org/apache/james/mailboxmanager/torque/MailboxExpression.java
> > ).
> >
> > a couple of questions:
> >
> > 1. would this be useful enough for independent implementation to
> > justify repackaging out of torque and into mailbox?
> > 2. would it be better to change the API
> >
> > ListResult[] list(String base, String expression, boolean subscribed)
> > throws MailboxManagerException;
> >
> > to accept expression objects capable (similar to MailboxExpression)
> > rather than strings?
> >
> > - robert
> >
>
> Nice :-)
>
> +1

if you fancy contributing interface and implementation for a
UserMetaDataRepository then 'subscribed' could be removed as well ;-)

- robert

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


Re: [MailboxAPI] MailboxManager.list

Posted by Zsombor <gz...@gmail.com>.
On 11/9/07, Robert Burrell Donkin <ro...@gmail.com> wrote:
>
> in order to properly support LIST and LSUB for the default torque
> implementation, i've committed a expression parser
> (
> http://svn.apache.org/repos/asf/james/server/trunk/torque-mailboxmanager-function/src/main/java/org/apache/james/mailboxmanager/torque/MailboxExpression.java
> ).
>
> a couple of questions:
>
> 1. would this be useful enough for independent implementation to
> justify repackaging out of torque and into mailbox?
> 2. would it be better to change the API
>
> ListResult[] list(String base, String expression, boolean subscribed)
> throws MailboxManagerException;
>
> to accept expression objects capable (similar to MailboxExpression)
> rather than strings?
>
> - robert
>

Nice :-)

+1


Zsombor