svn commit: r988001 - in /james/server/trunk: imapserver/src/main/java/org/apache/james/imapserver/ pop3server/src/test/java/org/apache/james/pop3server/ spring-deployment/src/main/config/james/

2010-08-22 Thread norman
Author: norman Date: Mon Aug 23 06:42:37 2010 New Revision: 988001 URL: http://svn.apache.org/viewvc?rev=988001&view=rev Log: Adjust code and xml to match current imap trunk Modified: james/server/trunk/imapserver/src/main/java/org/apache/james/imapserver/UserMetaDataRepositorySubscripter.ja

[jira] Updated: (JDKIM-18) jdkim verify(InputStream) doesn't clean up temp files until shutdown

2010-08-22 Thread Stefano Bagnara (JIRA)
[ https://issues.apache.org/jira/browse/JDKIM-18?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Stefano Bagnara updated JDKIM-18: - Fix Version/s: 0.2 > jdkim verify(InputStream) doesn't clean up temp files until shutdown > ---

[jira] Updated: (JDKIM-19) Change scope on 2 variables in DKIMSign to support subclassing

2010-08-22 Thread Stefano Bagnara (JIRA)
[ https://issues.apache.org/jira/browse/JDKIM-19?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Stefano Bagnara updated JDKIM-19: - Fix Version/s: 0.2 Affects Version/s: 0.2 > Change scope on 2 variables in DKIMSign to supp

[jira] Assigned: (JDKIM-19) Change scope on 2 variables in DKIMSign to support subclassing

2010-08-22 Thread Stefano Bagnara (JIRA)
[ https://issues.apache.org/jira/browse/JDKIM-19?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Stefano Bagnara reassigned JDKIM-19: Assignee: Stefano Bagnara > Change scope on 2 variables in DKIMSign to support subclassing >

Re: [IMAP] Remove org.apache.james.imap.store.Subscriper

2010-08-22 Thread Norman Maurer
Comments inline.. 2010/8/22 Eric Charles : >  one more inline :) > > On 22/08/2010 13:58, Norman Maurer wrote: >> >> Not 100%.. Subscriptions links users to Mailboxes. The users are just >> Strings and nothing more. So its not coupled to james server in any >> way. >> > Yeah, these are "soft links

[jira] Resolved: (IMAP-153) JCR Storage structure uses files as imap folders hiding the structure and exposing the underlying storage to users (potentially)

2010-08-22 Thread Norman Maurer (JIRA)
[ https://issues.apache.org/jira/browse/IMAP-153?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Norman Maurer resolved IMAP-153. Resolution: Won't Fix We dedicited to keep the current node structure > JCR Storage structure uses f

Re: [IMAP] Remove org.apache.james.imap.store.Subscriper

2010-08-22 Thread Eric Charles
one more inline :) On 22/08/2010 13:58, Norman Maurer wrote: Not 100%.. Subscriptions links users to Mailboxes. The users are just Strings and nothing more. So its not coupled to james server in any way. Yeah, these are "soft links". A "Subscription store" could be different from the "Mailbox

[jira] Resolved: (IMAP-188) renaming INBOX should move messages

2010-08-22 Thread Norman Maurer (JIRA)
[ https://issues.apache.org/jira/browse/IMAP-188?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Norman Maurer resolved IMAP-188. Fix Version/s: 0.1 Resolution: Fixed renaming INBOX now move the messages. I implemented it as

svn commit: r987895 - /james/imap/trunk/store/src/main/java/org/apache/james/imap/store/StoreMailboxManager.java

2010-08-22 Thread norman
Author: norman Date: Sun Aug 22 12:59:05 2010 New Revision: 987895 URL: http://svn.apache.org/viewvc?rev=987895&view=rev Log: renaming INBOX should move messages to new Mailbox (IMAP-188) Modified: james/imap/trunk/store/src/main/java/org/apache/james/imap/store/StoreMailboxManager.java Mod

[jira] Assigned: (IMAP-188) renaming INBOX should move messages

2010-08-22 Thread Norman Maurer (JIRA)
[ https://issues.apache.org/jira/browse/IMAP-188?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Norman Maurer reassigned IMAP-188: -- Assignee: Norman Maurer > renaming INBOX should move messages > -

[jira] Closed: (IMAP-108) Make it configurable to allow usage of Container managed Transaction

2010-08-22 Thread Norman Maurer (JIRA)
[ https://issues.apache.org/jira/browse/IMAP-108?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Norman Maurer closed IMAP-108. -- Resolution: Won't Fix No need > Make it configurable to allow usage of Container managed Transaction > -

svn commit: r987884 - in /james/imap/trunk: jcr/src/main/java/org/apache/james/imap/jcr/ jpa/src/main/java/org/apache/james/imap/jpa/ maildir/src/main/java/org/apache/james/imap/maildir/mail/ maildir/

2010-08-22 Thread norman
Author: norman Date: Sun Aug 22 12:13:40 2010 New Revision: 987884 URL: http://svn.apache.org/viewvc?rev=987884&view=rev Log: Straighten Non-/TransactionalMapper Hierarchy (IMAP-206). Thx to Tim-Christian Mundt for the patch :) Added: james/imap/trunk/store/src/main/java/org/apache/james/i

[jira] Resolved: (IMAP-206) Straighten Non-/TransactionalMapper Hierarchy

2010-08-22 Thread Norman Maurer (JIRA)
[ https://issues.apache.org/jira/browse/IMAP-206?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Norman Maurer resolved IMAP-206. Assignee: Norman Maurer Fix Version/s: 0.1 Resolution: Fixed merged in.. The patch wa

Re: [IMAP] Remove org.apache.james.imap.store.Subscriper

2010-08-22 Thread Norman Maurer
Hi, comments inline again ;) 2010/8/22 Eric Charles : >  See comments after. > Eric > > On 22/08/2010 10:12, Norman Maurer wrote: >>> >>> Which means, not only merge the SubscriptionManager into the >>> MailboxManager like you did, but also the SubscriptionMapper into the >>> MailboxMapper. That

Re: [IMAP] Remove org.apache.james.imap.store.Subscriper

2010-08-22 Thread Eric Charles
See comments after. Eric On 22/08/2010 10:12, Norman Maurer wrote: Which means, not only merge the SubscriptionManager into the MailboxManager like you did, but also the SubscriptionMapper into the MailboxMapper. That would make the separation between .user and .mail superfluous. Does that mak

[jira] Commented: (IMAP-206) Straighten Non-/TransactionalMapper Hierarchy

2010-08-22 Thread Eric Charles (JIRA)
[ https://issues.apache.org/jira/browse/IMAP-206?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12901143#action_12901143 ] Eric Charles commented on IMAP-206: --- I've svn up --revision 987444 and tried to apply the p

Re: [IMAP] Remove org.apache.james.imap.store.Subscriper

2010-08-22 Thread Norman Maurer
Hi Eric.. 2010/8/22 Eric Charles : >  Hi, > > So Tim, you was also ok with optin 2 and finally Norman reverted to option 1 > ? (not sure to be in line :) ?) > Seems so.. > I've looked at revision 987821: > - SubscriptionManager is not used anymore in MailboxManager hirerachy > (StoreMailboxMana

Re: [IMAP] Remove org.apache.james.imap.store.Subscriper

2010-08-22 Thread Norman Maurer
Hi Tim, 2010/8/21 Tim-Christian Mundt : > Norman, > > I have not yet reviewed your recent commit. However, I'd like to remark > that I was in favor of your last attempt. I just wanted to > _additionally_: ok :) > >> >>> I think we should also merge them which >> >>> would also simplify the pack

[jira] Created: (IMAP-207) Rename Document to Message and Mailbox to MessageManager

2010-08-22 Thread Eric Charles (JIRA)
Rename Document to Message and Mailbox to MessageManager Key: IMAP-207 URL: https://issues.apache.org/jira/browse/IMAP-207 Project: JAMES Imap Issue Type: Task Reporter: Er

Re: [IMAP] Remove org.apache.james.imap.store.Subscriper

2010-08-22 Thread Eric Charles
Back again. In DefaultProcessorChain, we know have createDefaultChain(chainEndProcessor, mailboxManager, subscriptionManager, statusResponseFactory) so mailbox and subscription can be seen on the same level, each playing role in the different processors. There are now 3 options: 1.- composit