HomePhabricator

Clean up various pieces of dead/obsolete Differential code

Description

Clean up various pieces of dead/obsolete Differential code

Summary:
Ref T2222.

  • Removes DifferentialTasksAttacher, which has had no callsites for a very long time.
  • Moves differential.getrevisioncomments off DifferentialCommentQuery.
  • Moves Releeph churn field off DifferentialCommentQuery.
  • Removes dead code in DifferentialRevisionViewController.
  • Removes DifferentialException (no references).
  • Removes DifferentialRevision->loadComments() (no callsites).
  • Removes DifferentialRevision->loadReviewedBy() (all callsites updated).
  • Removes DifferentialCommentQuery (all callsites updated).

Test Plan: Mostly a lot of grep.

Reviewers: btrahan

Reviewed By: btrahan

CC: aran

Maniphest Tasks: T2222

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

Details

Provenance
epriestleyAuthored on
epriestleyPushed on Mar 11 2014, 8:05 PM
Differential Revision
D8476: Clean up various pieces of dead/obsolete Differential code
Parents
rPa49fec39be84: Move lint/unit test warning code forward to Transactions
Branches
Unknown
Tags
Unknown

Event Timeline