commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From bugzi...@apache.org
Subject DO NOT REPLY [Bug 30973] - [email] [patch] HTML email with plain text alternative and attachments
Date Thu, 28 Oct 2004 19:25:17 GMT
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG 
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://issues.apache.org/bugzilla/show_bug.cgi?id=30973>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND 
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=30973

[email] [patch] HTML email with plain text alternative and attachments





------- Additional Comments From scott@minutestopost.com  2004-10-28 19:25 -------
This is likely because the patch on this issue moves the initialization of
primaryBodyPart from init() to getPrimaryBodyPart(). Because the patch wasn't
entirely applied (code not removed from init()), this code was duplicated in two
places.

Then, over at bug #31901, Corey noticed the duplicate code and removed it from
getPrimaryBodyPart(), now you are removing it again and primaryBodyPart isn't
initialized anywhere.

I'm all of this is clear as mud :), but I'd like to keep the initialization of
primaryBodyPart in getPrimaryBodyPart() so that it only happens if
primaryBodyPart is actually used. If its helpful, I can wait until all of the
patches are in and submit a new one for MultipartEmail.

---------------------------------------------------------------------
To unsubscribe, e-mail: commons-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: commons-dev-help@jakarta.apache.org


Mime
View raw message