Throw a more helpful error for bad Differential actions
ClosedPublic
Actions

Authored by epriestley on Mar 21 2015, 11:43 AM.

Details

Reviewers

joshuaspence
btrahan

Maniphest Tasks

T7611: Transaction type 'differential:action' is missing an internal apply implementation

Commits

Restricted Diffusion Commit
rP5001aadf4660: Throw a more helpful error for bad Differential actions

Summary

Ref T7611. This should let us figure out the root cause, hopefully.

Test Plan

iiam

Diff Detail

Repository

rP Phabricator

Branch

action1

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 4938
Build 4956: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

epriestley updated this revision to Diff 29150.Mar 21 2015, 11:43 AM

epriestley retitled this revision from to Throw a more helpful error for bad Differential actions.

epriestley updated this object.

epriestley edited the test plan for this revision. (Show Details)

epriestley added reviewers: joshuaspence, btrahan.

epriestley added a task: T7611: Transaction type 'differential:action' is missing an internal apply implementation.

Herald added a subscriber: epriestley. · View Herald TranscriptMar 21 2015, 11:43 AM

btrahan accepted this revision.Mar 23 2015, 3:59 PM

btrahan edited edge metadata.

This revision is now accepted and ready to land.Mar 23 2015, 3:59 PM

Closed by commit rP5001aadf4660: Throw a more helpful error for bad Differential actions (authored by epriestley, committed by epriestley). · Explain WhyMar 23 2015, 4:10 PM

This revision was automatically updated to reflect the committed changes.

btrahan mentioned this in T7611: Transaction type 'differential:action' is missing an internal apply implementation.Apr 6 2015, 6:52 PM

Revision Contents
Changeset List

Path

Size

src/

applications/

differential/

editor/

DifferentialTransactionEditor.php

5 lines

Diff 29150

View Options

src/applications/differential/editor/DifferentialTransactionEditor.php

Throw a more helpful error for bad Differential actionsClosedPublicActions