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
F13060639: D17248.diff
Fri, Apr 19, 6:09 PM
Unknown Object (File)
Mon, Apr 8, 12:10 PM
Unknown Object (File)
Sun, Mar 31, 5:07 PM
Unknown Object (File)
Sat, Mar 30, 4:00 AM
Unknown Object (File)
Feb 19 2024, 2:16 PM
Unknown Object (File)
Feb 19 2024, 2:57 AM
Unknown Object (File)
Feb 13 2024, 4:14 PM
Unknown Object (File)
Feb 3 2024, 2:11 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