Homestyx hydra
Diffusion hydra a7b2327c34ac

Fix an issue with "Auditors:" where an edge edit was used as a PHID list

Description

Fix an issue with "Auditors:" where an edge edit was used as a PHID list

Summary:
See https://discourse.phabricator-community.org/t/runtimeexception-during-import-of-commit/3801. When importing commits with "Auditors:", a raw transaction new value (with an edge edit map using a "+" key) may be passed as an unmentionable PHID list.

Instead, pass an actual PHID list.

Test Plan:

  • Pushed a commit with "Auditors: duck".
  • Ran daemons.
    • Before patch: umentionable PHID exception.
    • After patch: clean commit import.
  • Verified "duck" was added as an auditor.

Differential Revision: https://secure.phabricator.com/D21181

Details

Provenance
epriestleyAuthored on Apr 28 2020, 8:42 AM
sirocylPushed on Oct 16 2024, 5:49 AM
Parents
R1:befeb17f6ff1: Improve the construction of synthetic "comparison/intradiff" changesets
Branches
Unknown
Tags
Unknown

Event Timeline