KNL-65

Patch from Carl Hall
    Emails sent with attachments have a content type that reflects the text type of the email. The type should be multipart/mixed.

Thanks


git-svn-id: https://source.sakaiproject.org/svn/kernel/trunk@53118 66ffb92e-73f9-0310-93c1-f5514f145a0a

kernel/kernel-impl/src/main/java/org/sakaiproject/email/impl/BasicEmailService.java

@@ -435,9 +435,11 @@ else if (contentTypeHeader != null)
 
 			// if we have a full Content-Type header, set it NOW
 			// (after setting the body of the message so that format=flowed is preserved)
-			if (contentTypeHeader != null)
+			// if there attachments, the messsage type will default to multipart/mixed and should
+			// stay that way.
+			if ((attachments == null || attachments.size() == 0) && contentTypeHeader != null)
 			{
-				msg.addHeaderLine(contentTypeHeader);			
+				msg.addHeaderLine(contentTypeHeader);
 				msg.addHeaderLine(EmailHeaders.CONTENT_TRANSFER_ENCODING + ": quoted-printable");
 			}