Page MenuHomePhabricator

Editing Calendar forms can fail to build "newEditEngineSubtypeMap()" during transaction validation
Closed, ResolvedPublic

Description

See https://discourse.phabricator-community.org/t/editing-calendar-forms-causes-a-unable-to-resolve-method-neweditenginesubtypemap/3056.

PhabricatorEditEngineSubtypeTransaction->validateTransactions() is called on any EditEngine configuration (which is normal), but does not properly test if the engine supportsSubtypes() before it builds a newSubtypeMap(). This throws.

The validation logic should fail explicitly, and should only fail if the caller is actually trying to assign a subtype to an object which does not support subtypes.

Revisions and Commits

Restricted Differential Revision

Event Timeline

epriestley triaged this task as Normal priority.Aug 27 2019, 2:54 PM
epriestley created this task.
epriestley added a revision: Restricted Differential Revision.Aug 27 2019, 2:57 PM
epriestley closed this task as Resolved by committing Restricted Diffusion Commit.Aug 28 2019, 1:57 PM
epriestley added a commit: Restricted Diffusion Commit.