HomePhabricator

Migrate Differential revision edges to use modern `EdgeType` subclasses

Description

Migrate Differential revision edges to use modern EdgeType subclasses

Summary: Modernize Differential edges to subclass PhabricatorEdgeType. Largely based on D11045.

Test Plan: From previous experience, these changes are fairly trivial and safe. I poked around a little to make sure things looked reasonably okay.

Reviewers: epriestley, Blessed Reviewers

Reviewed By: epriestley, Blessed Reviewers

Subscribers: Korvin, Krenair, epriestley

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

Details

Auditors
Krenair
Provenance
joshuaspenceAuthored on
joshuaspencePushed on Jan 1 2015, 4:08 AM
Reviewer
Blessed Reviewers
Differential Revision
D11074: Migrate Differential revision edges to use modern `EdgeType` subclasses
Parents
rPc564a3c22a65: Fix method name
Branches
Unknown
Tags
Unknown

Event Timeline

Krenair added a subscriber: epriestley.
Krenair added a subscriber: Krenair.

Shouldn't DiffusionCommitHasRevisionEdgeType and DifferentialReviewerForRevisionEdgeType have get*String functions? Is this what is causing the Herald entries on https://secure.phabricator.com/project/edit/1308/ ?

This got fixed in rP2dea110 it seems.