I think I know why my implementation is not working. I am returning Message Sequence Numbers from the Mailbox.expunge() method, Which used to be what you were supposed to do in previous IMAP implementation. I will change to return UIDs and that should hopefully fix it.
/Martin -----Original Message----- From: Robert Burrell Donkin [mailto:[email protected]] Sent: 13 May 2009 16:26 To: James Developers List Subject: Re: EXPUNGE problem On Wed, May 13, 2009 at 4:06 PM, Martin.Bergljung <[email protected]> wrote: > Hi, > > > > The EXPUNGE does not work correctly as the MailboxEventAnalyser contains > a Set of expunged UIDs. MailboxEventAnalyser deals with UIDs. MSNs are converted in UidToMsnConverter since each client connection needs it's own perspective. (a complete PITA if you ask me, but it's hard to see any other way to do as the specification requires). since each UID is unique (except for UIDVALIDITY changes) a set should be ok, i think > It needs to be a List (of message sequence numbers in the end) as you > can have expunged 2 messages after each other such as: > > > > * 2 EXPUNGE (message 2 had the \Deleted flag set) > * 2 EXPUNGE (message 3 had the \Deleted flag set) > * 6 EXPUNGE (message 8 had the \Deleted flag set) there used to be a bug in AbstractMailboxProcessor#addExpungedResponses which did something similar but this should be fixed - robert --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
