enabled then no details of attachments are printed out. I added a patch for some additional local debugging that will print out the contents of "text" attachments and just print "BINARY" (for now) for binary contents. I don't think there's much need to actually get at the binary content, but it's nice to know that there are attachments that are coming through.
Alexandre Ballesté Crevillén February 14, 2017 at 6:48 AM
Perfect, so then it passed the QA.
Matthew Jones February 14, 2017 at 6:30 AM
@Alexandre Ballesté Crevillén Hmm, not sure how announcements adds attachments. I feel like an announcement doesn't contain the attachment in the body of the message normally and you actually have to click the announcement to go into the site to read it. So sounds like it's correct.
Alexandre Ballesté Crevillén February 14, 2017 at 5:43 AM
Tested on Sakai - 11.3-RC01 2017/02/10 https://qa1-us.nightly.sakaiproject.org/ I could see the BINARY when sending an email using the "Email tool" I couldn't see it using the "Announcements tool" Is it the correct behaviour?
Neal Caidin December 13, 2016 at 2:18 PM
Passed qa.
I tested with a binary file using the email tool.
I know I could have tested on nightly, but as a good exercise for me, I tested on a local Sakai. Build info: Sakai - 1c2c192 - Sakai 12-SNAPSHOT
When you have
testMode@org.sakaiproject.email.api.EmailService=true
enabled then no details of attachments are printed out. I added a patch for some additional local debugging that will print out the contents of "text" attachments and just print "BINARY" (for now) for binary contents. I don't think there's much need to actually get at the binary content, but it's nice to know that there are attachments that are coming through.