Page MenuHomePhabricator

Fix an issue where inline comments with only edit suggestions are considered empty

Authored by epriestley on May 23 2020, 3:18 PM.



Ref T13513. An inline is not considered empty if it has a suggestion, but some of the shared transaction code doesn't test for this properly.

Update the shared transaction code to be aware that application comments may have more complex emptiness rules.

Test Plan
  • Posted an inline with only an edit suggestion, comment went through.
  • Tried to post a normal empty comment, got an appropriate warning.

Diff Detail

rP Phabricator
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

epriestley created this revision.May 23 2020, 3:18 PM
epriestley requested review of this revision.May 23 2020, 3:19 PM