Page MenuHomePhabricator

Fix an issue when undoing mutiple inline comment deletions
ClosedPublic

Authored by epriestley on Mar 25 2021, 9:03 PM.
Tags
None
Referenced Files
F13081006: D21650.id51519.diff
Wed, Apr 24, 12:49 PM
Unknown Object (File)
Tue, Apr 23, 11:23 AM
Unknown Object (File)
Sun, Apr 21, 3:13 PM
Unknown Object (File)
Sat, Apr 20, 5:43 PM
Unknown Object (File)
Fri, Apr 19, 9:05 PM
Unknown Object (File)
Wed, Apr 17, 1:51 PM
Unknown Object (File)
Wed, Apr 10, 8:03 PM
Unknown Object (File)
Mon, Apr 8, 7:52 AM
Subscribers
None

Details

Summary

Ref T13559. If you create comments A and B, then delete comments A and B, then undo the deletion of A, the UI undoes the deletion of B instead.

This is becasue the undo rows are shipped down with a static scalar metadata reference. When copied multiple times to create multiple undo rows, they reference the same data object.

Preventing this in the general case is a problem with greater scope. For now, just avoid rendering these rows with any metadata so they don't alias a single data object.

Test Plan
  • Created comments A, B.
  • Deleted comments A, B.
  • Clicked "Undo" on A.
    • Before: Deletion of "B" undone.
    • After: Deletion of "A" undone.

Diff Detail

Repository
rP Phabricator
Branch
istate7
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 25327
Build 34969: Run Core Tests
Build 34968: arc lint + arc unit