Page MenuHomePhabricator

Partially support CustomFields in EditEngine
ClosedPublic

Authored by epriestley on Nov 30 2015, 10:25 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Apr 24, 11:58 PM
Unknown Object (File)
Wed, Apr 17, 2:36 PM
Unknown Object (File)
Fri, Apr 12, 5:59 PM
Unknown Object (File)
Thu, Apr 11, 5:32 AM
Unknown Object (File)
Mon, Apr 1, 3:14 PM
Unknown Object (File)
Sat, Mar 30, 12:56 AM
Unknown Object (File)
Fri, Mar 29, 10:08 AM
Unknown Object (File)
Mar 23 2024, 7:40 PM
Subscribers

Details

Summary

Ref T9132. This isn't perfect, but doesn't break any existing functionality. This stuff works:

  • Editing values.
  • Reordering fields.
  • All builtin field tyepes.

This stuff may not work yet:

  • Assigning custom field defaults.
  • Some conduit stuff.
  • Fully custom fields?
  • Locking/hiding fields? Didn't actually test this one.

I'll keep chipping away at that stuff. In some cases, it may be easier to convert all the CustomField apps first, although Differential might be a fair bit of work.

Test Plan

Created a bunch of custom fields of every avialable type and edited them.

Screen Shot 2015-11-30 at 2.21.41 PM.png (1×1 px, 189 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Partially support CustomFields in EditEngine.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
chad edited edge metadata.
This revision is now accepted and ready to land.Dec 1 2015, 12:39 AM
epriestley edited edge metadata.
  • Don't show misleading parameters in parameter prefilling view.
  • Don't show misleading parameters in Conduit.
  • Fix an issue with "Required" fields.
This revision was automatically updated to reflect the committed changes.
vrana added inline comments.
src/infrastructure/customfield/field/PhabricatorCustomField.php
1086

newStandardEditField doesn't accept any parameters.