Norman, you are right in that it was kinda double, so there should be some cleanup. My first attempt would have been to remove the subscription stuff from the MailboxManager (your option #1). The reason is that we always have a manager and its respective mapper. Now we have the MailboxManager with two Mappers. I think we should also merge them which would also simplify the package structure because we wouldn't need the .mail and .user packages anymore.
One more thing concerning naming and stuff: Now the DelegatingMailboxManager is not really delegating anymore. Is there any good reason we should keep it separate from StoreMailboxManager? If not I'd rather have a little bigger class but fewer hierarchy levels. Any thoughts? Tim Am Samstag, den 21.08.2010, 10:23 +0200 schrieb Norman Maurer: > I just committed the changes.. If anyone thinks its a bad idea we can > revert it anyway.. > > https://issues.apache.org/jira/browse/IMAP-205 > > Bye, > Norman > > 2010/8/21 Norman Maurer <nor...@apache.org>: > > Hi there, > > > > after looking again at the IMAP api I'm in favor of removing the > > org.apache.james.imap.store.Subscriper interface and merge the > > implementations with the MailboxManager implementations. Thats because > > the Subsciper interface has 3 methods, all of the methods are already > > in MailboxManager. So the MailboxManager just wraps the Subscriper > > implementation and delegate the call to it. > > > > So there are two solutions to this: > > > > 1) Remove the methods from MailboxManager and move the Subscriper > > interface to the mailbox api > > 2) Remove the Subscriper interface from store api and merge the > > implementations > > > > As I stated before I would prefer 2). > > > > WDYT ? > > > > Bye, > > Norman > > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org > For additional commands, e-mail: server-dev-h...@james.apache.org > --------------------------------------------------------------------- To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org For additional commands, e-mail: server-dev-h...@james.apache.org
