Page MenuHomePhabricator

Recognize remarkup custom fields as remarkup blocks
ClosedPublic

Authored by epriestley on Aug 21 2014, 11:40 PM.
Tags
None
Referenced Files
F15461356: D10335.id24880.diff
Tue, Apr 1, 7:45 AM
F15455819: D10335.id24880.diff
Sun, Mar 30, 5:02 AM
F15455817: D10335.id24881.diff
Sun, Mar 30, 5:02 AM
F15452489: D10335.id24881.diff
Sat, Mar 29, 5:15 AM
F15448853: D10335.diff
Fri, Mar 28, 7:11 AM
F15446977: D10335.id24881.diff
Thu, Mar 27, 9:11 PM
F15412919: D10335.id24880.diff
Wed, Mar 19, 3:46 PM
F15382819: D10335.id24880.diff
Fri, Mar 14, 2:35 PM
Subscribers

Details

Summary

Fixes T5916. Key insight here is that the screenshot shows a custom "Detail Solution / Notes" field, which is why this mojo doesn't work: custom remarkup fields don't emit their content for mention/file extraction.

Also fix a bug where multiple blocks with file PHIDs could be merged improperly, discarding some file PHIDs.

Test Plan

Added a custom remarkup field, added files to it, saw them attach to the task when changes were saved.

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

epriestley retitled this revision from to Recognize remarkup custom fields as remarkup blocks.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
btrahan edited edge metadata.
This revision is now accepted and ready to land.Aug 21 2014, 11:45 PM
epriestley updated this revision to Diff 24881.

Closed by commit rP34a90a0d2b4d (authored by @epriestley).