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, Dec 12, 2:39 PM
Unknown Object (File)
Thu, Nov 28, 1:04 PM
Unknown Object (File)
Sun, Nov 24, 6:48 AM
Unknown Object (File)
Sat, Nov 23, 10:25 PM
Unknown Object (File)
Nov 21 2024, 3:06 PM
Unknown Object (File)
Nov 21 2024, 8:25 AM
Unknown Object (File)
Nov 11 2024, 5:09 AM
Unknown Object (File)
Nov 10 2024, 8:13 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