HomePhabricator

Fix a ModularTransactions exception with custom fields that support change…

Description

Fix a ModularTransactions exception with custom fields that support change details

Summary: We're throwing here when we actually want to return null so we make it into custom field handling code. See Conpherence.

Test Plan: Found a failing task and re-executed it with bin/worker execute --id <id>; after this change, it didn't fatal.

Reviewers: chad

Reviewed By: chad

Differential Revision: https://secure.phabricator.com/D17051

Details

Provenance
epriestleyAuthored on Dec 14 2016, 1:45 AM
epriestleyPushed on Dec 14 2016, 2:21 AM
Reviewer
chad
Differential Revision
D17051: Fix a ModularTransactions exception with custom fields that support change details
Parents
rP7f99f2cde835: Add EditEngine + Modular Transactions for reviewers
Branches
Unknown
Tags
Unknown
Build Status
Buildable 14905
Build 19522: Run Core Tests