Page MenuHomePhabricator

Pholio's "replace" transaction does not attach files properly after refactoring
Closed, ResolvedPublic

Description

Steps:

  • Create a mock.
  • Update one of the images.
  • Observe that the image file does not become "Attached" to the mock.

Event Timeline

I've got most of a diff for this.

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.

epriestley closed this task as Resolved.Jun 2 2017, 11:17 AM

This appears to have been resolved by D18042. I'm letting the mail stuff fall by the wayside for now.