Page MenuHomePhabricator

Improve display behavior for write locks held by omnipotent users
ClosedPublic

Authored by epriestley on Jun 1 2021, 1:13 PM.
Tags
None
Referenced Files
F18993634: D21669.id51568.diff
Wed, Nov 19, 8:13 AM
F18806370: D21669.id51572.diff
Oct 18 2025, 7:10 PM
F18802691: D21669.id.diff
Oct 17 2025, 7:58 PM
F18793565: D21669.diff
Oct 16 2025, 9:38 PM
F18734422: D21669.id.diff
Sep 30 2025, 10:53 PM
F18704021: D21669.id.diff
Sep 28 2025, 5:58 AM
F18669081: D21669.diff
Sep 24 2025, 6:24 PM
F18669063: D21669.diff
Sep 24 2025, 6:20 PM
Subscribers
None

Details

Summary

Ref T13614. When an omnipotent user calls "synchronizeWorkingCopyBeforeWrite()", we record a WorkingCopyVersion record with a null "userPHID". The UI then renders this as "Unknown Object (????)".

Improve this behavior:

  • When no PHID is available, just render nothing in the UI (this doesn't seem meaningfully different from no version existing at all).
  • Allow callers to provide an acting user PHID, similar to Editor.

There's currently no way to perform this kind of write legitimately in the upstream, but T13614 is providing one.

Test Plan
  • Wrote a script that calls "synchronizeWorkingCopyBeforeWrite()" as the omnipotent user.
  • Ran script, saw "Unknown Object (????)" in the UI.
  • Applied UI fix, saw empty UI.
  • Applied "acting as" fix, modified script to act as the Diffusion application, ran script, saw "Diffusion" attribution in UI.

Screen Shot 2021-06-01 at 6.12.01 AM.png (131×374 px, 8 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

This revision was not accepted when it landed; it landed in state Needs Review.Jun 1 2021, 3:30 PM
This revision was automatically updated to reflect the committed changes.