Page MenuHomePhabricator

When a custom field is "disabled", that should take precedence over the field being "required"
Closed, ResolvedPublic

Description

See PHI1393. If you specify that a custom field is both "required" and "disabled" (via subtypes), the "required" attribute is currently stronger. This is less useful than making "disabled" stronger.

This configuration is meaningful and useful now that you can have different fields on "Bug" and "Fish" task subtypes, and requiring a "Swimminess" value on Fish but not on Bugs is perfectly sensible.

To reproduce simply:

  • Define a custom field in maniphest.custom-field-definitions that is and "required", then make it "disabled" on some subtype (say, subtype X) with subtype config.
  • Create a task of subtype X.

Desired behavior:

  • Field does not appear, no errors, creating a task works fine.

Actual behavior today:

  • Form submission fails with an error that the field is required (even though the field is not visible on the form and can not be filled out).

Revisions and Commits

Restricted Differential Revision

Event Timeline

epriestley created this task.
epriestley added a revision: Restricted Differential Revision.Aug 22 2019, 1:02 AM
epriestley closed this task as Resolved by committing Restricted Diffusion Commit.Aug 22 2019, 1:03 AM
epriestley added a commit: Restricted Diffusion Commit.