Page MenuHomePhabricator

Don't let users write summaries or test plans which will become ambiguous in commit messages
ClosedPublic

Authored by epriestley on Nov 12 2016, 3:14 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, May 31, 4:16 AM
Unknown Object (File)
Fri, May 30, 4:32 AM
Unknown Object (File)
Thu, May 29, 12:39 PM
Unknown Object (File)
Thu, May 29, 8:45 AM
Unknown Object (File)
May 8 2025, 10:40 AM
Unknown Object (File)
May 7 2025, 5:14 PM
Unknown Object (File)
Apr 29 2025, 4:35 AM
Unknown Object (File)
Apr 28 2025, 12:51 PM
Subscribers
None

Details

Summary

Ref T11085. To recreate the issue:

  • From the web UI, click "Edit Revision".
  • Write something like this as your "Summary" (i.e., put another field marker, like "Test Plan:", into the summary):

This is a test of the
Test Plan: field to see
if it works.

  • Save changes.

Later, when the summary is amended into a commit message, the parser will see two "Test Plan:" fields and fail to parse the message.

Instead, prevent users from making this edit.

Test Plan

Screen Shot 2016-11-12 at 7.10.43 AM.png (846×2 px, 112 KB)

Diff Detail

Repository
rP Phabricator
Branch
ambig1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 14457
Build 18844: Run Core Tests
Build 18843: arc lint + arc unit