Page MenuHomePhabricator
Differential D9289

Introduce DifferentialChangesetQuery and remove loadHunks()
ClosedPublic
Actions

Authored by epriestley on May 25 2014, 4:01 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Dec 22, 3:13 AM
Unknown Object (File)
Fri, Dec 20, 9:29 AM
Unknown Object (File)
Sun, Dec 15, 12:26 PM
Unknown Object (File)
Thu, Dec 12, 8:46 AM
Unknown Object (File)
Wed, Dec 11, 8:31 AM
Unknown Object (File)
Sun, Dec 8, 8:43 AM
Unknown Object (File)
Sun, Dec 8, 1:08 AM
Unknown Object (File)
Fri, Dec 6, 4:40 PM
View All Files
Subscribers
epriestley

Details

Reviewers
btrahan
Maniphest Tasks
T4045: Store diffs as binary, not UTF-8
T5179: Unearth Differential rendering technical debt
Commits
Restricted Diffusion Commit
rPbb306b58d553: Introduce DifferentialChangesetQuery and remove loadHunks()
Summary

Ref T4045. Ref T5179. This removes all non-Query hunk loads.

Test Plan
  • Viewed revisions.
  • Viewed standalone changesets.
  • Viewed raw old/new files.
  • Viewed vs diffs.
  • Enabled inline comments in mail and sent some transactions with inlines.
  • Called differential.getrawdiff.
  • Grepped for loadHunks().

Diff Detail

Repository
rP Phabricator
Branch
hunk4
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 697
Build 697: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

epriestley updated this revision to Diff 22054.May 25 2014, 4:01 PM
epriestley retitled this revision from to Introduce DifferentialChangesetQuery and remove loadHunks().
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
epriestley added tasks: T5179: Unearth Differential rendering technical debt, T4045: Store diffs as binary, not UTF-8.
Herald added a subscriber: epriestley. · View Herald TranscriptMay 25 2014, 4:01 PM
Harbormaster completed remote builds in B697: Diff 22054.May 25 2014, 4:02 PM
btrahan accepted this revision.Jun 3 2014, 10:12 PM
btrahan edited edge metadata.
This revision is now accepted and ready to land.Jun 3 2014, 10:12 PM
epriestley closed this revision.Jun 4 2014, 1:03 AM
epriestley updated this revision to Diff 22328.

Closed by commit rPbb306b58d553 (authored by @epriestley).

Revision Contents
Changeset List

Diff 22054

View Options

src/__phutil_library_map__.php

Loading...
View Options

src/applications/differential/controller/DifferentialChangesetViewController.php

Loading...
View Options

src/applications/differential/controller/DifferentialRevisionViewController.php

Loading...
View Options

src/applications/differential/editor/DifferentialTransactionEditor.php

Loading...
View Options

src/applications/differential/query/DifferentialChangesetQuery.php

Loading...
View Options

src/applications/differential/query/DifferentialDiffQuery.php

Loading...
View Options

src/applications/differential/storage/DifferentialChangeset.php

Loading...
Phabricator · Built by Phacility