- Create a mock.
- Update one of the images.
- Observe that the image file does not become "Attached" to the mock.
On top of this, I got some email from Pholio with odd HTML behavior, but I can't immediately reproduce it and I don't have any guesses about what's causing it so I'll assume I'm just crazy until someone else hits it.
Specifically, I got this (mail with HTML and text parts containing over-escaped HTML):
I also got this (mail with HTML and text parts containing remarkup that was not marked up):
In both cases, the HTML and text parts agree with one another -- so I don't think this is an HTML/text body issue exactly -- the parts are just wrong.
On review, PholioMockEditor->buildMailBody() is doing a lot of very, very deeply suspicious things which are almost certainly causing this, but they're all pre-existing.