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)
Wed, Dec 18, 6:49 AM
Unknown Object (File)
Fri, Dec 13, 11:56 AM
Unknown Object (File)
Tue, Dec 10, 6:43 AM
Unknown Object (File)
Fri, Nov 29, 2:08 AM
Unknown Object (File)
Tue, Nov 26, 9:07 PM
Unknown Object (File)
Thu, Nov 21, 5:31 PM
Unknown Object (File)
Nov 18 2024, 3:21 AM
Unknown Object (File)
Nov 13 2024, 9:33 PM
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