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)
Thu, Nov 21, 3:06 PM
Unknown Object (File)
Thu, Nov 21, 8:25 AM
Unknown Object (File)
Mon, Nov 11, 5:09 AM
Unknown Object (File)
Sun, Nov 10, 8:13 AM
Unknown Object (File)
Tue, Nov 5, 4:09 AM
Unknown Object (File)
Sun, Nov 3, 5:18 PM
Unknown Object (File)
Sun, Nov 3, 6:10 AM
Unknown Object (File)
Wed, Oct 30, 8:33 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
Branch
subfield2
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 21984
Build 30027: Run Core Tests
Build 30026: arc lint + arc unit