Page MenuHomePhabricator

Add basic, rough support for changing field behavior based on object subtype
ClosedPublic

Authored by epriestley on Feb 13 2019, 5:19 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, May 3, 8:02 AM
Unknown Object (File)
Thu, May 2, 8:46 AM
Unknown Object (File)
Wed, May 1, 1:06 AM
Unknown Object (File)
Thu, Apr 25, 2:00 AM
Unknown Object (File)
Fri, Apr 19, 3:01 AM
Unknown Object (File)
Fri, Apr 19, 3:01 AM
Unknown Object (File)
Fri, Apr 19, 3:01 AM
Unknown Object (File)
Thu, Apr 11, 4:12 AM
Subscribers

Details

Summary

Ref T13248. This will probably need quite a bit of refinement, but we can reasonably allow subtype definitions to adjust custom field behavior.

Some places where we use fields are global, and always need to show all the fields. For example, on /maniphest/, where you can search across all tasks, you need to be able to search across all fields that are present on any task.

Likewise, if you "export" a bunch of tasks into a spreadsheet, we need to have columns for every field.

However, when you're clearly in the scope of a particular task (like viewing or editing T123), there's no reason we can't hide fields based on the task subtype.

To start with, allow subtypes to override "disabled" and "name" for custom fields.

Test Plan
  • Defined several custom fields and several subtypes.
  • Disabled/renamed some fields for some subtypes.
  • Viewed/edited tasks of different subtypes, got desired field behavior.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable