Page MenuHomePhabricator
Differential D21232

Move the "Inline List" view to "DiffInlineCommentQuery"
ClosedPublic
Actions

Authored by epriestley on May 7 2020, 8:03 PM.
Tags
None
Referenced Files
F15514020: D21232.id50568.diff
Thu, Apr 17, 11:44 PM
F15464128: D21232.diff
Wed, Apr 2, 6:36 AM
F15460589: D21232.id50568.diff
Tue, Apr 1, 12:13 AM
F15459617: D21232.id50557.diff
Mon, Mar 31, 2:42 PM
F15455759: D21232.id.diff
Sun, Mar 30, 4:39 AM
F15453370: D21232.diff
Sat, Mar 29, 11:17 AM
F15400897: D21232.id50557.diff
Mar 17 2025, 2:34 PM
F15382731: D21232.diff
Mar 14 2025, 2:08 PM
View All Files
Subscribers
None

Details

Reviewers
None
Maniphest Tasks
T13513: Make schema changes to inlines to prepare for new inline features
Commits
rP983d77848b4c: Move the "Inline List" view to "DiffInlineCommentQuery"
Summary

Ref T13513. Continue removing usage sites for the obsolete "DifferentialInlineCommentQuery".

Test Plan

Viewed the inline list in Differential, saw sensible inlines.

Diff Detail

Repository
rP Phabricator
Branch
iquery9
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 24373
Build 33581: Run Core Tests
Build 33580: arc lint + arc unit

Event Timeline

epriestley created this revision.May 7 2020, 8:03 PM
Harbormaster completed remote builds in B24373: Diff 50557.May 7 2020, 8:04 PM
epriestley requested review of this revision.May 7 2020, 8:04 PM
This revision was not accepted when it landed; it landed in state Needs Review.May 7 2020, 11:07 PM
Closed by commit rP983d77848b4c: Move the "Inline List" view to "DiffInlineCommentQuery" (authored by epriestley). · Explain Why
This revision was automatically updated to reflect the committed changes.
epriestley added a commit: rP983d77848b4c: Move the "Inline List" view to "DiffInlineCommentQuery".

Revision Contents
Changeset List

PathSize
src/
applications/
differential/
controller/
9 lines
diffusion/
controller/
3 lines
3 lines
infrastructure/
diff/
query/
26 lines

Diff 50557

View Options

src/applications/differential/controller/DifferentialRevisionInlinesController.php

Loading...
View Options

src/applications/diffusion/controller/DiffusionCommitController.php

Loading...
View Options

src/applications/diffusion/controller/DiffusionDiffController.php

Loading...
View Options

src/infrastructure/diff/query/PhabricatorDiffInlineCommentQuery.php

Loading...
Phabricator · Built by Phacility