Page MenuHomePhabricator

Fix double-rendering of inline comments in mail
ClosedPublic

Authored by epriestley on Feb 13 2014, 8:17 PM.
Tags
None
Referenced Files
F19892047: D8216.id19554.diff
Fri, Mar 27, 8:12 PM
F19886647: D8216.id19545.diff
Sun, Mar 22, 11:20 AM
F19775598: D8216.diff
Feb 22 2026, 2:20 AM
F19517638: D8216.diff
Jan 15 2026, 1:28 PM
F19506931: D8216.id19545.diff
Jan 9 2026, 5:09 PM
F19506633: D8216.id19554.diff
Jan 9 2026, 4:49 PM
F19506412: D8216.id19546.diff
Jan 9 2026, 4:46 PM
F19147118: D8216.id19554.diff
Dec 10 2025, 4:00 PM
Subscribers

Details

Summary

Ref T2222. This is a tmp.differential-only issue. Inline comment transactions now have content, so we treat them like body text. We also render them separately as inline text. This produces mail where inlines are rendered twice.

Test Plan

Sent myself mail, saw only one copy of inlines.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

epriestley updated this revision to Unknown Object (????).Feb 13 2014, 8:19 PM

(Also fix missing string for "inline" action.)