Page MenuHomePhabricator

Fix two issues with embedding other fields inside "Summary" or "Test Plan" in Differential with the web UI
ClosedPublic

Authored by epriestley on Jan 25 2017, 7:25 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Jul 6, 5:41 PM
Unknown Object (File)
Sat, Jun 28, 9:13 PM
Unknown Object (File)
May 30 2025, 1:12 PM
Unknown Object (File)
May 29 2025, 8:59 AM
Unknown Object (File)
May 28 2025, 3:52 PM
Unknown Object (File)
May 27 2025, 9:00 PM
Unknown Object (File)
Apr 28 2025, 9:46 PM
Unknown Object (File)
Apr 27 2025, 3:05 PM
Subscribers
None

Details

Summary

Ref T11114. Converting to EditEngine caused us to stop running this validation, since these fields no longer subclass this parent. Restore the validation.

Also, make sure we check the first line of the value, too. After the change to make "Tests: xyz" a valid title, you could write silly summaries / test plans and escape the check if the first line was bogus.

Test Plan

Screen Shot 2017-01-25 at 11.22.29 AM.png (1×1 px, 174 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable