A small enhancement to exception management. I hope I'm done now!!
Vincenzo > -----Original Message----- > From: Vincenzo Gianferrari Pini > [mailto:[EMAIL PROTECTED] > Sent: gioved� 12 giugno 2003 19.15 > To: James Developers List > Subject: [no virus] RE:[PATCH]HasAttachment was ignoring some > attachments and reporting others inexistent > > > The overall catch now rethrows a MessagingException, to be > consistent with the new matcher/mailet exception handling. > > Vincenzo > > > -----Original Message----- > > From: Vincenzo Gianferrari Pini > > [mailto:[EMAIL PROTECTED] > > Sent: luned� 9 giugno 2003 1.47 > > To: List James-Dev > > Subject: [no virus] [PATCH]HasAttachment was ignoring some attachments > > and reporting others inexistent > > > > > > Noel, > > > > here is another patch: > > > > -------- > > HasAttachment was ignoring some attachments and reporting others > > inexistent: if there was a message with no inline text and an > > attachment, was erroneously not matching; if the message had > > inline text with HTML format and no attachments, was > erroneously matching. > > -------- > > > > It could be further refined: for instance, a pkcs7 inline text > > signed message with no > > attachment was and is reported as having it (in my case the > > public key is an attachment). But at this point, what is an attachment? > > > > I added also an overall catch of any throwable, being consistent > > with a previous patch to AttachmentFileNameIs and with what I > > have been saying in the "Separating internal errors from > > addressing errors in config.xml" postings in this list, but I > > think there should be some consensus on the approach to exception > > handling. > > > > Good night, > > > > Vincenzo > > > > >
HasAttachment.zip
Description: Macintosh archive
--------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
