HomePhabricator

Remove willWriteRevision/didWriteRevision hooks

Description

Remove willWriteRevision/didWriteRevision hooks

Summary:
Ref T2222. DifferentialRevisionEditor has no remaining callsites, but it has a bit of functionality which still needs to be ported forward. I'm going to rip it apart piece by piece.

This removes the willWriteRevision/didWriteRevision hooks. They are completely encapsulated by transactions now, except for a unique piece of branch/task logic, which I migrated forward.

Test Plan:

  • Lots of grep.
  • Created a new revision on branch T25, saw it associate with the task.

Reviewers: btrahan

Reviewed By: btrahan

CC: aran

Maniphest Tasks: T2222

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

Details

Provenance
epriestleyAuthored on
epriestleyPushed on Mar 11 2014, 8:05 PM
Reviewer
btrahan
Differential Revision
D8454: Remove willWriteRevision/didWriteRevision hooks
Parents
rPfbaa12440eac: Use DifferentialRevisionEditor in lipsum
Branches
Unknown
Tags
Unknown
Tasks
T2222: Implement ApplicationTransactions in Differential

Event Timeline