Page MenuHomePhabricator

Remove extra container tag on HandleListViews rendering from ModularTransactions in text mode
ClosedPublic

Authored by epriestley on Jan 9 2017, 4:19 PM.
Tags
None
Referenced Files
F15449230: D17162.id41270.diff
Fri, Mar 28, 9:00 AM
F15449108: D17162.id41270.diff
Fri, Mar 28, 8:27 AM
F15445948: D17162.diff
Thu, Mar 27, 4:41 PM
F15441675: D17162.diff
Wed, Mar 26, 7:19 PM
F15441576: D17162.id41273.diff
Wed, Mar 26, 6:51 PM
F15441575: D17162.id41270.diff
Wed, Mar 26, 6:50 PM
F15441574: D17162.id.diff
Wed, Mar 26, 6:50 PM
F15438135: D17162.diff
Tue, Mar 25, 11:35 PM
Subscribers
None

Details

Summary

Fixes T12082. Ref T11114. When modular transaction render a handle list, they use HandleListView, which has a text mode.

However, the HandleListView is a TagView, and currently TagViews always render a tag of some kind. Allow them to return null to decline to render any tag.

Test Plan
  • Added a pile of debugging stuff to ApplicationTransactionEditor to throw during mail generation.
  • Added a reviewer to a revision.
  • Used bin/worker execute --id ... to hit the mail generation repeatedly.
  • Before patch: mail generated with a <span>, even in text mode.
  • After patch: clean mail generation.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable