Page MenuHomePhabricator

Fix a bug with setting custom PHID list field values via Conduit and prepare for bulk edits
ClosedPublic

Authored by epriestley on Jan 19 2018, 2:23 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Apr 11, 10:50 AM
Unknown Object (File)
Sat, Apr 6, 12:19 AM
Unknown Object (File)
Sat, Mar 30, 12:36 AM
Unknown Object (File)
Mar 8 2024, 12:02 AM
Unknown Object (File)
Feb 3 2024, 5:34 PM
Unknown Object (File)
Jan 25 2024, 1:31 AM
Unknown Object (File)
Dec 26 2023, 8:50 PM
Unknown Object (File)
Dec 24 2023, 2:26 AM
Subscribers
None

Details

Summary

Ref T13025. Custom field transactions work somewhat unusually: the values sometimes need to be encoded. We currently do not apply this encoding correctly via Conduit.

For example, setting some custom PHID field to ["PHID-X-Y"] fails with a bunch of JSON errors.

Add an extra hook callback so that EditTypes can apply processing to transaction values, then apply the correct CustomField processing.

This only affects Conduit. In a future diff, this also allows bulk edit of custom fields to work correctly.

Test Plan

Added a custom field to Maniphest with a list of projects. Used Conduit to bulk edit it (which now works, but did not before). Used the web UI to bulk edit it.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable