Differential D11435

Implement PhabricatorDestructibleInterface for event triggers
ClosedPublic
Actions

Authored by epriestley on Jan 19 2015, 10:08 PM.

Details

Reviewers

btrahan

Maniphest Tasks

T6881: Implement subscription cancellation, past-due callbacks, and Merchant review of invoices in Phortune

Commits

Restricted Diffusion Commit
rPcccdc488834a: Implement PhabricatorDestructibleInterface for event triggers

Summary

Ref T6881. When stuff with triggers is destroyed, it should destroy the triggers.

Test Plan

Will test in Instances.

Diff Detail

Repository

rP Phabricator

Branch

trigger4

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 3919
Build 3932: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

epriestley updated this revision to Diff 27477.Jan 19 2015, 10:08 PM

epriestley retitled this revision from to Implement PhabricatorDestructibleInterface for event triggers.

epriestley updated this object.

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

epriestley added a reviewer: btrahan.

epriestley added a task: T6881: Implement subscription cancellation, past-due callbacks, and Merchant review of invoices in Phortune.

Herald added a subscriber: epriestley. · View Herald TranscriptJan 19 2015, 10:08 PM

btrahan accepted this revision.Jan 19 2015, 10:09 PM

btrahan edited edge metadata.

This revision is now accepted and ready to land.Jan 19 2015, 10:09 PM

Closed by commit rPcccdc488834a: Implement PhabricatorDestructibleInterface for event triggers (authored by epriestley, committed by epriestley). · Explain WhyJan 20 2015, 12:55 AM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

infrastructure/

daemon/

workers/

storage/

PhabricatorWorkerTrigger.php

22 lines

Diff 27477

View Options

src/infrastructure/daemon/workers/storage/PhabricatorWorkerTrigger.php

Implement PhabricatorDestructibleInterface for event triggersClosedPublicActions