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]
