Page MenuHomePhabricator

Move many "reviewers" readers to new storage
ClosedPublic

Authored by epriestley on Mar 20 2017, 8:24 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Dec 14, 4:27 PM
Unknown Object (File)
Wed, Dec 11, 10:22 AM
Unknown Object (File)
Tue, Dec 10, 3:43 PM
Unknown Object (File)
Mon, Dec 9, 9:11 AM
Unknown Object (File)
Sat, Dec 7, 1:49 PM
Unknown Object (File)
Thu, Dec 5, 12:01 PM
Unknown Object (File)
Thu, Dec 5, 8:04 AM
Unknown Object (File)
Wed, Dec 4, 8:07 PM
Subscribers
None

Details

Summary

Ref T10967.

When we query for revisions with particular reviewers, use the new table to drive the query.

When we load revisions for use in the application, also use the new table to drive the query.

This doesn't convert everything: there's some old loadRelationships() stuff still using the old table. But this moves the major stuff over.

(This also changes the icon for "commented" from a question mark to a speech bubble.)

Test Plan
  • Viewed revision lists and detail views on old and new code, saw identical outcomes.
  • Updated revisions, accepted/rejected/commented on revisions.
  • Hit the "Accepted Older" and "Commented Older" states by taking an action and then updating.
  • Grepped for removed methods (like getEdgeData() and getDiffID()).

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

chad added inline comments.
src/applications/differential/customfield/DifferentialCustomField.php
78

?

This revision is now accepted and ready to land.Mar 20 2017, 9:47 PM
src/applications/differential/customfield/DifferentialCustomField.php
78

It's possible that getActiveDiff() isn't loaded in some cases so we'd get a "thing not attached" exception. Since all we need it for is to pretty-up the UI I'm just blarpin' out.

This revision was automatically updated to reflect the committed changes.